Dynamic\Salsify\Traits\Yieldable

Complexity

Total Complexity

8

Size/Duplication

Total Lines	38
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	yieldSingle()	0	9	4
A	yieldKeyVal()	0	9	4

<?php

namespace Dynamic\Salsify\Traits;

/**
 * Trait Yieldable
 * @package Dynamic\Salsify\Traits
 */
trait Yieldable
{

    /**
     * @var string
     */
    public static $STOP_GENERATOR = 'stop';

    /**
     * @param array|iterable $list
     * @param callable|null $callback
     * @return \Generator
     */
    public function yieldSingle($list, $callback = null)
    {
        foreach ($list as $item) {
            $injected = (yield $item);

            if ($injected === static::$STOP_GENERATOR) break;
        }

        if (is_callable($callback)) $this->callback();

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

    }

    /**
     * @param array|iterable $list
     * @param callable|null $callback
     * @return \Generator
     */
    public function yieldKeyVal($list, $callback = null)
    {
        foreach ($list as $key => $val) {
            $injected = (yield $key => $val);

            if ($injected === static::$STOP_GENERATOR) break;
        }

        if (is_callable($callback)) $this->callback();
    }
}