Collections

Complexity

Total Complexity

2

Size/Duplication

Total Lines	15
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	1

1 Method

Rating	Name	Duplication	Size	Complexity
A	each()	0	8	2

<?php

For compatibility and reusability of your code, PSR1 recommends that a file should introduce either new symbols (like classes, functions, etc.) or have side-effects (like outputting something, or including other files), but not both at the same time. The first symbol is defined on line 9 and the first side effect is on line 2.

require_once dirname(__DIR__) . '/vendor/autoload.php';

/**
 * Class Item
 *
 * A simple item.
 */
class Item

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    public $position;
}

/**
 * Class ItemCollection
 *
 * A simple collection of items.
 */
class ItemCollection extends \RQuadling\TypedArray\TypedArray

PSR1 recommends that each class should be in its own file to aid autoloaders.

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    const ARRAY_TYPE = 'Item';
}

/**
 * Class Collections
 *
 * An example abstract collections class that implements an each() method.
 */
abstract class Collections extends \RQuadling\TypedArray\TypedArray

PSR1 recommends that each class should be in its own file to aid autoloaders.

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    /**
     * @param callable $callable
     * @return $this
     */
    public function each(callable $callable)
    {
        foreach ($this as $key => $value) {
            $callable($value, $key);
        }

        return $this;
    }
}

/**
 * Class ItemCollectionEx
 *
 * A extended collection of items.
 */
class ItemCollectionEx extends Collections

PSR1 recommends that each class should be in its own file to aid autoloaders.

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    const ARRAY_TYPE = 'Item';
}

/**
 * Create a new item collection and add a second item to it.
 */
$items = new ItemCollection([new Item]);
$items[] = new Item;

/**
 * Create an extended collection and add a second item to it.
 */
$itemsEx = new ItemCollectionEx([new Item]);
$itemsEx[] = new Item;

/**
 * Use the each() method to set the position property in the item.
 */
$itemsEx->each(function(Item $item, $key){
    $item->position = $key;
});

/**
 * Clone the extended items collection.
 */
$itemsEx2 = clone $itemsEx;

/**
 * Use the each() method to make further changes to the position property of
 * the cloned collection.
 */
$itemsEx2->each(function(Item $item, $key){
    $item->position += $key;
});

/**
 * Make a new simple collection using the same items as the cloned collection.
 */
$items2 = new ItemCollection( $itemsEx2);

/**
 * Show the results of all of this work.
 *
 * Pay special attention to the object #'s.
 */

/**
 * Firstly the simple collection #2[#3, #4]
 */
var_dump($items);

var_dump($items); looks like debug code. Are you sure you do not want to remove it? This might expose sensitive data.

/**
 * Secondly the extended collection #5[#6, #7]
 */
var_dump($itemsEx);

/**
 * Thirdly, the cloned extended collection #8[#11, #12]
 */
var_dump($itemsEx2);

/**
 * Finally, the copied simple collection #9[#11, #12]
 *
 * NOTE: These items are the same ones as in $itemsEx2.
 */
var_dump($items2);