Question

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

我想知道最干净，最容易遵循的方式来编写与我下面的代码类似的代码。

编辑：人们告诉我，我可以将空（$ _ POST [＆＃39; email＆＃39;]）移到验证功能。我无法做到这一点，因为我需要知道1）用户是否发布了数据，以及2）用户发布的数据是否有效。

例如，当用户首次进入注册页面时，他们没有发布任何数据，因此$ _POST [＆＃39; email＆＃39;]将生成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());
}

Answer 2

此时您不需要empty检查，将它们移至验证功能。

例如：

function validate_email($str) {
    if(empty($str)) {
      return false;
    }

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

$email = validate_email($_POST['email']);
if($email) { 
  // your code
}

Answer 3

这有点清洁：

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

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

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

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

if($errorMessage) echo $errorMessage;

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

3 个答案: