MrPrompt\Centercob\Received\Partial\Footer

Complexity

Total Complexity

5

Size/Duplication

Total Lines	68
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

Test Coverage

Coverage

100%

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	getRows()	0	4	1
A	setRows()	0	4	1
A	getSequence()	0	4	1
A	setSequence()	0	4	1

<?php
namespace MrPrompt\Centercob\Received\Partial;

use MrPrompt\ShipmentCommon\Base\Sequence;

/**
 * File footer
 *
 * @author Thiago Paes <mrprompt@gmail.com>
 */
class Footer
{
    /**
     * Line lenght
     *
     * @const int
     */
    const LENGTH = 1006;

    /**
     * Type of register
     *
     * @var string
     */
    const TYPE = 'Z';

    /**
     * @var int
     */
    private $rows;

    /**
     * @var Sequence
     */
    private $sequence;

    /**
     * Constructor
     * @param string $row
     */
    public function __construct($row)
    {
        $this->rows = substr($row, 1, 6);

The property $rows was declared of type integer, but substr($row, 1, 6) is of type string. Maybe add a type cast?

        $this->sequence = new Sequence((substr($row, 500, 6)));
    }

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

    /**
     * @param int $rows
     */
    public function setRows($rows = 0)
    {
        $this->rows = $rows;
    }

    /**
     * @return Sequence
     */
    public function getSequence()
    {
        return $this->sequence;
    }

    /**
     * @param Sequence $sequence
     */
    public function setSequence(Sequence $sequence)
    {
        $this->sequence = $sequence;
    }
}