refactor: random operation: use RandomIterableAggregate.

loophp · Apr 6, 2022 · 78e7a6f · 78e7a6f
1 parent 8afedbf
commit 78e7a6f
Showing 1 changed file with 14 additions and 8 deletions.
diff --git a/src/Operation/Random.php b/src/Operation/Random.php
@@ -11,12 +11,16 @@
 
 use Closure;
 use Generator;
+use loophp\iterators\LimitIterableAggregate;
+use loophp\iterators\RandomIterableAggregate;
 
 /**
  * @immutable
  *
  * @template TKey
  * @template T
+ *
+ * phpcs:disable Generic.Files.LineLength.TooLong
  */
 final class Random extends AbstractOperation
 {
@@ -35,14 +39,16 @@ static function (int $seed): Closure {
                      * @return Closure(iterable<TKey, T>): Generator<TKey, T>
                      */
                     static function (int $size) use ($seed): Closure {
-                        /** @var Closure(iterable<TKey, T>): Generator<TKey, T> $pipe */
-                        $pipe = (new Pipe())()(
-                            (new Shuffle())()($seed),
-                            (new Limit())()($size)(0)
-                        );
-
-                        // Point free style.
-                        return $pipe;
+                        return
+                            /**
+                             * @param iterable<TKey, T> $iterable
+                             *
+                             * @return Generator<TKey, T>
+                             */
+                            static function (iterable $iterable) use ($seed, $size): Generator {
+                                // Point free style.
+                                yield from new LimitIterableAggregate(new RandomIterableAggregate($iterable, $seed), 0, $size);
+                            };
                     };
             };
     }