__destruct()和__call()创建无限循环

时间:2015-08-22 17:28:26

标签: php oop methods closures destructor

我简化了我的代码,但我正在做的是这样的:

class App{

    protected $apps = [];

    public function __construct($name, $dependencies){
        $this->name = $name;

        $apps = [];
        foreach($dependencies as $dependName){
            $apps[$name] = $dependName($this); // returns an instance of App
        }
        $this->apps = $apps;
    }

    public function __destruct(){
        foreach($this->apps as $dep){
            $result = $dep->cleanup($this);
        }
    }

    public function __call($name, $arguments){
        if(is_callable([$this, $name])){
            return call_user_func_array([$this, $name], $arguments);
        }
    }
}


function PredefinedApp(){
    $app = new App('PredefinedApp', []);

    $app->cleanup = function($parent){
        // Do some stuff
    };
    return $app;
}

然后我创建一个这样的应用程序:

$app = new App('App1', ['PredefinedApp']);

它创建一个App实例,然后数组中的项创建任何已定义的新应用实例,并将它们放入内部应用程序数组中。

当我在主应用程序上执行析构函数时,它应该在所有子应用程序上调用cleanup()。但正在发生的事情是,它正在执行无限循环,我不确定为什么。

我注意到如果我发表评论call_user_func_array,那么__call()只会被调用一次,但它不会执行实际的closure

我也注意到,如果我在var_dump()中执行__call(),它会无休止地转储。如果我在var_dump()中执行了cleanup(),则会收到http 502错误。

2 个答案:

答案 0 :(得分:4)

因此,让我们浏览代码,看看这里发生了什么以及为什么:

01|    class App{ 
02|
03|        protected $apps = [];
04|
05|        public function __construct($name, $dependencies){
06|            $this->name = $name;
07|
08|            $apps = [];
09|            foreach($dependencies as $dependName){
10|                $apps[$name] = $dependName($this);
11|            }
12|            $this->apps = $apps;
13|        }
14|
15|        public function __destruct(){
16|            foreach($this->apps as $dep){
17|                $result = $dep->cleanup($this);
18|            }
19|        }
20|
21|        public function __call($name, $arguments){
22|            if(is_callable([$this, $name])){
23|                return call_user_func_array([$this, $name], $arguments);
24|            }
25|        }
26|    }
27|	
28|    function PredefinedApp(){
29|        $app = new App('PredefinedApp', []);
30|
31|        $app->cleanup = function($parent){
32|            //Do stuff like: echo "I'm saved";
33|        };
34|        return $app;
35|    }
36|		
37|    $app = new App('App1', ['PredefinedApp']);

注意:为代码的每一行添加了行号,因此我可以在下面的答案中引用这些行

问题

  1. 使用以下行创建:App的实例:

    $app = new App('App1', ['PredefinedApp']);  //Line: 37
  2. 调用构造函数:

    public function __construct($name, $dependencies){ /* Code here */ }  //Line: 05

    2.1。以下参数通过:

    • $name = "App1"
    • $dependencies = ["PredefinedApp"]
  3. 您使用此行将$name分配给$this->name

    $this->name = $name;  //Line: 06
  4. 使用空数组初始化$apps

    $apps = [];  //Line: 08
  5. 现在循环浏览$dependencies的每个元素,其中包含1个元素(["PredefinedApp"]

  6. 在循环中,您执行以下操作:

    6.1将函数调用的返回值赋给数组索引:

    $apps[$name] = $dependName($this);  //Line: 10
    //$apps["App1"] = PredefinedApp($this);

  7. 您调用该函数:

    PredefinedApp(){ /* Code here */}  //Line: 28
  8. 现在再次创建App的新实例PredefinedApp()与之前相同(第2 - 6点,在构造函数中期望您有其他变量值+您不输入循环,因为数组是空的)

  9. 您将closure分配给类属性:

    $app->cleanup = function($parent){  //Line: 31
        //Do stuff like: echo "I'm saved";
    };
  10. 您将返回App的新创建对象:

    return $app;  //Line: 34
  11. 这里已经是__destruct() gets called,因为当函数结束时,refcount会转到zval的0并触发__destruct()。但由于$this->apps为空,因此没有任何事情发生。

  12. 返回该函数中新创建的对象并将其分配给数组索引(注意:我们从函数返回到6.1):

    $apps[$name] = $dependName($this);  //Line: 10
    //$apps["App1"] = PredefinedApp($this);

  13. 构造函数以将本地数组赋值给class属性结束:

    $this->apps = $apps;  //Line: 12
  14. 现在整个脚本结束了(我们已完成第37行)!这意味着对象$app __destruct() is triggered的原因与函数$app

  15. PredefinedApp()的原因相同
  16. 这意味着你现在循环遍历来自$this->apps的每个元素,它只保存函数的返回对象:

    public function __destruct(){  //Line: 15
        foreach($this->apps as $dep){
            $result = $dep->cleanup($this);
        }
    }
    Array(
        "App1" => App Object
            (
                [apps:protected] => Array
                    (
                    )
    
                [name] => PredefinedApp
                [cleanup] => Closure Object
                    (
                        [parameter] => Array
                            (
                                [$parent] => <required>
                            )
    
                    )
    
            )
    )
    
  17. 对于每个元素(这里只有1个),执行:

    $result = $dep->cleanup($this);  //Line: 17

    But you don't call the closure! It tries to call a class method.所以没有cleanup类方法,它只是一个类属性。这意味着__call() gets invoked

    public function __call($name, $arguments){  //Line: 21
        if(is_callable([$this, $name])){
            return call_user_func_array([$this, $name], $arguments);
        }
    }
  18. $arguments包含自身($this)。 is_callable([$this, $name])TRUE,因为cleanup可以作为闭包调用。

  19. 所以现在我们进入了无穷无尽的东西,因为:

    return call_user_func_array([$this, $name], $arguments);  //Line: 23

    执行,然后看起来像这样:

    return call_user_func_array([$this, "cleanup"], $this);

    然后又尝试调用cleanup作为方法,再次调用__call()等等......

  20. 所以在整个剧本结束时,洞穴灾难开始了。但是我有一些好消息,听起来很复杂,解决方案要简单得多!

    解决方案

    只需改变:

    return call_user_func_array([$this, $name], $arguments);  //Line: 23

    用这个:

    return call_user_func_array($this->$name, $arguments);
                               //^^^^^^^^^^^ See here
    

    因为通过这样改变它你不会尝试调用方法,而是关闭。所以,如果你也提出:

    echo "I'm saved";
    
    分配时,在闭包中

    ,你将得到输出结果:

    I'm saved
    

答案 1 :(得分:3)

使用以下内容替换__call()函数将阻止您看到的递归:

public function __call( $method, $arguments ) {
    if ( $this->{$method} instanceof Closure ) {
        return call_user_func_array( $this->{$method}, $arguments );
    } else {
        throw new Exception( "Invalid Function" );
    }
}

有关详细信息,请参阅@ Rizier123的答案,但TLDR版本为:

问题是您首先调用cleanup()作为方法调用$dep->cleanup()调用__call()。如果您然后使用call_user_func_array()并传递[$this, $method],则将其作为方法调用,从而再次调用_call()并触发循环(并且从不实际调用{{ 1}}方法),而使用cleanup()会将其作为$this->{$method}调用并阻止循环。