Graze\DataFile\Format\Formatter\FormatterFactory

Complexity

Total Complexity

5

Size/Duplication

Total Lines	38
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
B	getFormatter()	0	30	5

<?php
/**
 * This file is part of graze/data-file
 *
 * Copyright (c) 2016 Nature Delivered Ltd. <https://www.graze.com>
 *
 * For the full copyright and license information, please view the LICENSE
 * file that was distributed with this source code.
 *
 * @license https://github.com/graze/data-file/blob/master/LICENSE.md
 * @link    https://github.com/graze/data-file
 */

namespace Graze\DataFile\Format\Formatter;

use Graze\DataFile\Format\CsvFormatInterface;
use Graze\DataFile\Format\FormatInterface;
use Graze\DataFile\Format\JsonFormatInterface;
use InvalidArgumentException;

class FormatterFactory implements FormatterFactoryInterface
{
    /**
     * @param FormatInterface $format
     *
     * @return FormatterInterface

Should the return type not be CsvFormatter|null|JsonFormatter?

     */
    public function getFormatter(FormatInterface $format)
    {
        switch ($format->getType()) {
            case 'csv':
                if ($format instanceof CsvFormatInterface) {
                    return new CsvFormatter($format);
                } else {
                    throw new InvalidArgumentException(
                        "Format indicates it is csv but does not implement CsvFormatInterface"
                    );
                }

                // this will never be called but phpcf doesn't understand the concept of exceptions
                break;

// this will never be called but phpcf doesn't understand the concept of exceptions break; does not seem to be reachable.

            case 'json':
                if ($format instanceof JsonFormatInterface) {
                    return new JsonFormatter($format);
                } else {
                    throw new InvalidArgumentexception(
                        "Format indicates it is json but does not implement JsonFormatInterface"
                    );
                }

                // this will never be called but phpcf doesn't understand the concept of exceptions
                break;

// this will never be called but phpcf doesn't understand the concept of exceptions break; does not seem to be reachable.

            default:
                throw new InvalidArgumentException("Supplied format: {$format->getType()} is unknown");
        }
    }
}