Почему не работает скрипт?

Question

[0ldn0mad]

Добрый день.
Тренируюсь в написании скрипта для проверки пароля.
В форме пишу хоть правильный пароль, хоть не правильный, скрипт все равно выводит "Доступ открыт". Что не так?

<?php
$password = 'd8578edf8458ce06fbc5bb76a58c5ca4';
	if (!empty($_REQUEST['passwd']) && md5($_REQUEST['passwd'] == $password)) {
		echo "Доступ открыт";
		
	}else{
		if (!empty($_REQUEST['passwd']) && md5($_REQUEST['passwd'] != $password)){
			echo "Нет доступа";
		}
	}
?>

Comments

[SagePtr]

А если empty($_REQUEST['passwd']) - то никакое условие у вас не сработает

[Довольный Айтишникъ]

И вообще, на вариант нет доступа после else как мне кажется вообще if не нужен. Если первую. проверку не прошел то смысл проверять что либо еще?

[0ldn0mad]

Довольный Жизнью,
Это реакция на введеный, но не правильно введеный пароль

[Довольный Айтишникъ]

0ldn0mad, понимаю, но в первом условии вы выяснили что пароль неверный и если нет задачи выяснять вводил ли пользователь пароль или отправил пустое поле, то второй if смысла не имеет

Answer 1

[Михаил Бобков]

md5($_REQUEST['passwd'] == $password) => md5($_REQUEST['passwd']) == $password

Иначе в условие вставляется результат md5 (дает true)

Comments

[0ldn0mad]

Спасибо огромное, заработало!

Answer 2

[SagePtr]

Рекомендую сразу учиться пользоваться нормальными функциями, вместо дырявой MD5:
php.net/manual/ru/function.password-hash.php
php.net/manual/ru/function.password-verify.php

Comments

[0ldn0mad]

Подскажите, пожалуйста, исключительно в моем примере как применить данные функции?

[SagePtr]

0ldn0mad, сначала посчитать хэш пароля функцией password_hash (считать можно на любом компьютере, не обязательно на сервере).
А потом также в программе подставить его в переменную $password, но проверять как-то так:
!empty($_REQUEST['passwd']) && password_verify($_REQUEST['passwd'], $password)
В реальности конечно же пароли пользователей будут лежать не в коде скрипта в переменной, а в базе данных, к примеру (у каждого юзера свой пароль).
Или работу с суперглобальным массивом напрямую вообще убрать:
if(password_verify(filter_input(INPUT_POST, 'passwd'), $password) { ... доступ разрешён ... };