3

假设我正在创建一个注册表单。我有如下代码，由于所有嵌套的 if 语句，很难管理。

我想知道编写代码的最简洁和最容易遵循的方法，其功能类似于我下面的代码。

编辑：人们告诉我，我可以将 empty($_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;
4

3 回答 3

6

每当您嵌套if()s 时，您都可以“从里到外”翻转逻辑：

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());
}
于 2013-06-21T22:40:58.737 回答
4

您此时不需要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
}
于 2013-06-21T22:26:26.657 回答
2

这有点干净：

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;
于 2013-06-21T22:29:44.793 回答