Cocur\Chain\Link\Chunk

Complexity

Total Complexity

5

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Test Coverage

Coverage

0%

Importance

Changes	2
Bugs	1	Features	0

1 Method

Rating	Name	Duplication	Size	Complexity
A	chunk()	0	15	5

<?php

namespace Cocur\Chain\Link;

/**
 * Chunk.
 *
 * @author    Nicolas Reynis
 */
trait Chunk
{
    /**
     * @param int   $size
     * @param array $options options, including:
     *                       bool `preserveKeys` to prevent reindexing, default to false
     *                       bool `decorate` to generate chains instead of arrays, default tu true
     *
     * @return self
     */
    public function chunk(int $size, array $options = []): self
    {
        if (!empty($options['preserveKeys'])) {
            $this->array = array_chunk($this->array, $size, $options['preserveKeys']);

The property array does not exist. Although not strictly required by PHP, it is generally a best practice to declare properties explicitly.

        } else {
            $this->array = array_chunk($this->array, $size);
        }

        if (empty($options['decorate']) || true === $options['decorate']) {
            foreach ($this->array as $index => $chunk) {
                $this->array[$index] = new static($chunk);

The call to Cocur\Chain\Link\Chunk::__construct() has too many arguments starting with $chunk.

            }
        }

        return $this;
    }
}