tarsana / functional

try.php

Spacing +1 added lines, -2 removed lines

@@ -2,8 +2,7 @@
 
 function numberOfArgs($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->getNumberOfParameters();
 }
 

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) {