PHP - 避免使用嵌套的if语句 - 更清晰的编码方式

Question

假设我正在创建一个注册表单.我有类似下面的代码,因为所有嵌套的if语句都很难管理.

我想知道最干净,最简单的方法来编写与我下面的代码类似的代码.

编辑:人们告诉我,我可以将空($ _ POST ['email'])移动到验证功能.我不能这样做,因为我需要知道1)用户是否发布了数据,以及2)用户发布的数据是否有效.

例如,当用户首次进入注册页面时,他们没有发布任何数据,因此$ _POST ['email']将生成PHP警告,因为它们不存在.这就是我在验证之前检查数据是否已发布的原因.

这有意义吗？

function validate_email($str) {
    $str = trim(strtolower($str));
    if(!filter_var($str, FILTER_VALIDATE_EMAIL)) {
      return false;
    } else {
      return $str;
    }
}

function validate_password($str) {
    $str = trim($str);
    if(strlen($str) < 5 || strlen($str) > 70) {
        return false;
    } else {
        return $str;
    }
}

$email = false;
$password = false;
$errorMessage = false;

if(!empty($_POST['email'])) {
    $email = validate_email($_POST['email']);
    if($email) {
        if(!empty($_POST['password'])) {
            $password = validate_password($_POST['password']);
            if($password) {
                createNewUser($email,$password);
            } else {
                $errorMessage = "The password is not valid";
            }
        } else {
            $errorMessage = "The password is not valid";
        }
    } else {
        $errorMessage = "Email address is invalid";
    }
} else {
    $errorMessage = "Email address is invalid";
}

if($errorMessage) echo $errorMessage;

Answer 1

每当你有嵌套if()的时候,你可以"从里到外"翻转逻辑:

if (A)
  if (B)
    if (C)
      final()

改成:

 if (!A) return
 if (!B) return
 if (!C) return
 final()

在你的情况下,你可以抛出异常而不是返回.

try {
  validateAndCreateNewUser();
}
catch(ValidationError $e) {
  display($e->getMessage());
}