SPSS\Sav\Record\Info\VariableAttributes

Complexity

Total Complexity

9

Size/Duplication

Total Lines	51
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	2
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	write()	0	19	5
A	read()	0	13	4

<?php

namespace SPSS\Sav\Record\Info;

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

class VariableAttributes extends Info
{
    const SUBTYPE = 18;

    /**
     * @var array
     */
    public $data = [];

    /**
     * @param Buffer $buffer
     */
    public function read(Buffer $buffer)
    {
        parent::read($buffer);
        $data = $buffer->readString($this->dataSize * $this->dataCount);
        foreach (explode('/', $data) as $item) {

It seems like $data can also be of type false; however, parameter $string of explode() does only seem to accept string, maybe add an additional type check?

            list($var, $value) = explode(':', $item);
            if (preg_match_all('#(.+)\((.+)\)#Uis', $value, $matches)) {
                $this->data[$var] = [];
                foreach ($matches[1] as $key => $val) {
                    $this->data[$var][$val] = trim(trim($matches[2][$key]), '\'');
                }
            } else {
                $this->data[$var] = $value;
            }
        }
    }

    /**
     * @param Buffer $buffer
     */
    public function write(Buffer $buffer)
    {
        $lines = [];
        foreach ($this->data as $var => $value) {
            if (is_array($value)) {
                $_tmpString = '';
                foreach ($value as $key => $val) {
                    $_tmpString .= sprintf("%s('%s'\n)", $key, $val);
                }
                $value = $_tmpString;
            }
            $lines[] = sprintf('%s:%s', $var, $value);
        }

        if ($lines) {
            $data = implode('/', $lines);
            $this->dataCount = mb_strlen($data);
            parent::write($buffer);
            $buffer->writeString($data);
        }
    }
}