Didatus\TextCaptcha\Generator\Arithmetical

Complexity

Total Complexity

11

Size/Duplication

Total Lines	93
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	getQuestionString()	0	22	3
A	calculateAnswer()	0	16	4
A	getTextCaptcha()	0	7	2
A	generateQuestion()	0	12	1
A	__construct()	0	3	1

<?php

namespace Didatus\TextCaptcha\Generator;

use Didatus\TextCaptcha\TextCaptcha;
use Symfony\Component\Translation\Translator;

class Arithmetical implements GeneratorInterface
{
    private $translator;

    private $numbers = [
        'zero',
        'one',
        'two',
        'three',
        'four',
        'five',
        'six',
        'seven',
        'eight',
        'nine',
    ];

    private $arithemticalFunctions = [
        '+' => 'plus',
        '-' => 'minus',
    ];

    private $captcha;

    public function __construct(Translator $translator)
    {
        $this->translator = $translator;
    }

    public function getTextCaptcha(): TextCaptcha
    {
        if (is_null($this->captcha)) {
            $this->generateQuestion();
        }

        return $this->captcha;
    }

    private function generateQuestion()
    {
        $questionParts = [];
        $questionParts[] = rand(0, 9);
        $questionParts[] = array_rand($this->arithemticalFunctions);
        $questionParts[] = rand(0, 9);
        $questionParts[] = array_rand($this->arithemticalFunctions);
        $questionParts[] = rand(0, 9);

        $this->captcha = new TextCaptcha(
            $this->getQuestionString($questionParts),
            $this->calculateAnswer($questionParts)
        );
    }

    private function getQuestionString(array $questionParts):string
    {
        $question = '';
        foreach ($questionParts as $key => $part) {
            if ($key % 2 == 0) {
                $question .= $this->translator->trans($this->numbers[$part]);
            } else {
                $question .= $this->translator->trans($this->arithemticalFunctions[$part]);
            }
            $question .=  ' ';
        }

        /*
        $question = '';
        $question .= $this->numbers[$questionParts[0]];
        $question .= ' ' . $this->arithemticalFunctions[$questionParts[1]] . ' ';
        $question .= $this->numbers[$questionParts[2]];
        $question .= ' ' . $this->arithemticalFunctions[$questionParts[3]] . ' ';
        $question .= $this->numbers[$questionParts[4]];
        */

        return rtrim($question);
    }

    private function calculateAnswer(array $questionParts): string
    {
        $result = array_shift($questionParts);

        for ($i = 0; $i < count($questionParts); $i=$i+2) {

It seems like you are calling the size function count() as part of the test condition. You might want to compute the size beforehand, and not on each iteration.

            switch($questionParts[$i]) {
                case '+':
                    $result = $result + $questionParts[$i+1];
                    break;
                case '-':
                    $result = $result - $questionParts[$i+1];
                    break;
            }
        }

        return $result;
    }
}