Ktomk\Pipelines\File\Definitions\Services

Complexity

Total Complexity

7

Size/Duplication

Total Lines	61
Duplicated Lines	0 %

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	1

4 Methods

Rating	Name	Duplication	Size	Complexity
A	parseServices()	0	7	3
A	count()	0	8	2
A	__construct()	0	5	1
A	parseNamedService()	0	3	1

<?php

/* this file is part of pipelines */

namespace Ktomk\Pipelines\File\Definitions;

use Countable;
use Ktomk\Pipelines\File\ParseException;

/**
 * Class Services
 *
 * @package Ktomk\Pipelines\File\Definitions
 */
class Services implements Countable
{
    /**
     * @var array
     */
    private $array;

    /**
     * @var array|Service
     */
    private $services = array();

    /**
     * Services constructor.
     *
     * @param array $array
     */
    public function __construct(array $array)
    {
        $this->parseServices($array);

        $this->array = $array;
    }

    /**
     * @return int
     */
    public function count()
    {
        $count = count($this->services);

It seems like $this->services can also be of type Ktomk\Pipelines\File\Definitions\Service; however, parameter $var of count() does only seem to accept Countable|array, maybe add an additional type check?

        // special docker service does not count
        isset($this->services['docker']) && $count--;

        return $count;
    }

    /**
     * @param array $array
     *
     * @return void
     */
    private function parseServices(array $array)
    {
        foreach ($array as $name => $service) {
            if (!is_string($name)) {
                throw new ParseException(sprintf('Invalid service definition name: %s', var_export($name, true)));
            }
            $this->services[$name] = $this->parseNamedService($name, $service);
        }
    }

    /**
     * @param string $name
     * @param array $service
     *
     * @return Service
     */
    private function parseNamedService($name, array $service)
    {
        return new Service($name, $service);
    }
}