ByJG\AnyDataset\Core\Formatter\BaseFormatter

Complexity

Total Complexity

4

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Test Coverage

Coverage

85.71%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	3
A	saveToFile()	0	3	1

<?php

namespace ByJG\AnyDataset\Core\Formatter;

use \ByJG\AnyDataset\Core\AnyDataset;
use \ByJG\AnyDataset\Core\Row;
use InvalidArgumentException;

abstract class BaseFormatter
{
    /**
     * @var AnyDataset
     */
    protected $object;

    abstract public function raw();

    abstract public function toText();

    public function saveToFile($filename)
    {
        file_put_contents($filename, $this->toText());
    }

    /**
     * $object AnyDataset|Row
     */
    public function __construct($object)
    {
        if (!($object instanceof AnyDataset) && !($object instanceof Row)) {
            throw new InvalidArgumentException("Constructor must have an AnyDataset or Row instance in the argument");
        }
        $this->object = $object;

It seems like $object can also be of type ByJG\AnyDataset\Core\Row. However, the property $object is declared as type ByJG\AnyDataset\Core\AnyDataset. Maybe add an additional type check?

    }
}