LegalThings\DataEnricher\Processor\Sum

Complexity

Total Complexity

11

Size/Duplication

Total Lines	65
Duplicated Lines	52.31 %

Coupling/Cohesion

Components	1
Dependencies	2

Test Coverage

Coverage

77.78%

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	applyToNode()	14	14	4
B	resolve()	20	20	6
A	execute()	0	6	1

<?php

namespace LegalThings\DataEnricher\Processor;

use LegalThings\DataEnricher\Node;
use LegalThings\DataEnricher\Processor;

/**
 * Calculate sum of the numbers in an array
 */
class Sum implements Processor
{
    use Processor\Implementation;
    
    /**
     * Apply processing to a single node
     * 
     * @param Node $node
     */
    public function applyToNode(Node $node)

This method seems to be duplicated in your project.

    {
        $instruction = $node->getInstruction($this);
        
        $list = $this->resolve($instruction);
        
        if (isset($list) && !is_array($list)) {
            throw new \Exception("Unable to apply {$this->property} processing instruction:"
                . " Expected an array, got a " . (is_object($list) ? get_class($list) . ' ' : '') . gettype($list));
        }
        
        $result = $this->execute((array)$list);
        $node->setResult($result);
    }
    
    /**
     * Resolve processing nodes in the instruction
     * 
     * @param array $list
     * @return array
     */
    public function resolve($list)

This method seems to be duplicated in your project.

    {
        if ($list instanceof Node) {
            $list = $list->getResult();
        }
        
        if (is_array($list)) {
            foreach ($list as &$item) {
                if ($item instanceof Node) {
                    $item = $item->getResult();
                }

                if ($item instanceof \Traversable) {
                    $item = iterator_to_array($item);
                }
            }
        }
        
        return $list;
    }
    
    /**
     * Calculate sum
     * 
     * @param array $list
     * 
     * @return number
     */
    protected function execute(array $list)
    {
        $sum = array_sum($list);
        
        return $sum;
    }
}