SPSS\Sav\Record\Document

Complexity

Total Complexity

11

Size/Duplication

Total Lines	84
Duplicated Lines	0 %

Test Coverage

Coverage

50%

Importance

Changes	1
Bugs	0	Features	0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	read()	0	5	2
A	append()	0	4	2
A	toArray()	0	3	1
A	offsetSet()	0	3	1
A	write()	0	6	2
A	offsetGet()	0	3	1
A	offsetUnset()	0	3	1
A	offsetExists()	0	3	1

<?php

namespace SPSS\Sav\Record;

use SPSS\Buffer;
use SPSS\Sav\Record;

class Document extends Record implements \ArrayAccess
{
    const TYPE = 6;
    const LENGTH = 80;

    /**
     * @var array
     */
    protected $lines = [];

    /**
     * @param Buffer $buffer
     */
    public function read(Buffer $buffer)
    {
        $count = $buffer->readInt();
        for ($i = 0; $i < $count; $i++) {
            $this->lines[] = trim($buffer->readString(self::LENGTH));

It seems like $buffer->readString(self::LENGTH) can also be of type false; however, parameter $str of trim() does only seem to accept string, maybe add an additional type check?

        }
    }

    /**
     * @param Buffer $buffer
     */
    public function write(Buffer $buffer)
    {
        $buffer->writeInt(self::TYPE);
        $buffer->writeInt(count($this->lines));
        foreach ($this->lines as $line) {
            $buffer->writeString((string) $line, self::LENGTH);
        }
    }

    /**
     * @return array
     */
    public function toArray()
    {
        return $this->lines;
    }

    /**
     * @param array $lines
     */
    public function append($lines)
    {
        foreach ($lines as $line) {
            $this->lines[] = $line;
        }
    }

    /**
     * @param mixed $offset
     * @return bool
     */
    public function offsetExists($offset)
    {
        return isset($this->lines[$offset]);
    }

    /**
     * @param mixed $offset
     * @return mixed
     */
    public function offsetGet($offset)
    {
        return $this->lines[$offset];
    }

    /**
     * @param mixed $offset
     * @param mixed $value
     */
    public function offsetSet($offset, $value)
    {
        $this->lines[$offset] = $value;
    }

    /**
     * @param mixed $offset
     */
    public function offsetUnset($offset)
    {
        unset($this->lines[$offset]);
    }
}