我简化了我的代码,但我正在做的是这样的:
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
错误。
答案 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']);
注意:为代码的每一行添加了行号,因此我可以在下面的答案中引用这些行
使用以下行创建:App
的实例:
$app = new App('App1', ['PredefinedApp']); //Line: 37
调用构造函数:
public function __construct($name, $dependencies){ /* Code here */ } //Line: 05
2.1。以下参数通过:
$name
= "App1"
$dependencies
= ["PredefinedApp"]
您使用此行将$name
分配给$this->name
:
$this->name = $name; //Line: 06
使用空数组初始化$apps
:
$apps = []; //Line: 08
现在循环浏览$dependencies
的每个元素,其中包含1个元素(["PredefinedApp"]
)
在循环中,您执行以下操作:
6.1将函数调用的返回值赋给数组索引:
$apps[$name] = $dependName($this); //Line: 10 //$apps["App1"] = PredefinedApp($this);
您调用该函数:
PredefinedApp(){ /* Code here */} //Line: 28
现在再次创建App
的新实例PredefinedApp()
与之前相同(第2 - 6点,在构造函数中期望您有其他变量值+您不输入循环,因为数组是空的)
您将closure分配给类属性:
$app->cleanup = function($parent){ //Line: 31 //Do stuff like: echo "I'm saved"; };
您将返回App
的新创建对象:
return $app; //Line: 34
这里已经是__destruct()
gets called,因为当函数结束时,refcount
会转到zval
的0并触发__destruct()
。但由于$this->apps
为空,因此没有任何事情发生。
返回该函数中新创建的对象并将其分配给数组索引(注意:我们从函数返回到6.1):
$apps[$name] = $dependName($this); //Line: 10 //$apps["App1"] = PredefinedApp($this);
构造函数以将本地数组赋值给class属性结束:
$this->apps = $apps; //Line: 12
现在整个脚本结束了(我们已完成第37行)!这意味着对象$app
__destruct()
is triggered的原因与函数$app
PredefinedApp()
的原因相同
这意味着你现在循环遍历来自$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>
)
)
)
)
对于每个元素(这里只有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); } }
$arguments
包含自身($this
)。 is_callable([$this, $name])
为TRUE
,因为cleanup
可以作为闭包调用。
所以现在我们进入了无穷无尽的东西,因为:
return call_user_func_array([$this, $name], $arguments); //Line: 23
执行,然后看起来像这样:
return call_user_func_array([$this, "cleanup"], $this);
然后又尝试调用cleanup
作为方法,再次调用__call()
等等......
所以在整个剧本结束时,洞穴灾难开始了。但是我有一些好消息,听起来很复杂,解决方案要简单得多!
只需改变:
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}
调用并阻止循环。