update hook handlers` arguments

Author: toplan

README.md

@@ -13,7 +13,7 @@ lightweight and powerful task load balancing for php
 # Install
 
 ```php
-    composer require 'toplan/task-balancer:~0.2.1'
+    composer require 'toplan/task-balancer:~0.2.2'
 ```
 
 # Usage
@@ -173,12 +173,12 @@ get data value of task instance.
 
 | Hook name | handler arguments | influence of the last handler`s return value |
 | --------- | :----------------: | :-----: |
-| beforeCreateDriver | $task, $preReturn, $index, $count | no effect |
-| afterCreateDriver | $task, $preReturn, $index, $count | no effect |
-| beforeRun | $task, $preReturn, $index, $count | if `false` will stop run task and return `false` |
-| beforeDriverRun | $task, $preReturn, $index, $count | no effect |
-| afterDriverRun | $task, $preReturn, $index, $count | no effect |
-| afterRun | $task, $taskResult, $preReturn, $index, $count | if not boolean will override result value |
+| beforeCreateDriver | $task, $preReturn, $index, $handlers | no effect |
+| afterCreateDriver | $task, $preReturn, $index, $handlers | no effect |
+| beforeRun | $task, $preReturn, $index, $handlers | if `false` will stop run task and return `false` |
+| beforeDriverRun | $task, $preReturn, $index, $handlers | no effect |
+| afterDriverRun | $task, $preReturn, $index, $handlers | no effect |
+| afterRun | $task, $taskResult, $preReturn, $index, $handlers | if not boolean will override result value |
 
 ###Use Hooks
 
@@ -201,24 +201,24 @@ get data value of task instance.
 
 ```php
 //example
-$task->beforeRun(function($task, $preReturn, $index, $count){
+$task->beforeRun(function($task, $preReturn, $index, $handlers){
     //what is $preReturn?
     $preReturn == null; //true
     //what is $index?
     $index == 0; //true
-    //what is $count?
-    echo $count; //2
+    //what is $handlers?
+    echo count($handlers); //2
     //do something..
     return 'beforeRun_1';
 }, false);
 
-$task->beforeRun(function($task, $preReturn, $index, $count){
+$task->beforeRun(function($task, $preReturn, $index, $handlers){
     //what is $preReturn?
     $preReturn == 'beforeRun_1'; //true
     //what is $index?
     $index == 1; //true
-    //what is $count?
-    echo $count; //2
+    //what is $handlers?
+    echo count($handlers); //2
     //do other something..
 }, false);
 ```

composer.json

@@ -2,7 +2,7 @@
   "name": "toplan/task-balancer",
   "description": "lightweight and powerful task load balancing for php (like the nginx load balancing)",
   "license": "MIT",
-  "version": "0.2.1",
+  "version": "0.2.2",
   "keywords": ["task", "balance", "load balancing", "balancer"],
   "authors": [
     {

src/TaskBalancer/Task.php

@@ -524,11 +524,10 @@ protected function callHookHandler($hookName, $data = null)
         if (array_key_exists($hookName, $this->handlers)) {
             $handlers = $this->handlers[$hookName] ?: [];
             $result = null;
-            $count = count($handlers);
             foreach ($handlers as $index => $handler) {
                 $handlerArgs = $data === null ?
-                               [$this, $result, $index, $count]:
-                               [$this, $data, $result, $index, $count];
+                               [$this, $result, $index, $handlers]:
+                               [$this, $data, $result, $index, $handlers];
                 $result = call_user_func_array($handler, $handlerArgs);
             }
             if ($result === null) {

test/demo2.php

@@ -16,36 +16,36 @@
                     print_r('run work! by '.$driver->name.'<br>');
                 });
 
-    $task->beforeRun(function($task, $preReturn, $index){
+    $task->beforeRun(function($task, $preReturn, $index, $handlers){
         print_r("before run ---$preReturn-----$index<br>");
         return 11;
     });
 
-    $task->beforeRun(function($task, $preReturn, $index){
+    $task->beforeRun(function($task, $preReturn, $index, $handlers){
         print_r("before run ---$preReturn-----$index<br>");
         return 22;
-    }, true);
+    }, false);
 
-    $task->beforeRun(function($task, $preReturn, $index){
+    $task->beforeRun(function($task, $preReturn, $index, $handlers){
         print_r("before run ---$preReturn-----$index<br>");
     });
 
-    $task->hook('beforeDriverRun', function($task, $preReturn, $index){
+    $task->hook('beforeDriverRun', function($task, $preReturn, $index, $handlers){
         print_r("before driver run ---$preReturn-----$index<br>");
         return [1];
     });
 
-    $task->hook('beforeDriverRun', function($task, $preReturn, $index){
+    $task->hook('beforeDriverRun', function($task, $preReturn, $index, $handlers){
         print_r("before driver run ---".implode(',', $preReturn)."-----$index<br>");
         return [1,2];
     }, true);
 
-    $task->hook('beforeDriverRun', function($task, $preReturn, $index){
+    $task->hook('beforeDriverRun', function($task, $preReturn, $index, $handlers){
         print_r("before driver run ---".implode(',', $preReturn)."-----$index<br>");
         return [1,2,3];
     });
 
-    $task->afterRun(function($task, $results){
+    $task->afterRun(function($task, $results, $preReturn, $index, $handlers){
         print_r("after run --------!<br>");
     });
 });