rtens\domin\delivery\web\renderers\charting\coloring\MixedColorProvider

Complexity

Total Complexity

3

Size/Duplication

Total Lines	28
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	13	1
A	next()	0	6	2

<?php
namespace rtens\domin\delivery\web\renderers\charting\coloring;

use rtens\domin\parameters\Color;

class MixedColorProvider implements ColorProvider {

    private $colors;

    public function __construct() {
        $this->colors = [
            Color::BLUE(),
            Color::GREEN(),
            Color::RED(),
            Color::PURPLE(),
            Color::ORANGE(),
            Color::BROWN(),
            Color::PINK(),
            Color::YELLOW(),
            Color::GRAY()
        ];
    }

    /**
     * @return Color
     */
    public function next() {
        if (!$this->colors) {

The expression $this->colors 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.

            return Color::RANDOM();
        }
        return array_shift($this->colors);
    }
}