构建条件逻辑的最佳方法是什么？

Question

我发现有许多不同的方法来构建条件逻辑。据我所知，只要我们设置错误来结束脚本（或者您可以想象相同的示例，但在函数中带有返回），那么以下示例是相同的：

示例 1

if($condition1) {
    trigger_error("The script is now terminated");
    }

if($condition2) {
    trigger_error("The script is now terminated");
    }

echo "If either condition was true, we won't see this printed";

示例 2

if(!$condition1) {
    if(!$condition2) {
        echo "If either condition was true, we won't see this printed";
        }
    else {
        trigger_error("The script is now terminated");
        }
    }
else {
    trigger_error("The script is now terminated");
    }

示例 3

if($condition1) {
    trigger_error("The script is now terminated");
    }
else {
    if($condition2) {
        trigger_error("The script is now terminated");
        }
    else {
        echo "If either condition was true, we won't see this printed";
        }
    }

示例 4 -- 改编自 Fraser 的回答

function test($condition) { 
    if($condition) {
        trigger_error("The script is now terminated");
        }   
    }

test($condition1);

test($condition2);

echo "If either condition was true, we won't see this printed";

就个人而言，我倾向于像示例 1 那样编写代码。这是因为我觉得通过检查条件以这种方式结束脚本（或函数），我可以清楚地定义脚本执行的内容和未执行的内容，即条件执行之前的所有内容以及该行之后的所有内容尚未执行。这意味着当我在第 147 行遇到错误时，我立即知道发生了什么，帮助我更快地找到错误。此外，如果我突然意识到我需要在 $condition1 之前测试 $condition2，我可以通过简单的复制粘贴进行更改。

我看到很多代码都是像示例 2 中那样编写的，但对我来说，这似乎调试起来要复杂得多。这是因为，当嵌套变得太大时，错误将在底部的某个较远的行处触发，并与由大量嵌套代码引起的条件分开。此外，改变条件序列可能会变得更加混乱。

您可以混合这两种样式，例如示例 3，但这似乎会使问题变得过于复杂，因为所有“其他”本质上都是多余的。

我错过了什么吗？构造条件代码的最佳方法是什么？还有比这些例子更好的方法吗？ 在某些特定情况下，一种风格可能优于另一种风格？

编辑：示例 4 看起来很有趣，但我没有考虑过。您还可以传入错误消息作为第二个参数。

谢谢！

PS 请记住，我可能需要在检查 $condition1 和 $condition2 之间执行一些任意步骤，因此任何替代方案都必须适应这一点。否则，还有更好的选择，例如 if($condition1 || $condition2)。

Answer 1

我属于示例1阵营。根据经验，所需的缩进越少越好。

// Exit early if there are errors.
if ($n < 0) {
    die "bad n: $n";
}

// Handle trivial cases without fuss.
if ($n == 0) {
    return 0;
}

/* Now the meat of the function. */
$widget->frob($n);
foreach ($widget->blaxes as $blax) {
    checkFrobbingStatus($blax);
}
// ...And another 20 lines of code.

当您使用 if/else 并将成功代码和错误代码放在并行部分中时，您会使两个代码块看起来相等。实际上，应该不再强调边缘情况和错误条件。通过有意尽早处理错误，然后不将“重要”代码放入 else 子句中，我觉得这样可以在视觉上更清晰地显示重要代码的位置。

“这是所有的先决条件。现在这是好东西。”

Answer 2

就我个人而言，我讨厌嵌套的 if-else 语句，所以从你的例子中对我来说#1。我会考虑的另一个选项如下所示。

function test($condition) { 
  if($condition) {
    trigger_error("The script is now terminated");
  }   
}

test($condition1);

//do stuff...

test($condition2);

//passed the tests

编辑：我越想越觉得函数式方法是迄今为止最好的方法，因为它不需要编写多次测试条件的逻辑。它还具有更高的可读性，因为很明显您正在“测试”条件（只要您为函数指定一个有意义的名称）。此外，正如问题编辑中所指出的，将其他参数传递给函数也很简单。 IE

function test($c, $msg) { 
  if($c) {
    trigger_error($msg);
  }   
}

test($condition1, "condition1 error");
test($condition2, "condition2 error");

Answer 3

#1 是迄今为止最清晰的。但是，如果以某种方式将之前结束执行的事情更改为执行其他操作，那么它就会中断。

最好还是选择#1，但要确保用于“停止”的东西被明确命名以表明它确实停止事情，这样在10年内维护你的代码的人就不会'不要因为改变而意外地破坏它。

