NeedleProject\FileIo\Factory\ContentFactory

Complexity

Total Complexity

5

Size/Duplication

Total Lines	53
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Test Coverage

Coverage

100%

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
B	create()	0	24	5

<?php
/**
 * This file is part of the NeedleProject\FileIo package.
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 */
namespace NeedleProject\FileIo\Factory;

use NeedleProject\FileIo\Content\Content;
use NeedleProject\FileIo\Content\ContentInterface;
use NeedleProject\FileIo\Content\JsonContent;
use NeedleProject\FileIo\Content\YamlContent;

/**
 * Class ContentFactory
 *
 * @package NeedleProject\FileIo\Factory
 * @author Adrian Tilita <adrian@tilita.ro>
 * @copyright 2017 Adrian Tilita
 * @license https://opensource.org/licenses/MIT MIT Licence
 */
class ContentFactory
{
    /**
     * @const string
     */
    const EXT_TXT = 'txt';

    /**
     * @const string
     */
    const EXT_JSON = 'json';

    /**
     * @const string
     */
    const EXT_YAML = 'yaml';

    /**
     * @const string
     */
    const EXT_YML = 'yml';

    /**
     * Create a ContentInterface based on the extension and string content
     * @param string $extension
     * @param string $content
     * @return \NeedleProject\FileIo\Content\ContentInterface

Should the return type not be null|Content?

     */
    public function create($extension, $content)
    {
        switch ($extension) {
            /**
             * Create JSON Content
             */
            case static::EXT_JSON:
                return new JsonContent($content);
                break;

break is not strictly necessary here and could be removed.

            /**
             * Create YAML Content
             */
            case static::EXT_YAML:
            case static::EXT_YML:
                return new YamlContent($content);
                break;

break is not strictly necessary here and could be removed.

            /**
             * Default TXT Content
             */
            case static::EXT_TXT:
            default:
                return new Content($content);
        }
    }
}