PhpYacc\Lalr\Conflict\ShiftReduce

Complexity

Total Complexity

4

Size/Duplication

Total Lines	51
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	isShiftReduce()	0	4	1
A	state()	0	4	1
A	reduce()	0	4	1

<?php
/**
 * This file is part of PHP-Yacc package.
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */
declare(strict_types=1);

namespace PhpYacc\Lalr\Conflict;

use PhpYacc\Grammar\State;
use PhpYacc\Grammar\Symbol;
use PhpYacc\Lalr\Conflict;

/**
 * Class ShiftReduce.
 */
class ShiftReduce extends Conflict
{
    /**
     * @var State
     */
    protected $state;

    /**
     * @var int
     */
    protected $reduce;

    /**
     * ShiftReduce constructor.
     *
     * @param State         $state
     * @param int           $reduce
     * @param Symbol        $symbol
     * @param Conflict|null $next
     */
    public function __construct(State $state, int $reduce, Symbol $symbol, Conflict $next = null)
    {
        $this->state = $state;
        $this->reduce = $reduce;
        parent::__construct($symbol, $next);

It seems like $next defined by parameter $next on line 39 can also be of type object<PhpYacc\Lalr\Conflict>; however, PhpYacc\Lalr\Conflict::__construct() does only seem to accept null|object<self>, maybe add an additional type check?

    }

    /**
     * @return bool
     */
    public function isShiftReduce(): bool
    {
        return true;
    }

    /**
     * @return State
     */
    public function state(): State
    {
        return $this->state;
    }

    /**
     * @return int
     */
    public function reduce(): int
    {
        return $this->reduce;
    }
}