Balloon\Reader\FileReader

Complexity

Total Complexity

17

Size/Duplication

Total Lines	157
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes	1
Bugs	0	Features	0

13 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	6	1
A	read()	0	7	2
A	write()	0	5	1
A	isExistingFile()	0	7	2
A	isExistingDir()	0	7	2
A	getFilePath()	0	4	1
A	getDir()	0	4	1
A	useIncludePath()	0	4	1
A	getContext()	0	4	1
A	setFilePath()	0	4	1
A	setUseIncludePath()	0	4	1
A	setContext()	0	4	1
A	createDir()	0	7	2

<?php
namespace Balloon\Reader;

/**
 * Class FileReader
 * @package Balloon\Reader
 * @author Raphaël Lefebvre <raphael@raphaellefebvre.be>
 */
class FileReader implements IFileReader
{
    /**
     * @var string
     */
    private $filePath;

    /**
     * @var bool
     */
    private $useIncludePath;

    /**
     * @var resource
     */
    private $context;

    /**
     * @var bool
     */
    private $exists = false;

    /**
     * @param string $filePath
     * @param bool $useIncludePath
     * @param resource $context

Should the type for parameter $context not be resource|null?

     */
    public function __construct($filePath, $useIncludePath = false, $context = null)
    {
        $this->setFilePath($filePath);
        $this->setUseIncludePath($useIncludePath);
        $this->setContext($context);

It seems like $context defined by parameter $context on line 36 can also be of type null; however, Balloon\Reader\FileReader::setContext() does only seem to accept resource, maybe add an additional type check?

    }

    /**
     * @return string
     */
    public function read()
    {
        if(!$this->isExistingFile()){
            return '';
        }
        return file_get_contents($this->getFilePath(), $this->useIncludePath(), $this->getContext());
    }

    /**
     * @param string $data
     * @param int $mode
     * @return int
     */
    public function write($data, $mode = 0)
    {
        $this->createDir();
        return file_put_contents($this->getFilePath(), $data, $mode, $this->getContext());
    }

    /**
     * @return bool
     */
    public function isExistingFile()
    {
        if (!$this->exists) {
            $this->exists = file_exists($this->getFilePath());
        }
        return $this->exists;
    }

    /**
     * @return bool
     */
    public function isExistingDir()
    {
        if (!$this->exists) {
            $this->exists = is_dir($this->getDir());
        }
        return $this->exists;
    }

    /**
     * Getter of $filePath
     *
     * @return string
     */
    public function getFilePath()
    {
        return $this->filePath;
    }

    /**
     * @return string
     */
    public function getDir()
    {
        return dirname($this->getFilePath());
    }

    /**
     * Getter of $useIncludePath
     *
     * @return boolean
     */
    public function useIncludePath()
    {
        return $this->useIncludePath;
    }

    /**
     * Getter of $context
     *
     * @return resource
     */
    public function getContext()
    {
        return $this->context;
    }

    /**
     * Setter of $filePath
     *
     * @param string $filePath
     */
    private function setFilePath($filePath)
    {
        $this->filePath = (string)$filePath;
    }

    /**
     * Setter of $useIncludePath
     *
     * @param boolean $useIncludePath
     */
    private function setUseIncludePath($useIncludePath)
    {
        $this->useIncludePath = (boolean)$useIncludePath;
    }

    /**
     * Setter of $context
     *
     * @param resource $context
     */
    private function setContext($context)
    {
        $this->context = $context;
    }

    /**
     *
     */
    private function createDir()
    {
        if (!$this->isExistingDir()) {
            mkdir($this->getDir(), 0777, true);
            $this->exists = true;
        }
    }
}