Equip\Command\AbstractCommand

Complexity

Total Complexity

7

Size/Duplication

Total Lines	55
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

4 Methods

Rating	Name	Duplication	Size	Complexity
A	options()	0	14	4
A	withOptions()	0	7	1
A	addOptions()	0	7	1
A	hasOption()	0	4	1

<?php

namespace Equip\Command;

abstract class AbstractCommand implements CommandInterface
{
    /**
     * @var array
     */
    private $options = [];

    /**
     * @inheritDoc
     */
    public function options()
    {
        $required = $this->requiredOptions();

        if ($required) {

The expression $required of type array is implicitly converted to a boolean; are you sure this is intended? If so, consider using ! empty($expr) instead to make it clear that you intend to check for an array without elements.

            foreach ($required as $key) {
                if (!isset($this->options[$key])) {
                    throw CommandException::missingOption($key);
                }
            }
        }

        return $this->options;
    }

    /**
     * @inheritDoc
     */
    public function withOptions(array $options)
    {
        $copy = clone $this;

Equals sign not aligned with surrounding assignments; expected 10 spaces but found 1 space

        $copy->options = $options;

        return $copy;
    }

    /**
     * @inheritDoc
     */
    public function addOptions(array $options)
    {
        $copy = clone $this;

Equals sign not aligned with surrounding assignments; expected 10 spaces but found 1 space

        $copy->options = array_replace($copy->options, $options);

        return $copy;
    }

    /**
     * @inheritDoc
     */
    public function hasOption($name)
    {
        return isset($this->options[$name]);
    }
}