Функция не принимает значение из переменной вне функции, почему?

Question

[podushka]

Всем здравствуйте)
Есть у меня 2 файла
file1.php:

<?php
session_start();
include('../sql.php');
$user_uid = $_SESSION['auth']['uid'];
$query = mysqli_query($connection, "SELECT * FROM users WHERE `uid` = '$user_uid'");
$result = mysqli_fetch_assoc($query);

function check_user_role($max_role) {
    if($result['role'] > $max_role) {
        return 0;
    } else {
        return 1;
    }
}
?>

file2.php:

<?php
include('../check_role.php');
include('../../sql.php');

echo check_user_role(3);
?>

суть в том, что file2 обращается к функции в file1, которая определяет роль пользователя и возвращает 1, если роль соответствует требованию, и 0 если нет. Но проблема в том, что функция принимает только $max_role, я пытался вывести значение базы данных внутри функции, но они ничего не возвращала. как можно решить данную проблему?
Видел решение с передачей уже ответа от бд в функцию:
file2.php:

$user_uid = $_SESSION['auth']['uid'];
$query = mysqli_query($connection, "SELECT * FROM users WHERE `uid` = '$user_uid'");
$result = mysqli_fetch_assoc($query);
echo check_user_role(3, $result);

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

Answer 1

[AUser0]

А где у вас в коде include("file1.php"); ?
Зачем подгружаете sql.php в каждом файле? Есть же include_once().
Ну и наконец: $user_uid, $query, $result должны по логике находиться или внутри функции, или в файле file2.php

Comments

[podushka]

файл называется вообще по другому, подключается в file2:
include('../check_role.php');
я также пробовал вставлять запрос в бд прям внутрь функции, но это тоже не работало. а запрос в file2 вставлять не особо хочется, потому что функция вызывается в многих файлах, поэтому лучше сразу в file1 запрос делать (если конечно так вообще можно)

[AUser0]

Пробуйте так:

function check_user_role($sql_conn, $max_role) {
    $user_uid = mysqli_escape_string($sql_conn, $_SESSION['auth']['uid']);
    $query = mysqli_query($sql_conn, "SELECT * FROM users WHERE `uid` = '{$user_uid}'");
    $result = mysqli_fetch_assoc($query);
    if($result['role'] > $max_role) {
        return 0;
    } else {
        return 1;
    }
}

echo check_user_role($connection, 3);

Разумеется $connection должно уже существовать и быть сами понимаете чем.

[podushka]

AUser0, спасибо, позже попробую и напишу результат)

[AUser0]

podushka, и загляните в errorlog файл вашего HTTP сервера, увидите мнооооого ошибок из PHP, которых быть категорически не должно!

[podushka]

AUser0, огромное спасибо, это действительно работает!))

Answer 2

[ThunderCat]

Учим основы: Область видимости переменных в php

Comments

[podushka]

спасибо, $GLOBALS действительно работает, только как-то странно. пишу значит так:

$test = 123;

function check_user_role($max_role) {
    return $GLOBALS['test'];
}

то функция возвращает значение переменной, но если я сделаю так:

$test = $result['role'];

function check_user_role($max_role) {
    return $GLOBALS['test'];
}

то функция уже ничего не возвращает(

[ThunderCat]

podushka, во первых использовать $GLOBALS плохая практика, во вторых все будет работать одинаково в одинаковых условиях (очевидно что код не приведен полностью, пруф, в частности по этому работа с глобальными переменными считается плохой практикой).

Логичнее не издеваться над кодом, а передать вторым аргументом $result['role'].
function check_user_role($max_role, $role) {...

[Дмитрий]

echo min($maxRole, $resultRole);