循环中带有布尔运算的PHP错误

Question

在循环中使用布尔运算的 PHP 中似乎存在错误。

在这个特定的示例中，我测试是否设置了所有重要的表单字段（由对象表示）。 var_dump 用于调试目的。

$allset = true;
foreach ($forms as $one):
    $allset = $one->wasSet() and $allset;
endforeach;
var_dump($allset);
foreach ($more as $one):
    $allset = $one->wasSet() and $allset;
endforeach;
var_dump($allset);
$allset = $iwasthere->wasSet() and $allset;
var_dump($allset);

此代码可能会失败。可能第一个转储返回 false，第二个转储返回 true。

我的问题。

我怎样才能避免这个错误并拥有简洁干净的代码？

---

附言

我现在使用if 块。

Answer 1

这里的问题是= 运算符有一个higher precedence 而不是and 运算符。所以赋值表达式只是逻辑表达式的一部分，整个表达式等价于：

($allset = $one->wasSet()) and $allset;

要么将逻辑表达式放在括号中，要么使用&& 代替and：

$allset = ($one->wasSet() and $allset);
$allset = $one->wasSet() && $allset;

Answer 2

不应该有

$allset = true;

在第二个 foreach 循环之前？

Answer 3

（以下代码仅与 php 5 兼容）

您的代码有另一个错误：

$allset = $iwasthere->wasSet(); // this overwrites all previous checks

应该是

$allset = $iwasthere->wasSet() && $allset;

如果你有一个表单类而不是字段数组会更好

类形式{ 受保护的 $fields = array(); 函数添加字段（$字段）{ $this->fields = $field; 返回$字段； } 函数 allSet(){ $allSet = 真； foreach($this->fields as $field){ $allSet = $allSet && $field->wasSet(); } 返回 $allSet; } }

添加字段，您可以（选项一）扩展类：

RegisterForm extends Form{
  function __construct(){
    $name = $this->addField(new FormField('name'));
    $action = $this->addField(new FormField('action'));
    $action->hidden = true;
  }
}

然后你像这样使用它：

$form = new RegisterForm();

或者（选项二）您可以在创建表单后添加字段

$form = new Form();
$name = $form->addField(new FormField('name'));

....

然后使用非常简单：

....

if(!$form->allSet()){ // that looks like clean code
  ...error...
}

如果您有 2 个表格：

$allSet = $form1->allSet() && $form2->allSet();
if( !$allSet ){
  ...error...
}

如果你需要重复两次相同的循环，你的代码已经很枯燥​​了