Lexer::getNextToken() - Code Metrics - Inspection of "Merge branch 'master' of github.com:drumser/arithm..." - drumser/arithmetic-parser - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 633b25...e5131a )

by Alexander

created 2019-08-28 17:27 UTC

Lexer::getNextToken() C

↳ Parent: Lexer

Complexity

Conditions	14
Paths	24

Size

Total Lines	58
Code Lines	36

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	14
eloc	36
c	1
b	0
f	0
nc	24
nop	0
dl	0
loc	58
rs	6.2666

How to fix Long Method Complexity

<?php


namespace App;


use App\Exception\LexerException;
use App\Exception\UnknownIdentifier;

class Lexer
{
    /**
     * @var string
     */
    private $input;
    /**
     * @var string|null
     */
    private $currentChar;
    /**
     * @var int
     */
    private $pos = 0;

    /**
     * Lexer constructor.
     * @param string $input
     * @throws LexerException
     */
    public function __construct(
        string $input
    )
    {
        $this->input       = $input;
        $this->currentChar = $this->input[$this->pos];
        if ($this->currentChar === '') {
            throw new LexerException('Lexer input is empty');
        }
    }

    private function isAlpha(string $char): bool
    {
        return (bool)preg_match('/[a-zA-Z_]/', $char);
    }

    /**
     * @return Token
     * @throws UnknownIdentifier
     */
    private function identifier(): Token
    {
        $identifier = '';
        while ($this->isAlpha($this->currentChar)) {
        while ($this->isAlpha(/** @scrutinizer ignore-type */ $this->currentChar)) {
            $identifier .= $this->currentChar;
            $this->advance();
        }

        if (isset(FunctionModel::BUILT_IN_FUNCTIONS[$identifier])) {
            return new Token(Token::FUNCTION_CALL, $identifier);
        }

        throw new UnknownIdentifier('Unknown identifier ' . $identifier);
    }

    /**
     * @return Token
     * @throws LexerException
     * @throws UnknownIdentifier
     */
    public function getNextToken(): Token
    {
        while ($this->currentChar !== null) {
            if ($this->isAlpha($this->currentChar)) {
                return $this->identifier();
            }

            if ($this->currentChar === ' ') {
                $this->skipWhitespace();
            }

            if ($this->currentChar === ',') {
                $this->advance();
                return new Token(Token::COMMA, ',');
            }

            if ($this->currentChar === '+') {
                $this->advance();
                return new Token(Token::PLUS, '+');
            }
            if ($this->currentChar === '-') {
                $this->advance();
                return new Token(Token::MINUS, '-');
            }
            if ($this->currentChar === '^') {
                $this->advance();
                return new Token(Token::POWER, '^');
            }
            if ($this->currentChar === '*') {
                if ($this->peek() === '*') {
                    $this->advance();
                    $this->advance();
                    return new Token(Token::POWER, '**');
                }

                $this->advance();
                return new Token(Token::MUL, '*');
            }
            if ($this->currentChar === '/') {
                $this->advance();
                return new Token(Token::REALDIV, '/');
            }

            if ($this->currentChar === '(') {
                $this->advance();
                return new Token(Token::LPAREN, '(');
            }
            if ($this->currentChar === ')') {
                $this->advance();
                return new Token(Token::RPAREN, ')');
            }

            if (is_numeric($this->currentChar)) {
                return $this->parseNumber();
            }
        }

        return new Token(Token::EOF, Token::EOF);
    }

    private function skipWhitespace(): void
    {
        if ($this->currentChar === ' ') {
            $this->advance();
        }
    }

    private function advance(): void
    {
        $this->pos++;
        $this->currentChar = $this->input[$this->pos] ?? null;
    }

    private function peek(): ?string
    {
        $pos = $this->pos + 1;
        return $this->input[$pos] ?? null;
    }

    /**
     * @return Token
     * @throws LexerException
     */
    private function parseNumber(): Token
    {
        if ($this->currentChar === null) {
            throw new LexerException('Cannot found current char');
        }
        $number = '';
        while (is_numeric($this->currentChar) || ($number !== '' && $this->currentChar === '.')) {
            $number .= $this->currentChar;
            $this->advance();
        }

        if ($this->validateNumber($number) === false) {
            throw new LexerException('Parsed number not valid');
        }

        if (strpos($number, '.')) {
            return new Token(Token::REAL, (float)$number);
        }

        return new Token(Token::INT, (int)$number);
    }

    private function validateNumber(string $number): bool
    {
        return (bool)preg_match('/^[\d]+\.?[\d]*/', $number);
    }
}

drumser / arithmetic-parser

Push — master ( 633b25...e5131a )

Lexer::getNextToken() C

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Duplication Side-by-Side

Filter issues like