Почему в одном файле вызов функции с параметром работает, а в другом нет?

Question

[freeman0204]

Вот функция:

function directory_val($val) {
  $id = $_GET['id'];
$result = mysql_query("SELECT * FROM directory WHERE id = $id");
if ($val == "title") {
    while ($row = mysql_fetch_assoc($result))  {
      echo $row['title'];
      }
}

elseif ($val == "description") {
    while ($row = mysql_fetch_assoc($result))  {
      echo $row['description'];
      }
}

elseif ($val == "id") {
    while ($row = mysql_fetch_assoc($result))  {
      echo $row['id'];
      }
}
}

<?php directory_val('title'); ?>

вызываю ее в одном файле, в значении value работает, вызываю просто в другом файле выдает ошибку. Но файлы находятся в разных папках, может из за этого? Хотя и там и там есть подключение к БД и к файлу functions.php.

Warning: mysql_fetch_assoc() expects parameter 1 to be resource, boolean given in /home/freeman/domains/spilart.dp.ua/public_html/domolux/admin/functions.php on line 56

функция что выше (function directory_val($val)) как раз находится в файле functions.php.
Может это связано с путями, или с подключением к БД? Очень часто такая ошибка.

Answer 1

[Кузьмина Мария]

Скорее всего, mysql_query возвращает false.
Попробуйте написать:
$result = mysql_query("SELECT * FROM directory WHERE id = $id") or die("Invalid query: " . mysql_error());

Проверьте, что лежит в переменной $id.

Вообще, нехорошо в функции брать значение переменной из $_GET параметра, лучше сделать $id аргументом функции:

function directory_val($id, $val) {
...
}

А вызывать так:

$id = (int) $_GET['id'];
directory_val($id, 'title');

К тому же - зачем вам цикл по строкам, если у вас id уникальный?
Напишите хотя бы так:

$result = mysql_query("SELECT * FROM directory WHERE id = $id LIMIT 1") or die("Invalid query: " . mysql_error());
$row = mysql_fetch_assoc($result);
if(key_exists($val, $row)) {
     echo $row[$val];
}

Comments

[freeman0204]

Написал ваш код, выдает такое:

Warning: key_exists() expects parameter 2 to be array, boolean given in /home/freeman/domains/spilart.dp.ua/public_html/domolux/admin/functions.php on line 55
возможно тогда причина не в этом?

[Кузьмина Мария]

Артур Черешнюк: да, надо добавить еще проверку, что $row - массив:
if(is_array($row) && key_exists($val, $row)) {
echo $row[$val];
}

а дописали or die('...') после mysql_query? Никакую ошибку не выдает?

[Кузьмина Мария]

Артур Черешнюк: у вас точно в обоих случаях передается параметр $_GET['id']?

[freeman0204]

Кузьмина Мария: что то я запутался совсем) смотрите вот мой слайдер и те функции что я использую, они ниже https://jsfiddle.net/5au5bc6m/ как видите тот блок что в foreach нормально отрабатывает. А блок что ниже prntscr.com/d9xofn он связан с атрибутом data-title. Вообщем при данном раскладе, почти все хорошо кроме того что id рубрики все время 34. Хотя сами названия рубрик меняются благодаря data-title="<?php echo $value['title'] ?> а вот с id неполадки.

У меня есть аналогичный сладйер https://jsfiddle.net/ab6h55da/ и там foreach все хорошо отрабатывает. А в первом слайдере если взять весь слайдер вместе с этим блоком prntscr.com/d9xofn в foreach то верстка ломается, вот и приходится как то по другому его делать.

[freeman0204]

Кузьмина Мария: когда я убрал WHERE id = $id то ошибка исчезла.

[Кузьмина Мария]

Артур Черешнюк: Мне кажется, что вы запутались немного. Смотрите, у вас есть 2 функции, одна возвращает список строк из БД:
function our_directory() {
$result = mysql_query("SELECT * FROM directory ORDER BY id DESC") or die("Invalid query: " . mysql_error());
while ($row = mysql_fetch_assoc($result)) {
$count++; // нужно ставить перед $res_array
$res_array[$count] = $row;
}
return $res_array;
}
Эта функция тоже немного некорректная, у вас нет инициализации переменных, которые вы используете внутри функции, нужно сделать так:
function our_directory() {
$res_array = [];
$count = 0;
$result = mysql_query("SELECT * FROM directory ORDER BY id DESC") or die("Invalid query: " . mysql_error());
while ($row = mysql_fetch_assoc($result)) {
$count++; // нужно ставить перед $res_array
$res_array[$count] = $row;
}
return $res_array;
}

В функции directory_val вы хотите получить значение одного поля одной конкретной записи из БД с заданным идентификатором id.
Здесь не нужны никакие циклы:

function directory_val($id, $val) {
$result = mysql_query("SELECT * FROM `directory` WHERE id = $id LIMIT 1") or die("Invalid query: " . mysql_error());
$row = mysql_fetch_assoc($result);
if(is_array($row) && key_exists($val, $row)) {
echo $row[$val];
}
}

$id = (int) $_GET['id'];
directory_val($id, 'title');

Т.е. вы выводите на экран значение поля title записи из таблицы directory с ID, переданным в get параметре id.

Но такой вариант вообще говоря непрактичен, т.к. будет обращаться к БД каждый раз, когда вы хотите вывести значение какого-то поля записи.

Еще лучше сделать так:
function getDirectoryByID($id) {
$id = (int) $id;
$result = mysql_query("SELECT * FROM `directory` WHERE id = $id LIMIT 1") or die("Invalid query: " . mysql_error());
$row = mysql_fetch_assoc($result);
return $row;
}

$id = (int) $_GET['id'];
$directory = getDirectoryByID($id);

if(is_array($row) && !empty($row)) {
echo $directory['id'] . '
';
echo $directory['title'] . '
';
//...
}

[freeman0204]

Кузьмина Мария: Спасибо за ответы. Можете принят мой контакт в скайпе?