Не могу авторизоваться, почему?

Question

[GirlInc]

<?php
$login = filter_var(trim($_POST['login']),
FILTER_SANITIZE_STRING);
$pass = filter_var(trim($_POST['pass']),
FILTER_SANITIZE_STRING);

$pass = md5($pass."ghjsfkld2345");

    $mysql = new mysqli('localhost','root','root','automoykaBase'); 

    $result = $mysql->query("SELECT * FROM `users` WHERE `login` =
    '$login' AND `pass` = '$pass '");
    $user = $result ->fetch_assoc();
    if(count($user) == 0) {
    echo "Такой пользователь не найден";
    exit();
    }

    setcookie('user', $user['name'], time() + 3600, '/');

    $mysql->close();

    header('Location: Личный-кабинет.html');
?>

Comments

[humoured]

Тут не кружок телепатов.

[Дмитрий]

Ну наверное надо ошибки посмотреть

[Adamos]

В запросе пробел после $pass. Всего-то.

[GirlInc]

Adamos,

Warning: count(): Parameter must be an array or an object that implements Countable in D:\OSPanel\domains\Site1\auth.php on line 14

Такой пользователь не найден

[GirlInc]

Дмитрий, Warning: count(): Parameter must be an array or an object that implements Countable in D:\OSPanel\domains\Site1\auth.php on line 14
Такой пользователь не найден

[GirlInc]

humoured, Warning: count(): Parameter must be an array or an object that implements Countable in D:\OSPanel\domains\Site1\auth.php on line 14
Такой пользователь не найден

[Wexter]

Оформите код в тег code

[Adamos]

GirlInc, хороший повод прочитать документацию к функции fetch_assoc

[Алан Гибизов]

GirlInc, оборачивайте traceback и прочие warning в вопросе и комментариях тегом code для корректного отображения.

Answer 1

[ThunderCat]

так, давайте по порядку:

login = filter_var(trim($_POST['login']), FILTER_SANITIZE_STRING);
$pass = filter_var(trim($_POST['pass']), FILTER_SANITIZE_STRING);

Во первых - зачем? Вы здесь не вносите никаких данных в систему, вы просто проверяете данные от пользователя с данными в бд.
Во вторых - если у вас есть правила для логина и пароля - надо их и применять, а не втыкать абсолютно бесполезные фильтры.

$pass = md5($pass."ghjsfkld2345");используйте встроенные функции, password_hash и password_verify для работы с паролями.

$result = $mysql->query("SELECT * FROM `users` WHERE `login` = '$login' AND `pass` = '$pass '");

Читаем про подготовленные выражения. Без них рано или поздно словите инъекцию. И начинать надо сейчас, на этапе обучения, и привыкнуть что по другому с переменными в запросах не работают. Так же, по уму подключение желательно вынести отдельно и добавить настройки. Рекомендую использовать PDO драйвер подключения к бд, он работает чуть быстрее и по функционалу побогаче.

Warning: count(): Parameter must be an array or an object that implements Countable in D:\OSPanel\domains\Site1\auth.php on line 14

по тому что читаем доку -

fetch_assoc — Fetch the next row of a result set as an associative array

Соответственно, если запрос пустой, то фетчить нечего. Для проверки существует num_rows.

Выводит: "Такой пользователь не найден." Но в БД такой пользователь есть зарегестрированный, все данные авторизации правильные, а он выводит, что нету такого пользователя.

Программу не на*бешь. Если говорит что нету, значит нету. А если не находит то что есть, значит вы не правильно ищете, что легко проверить. Вангую что запрос будет совершенно не таким как вы ожидаете:

...
$query = "SELECT * FROM `users` WHERE `login` = '$login' AND `pass` = '$pass'";
var_dump('check: ',$login, $pass,$query); 
$result = $mysql->query($query);
...

UPD: абсолютно верно подмечено Adamos - `pass` = '$pass ' никогда ничего не найдет из-за пробела в условии.

Answer 2

[aryzhanki]

$login = filter_var(trim($_POST['login']),
FILTER_SANITIZE_STRING);

$login = filter_input(INPUT_POST, 'login');

$pass = md5($pass."ghjsfkld2345");

ИЗУЧИТЬ

а так-то видимо условие не выполнилось или редирект туда, где кука не действует или нет проверки на её наличие там

Comments

[GirlInc]

Выводит: "Такой пользователь не найден."
Но в БД такой пользователь есть зарегестрированный, все данные авторизации правильные, а он выводит, что нету такого пользователя.

<?php
$login = filter_input(INPUT_POST, 'login');
$pass = filter_var(trim($_POST['pass']),
FILTER_SANITIZE_STRING);

$pass = md5($pass."ghjsfkld2345");

    $mysql = new mysqli('localhost','root','root','automoykaBase'); 

    $result = $mysql->query("SELECT * FROM `users` WHERE `login` =
    '$login' AND `pass` = '$pass'");
    $user = $result ->fetch_assoc();
    if(($user) == 0) {
    echo "Такой пользователь не найден";
    exit();
    }

    setcookie('user', $user['name'], time() + 3600, '/');

    $mysql->close();

    header('Location: Личный-кабинет.html');
?>

[aryzhanki]

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

[aryzhanki]

GirlInc,

в БД такой пользователь есть зарегестрированный, все данные авторизации правильные

моё мнение: ты дофига чисток делаешь :) поэтому что-то не сходится

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

[GirlInc]

aryzhanki, ты имеешь ввиду убрать filter_input, filter_var? :)

[aryzhanki]

GirlInc, filter_var убрать, использовать filter_input без фильтра очистки, filter_input в отличие от filter_var ещё проверит пришла ли переменная именно из вне нужным методом - детектирует метод

[GirlInc]

aryzhanki, filter_var убрал, но пользователя при авторизации всё равно не находит.

<?php
$login = filter_input(INPUT_POST, 'login');

$pass = md5($pass."ghjsfkld2345");

    $mysql = new mysqli('localhost','root','root','automoykaBase'); 

    $result = $mysql->query("SELECT * FROM `users` WHERE `login` =
    '$login' AND `pass` = '$pass'");
    $user = $result ->fetch_assoc();
    if(($user) == 0) {
    echo "Такой пользователь не найден";
    exit();
    }

    setcookie('user', $user['name'], time() + 3600, '/');

    $mysql->close();

    header('Location: Личный-кабинет.html');
?>

[Алексей Уколов]

Попробуйте использовать волшебное заклинание: var_dump($login, $pass, $user, $user == 0); die;
Оно даст точный ответ на вопрос что там у вас не сходится.