0

У меня есть код:

$login = $_POST['login'];
$password = $_POST['psw'];
$sql = "SELECT * FROM `Users` WHERE Name='$login' AND Password='$password'";
$result = mysqli_query($conn, $sql);

$res = $result->fetch_assoc();

if (empty($res) == 1) {
    echo "Неверный логин или пароль!";
}
else {
    while($row = $res){
        echo  $row['Name'];
        echo  ' - ';
        echo  $row['Surname'];
        echo  '<br>';
    }
}

Если я использую $result->fetch_assoc() через переменную $res, то при вводе верного пароля бесконечно выводится один и тот же результат из базы данных. При прямом использовании всё было нормально, пока не добавил скрипт проверки "Неверный логин или пароль".

1
  • 2
    В таком варианте $res не меняется, поэтому и получается бесконечный цикл в while($row = $res).
    – Knyaz71
    5 фев в 14:01

2 ответа 2

0

Потому что вы один раз получили $res и больше он у вас никогда не меняется. Его нужно каждый раз вызывать в цикле. Вопрос следующий. Я так понимаю, вы собираетесь выводить только одного юзера. Зачем вам вообще здесь какой-то цикл?

$login = mysqli_real_escape_string($conn, $_POST['login']);
$password = mysqli_real_escape_string($conn, $_POST['psw']);
$sql = "SELECT * FROM `Users` WHERE Name='$login' AND Password='$password'";
$result = mysqli_query($conn, $sql);
$res = $result->fetch_assoc();
if (!$res) {
    echo "Неверный логин или пароль!";
}
else {
    echo  $res['Name'];
        echo  ' - ';
        echo  $res['Surname'];
        echo  '<br>';
}

И всегда экранируйте принимаемые значения в SQL-запросах, если не хотите например базу потом из бэкапа восстанавливать

3
  • 1
    У вас будет ошибка в 6 строке. $res не найдена...
    – Knyaz71
    6 фев в 7:21
  • @Knyaz71 спасибо что заметили. Поправил 6 фев в 9:29
  • У меня суть проекта - это уязвимая для SQL-иньекции страница, поэтому использую цикл (чтобы выводились все подходящие значения). При втором обращении к $result->fetch_assoc() ничего не выводится, поэтому использую через переменную
    – Oladik
    6 фев в 15:35
0

Мне помогла функция mysqli_num_rows(), таким образом у меня есть код:

$login = $_POST['login'];
$password = $_POST['psw'];
$sql = "SELECT * FROM `Users` WHERE Name='$login' AND Password='$password'";
$result = mysqli_query($conn, $sql);

$count = mysqli_num_rows($result);

if($count == 0) {
    echo "Неверный логин или пароль!";
}
else {
    while($row = $result->fetch_assoc()){
        echo  $row['Name'];
        echo  ' - ';
        echo  $row['Surname'];
        echo  '<br>';
    }
}

который исправно работает. Обратите внимание, что скрипт уязвим.

1
  • В текущем виде ваш ответ непонятен. Пожалуйста, нажмите править под сообщением, чтобы добавить больше подробностей, которые помогут другим понять, как он отвечает на заданный вопрос. Вы можете найти больше информации о том, как писать хорошие ответы в Справке. 6 фев в 20:27

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.