Balloon\Reader\Factory\FileReaderFactory

Complexity

Total Complexity

2

Size/Duplication

Total Lines	31
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	5	1
A	create()	0	4	1

<?php
namespace Balloon\Reader\Factory;

use Balloon\Reader\FileReader;
use Balloon\Reader\IFileReader;

/**
 * Class FileReaderFactory
 * @package Balloon\Reader\Factory
 * @author Raphaël Lefebvre <raphael@raphaellefebvre.be>
 */
class FileReaderFactory implements IFileReaderFactory
{
    /**
     * @var bool
     */
    private $useIncludePath;

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

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

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

     */
    public function __construct($useIncludePath = false, $context = null)
    {
        $this->useIncludePath = $useIncludePath;
        $this->context = $context;
    }

    /**
     * @param $filePath
     * @return IFileReader

Consider making the return type a bit more specific; maybe use FileReader.

     */
    public function create($filePath)
    {
        return new FileReader($filePath, $this->useIncludePath, $this->context);
    }
}