Почему не работает функция?

Question

[Евгений Иванов]

Код прекрасно работает "вне функции".

$select_result=mysql_query("SELECT `api_key` FROM `users` WHERE `id`='$all_about_user_array[id]' LIMIT 1");
$row=mysql_fetch_assoc($select_result);
$api_key=$row['api_key'];
echo"$api_key";

Но оформленный в виде функции - нет.

function user_api_key()
{
$select_result=mysql_query("SELECT `api_key` FROM `users` WHERE `id`='$all_about_user_array[id]' LIMIT 1");
$row=mysql_fetch_assoc($select_result);
$api_key=$row['api_key'];
return $api_key;
}
echo user_api_key();

Почему не работает функция?

Answer 1

[Станислав Б]

$all_about_user_array[id]
потерялся

а вообще ваш код работает далеко не прекрасно.
на дворе 2018 год.
функции mysql_* устарели начиная с версии 5.5 (а ей уже больше 4х лет)
и что за id? константа?

Answer 2

[riot26]

$all_about_user_array['id'] откуда в функции взяться? Нужно передавать

function user_api_key($user_id)
{
    $select_result=mysql_query("SELECT `api_key` FROM `users` WHERE `id`='$user_id' LIMIT 1");
    $row=mysql_fetch_assoc($select_result);
    $api_key=$row['api_key'];
    return $api_key;
}
echo user_api_key($all_about_user_array['id']);

P.S. Mysql-расширение для работы с БД устарело и использовать его нельзя. Подробнее в красной плашке здесь

Comments

[Дмитрий Энтелис]

хоть бы escape какой вставили...
он же так и будет такой чудо код фигачить...

Answer 3

[Антон Иванов]

Поясните, пожалуйста, что именно не работает. Не выводит на экран api_key?
Вместо echo сделайте так
var_dump(user_api_key());
Что дал вывод?

Comments

[Евгений Иванов]

null

[Евгений Иванов]

кажется разобрался. область видимости не та.

Answer 4

[Дмитрий Энтелис]

1. учимся дебажить.
если что-то не работает, нужно по шагам проверить все.
что передается в каждую функцию.
что возвращается из каждой функции.

2. перестаем использовать mysql_, переходим на mysqli_ / pdo

3. обновляем версию PHP до актуальной

PS
ответ на вопрос - потому что внутри функции не определен $all_about_user_array['id'].
его нужно туда передать агргументом

PPS
Вместо

mysql_query("SELECT `api_key` FROM `users` WHERE `id`='$all_about_user_array[id]' LIMIT 1");

пишите хотя бы

mysql_query('SELECT `api_key` FROM `users` WHERE `id` = ' . (int)$all_about_user_array[id] . ' LIMIT 1');

Comments

[Александр]

Раз учите, то учите сразу через mysqli_prepare. :)

[Евгений Иванов]

пишите хотя бы (int)$all_about_user_array[id]
а смысл? Вы приводите значение к числу и вставляете его в строку.
Запрос - строка. Какая разница что там в элементе массива. Это будет строкой.

[Евгений Иванов]

run through
И? Вы это кому говорите?
Запрос строка. Очевидно. Выше мне предлагают привести переменную к типу int. А потом объединить её со строкой.
Зачем - спросите у человека выше. Не мой код.
А в конечном итоге, мы всё равно получим строку - т.к. запрос есть строка.

Мне оно надо - приводить переменную к типу инт, а потом объединять её со строкой - что приведет эту переменную к типу string? Нет. Ну я это и не делаю.

[imhuman]

Евгений Иванов, вопрос безопасности, а не приведения типов, вам же написали в этом ключе

[Станислав Б]

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

[Евгений Иванов]

Вопрос безопасности и возможности SQL инъекций решается отдельно, а никак не на этапе запроса. И явно не приведением переменной к типу int.
//Иконка как бы намекает )

[Станислав Б]

Евгений Иванов, Если договоритесь с хостером поставить иконку на сервер, то тогда да, на инъекции можно не обращать внимание

[Дмитрий Энтелис]

Евгений Иванов, Евгений, без обид - но вы задаете вопрос не того уровня сложности что бы спорить.
Вопрос защиты от SQL инъекций ДОЛЖЕН решаться как раз на уровне кода модели.
Если где-то подразумевается число - лучше делать (int).
Если где-то подразумевается срока - нужно делать хотя бы mysql_real_escape_string.
Ну и про PDO выше написали.