Document::toArray() - Code Metrics - Inspection of "Code cleanup, use private variables instead of pub..." - collecthor/spss - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Push — master ( 386efc...fcbe67 )

by Sam

created 2020-01-31 08:59 UTC

Document::toArray() A

↳ Parent: Document

Complexity

Conditions	1
Paths	1

Size

Total Lines	3
Code Lines	1

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	2

Importance

Changes

Metric	Value
cc	1
eloc	1
c	0
b	0
f	0
nc	1
nop	0
dl	0
loc	3
ccs	0
cts	3
cp	0
crap	2
rs	10

<?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));
            $this->lines[] = trim(/** @scrutinizer ignore-type */ $buffer->readString(self::LENGTH));
        }
    }

    /**
     * @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]);
    }
}


collecthor / spss

Push — master ( 386efc...fcbe67 )

Document::toArray() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like