Как проверить одну переменную или две переменные в PHP - PullRequest
Новая
       6

Как проверить одну переменную или две переменные в PHP

0 голосов
/ 10 октября 2018

У меня есть две переменные: mobile и email, теперь я хочу проверить оба, но я хочу, чтобы пользователь оставил пустым одно из полей, если у пользователя его нет, например, если пользователь не хочет регистрироваться в своей электронной почте, тогда он можетперейдите на номер мобильного телефона для регистрации, и наоборот, это мой код проверки 

<?php

$emailError    = "";
$fullnameError = "";
$usernameError = "";
$passwordError = "";
$mobileerror = "";
$errors        = 0;

if ((isset($_POST['submit']))) {

    $email    = strip_tags($_POST['email']);
    $fullname = strip_tags($_POST['fullname']);
    $username = strip_tags($_POST['username']);
    $password = strip_tags($_POST['password']);
    $mobile = strip_tags($_POST['mobile']);



    $fullname_valid = $email_valid = $mobile_valid = $username_valid = $password_valid = false;



    if (!empty($fullname)) {

        if (strlen($fullname) > 2 && strlen($fullname) <= 30) {
            if (!preg_match('/[^a-zA-Z\s]/', $fullname)) {
                $fullname_valid = true;

                # code...
            } else {
                $fullnameError = "fullname can contain only alphabets <br>";
                $errors++;
            }
        } else {
            $fullnameError = "fullname must be 2 to 30 char long <br>";
            $errors++;
        }
    } else {
        $fullnameError = "fullname can not be blank <br>";
        $errors++;
    }



        if (filter_var($email, FILTER_VALIDATE_EMAIL)) {

            $query2 = "SELECT email FROM users WHERE email = '$email'";
            $fire2 = mysqli_query($con, $query2) or die("can not fire  query" . mysqli_error($con));
            if (mysqli_num_rows($fire2) > 0) {
                $emailError = $email . "is already taken please try another one<br> ";
            } else {
                $email_valid = true;
            }
            # code...
        } else {
            $emailError = $email . "is an invalid email address <br> ";
            $errors++;
        }
        # code...

   if ($mobile) {
      $query4 = "SELECT mobile FROM users WHERE mobile = '$mobile'";
            $fire4 = mysqli_query($con, $query4) or die("can not fire  query" . mysqli_error($con));
            if (mysqli_num_rows($fire4) > 0) {
                $mobileerror =  "is already taken please try another one<br> ";
            } else {
                $mobile_valid = true;
            }
   }

    if (!empty($username)) {

        if (strlen($username) > 4 && strlen($username) <= 15) {
            if (!preg_match('/[^a-zA-Z\d_.]/', $username)) {


                $query = "SELECT username FROM users WHERE username = '$username'";
                $fire = mysqli_query($con, $query) or die("can not fire  query" . mysqli_error($con));


                if (mysqli_num_rows($fire) > 0) {
                    $usernameError = '<p style="color:#cc0000;">username already taken</p>';
                    $errors++;
                } else {
                    $username_valid = true;
                }

            } else {
                $usernameError = "username can contain only alphabets <br>";
                $errors++;
            }
        } else {
            $usernameError = "username must be 4 to 15 char long <br>";
            $errors++;
        }
    } else {
        $usernameError = "username can not be blank <br>";
        $errors++;
    }

    if (!empty($password)) {
        if (strlen($password) >= 5 && strlen($password) <= 15) {
            $password_valid = true;
            $password       = md5($password);

            # code...
        } else {
            $passwordError = $password . "password must be between 5 to 15 character long<br>";
            $errors++;
        }
        # code...
    } else {
        $passwordError = "password can not be blank <br>";
        $errors++;
    }

    //if there's no errors insert into database
    if ($errors <= 0) {
        if ($fullname_valid && ($email_valid || $mobile_valid )&& $password_valid && $username_valid) {
            $query = "INSERT INTO users(fullname,email,username,password,avatar_path) VALUES('$fullname','$email','$username','$password','avatar.jpg')";
            $fire = mysqli_query($con, $query) or die("can not insert data into database" . mysqli_error($con));
            if ($fire) {


                header("Location: dashboard.php");
            }
        }

    }
}
?>

Теперь, когда я использую электронную почту и оставляю пустой мобильный телефон, код работает нормально, но когда я использую электронную почту и оставляю пустой мобильный телефон, возникает ошибка, какрешить эту проблему

Ответы [ 2 ]

0 голосов
/ 10 октября 2018

теперь, когда я использую электронную почту и оставляю пустой мобильный телефон, код работает нормально, но когда я использую электронную почту и оставляю пустой мобильный телефон, возникает ошибка

у вас есть: 

if (!empty($fullname)) {}
if (filter_var($email, FILTER_VALIDATE_EMAIL)) {}
if ($mobile) {}
if (!empty($username)) {}
if (!empty($password)) {}

Чтобы устранить ошибку, попробуйте добавить 

if (!empty($mobile)) {

Кроме того, я бы посоветовал еще немного обернуть операторы.Вам нужен только один для сбоя, чтобы остановить ввод.Вы можете сделать что-то вроде этого: 

$mobileOrEmail = false;
if (!empty($fullname) && !empty($username) && !empty($password) {
  //check fullname, username and password
  if (!empty($mobile) {
    //check mobile, if it passes
    $mobileOrEmail = true;
  }
  if (filter_var($email, FILTER_VALIDATE_EMAIL)) {
    //check email, if it passes
    $mobileOrEmail = true;
  }
  if (!$mobileOrEmail) $errors++;
} else {
  //missing input values
  $errors++;
}

Лично я бы создал функцию для каждого поля ввода. 

function checkUsername($username){
  //check username
  return true;
}
function checkEmail($email) {
  //check email
  return true;
}
....

, тогда вы можете запустить 

if (checkUsername($username) && checkPassword($password)
    && checkFullname($fullname) && (checkEmail($email) || checkEmail($email)) {
    //user input correct
 } else {
   //user input failed
 }

Просто чтобы придать ему больше структуры

0 голосов
/ 10 октября 2018

Используйте еще один флаг $isValid_email_mobile = FALSE;

Когда поток управления входит в if (filter_var($email, FILTER_VALIDATE_EMAIL)), тогда на УСПЕХЕ просто установите $ isValid_email_mobile = TRUE;То же самое будет, если элемент управления снова войдет в состояние if ($mobile) при УСПЕХЕ, установите его как $isValid_email_mobile = TRUE;

When $ isValid_email_mobile = FALSE;становится ИСТИНОЙ, тогда вы знаете, что поле / переменная прошло ваше требование и готово к БД INSERT

Тогда

В вашем последнем IF состоянии, когда вы пытаетесь вставить INSERT, просто измените IF условие к следующему 

IF ($fullname_valid &&  $isValid_email_mobile && $password_valid && $username_valid)

Еще одна вещь, когда вы используете логику флага, всегда устанавливайте свой флаг на какое-то значение по умолчанию перед его использованием.

...