PragmaRX\Random\Generators\ArrayGenerator

Complexity

Total Complexity

4

Size/Duplication

Total Lines	43
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	0

Test Coverage

Coverage

100%

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	count()	0	6	1
A	generateArray()	0	16	3

<?php

namespace PragmaRX\Random\Generators;

trait ArrayGenerator
{
    protected $array = false;

    protected $items = [];

    protected $count = 1;

    /**
     * Set the array items count.
     *
     * @param $count
     * @return $this
     */
    public function count($count)
    {
        $this->count = $count;

        return $this;
    }

    /**
     * Generate random array elements.
     *
     * @return array
     */
    protected function generateArray()
    {
        $result = [];

        $last = count($this->items)-1;

        for ($counter = 1; $counter <= $this->count; $counter++) {
            $result[] = $this->items[$this->generateRandomInt(0, $last)];

It seems like generateRandomInt() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }

        if ($this->count == 1) {
            return $result[0];
        }

        return $result;
    }
}