nozavroni / collections

src/Collection/LList.php

Doc Comments +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
     /**
      * LList constructor.
      *
-     * @param array|Traversable $data The list constructor
+     * @param string[] $data The list constructor
      */
     public function __construct($data)
     {

Unused Use Statements +5 added lines, -8 removed lines

@@ -9,19 +9,16 @@
  */
 namespace Noz\Collection;
 
-use InvalidArgumentException;
-
 use Countable;
-use Traversable;
-use SplDoublyLinkedList;
-
-use Noz\Contracts\Structure\Listable;
-use Noz\Contracts\Immutable;
+use InvalidArgumentException;
 use Noz\Contracts\Arrayable;
+use Noz\Contracts\Immutable;
 use Noz\Contracts\Invokable;
-
+use Noz\Contracts\Structure\Listable;
 use Noz\Traits\IsArrayable;
 use Noz\Traits\IsImmutable;
+use SplDoublyLinkedList;
+use Traversable;
 
 use function
     Noz\to_array,

Spacing +1 added lines, -1 removed lines

@@ -92,7 +92,7 @@
     public function __invoke()
     {
         $list = new LList([
-            'abc','def','ghi','jkl','mno'
+            'abc', 'def', 'ghi', 'jkl', 'mno'
         ]);
         $list(); // equivalent to pop()
         $list($val); // equivalent to push()

src/Collection/Stack.php

Unused Use Statements +2 added lines, -8 removed lines

@@ -9,17 +9,11 @@
  */
 namespace Noz\Collection;
 
-use BadMethodCallException;
-
 use Countable;
-use Traversable;
-use SplFixedArray;
-
-use Noz\Contracts\Structure\Listable;
-use Noz\Contracts\Immutable;
 use Noz\Contracts\Arrayable;
+use Noz\Contracts\Immutable;
 use Noz\Contracts\Invokable;
-
+use Noz\Contracts\Structure\Listable;
 use Noz\Traits\IsImmutable;
 
 use function Noz\to_array;

src/Collection/Sequence.php

Indentation +6 added lines, -10 removed lines

@@ -83,16 +83,12 @@ 
 
     /**
      * Invoke sequence.
-
      * A sequence is invokable as if it were a function. This allows some pretty useful functionality such as negative
      * indexing, sub-sequence selection, etc. Basically, any way you invoke a sequence, you're going to get back either
      * a single value from the sequence or a subset of it.
-
      * @internal param mixed $funk Either a numerical offset (positive or negative), a range string (start:end), or a
      * callback to be used as a filter.
-
      * @return mixed
-
      * @todo Put all the slice logic into a helper function or several
      */
     public function __invoke()
@@ -360,12 +356,12 @@ 
      *
      * @return Sequence
      */
-     public function prepend($item)
-     {
-         $arr = $this->getData();
-         array_unshift($arr, $item);
-         return new static($arr);
-     }
+        public function prepend($item)
+        {
+            $arr = $this->getData();
+            array_unshift($arr, $item);
+            return new static($arr);
+        }
 
     /**
      * Append item to collection.

Spacing +2 added lines, -2 removed lines

@@ -103,7 +103,7 @@ 
             $count = $this->count();
             if (count($args)) {
                 // if there are more args...
-                $length =  array_shift($args);
+                $length = array_shift($args);
             }
             if (Str::contains($offset, static::SLICE_DELIM)) {
                 list($start, $length) = get_range_start_end($offset, $count);
@@ -413,7 +413,7 @@ 
     public function isEmpty(callable $funk = null)
     {
         if (!is_null($funk)) {
-            return $this->fold(function ($carry, $val) use ($funk) {
+            return $this->fold(function($carry, $val) use ($funk) {
                 return $carry && $funk($val);
             }, true);
         }