3

登録フォームを作成しているとします。以下のようなコードがあり、ネストされたすべての if ステートメントのために管理が困難です。

以下にあるものと同様に機能するコードを記述するための、最もクリーンで簡単な方法を知りたいです。

EDIT : 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 に答える