这是一个可接受的使用eval（）吗？

Question

我试图使用一个内爆字符串作为运算符。作为一个PHP新手，到目前为止，eval（）是唯一可行的选项。

我已阅读eval()的其他问题。在大多数情况下，人们试图允许用户输入实际的代码，这不是我在这里做的。

这就是：

/* 
* Get user choices from a multiple-select customizer custom control. 
* Setting's choice values are the names of WordPress conditional tags. 
* Each choice set is given as 'key'=>'label' NOT 'key'=>'function' 
* so I'm limited to using strings here, e.g. is_front_page, is_singular. 
* The result will be a linear array, e.g. array(is_front_page, is_singular). 
*/ 
$conditions = get_theme_mod('theme_setting'); 

/* 
* Here's where my trouble begins. 
* I need to append `()` to each array value using array_walk. */ 
array_walk($conditions, function(&$value, $key) { $value = $value . '()'; }); 

/* 
* Next I implode the array values to insert the or operator 
* So instead of "is_front_page, is_singular", I'd end up with 
* "is_front_page() || is_singular()" 
*/ 
$conditions = implode(' || ', $conditions); 

/* The previous step is NOT usable, in my novice experience, hence eval() */ 
eval('$conditions = ' . $conditions . ';'); 

/* Eval makes the following usable */ 
if($conditions) { 
    // do stuff here 
} 

我希望这是可以接受的，因为我不会允许来自用户代码输入，我的主题设置为静态的，所以我实际上不能这样做$conditions === true为解决方法。

即使可以接受，请让我知道如果您有任何建议如何改善它。

Answer 1

Nooo ...你的想法太宽泛了。你有一个函数名称数组，你跳得太远，不能执行它们作为自由形式代码。

其实功能名称是有效的回调和那些更安全和更容易执行call_user_func()。所以简单地array_map('call_user_func', $conditions)会将所有的回调变成它们的返回值。

但请注意，您所需的条件是OR型。我们不需要每回调运行，我们只需要执行它们，直到我们获得第一个true之一。

这可以表示为：

$result = array_reduce($callbacks, function ($carry, $callback) { 
    return $carry ?: (boolean) call_user_func($callback); 
}, false);