Answer 4

我认为你的方法（示例1）在这种情况下是最有效的。但是，有时您不希望任何条件停止执行，而只想在 condition1 为 false 时执行 condition2。在这些情况下，else 或 elseif 效果很好。

Answer 5

我建议在任何可能出现错误的部分上使用“try”子句，并在每次发生错误时使用“抛出“错误描述””（如示例#1）。
这样，您就可以在程序中包含一次错误报告代码（在“catch”子句中），并且将代码拆分为函数将不会是重写错误处理的麻烦。

Answer 6

我更喜欢这种风格，如果任何一个条件块发生更改，这种风格都不会中断，因此它们不会退出执行。

if($condition1) {
    trigger_error("The script is now terminated");
}
if($condition2) {
    trigger_error("The script is now terminated");
}

if (!$condition1 && !$condition2) {
  echo "If either condition was true, we won't see this printed"; 
}

编辑：错过了 PS，因此更新了代码以匹配完整的问题详细信息。

Answer 7

到目前为止，我总体上同意 Amber因为你的第一个选项似乎是最清晰的。这是我一直在与自己斗争的事情 - 到目前为止，我偶然发现的唯一推理如下：

• 在阅读线性脚本时，第一种形式是最清晰的，因此非常适合简单的脚本
• 当您需要确保整洁时，第二种形式是最干净的/ 清理操作

我提到第二个是因为这是一个棘手的问题。每个脚本可能是一个更大系统的一部分，事实上，您注入“bail out”代码的脚本元素可能会被多个地方调用。加入一些面向对象，你就会得到一个真正有潜力的泡菜。

我可以推荐的最佳经验法则是，如果您的脚本简单且线性，或者您正在进行快速原型设计，那么您需要使用第一种形式并在此时终止执行。任何更复杂或“企业式”的东西都将受益于（至少）模块化的重新设计，这样您就可以隔离方法和调用堆栈 - 以及可能的 OO 构建的封装。

随着当今一些更强大的调试和跟踪工具的出现，它变得更多的是个人风格而不是必要性的问题。您可能考虑的另一种选择是在每个救助区域之前（也可能之后）在注释中添加信息，以明确如果满足标准（或失败），替代方案是什么。

编辑：

我会说Fraser 的答案对于封装来说是最干净的。我唯一要补充的是，您可能会受益于将对象或哈希数组传递到标准“保释，我死了”方法，这样您就可以修改函数可用的信息，而无需始终更改参数列表（非常烦人...）。

也就是说，在生产系统中要小心，您可能需要在中间状态下清理资源。

Answer 8

我也更喜欢#1。

此外，我真的很喜欢在条件语句中分配变量

，例如

if ( !$userName = $user->login() ) {
    die('could not log in');
}

echo "Welcome, $username";

我通常发现，在第一次编写代码时，我最终会得到一些混乱的嵌套条件语句，因此通常在第二遍期间我会返回并清理内容，尽可能多地取消嵌套条件语句。

除了看起来更整洁之外，我发现它在概念上更容易理解代码，您不必在心里跟踪分支逻辑。

对于无法删除且包含大量过程代码的分支逻辑，我通常最终将其放入函数/类方法中——理想情况下，这样我就可以在一个屏幕上看到正在发生的所有分支逻辑，但可以修改其中之一行动或逻辑不会破坏另一个。

Answer 9

实施例5

if($condition1 || $condition2)
{
    echo "If either condition was true, we won't see this printed";
}else
{
    trigger_error("The script is now terminated");
}

实施例6

function allAreTrue()
{
    foreach(func_get_args() as $check)
    {
       if(!$check)
       {
           return false;
       }
    }
    return true;
}

if(allAreTrue(true,true,$condition1,$condition2,false))
{
   exit("Invalid Arguments");
}

//Continue 

Answer 10

构造条件逻辑的最佳方法是遵循逻辑本身。

如果你有依赖关系，比如说，第一个条件的失败将使其他条件变得不必要是一回事。然后return、goto、嵌套条件和异常由您选择。

如果您要对测试做出决定，请说它

if (!isset($_GET['id'])) { 
  //listing part: 
} else { 
  // form displaying part: 
}  

是 else、elseif 和 case 领域。

等等
首先确定你的程序逻辑，然后编写它的逻辑。

并且 trigger_error() 与条件无关。这是调试功能，而不是与程序逻辑相关的功能。