byrokrat\autogiro\Visitor\PayeeVisitor

Complexity

Total Complexity

7

Size/Duplication

Total Lines	59
Duplicated Lines	50.85 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	beforeAutogiroFile()	0	5	1
A	beforePayeeBankgiro()	15	15	3
A	beforePayeeBgcNumber()	15	15	3

<?php
/**
 * This file is part of byrokrat\autogiro.
 *
 * byrokrat\autogiro is free software: you can redistribute it and/or
 * modify it under the terms of the GNU General Public License as published
 * by the Free Software Foundation, either version 3 of the License, or
 * (at your option) any later version.
 *
 * byrokrat\autogiro is distributed in the hope that it will be useful,
 * but WITHOUT ANY WARRANTY; without even the implied warranty of
 * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
 * GNU General Public License for more details.
 *
 * You should have received a copy of the GNU General Public License
 * along with byrokrat\autogiro. If not, see <http://www.gnu.org/licenses/>.
 *
 * Copyright 2016-18 Hannes Forsgård
 */

declare(strict_types = 1);

namespace byrokrat\autogiro\Visitor;

use byrokrat\autogiro\Tree\PayeeBankgiro;
use byrokrat\autogiro\Tree\PayeeBgcNumber;

/**
 * Validate that payee bankgiro and BGC customer number are constant within tree
 */
class PayeeVisitor extends ErrorAwareVisitor
{
    /**
     * @var string Payee bankgiro account number
     */
    private $payeeBg;

    /**
     * @var string Payee BGC customer number
     */
    private $payeeBgcNr;

    /**
     * Reset payee bankgiro and customer number before a new file is traversed
     */
    public function beforeAutogiroFile(): void
    {
        $this->payeeBg = '';
        $this->payeeBgcNr = '';
    }

    /**
     * Validate payee bankgiro number
     */
    public function beforePayeeBankgiro(PayeeBankgiro $node): void

This method seems to be duplicated in your project.

    {
        if (!$this->payeeBg) {
            $this->payeeBg = $node->getValue();
        }

        if ($node->getValue() != $this->payeeBg) {
            $this->getErrorObject()->addError(
                "Non-matching payee bankgiro numbers (expecting: %s, found: %s) on line %s",
                $this->payeeBg,
                $node->getValue(),
                (string)$node->getLineNr()
            );
        }
    }

    /**
     * Validate payee BGC customer number
     */
    public function beforePayeeBgcNumber(PayeeBgcNumber $node): void

This method seems to be duplicated in your project.

    {
        if (!$this->payeeBgcNr) {
            $this->payeeBgcNr = $node->getValue();
        }

        if ($node->getValue() != $this->payeeBgcNr) {
            $this->getErrorObject()->addError(
                "Non-matching payee BGC customer numbers (expecting: %s, found: %s) on line %s",
                $this->payeeBgcNr,
                $node->getValue(),
                (string)$node->getLineNr()
            );
        }
    }
}