Sufir\Calc\Calc

Complexity

Total Complexity

18

Size/Duplication

Total Lines	128
Duplicated Lines	20.31 %

Coupling/Cohesion

Components	1
Dependencies	1

Test Coverage

Coverage

0%

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
C	evaluate()	0	53	10
A	registerFunction()	13	13	1
A	registerVariable()	13	13	1
B	calc()	0	17	6

<?php

/**
 * Calc.php
 *
 * @date 28.03.2015 0:54:24
 * @copyright Sklyarov Alexey <sufir@mihailovka.info>
 */

namespace Sufir\Calc;

use Sufir\Calc\Token\AbstractToken;

/**
 * Calc
 *
 * Description of Calc
 *
 * @author Sklyarov Alexey <sufir@mihailovka.info>
 * @package Sufir\Calc
 */
class Calc
{
    protected $functions = [];

    protected $variables = [];

    /**
     *
     * @param \Sufir\Calc\Token\AbstractToken[] $tokens
     */
    public function evaluate(array $tokens)
    {
        $stack = new \SplStack;
        foreach ($tokens as $token) {
            // Если на вход подан операнд, он помещается на вершину стека
            if ($token->isNumber()) {
                $stack->push($token);

                // Если на вход подан знак операции
            } elseif ($token->isOperator()) {
                if ($stack->count() < 2) {
                    throw new \Exception('Ошибочка вышла! Недостаточно операндов для бинарного оператора: ' . $token);

                // операция выполняется над требуемым количеством значений,
                // извлечённых из стека, взятых в порядке добавления
                } elseif ($stack->count() > 1) {
                    $second = $stack->pop();
                    $first = $stack->pop();

                    $result = $this->calc((string) $token, (string) $first, (string) $second);

                    // Результат выполненной операции кладётся на вершину стека
                    $stack->push($result);
                }

                // Если на вход подана функция
            } elseif ($token->isFunction()) {
                if (!isset($this->functions[$token->getValue()])) {
                    throw new \Exception('Не найдена функция: ' . $token->getValue());
                }

                $countOfArguments = $this->functions[$token->getValue()]['args'];

                if ($stack->count() < $countOfArguments) {
                    throw new \Exception('Ошибочка вышла! Недостаточно аргументов для функции: ' . $token);
                }

                $arguments = array();

                for ($idx = 0; $idx < $countOfArguments; $idx++) {
                    $arguments[] = ((string) $stack->pop()) - 0;
                }

                $arguments = array_reverse($arguments);

                $result = call_user_func_array($this->functions[$token->getValue()]['func'], $arguments);

                $stack->push($result);
            }
        }

        return $stack->top();
    }

    /**
     *
     * @param string $name
     * @param \Closure $callable
     * @return \Sufir\Calc\Calc
     */
    public function registerFunction($name, \Closure $callable)

This method seems to be duplicated in your project.

    {
        $objReflector = new \ReflectionObject($callable);
        $reflector = $objReflector->getMethod('__invoke');
        $parameters = $reflector->getParameters();

        $this->functions[$name] = array(
            'args' => count($parameters),
            'func' => $callable,
        );

        return $this;
    }

    /**
     *
     * @param string $name
     * @param \Closure $callable

There is no parameter named $callable. Was it maybe removed?

     * @return \Sufir\Calc\Calc
     */
    public function registerVariable($name, $value)

The parameter $value is not used and could be removed.

This method seems to be duplicated in your project.

    {
        $objReflector = new \ReflectionObject($callable);

The variable $callable does not exist. Did you forget to declare it?

        $reflector = $objReflector->getMethod('__invoke');
        $parameters = $reflector->getParameters();

        $this->functions[$name] = array(
            'args' => count($parameters),
            'func' => $callable,
        );

        return $this;
    }

    /**
     * @param string $operator
     * @param integer|float $firstOperand
     * @param integer|float $secondOperand
     * @return int
     */
    protected function calc($operator, $firstOperand, $secondOperand)

Please use __construct() instead of a PHP4-style constructor that is named after the class.

    {
        switch ($operator) {
            case '^':
                return pow($firstOperand, $secondOperand);
            case '*':
                return $firstOperand * $secondOperand;
            case '/':
                return $firstOperand / $secondOperand;
            case '+':
                return $firstOperand + $secondOperand;
            case '-':
                return $firstOperand - $secondOperand;
            default:
                return 0;
        }
    }
}