PHP - 嵌套 IF 语句

Question

关闭。这个问题是 opinion-based 。目前不接受答案。

---

想要改进这个问题吗？更新问题，以便 editing this post 可以用事实和引文来回答它。 .

已关闭 4 年前。

Improve this question

可以使用哪些控制结构来代替多个嵌套 IF 语句。

例如:

function change_password($email, $password, $new_password, $confirm_new_password)
{
    if($email && $password && $new_password && $confirm_new_password)
    {
        if($new_password == $confirm_new_password)
        {
            if(login($email, $password))
            {
                if(set_password($email, $new_password))
                {
                    return TRUE;
                }
            }
        }
    }
}       

该函数的使用方式如下:

if(!change_password($email, $password, $new_password, $confirm_new_password)
{
    echo 'The form was not filled in correctly!';
    exit;
}

我这样调用我的所有函数，我想知道我的编码风格是否有问题。我有疑问，因为如果我遵循这个设计，那么这意味着我编写的每个函数都将与 IF 嵌套，检查每个阶段是否有错误。这是其他人做的吗？

我没有看到很多其他脚本是这样写的，嵌套的 IF 形成一个三角形，并且只在中间有所需的结果。如果没有达到中间，那么就会出现问题。

这是一个好的函数结构吗？

Answer 1

嵌套太深通常是一个坏主意 - 这是意大利面条逻辑并且难以遵循。由于每个验证步骤都取决于前一个阶段是否成功，因此根本不要嵌套 - 只需在某个阶段失败时退出即可:

function change_password(blah blah blah) {
   if (!$condition1) {
      return false;
   }
   if (!$condition2) {
      return false;
   }
   etc....


   // got here, must have succeeded
   return true;
}

这清楚地表明了逻辑顺序是什么。