Возникла проблема с запросом к бд, в чем проблема?

Question

[Никита Братский]

Уже давно мучаюсь над проблемой с запросом к бд. Запрос верный, в бд находит записи, однако доставать данные из бд не хочет и при дампе переменной $query, выводит Null либо:

object(mysqli_result)#1 (5) { ["current_field"]=> int(0) ["field_count"]=> int(4) ["lengths"]=> NULL ["num_rows"]=> int(1) ["type"]=> int(0) }

Вот мой код:

if (isset($_POST['submit_auth'])) {
if (!isset($_COOKIE['UserLogin'])) {
$login = $_POST['login'];
$password = $_POST['password'];
$con = mysqli_connect("localhost","login","password","table") or die('Не удалось подключиться');
$query = mysqli_query($con, "SELECT * FROM table WHERE `login` = '$login'");
var_dump($query);
$hash = $query->fetch_assoc()['password'];
$query = mysqli_fetch_assoc($query);
if ($login == $query['login'] or $login == $query['mail']) {
if (password_verify($password,$hash)) {
setcookie('UserLogin', $query['login']);
echo '<script type="text/javascript">window.location.href="/"</script>';

}else{
echo "<p class='RU-font-family' style='color:red;'>Вы ввели неверный пароль</p>";
print_r($query['password']);
}
}else{
echo "<p class='RU-font-family' style='color:red;'>Вы ввели неверные данные</p>";
$text = "SELECT * FROM users WHERE login = '$login'";
var_dump($text);
}
}
}

Comments

[Lander]

$con = mysqli_connect("localhost","login","password","table") or die('Не удалось подключиться');

Точно не ТАБЛИЦУ указываете в "table"? Там имя базы должно быть.

[Никита Братский]

Lander, точно

Answer 1

[dtBlack]

$hash = $query->fetch_assoc()['password'];
$query = mysqli_fetch_assoc($query);

Уточните в первую очередь, что вы пытаетесь получить этими строками кода? Зачем вызывать одно и тоже в двух разных формах?

Так же вопрос к именованию переменных, что происходит по коду с $query?

Что вам мешает вызвать один раз $data = $query->fetch_assoc() и далее работать с $data.

Подумайте внимательно что происходит при первом вызове fetch_assoc() и повторном вызове?

Возможно раньше работало так как у вас было две записи возвращаемых запросом а теперь одна?

Comments

[Никита Братский]

С вашим кодом все заработало

[dtBlack]

Шустро ты проверил... Надеюсь понял в чем была ошибка?

PS: Мой совет - не используй одно имя переменой для разных по сути вещей.
У тебя по коду выходит, что сначала в $query находится объект mysqli_result, а потом в нем оказывается массив с данными. Это очень сильно усложняет чтение кода и поиск ошибок.

[Никита Братский]

dtBlack, все равно откатилось до ошибки, на один раз работает почему-то

[Никита Братский]

dtBlack, и не создается куки, выдает это: Warning: Cannot modify header information - headers already sent by

[dtBlack]

Никита Братский Хм... не понял.
В принципе у тебя код не правильно написан. Нет даже самых нужных проверок.

Еще вопрос в чем соль строки if ($login == $query['login'] or $login == $query['mail']) { ? У тебя ведь всегда $login == $query['login'] см свой запрос и никогда не сработает проверка $login == $query['mail'] см тот же запрос.

Еще попробуй почитать про редиректы в PHP.

Чуть более верно твой код будет выглядеть так:

if (isset($_POST['submit_auth'])) {
    if (!isset($_COOKIE['UserLogin'])) {
        $login = $_POST['login'];
        $password = $_POST['password'];
        $con = mysqli_connect("localhost","login","password","table") or die('Не удалось подключиться');
        $query = mysqli_query($con, "SELECT * FROM table WHERE `login` = '$login'");
        if ($query){//Проверка успешности запроса
            var_dump($query);
            $user_data = $query->fetch_assoc();
            
            if (!empty($user_data)) {//Проверим нашелся ли такой пользователь
                $hash = $user_data['password'];
                if (password_verify($password,$hash)) {
                    setcookie('UserLogin', $user_data['login']);
                    echo '<script type="text/javascript">window.location.href="/"</script>';
                }else{
                    echo "<p class='RU-font-family' style='color:red;'>Вы ввели неверный пароль</p>";
                    print_r($password);
                }
            }else{
                echo "<p class='RU-font-family' style='color:red;'>Вы ввели неверные данные</p>";
                $text = "SELECT * FROM users WHERE login = '$login'";
                var_dump($text);
            }
        }
    }
}

PS: особенно веселая строка print_r($query['password']);

[Никита Братский]

dtBlack, if ($login == $query['login'] or $login == $query['mail']) {
В поле ввести можно либо логин, либо почту. Код, который приведен выше не работает, никаких уведомлений о работе нет и ридеректа

[dtBlack]

Никита Братский, Насчет той ошибки тебе стоило гугл спросить... У тебя сайт отправил уже заголовки, то есть ты уже вывел что то в браузер.

И вообще зачем тебе куки? Чем сессия не подходит?

Я вообще себя хакером ощущаю... Так и хочется воткнуть sql инъекцию и авторизовавшиcь как user заменить имя пользователя в куках на admin/

PS: Почитай про mysqli_real_escape_string

[Никита Братский]

dtBlack, сессии долго не живут, тем более есть ssl сертификат

[dtBlack]

Никита Братский, Я очень сильно надеюсь, что это курсовик или учебный проект, а не реальный сайт...
1. Время жизни сессии настраивается.
2. Куки могут редактироваться на компьютере. Я авторизуюсь на твоем сайте как user, а потом изменю имя в куках на admin. Что будет тогда?
3. Твоим правилом про создании сайта должно быть "Пользователь хочет сделать каку, нужно не дать ему этого". Всегда обрабатывай всю инфу пришедшую от пользователя особено если потом воткнешь ее в запрос к бд.
4. Время жизни авторизации и не должно быть слишком большим. 2-3 часа обычно вполне хватает.
5. Для работы в качестве логина с почтой нужно изменить запрос на "SELECT * FROM table WHERE `login` = '$login' OR `mail` = '$login'"
6. А почему что то должно выводиться? Ты куки грохнул которые создал? Добавь секции else с выводом ко всем IF и смотри где прекратилось выполнение.

Answer 2

[Александр Ivanov]

$con = mysqli_connect("localhost","login","password","table") or die('Не удалось подключиться');
заменить на 
$con = mysqli_connect("localhost",$login,$password,$table) or die('Не удалось подключиться');

Необходимо завести переменную $table, $table = "Нужная таблица"

Comments

[Никита Братский]

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

[Александр Ivanov]

Никита Братский, Если логин и пароль у базы login и password, а таблица названа table, тогда все будет работать.

[Никита Братский]

Александр Ivanov, логин совпадает с базой данных, это не моя идея, это идея хостинга

[Никита Братский]

Александр Ivanov, я могу скинуть код таблицы MySQL

[Александр Ivanov]

Никита Братский,

mysqli_connect("localhost","login","password","table")

В данной строке необходимо указать ваш личный логин и пароль от хостинга и название нужной вам таблицы. В целях безопасности эти данные лучше не разглашать.

mysqli_connect("localhost",$login,$password,"table")

Минимум нужно сделать так. Вы же передаете постом данные в переменную. А потом нужно использовать их при подключении к БД.

[Никита Братский]

Александр Ivanov, это были ненастоящие логин и пароль