Gorynych\Http\Formatter\FormatterFactory

Complexity

Total Complexity

5

Size/Duplication

Total Lines	39
Duplicated Lines	0 %

Test Coverage

Coverage

92.31%

Importance

Changes	2
Bugs	0	Features	0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	parseFormatters()	0	3	1
A	__construct()	0	3	1
A	create()	0	12	3

<?php
/**
 * Copyright (c) 2020.
 * @author Paweł Antosiak <contact@pawelantosiak.com>
 */

declare(strict_types=1);

namespace Gorynych\Http\Formatter;

use Gorynych\Http\Exception\NotAcceptableHttpException;
use Symfony\Component\Config\FileLocatorInterface;
use Symfony\Component\Yaml\Yaml;

final class FormatterFactory
{
    /** @var string[] */
    private array $formatters;

    public function __construct(FileLocatorInterface $fileLocator)
    {
        $this->formatters = $this->parseFormatters($fileLocator->locate('formatters.yaml'));

It seems like $fileLocator->locate('formatters.yaml') can also be of type array; however, parameter $file of Gorynych\Http\Formatter\FormatterFactory::parseFormatters() does only seem to accept string, maybe add an additional type check?

    }

    /**
     * Creates formatter for first acceptable format
     *
     * @param string ...$acceptableFormats
     * @return FormatterInterface
     * @throws NotAcceptableHttpException if proper formatter does not exist
     */
    public function create(string ...$acceptableFormats): FormatterInterface
    {
        foreach ($acceptableFormats as $format) {
            if (true === array_key_exists($format, $this->formatters)) {
                return new $this->formatters[$format];
            }
        }

        throw new NotAcceptableHttpException(
            sprintf(
                'Unable to provide an acceptable representation. Available alternatives: %s.',
                implode(', ', array_keys($this->formatters))
            )
        );
    }

    /**
     * @return string[]
     */
    private function parseFormatters(string $file): array
    {
        return Yaml::parseFile($file)['formatters'];
    }
}