chillerlan\Filereader\Directory

Complexity

Total Complexity

15

Size/Duplication

Total Lines	101
Duplicated Lines	18.81 %

Coupling/Cohesion

Components	1
Dependencies	4

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	change()	0	5	1
A	read()	0	24	5
A	create()	8	8	3
A	delete()	8	8	3
A	rename()	3	8	2

<?php
/**
 * Class Directory
 *
 * @filesource   Directory.php
 * @created      03.03.2017
 * @package      chillerlan\Filereader
 * @author       Smiley <smiley@chillerlan.net>
 * @copyright    2017 Smiley
 * @license      MIT
 */

namespace chillerlan\Filereader;

use chillerlan\Filereader\Drivers\FSDriverInterface;

class Directory extends FSAbstract{

	/**
	 * Directory constructor.
	 *
	 * @param \chillerlan\Filereader\Drivers\FSDriverInterface $driver
	 * @param string                                           $path
	 */
	public function __construct(FSDriverInterface $driver, string $path){
		parent::__construct($driver);

		$this->path = $path;
	}

	/**
	 * @param string $path
	 *
	 * @return \chillerlan\Filereader\Directory
	 */
	public function change(string $path):Directory{
		$this->path = $path;

		return $this;
	}

	/**
	 * Reads a directory and returns the contents as an array of \stdClass
	 *
	 * @return array
	 * @throws \chillerlan\Filereader\FilereaderException
	 */
	public function read():array{

		if(!$this->filereader->isDir($this->path)){
			throw new FilereaderException('Directory not found: '.$this->path);
		}

		$dir = scandir($this->path);

		$filelist = [];

		if(is_array($dir)){

			foreach($dir as $file){

				if(in_array($file, ['.', '..'], true)){
					continue;
				}

				$filelist[] = new File($this->filereader, $this, $file);
			}
		}

		return $filelist;
	}

	/**
	 * @param string|null $subdir
	 *
	 * @return bool
	 */
	public function create(string $subdir = null):bool{

This method seems to be duplicated in your project.

		if($subdir && $this->filereader->isDir($this->path)){

The expression $subdir of type null|string is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== null instead.

			return $this->filereader->makeDir($this->path.DIRECTORY_SEPARATOR.$subdir);
		}

		return $this->filereader->makeDir($this->path);
	}

	/**
	 * @param string|null $subdir
	 *
	 * @return bool
	 */
	public function delete(string $subdir = null):bool{

This method seems to be duplicated in your project.

		if($subdir && $this->filereader->isDir($this->path)){

The expression $subdir of type null|string is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== null instead.

			return $this->filereader->deleteDir($this->path.DIRECTORY_SEPARATOR.$subdir);
		}

		return $this->filereader->deleteDir($this->path);
	}

	/**
	 * @param string $newname
	 * @param bool   $overwrite
	 *
	 * @return \chillerlan\Filereader\Directory
	 * @throws \chillerlan\Filereader\FilereaderException
	 */
	public function rename(string $newname, bool $overwrite = true):Directory{

		if(!$this->filereader->rename($this->path, $newname, $overwrite)){

This code seems to be duplicated across your project.

			throw new FilereaderException('cannot rename '.$this->path.' to '.$newname); // @codeCoverageIgnore
		}

		return $this->change($newname);
	}

}