tarsana / functional

src/functions.php

Doc Comments +1 added lines, -3 removed lines

@@ -80,7 +80,7 @@ 
  * @param  callable $fn
  * @param  int $argsCount
  * @param  array  $boundArgs
- * @return callable
+ * @return \Closure
  */
 function _curried_function($fn, $argsCount, $boundArgs = []) {
     return function() use($fn, $argsCount, $boundArgs) {
@@ -152,8 +152,6 @@ 
  * ```
  *
  * @signature (*... -> a) -> [*] -> a
- * @param  callable $fn
- * @param  array    $args
  * @return mixed
  */
 function apply() {

Spacing +4 added lines, -5 removed lines

@@ -28,8 +28,7 @@ 
  */
 function _number_of_args($fn) {
     $reflector = is_array($fn) ?
-        new \ReflectionMethod($fn[0], $fn[1]) :
-        new \ReflectionFunction($fn);
+        new \ReflectionMethod($fn[0], $fn[1]) : new \ReflectionFunction($fn);
     return $reflector->getNumberOfRequiredParameters();
 }
 
@@ -59,7 +58,7 @@ 
     $addArgument = function($currentBoundArgs, $arg) use($fnArgsCount) {
         $currentBoundArgsCount = count($currentBoundArgs);
         $placeholderPosition = 0;
-        while($placeholderPosition < $currentBoundArgsCount && !_is_placeholder($currentBoundArgs[$placeholderPosition]))
+        while ($placeholderPosition < $currentBoundArgsCount && !_is_placeholder($currentBoundArgs[$placeholderPosition]))
             $placeholderPosition ++;
         if ($currentBoundArgsCount < $fnArgsCount || $placeholderPosition == $currentBoundArgsCount) {
             $currentBoundArgs[] = $arg;
@@ -179,9 +178,9 @@ 
  */
 function pipe() {
     $fns = func_get_args();
-    if(count($fns) < 1)
+    if (count($fns) < 1)
         throw new InvalidArgument("pipe() requires at least one argument");
-    return curry(function () use ($fns) {
+    return curry(function() use ($fns) {
         $result = _apply(array_shift($fns), func_get_args());
         foreach ($fns as $fn) {
             $result = $fn($result);

Braces +9 added lines, -6 removed lines

@@ -59,8 +59,9 @@ 
     $addArgument = function($currentBoundArgs, $arg) use($fnArgsCount) {
         $currentBoundArgsCount = count($currentBoundArgs);
         $placeholderPosition = 0;
-        while($placeholderPosition < $currentBoundArgsCount && !_is_placeholder($currentBoundArgs[$placeholderPosition]))
-            $placeholderPosition ++;
+        while($placeholderPosition < $currentBoundArgsCount && !_is_placeholder($currentBoundArgs[$placeholderPosition])) {
+                    $placeholderPosition ++;
+        }
         if ($currentBoundArgsCount < $fnArgsCount || $placeholderPosition == $currentBoundArgsCount) {
             $currentBoundArgs[] = $arg;
         } else { // There is a placeholder and number of bound args > $fnArgsCount
@@ -87,8 +88,9 @@ 
         $boundArgs = _merge_args($argsCount, $boundArgs, func_get_args());
         $numberOfPlaceholders = count(array_filter($boundArgs, _f('_is_placeholder')));
         $numberOfGivenArgs = count($boundArgs) - $numberOfPlaceholders;
-        if ($numberOfGivenArgs >= $argsCount)
-            return call_user_func_array($fn, $boundArgs);
+        if ($numberOfGivenArgs >= $argsCount) {
+                    return call_user_func_array($fn, $boundArgs);
+        }
         return _curried_function($fn, $argsCount, $boundArgs);
     };
 }
@@ -179,8 +181,9 @@ 
  */
 function pipe() {
     $fns = func_get_args();
-    if(count($fns) < 1)
-        throw new InvalidArgument("pipe() requires at least one argument");
+    if(count($fns) < 1) {
+            throw new InvalidArgument("pipe() requires at least one argument");
+    }
     return curry(function () use ($fns) {
         $result = _apply(array_shift($fns), func_get_args());
         foreach ($fns as $fn) {

src/list.php

Spacing +16 added lines, -17 removed lines

@@ -56,7 +56,7 @@ 
  * @return array
  */
 function filter() {
-    $filter = function($fn, $list){
+    $filter = function($fn, $list) {
         return array_values(array_filter($list, $fn));
     };
     return apply(curry($filter), func_get_args());
@@ -78,7 +78,7 @@ 
  * @return array
  */
 function reduce() {
-    $reduce = function($fn, $initial, $list){
+    $reduce = function($fn, $initial, $list) {
         return array_reduce($list, $fn, $initial);
     };
     return apply(curry($reduce), func_get_args());
@@ -104,7 +104,7 @@ 
  * @return array
  */
 function each() {
-    $each = function($fn, $list){
+    $each = function($fn, $list) {
         foreach ($list as $item) {
             apply($fn, [$item]);
         }
@@ -128,7 +128,7 @@ 
  * @return mixed
  */
 function head($list) {
-    if(is_string($list))
+    if (is_string($list))
         return substr($list, 0, 1);
     return (count($list) > 0)
         ? $list[0]
@@ -150,7 +150,7 @@ 
  * @return mixed
  */
 function last($list) {
-    if(is_string($list))
+    if (is_string($list))
         return substr($list, -1);
     return (count($list) > 0)
         ? $list[count($list) - 1]
@@ -173,7 +173,7 @@ 
  * @return array
  */
 function init($list) {
-    if(is_string($list))
+    if (is_string($list))
         return (strlen($list) > 1)
             ? substr($list, 0, strlen($list) - 1)
             : '';
@@ -198,7 +198,7 @@ 
  * @return array
  */
 function tail($list) {
-    if(is_string($list))
+    if (is_string($list))
         return (strlen($list) > 1)
             ? substr($list, 1)
             : '';
@@ -322,8 +322,7 @@ 
  */
 function concatAll() {
     $concatAll = function($lists) {
-        return length($lists) == 0 ? [] :
-            reduce(concat(), head($lists), tail($lists));
+        return length($lists) == 0 ? [] : reduce(concat(), head($lists), tail($lists));
     };
     return apply(curry($concatAll), func_get_args());
 }
@@ -400,7 +399,7 @@ 
  * @return array
  */
 function append() {
-    $append = function ($item, $list) {
+    $append = function($item, $list) {
         return concat($list, [$item]);
     };
     return apply(curry($append), func_get_args());
@@ -447,7 +446,7 @@ 
         $length = length($list);
         if ($count > $length || $count < -$length)
             return [];
-        if(is_string($list)) {
+        if (is_string($list)) {
             return ($count >= 0)
                 ? substr($list, 0, $count)
                 : substr($list, $count);
@@ -703,9 +702,9 @@ 
  */
 function slices() {
     $slices = function($size, $list) {
-        if(empty($list))
+        if (empty($list))
             return is_string($list) ? '' : [];
-        if(length($list) <= $size)
+        if (length($list) <= $size)
             return [$list];
         return prepend(take($size, $list), slices($size, remove($size, $list)));
     };
@@ -778,7 +777,7 @@ 
 function findLastIndex() {
     $findLastIndex = function($predicate, $list) {
         foreach (reverse(toPairs($list)) as $pair) {
-            if($predicate($pair[1]))
+            if ($predicate($pair[1]))
                 return $pair[0];
         }
         return null;
@@ -851,7 +850,7 @@ 
     $indexOf = function($item, $list) {
         if (is_string($list)) {
             $index = strpos($list, $item);
-            return (-1 == $index)
+            return (- 1 == $index)
                 ? null
                 : $index;
         }
@@ -880,7 +879,7 @@ 
     $lastIndexOf = function($item, $list) {
         if (is_string($list)) {
             $index = strrpos($list, $item);
-            return (-1 == $index)
+            return (- 1 == $index)
                 ? null
                 : $index;
         }
@@ -968,7 +967,7 @@ 
     $groupBy = function($fn, $list) {
         return reduce(function($result, $item) use($fn) {
             $index = $fn($item);
-            if (! isset($result[$index]))
+            if (!isset($result[$index]))
                 $result[$index] = [];
             $result[$index][] = $item;
             return $result;

Braces +39 added lines, -26 removed lines

@@ -128,8 +128,9 @@ 
  * @return mixed
  */
 function head($list) {
-    if(is_string($list))
-        return substr($list, 0, 1);
+    if(is_string($list)) {
+            return substr($list, 0, 1);
+    }
     return (count($list) > 0)
         ? $list[0]
         : null;
@@ -150,8 +151,9 @@ 
  * @return mixed
  */
 function last($list) {
-    if(is_string($list))
-        return substr($list, -1);
+    if(is_string($list)) {
+            return substr($list, -1);
+    }
     return (count($list) > 0)
         ? $list[count($list) - 1]
         : null;
@@ -173,10 +175,11 @@ 
  * @return array
  */
 function init($list) {
-    if(is_string($list))
-        return (strlen($list) > 1)
+    if(is_string($list)) {
+            return (strlen($list) > 1)
             ? substr($list, 0, strlen($list) - 1)
             : '';
+    }
     return (count($list) > 1)
         ? array_slice($list, 0, count($list) - 1)
         : [];
@@ -198,10 +201,11 @@ 
  * @return array
  */
 function tail($list) {
-    if(is_string($list))
-        return (strlen($list) > 1)
+    if(is_string($list)) {
+            return (strlen($list) > 1)
             ? substr($list, 1)
             : '';
+    }
     return (count($list) > 1)
         ? array_slice($list, 1)
         : [];
@@ -302,8 +306,9 @@ 
         $t1 = toString($list1);
         $t2 = toString($list2);
         // echo "Concating {$t1} and {$t2}", PHP_EOL;
-        if (is_string($list1) && is_string($list2))
-            return $list1 . $list2;
+        if (is_string($list1) && is_string($list2)) {
+                    return $list1 . $list2;
+        }
         return array_merge($list1, $list2);
     };
     return apply(curry($concat), func_get_args());
@@ -377,8 +382,9 @@ 
 function insertAll() {
     $insertAll = function($position, $items, $list) {
         $length = length($list);
-        if ($position < 0)
-            $position = $length + $position;
+        if ($position < 0) {
+                    $position = $length + $position;
+        }
         return ($position >= $length) 
             ? concat($list, $items) 
             : concatAll([take($position, $list), $items, remove($position, $list)]);
@@ -445,8 +451,9 @@ 
 function take() {
     $take = function($count, $list) {
         $length = length($list);
-        if ($count > $length || $count < -$length)
-            return [];
+        if ($count > $length || $count < -$length) {
+                    return [];
+        }
         if(is_string($list)) {
             return ($count >= 0)
                 ? substr($list, 0, $count)
@@ -571,8 +578,9 @@ 
 function remove() {
     $remove = function($count, $list) {
         $length = length($list);
-        if ($count > $length || $count < -$length)
-            return [];
+        if ($count > $length || $count < -$length) {
+                    return [];
+        }
         return ($count > 0)
             ? take($count - $length, $list)
             : take($count + $length, $list);
@@ -703,10 +711,12 @@ 
  */
 function slices() {
     $slices = function($size, $list) {
-        if(empty($list))
-            return is_string($list) ? '' : [];
-        if(length($list) <= $size)
-            return [$list];
+        if(empty($list)) {
+                    return is_string($list) ? '' : [];
+        }
+        if(length($list) <= $size) {
+                    return [$list];
+        }
         return prepend(take($size, $list), slices($size, remove($size, $list)));
     };
     return apply(curry($slices), func_get_args());
@@ -752,8 +762,9 @@ 
 function findIndex() {
     $findIndex = function($predicate, $list) {
         foreach ($list as $key => $value) {
-            if ($predicate($value))
-                return $key;
+            if ($predicate($value)) {
+                            return $key;
+            }
         }
         return null;
     };
@@ -778,8 +789,9 @@ 
 function findLastIndex() {
     $findLastIndex = function($predicate, $list) {
         foreach (reverse(toPairs($list)) as $pair) {
-            if($predicate($pair[1]))
-                return $pair[0];
+            if($predicate($pair[1])) {
+                            return $pair[0];
+            }
         }
         return null;
     };
@@ -968,8 +980,9 @@ 
     $groupBy = function($fn, $list) {
         return reduce(function($result, $item) use($fn) {
             $index = $fn($item);
-            if (! isset($result[$index]))
-                $result[$index] = [];
+            if (! isset($result[$index])) {
+                            $result[$index] = [];
+            }
             $result[$index][] = $item;
             return $result;
         }, [], $list);

src/Stream.php

Spacing +20 added lines, -20 removed lines

@@ -81,7 +81,7 @@ 
      * @param  mixed $data
      * @return Stream
      */
-    public static function of ($data)
+    public static function of($data)
     {
         $data = func_get_args();
         if (count($data) == 1)
@@ -96,7 +96,7 @@ 
      * @param  array $operations
      * @return Stream
      */
-    protected static function with ($data, $operations, $type)
+    protected static function with($data, $operations, $type)
     {
         return new Stream($data, $operations, $type);
     }
@@ -107,7 +107,7 @@ 
      * @param  array $operations
      * @return array|Error
      */
-    protected static function optimize ($operations)
+    protected static function optimize($operations)
     {
         // TODO: ...
         return $operations;
@@ -120,11 +120,11 @@ 
      * @param  mixed $data
      * @return mixed|Error
      */
-    protected static function execute ($operations, $data)
+    protected static function execute($operations, $data)
     {
         if (length($operations) == 0)
             return $data;
-        $operations = apply(_f('pipe'), map(function($operation){
+        $operations = apply(_f('pipe'), map(function($operation) {
             if ($operation['name'] == 'apply') {
                 return $operation['args'];
             }
@@ -142,7 +142,7 @@ 
      * @param  string $type
      * @return bool
      */
-    protected static function canApply ($operation, $type)
+    protected static function canApply($operation, $type)
     {
         return isset(Stream::$transformations[$operation]) && (
             $type == 'Unknown' ||
@@ -158,7 +158,7 @@ 
      * @param  string $type
      * @return bool
      */
-    protected static function returnOf ($operation, $type)
+    protected static function returnOf($operation, $type)
     {
         return isset(Stream::$transformations[$operation][$type])
             ? Stream::$transformations[$operation][$type]
@@ -173,12 +173,12 @@ 
      * @param  Stream $stream
      * @return Stream
      */
-    protected static function apply ($operation, $args, $stream)
+    protected static function apply($operation, $args, $stream)
     {
         if ($stream->type == 'Error') {
             return Stream::of(Error::of("Could not apply {$operation} to {$stream->type}", $stream->data));
         }
-        if (! Stream::canApply($operation, $stream->type)) {
+        if (!Stream::canApply($operation, $stream->type)) {
             $data = toString($stream->data);
             return Stream::of(Error::of("Could not apply {$operation} to {$stream->type}({$data})"));
         }
@@ -194,7 +194,7 @@ 
      *
      * @param mixed $data
      */
-    protected function __construct ($data, $operations, $type)
+    protected function __construct($data, $operations, $type)
     {
         $this->data = $data;
         $this->type = $type;
@@ -232,7 +232,7 @@ 
      * @signature Stream(a) -> a
      * @return mixed
      */
-    public function get ()
+    public function get()
     {
         if ($this->type == 'Error')
             return $this->data;
@@ -251,7 +251,7 @@ 
      * @param  callable $fn
      * @return Stream
      */
-    public function map (callable $fn)
+    public function map(callable $fn)
     {
         return Stream::apply('map', [$fn], $this);
     }
@@ -268,7 +268,7 @@ 
      * @param  callable $predicate
      * @return Stream
      */
-    public function filter (callable $predicate)
+    public function filter(callable $predicate)
     {
         return Stream::apply('filter', [$predicate], $this);
     }
@@ -286,7 +286,7 @@ 
      * @param mixed $initial
      * @return Stream
      */
-    public function reduce ($fn, $initial)
+    public function reduce($fn, $initial)
     {
         return Stream::apply('reduce', [$fn, $initial], $this);
     }
@@ -305,7 +305,7 @@ 
      * @param callable $fn
      * @return Stream
      */
-    public function chain ($fn)
+    public function chain($fn)
     {
         return Stream::apply('chain', [$fn], $this);
     }
@@ -325,7 +325,7 @@ 
      * @signature Stream(String) -> Number
      * @return Stream
      */
-    public function length ()
+    public function length()
     {
         return Stream::apply('length', [], $this);
     }
@@ -346,7 +346,7 @@ 
      * @param int $number
      * @return Stream
      */
-    public function take ($number)
+    public function take($number)
     {
         return Stream::apply('take', [$number], $this);
     }
@@ -367,7 +367,7 @@ 
      * @param callable $fn
      * @return Stream
      */
-    public function then ($fn)
+    public function then($fn)
     {
         $result = $this;
         foreach (func_get_args() as $fn) {
@@ -417,7 +417,7 @@ 
      * @param  mixed|null $args...
      * @return Stream
      */
-    public function call ($method)
+    public function call($method)
     {
         $args = tail(func_get_args());
         return Stream::apply('apply', function($data) use($method, $args) {
@@ -471,7 +471,7 @@ 
      * @param  mixed|null $args...
      * @return Stream
      */
-    public function run ($method)
+    public function run($method)
     {
         $args = tail(func_get_args());
         return Stream::apply('apply', function($data) use($method, $args) {