Проверка логина и пароля для входа на сайт

Что значит не работает - он делает ровно то что вы написали

Опишите пожалуйста, что, по-вашему, он должен делать?

 

PHP:

1. <?php
2.  
3. $_POST['login'] = 'Vova';
4. $_POST['password'] = '1111';
5.  
6. if (isset($_POST["login"]) && ($_POST["password"])){
7.     echo "thanks for data!";
8.     user_data_check($_POST["login"], $_POST["password"]);
9. }
10.  
11. function user_data_check($login, $password){
12.     $arr= ["Rita"=> "5050", "Nick" => "6666", "Vova" => "999"];
13.     foreach ($arr as $key => $val){
14.         if ($key == $login){
15.             echo $key . "проверка пройдена";;
16.         }
17.     }
18. }

Вы не поверите, но у меня ваш код прекрасно работает. Это выведет на экран

Код (Text):

1. thanks for data!Vovaпроверка пройдена

 

PHP:

1. <?php
2.  
3. function user_data_check( $LOGIN, $PASS )
4. {
5.    $arr =
6.    [
7.      'Rita' => "5050",
8.      'Nick' => "6666",
9.      'Vova' => "999"
10.    ];
11.  
12.    if ( !in_array ( $PASS, $arr ) )
13.    {
14.      echo 'Passworld error';
15.    }
16.    elseif ( !isset ( $arr[$LOGIN] ) )
17.    {
18.      echo 'Login error';
19.    }
20.    else
21.    {
22.      echo "{$LOGIN}проверка пройдена";
23.    }
24. }
25.  
26. if ( !empty ( $_POST['login'] ) && !empty ( $_POST['passworld'] ) )
27. {
28.    echo 'thanks for data!<br>' . PHP_EOL;
29.    user_data_check ( $_POST['login'], $_POST['passworld'] );
30. }
31. else
32. {
33.    header ( 'location: /buy.php' );
34. }

--- Добавлено ---

Спойлер: ...

Если просто тупо скопировать, то на врятли пойдет вам это на пользу.

 

Рано или поздно поймете, что ни черта код не работает (с)

 

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

Сейчас, у вас функция user_data_check что-то проверяет и выводит данные в html, по-сути на экран пользователю
Если бы ваша функция вместо того чтобы что-то выводить на экран просто возвращала результат авторизации (true или false), а вот вызывающий код уже решал что делать то было бы всё намного симпатичнее.

Это несложно для понимания (сложнее описать этот процесс словами), но если вы освоите этот приём, ваш код будет на порядок "чище" и "изящнее"

Второй момент: вы в цикле проходите по всем пользователям и ищете соответствие. Если у вас немного пользователей, скажем порядка 100 или 1000, то такой подход вполне имеет место быть. Но если пользователей становится больше, то лучше обращаться к вашем ассоциативному массиву напрямую.

Код (PHP):

1. <?php
2. if (isset($arr[$login]) === false) {
3.     // пользователь не найден
4.     return false;
5. }
6.  
7. $pwd = $arr[$login];
8. if ($pwd !== $password) {
9.     // пароль не подходит
10.     return false;
11. }
12.  
13. // Авторизация прошла успешно
14. return true;

 

конечно