ntentan\utils\filesystem\FileCollection

Complexity

Total Complexity

20

Size/Duplication

Total Lines	96
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

14 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	getInstance()	0	11	3
A	rewind()	0	4	1
A	current()	0	4	1
A	key()	0	4	1
A	next()	0	4	1
A	valid()	0	4	1
A	offsetSet()	0	9	2
A	offsetExists()	0	4	1
A	offsetGet()	0	4	2
A	offsetUnset()	0	4	1
A	moveTo()	0	6	2
A	getSize()	0	7	1
A	copyTo()	0	6	2

<?php


namespace ntentan\utils\filesystem;


use ntentan\utils\Filesystem;

class FileCollection implements \Iterator, \ArrayAccess, FileInterface

There is at least one abstract method in this class. Maybe declare it as abstract, or implement the remaining methods: delete, getPath

{
    private $paths;
    private $iteratorIndex;
    private $instances;

    public function __construct($paths)
    {
        $this->paths = $paths;
        $this->iteratorIndex = 0;
    }

    private function getInstance($index)

The return type could not be reliably inferred; please add a @return annotation.

    {
        if(!isset($this->instances[$index])) {
            if(is_dir($this->paths[$index])) {
                $this->instances[$index] = Filesystem::directory($this->paths[$index]);
            } else {
                $this->instances[$index] = Filesystem::file($this->paths[$index]);
            }
        }
        return $this->instances[$index];
    }

    public function rewind()
    {
        $this->iteratorIndex = 0;
    }

    public function current()
    {
        return $this->getInstance($this->iteratorIndex);
    }

    public function key()
    {
        return $this->iteratorIndex;
    }

    public function next()
    {
        $this->iteratorIndex++;
    }

    public function valid()
    {
        return isset($this->paths[$this->iteratorIndex]);
    }

    public function offsetSet($index, $path)
    {
        if(is_null()) {
            $this->paths[] = $path;
        } else {
            $this->paths[$index] = $path;
            unset($this->instances[$index]);
        }
    }

    public function offsetExists($index)
    {
        return isset($this->paths[$index]);
    }

    public function offsetGet($index)
    {
        return isset($this->paths[$index]) ? $this->paths[$index] : null;
    }

    public function offsetUnset($index)
    {
        unset($this->paths[$index]);
    }

    public function moveTo(string $destination): void
    {
        foreach($this as $file) {
            $file->moveTo($destination);
        }
    }

    public function getSize(): int
    {
        return array_reduce($this,
            function($carry, $item){
                $carry += $item->getSize();
            }, 0);
    }

    public function copyTo(string $destination): void
    {
        foreach($this as $file) {
            $this->copyTo($destination);
        }
    }
}