Почему mysql выдает такую ошибку?

Question

[Kejin]

Вставил код, теперь mysql запрос выдает ошибку, что в 135 строке что то не так.
Ошибка Warning: mysql_num_rows() expects parameter 1 to be resource, boolean given in R:\home\ProectMagazine\www\view_content.php on line 135
Вот собственно код из этого файла

while ($row1 = mysql_fetch_array($result1));


 $result = mysql_query("SELECT * FROM uploads_images WHERE products_id='$id'",$link);

If (mysql_num_rows($result) > 0) <- 135 строка

{
$row = mysql_fetch_array($result);
echo '<div id="block-img-slide">
      <ul>';
do
{
    
$img_path = './uploads_images/'.$row["image"];
$max_width = 70; 
$max_height = 70; 
 list($width, $height) = getimagesize($img_path); 
$ratioh = $max_height/$height; 
$ratiow = $max_width/$width; 
$ratio = min($ratioh, $ratiow); 

$width = intval($ratio*$width); 
$height = intval($ratio*$height);    
    
    
echo '
<li>
<a class="image-modal" href="#image'.$row["id"].'"><img src="'.$img_path.'" width="'.$width.'" height="'.$height.'" /></a>
</li>
<a style="display:none;" class="image-modal" rel="group" id="image'.$row["id"].'" ><img  src="./uploads_images/'.$row["image"].'" /></a>
';
}

Comments

[Максим Тимофеев]

Зачем столько кода и почему не указали какая из этих строк 135-я?

Answer 1

[Arman]

если так сделать?

//...
$result = mysql_query("SELECT * FROM uploads_images WHERE products_id='$id'",$link);
if($result === false) {
    die(mysql_error($link));
}

Comments

[Kejin]

Спасибо, нашел ошибку

Answer 2

[Антон Антон]

php.net/mysql-query

For SELECT, SHOW, DESCRIBE, EXPLAIN and other statements returning resultset, mysql_query() returns a resource on success, or FALSE on error.

В данном случае очень подозрительно ='$id'
во первых - это может быть путь к sql injection, во вторых - подозрительные кавычки. попробуйте вывести фактический запрос и выполнить его в phpmyadmin.

Ну и mysql - deprecated, используйте PDO/mysqli. И не используйте запросы в цикле.

Comments

[Kejin]

Если не сложно, как в phpmyadmin вывести запрос?

[Антон Антон]

Kejin: вывести через echo на страницу, потом скопировать в phpmyadmin

Answer 3

[nozzy]

У тебя что-то не так с запросом,
попробуй:

$result = mysql_query("SELECT * FROM uploads_images WHERE products_id='$id'",$link) or die(mysql_error());

Answer 4

[Василий Назаров]

1. Никогда больше не используйте mysql_...()
Эта либа запрещена к использованию и не работает в PHP7.

2. Не используйте запросы в цикле, делайте 1 запрос с условием "WHERE IN ()", потом уже перебирайте все полученные записи в цикле.

3. Используйте echo mysql_error(), чтобы определить, в чём ошибка.

4. Скорее всего (судя по коду) у вас на момент вызова
$result = mysql_query("SELECT * FROM uploads_images WHERE products_id='$id'",$link);
не установлено значение '$id

Comments

[nozzy]

А где здесь запросы в цикле?

[Антон Антон]

nozzy: в данном случае, если внимательно приглядеться, действительно, нет запросов в цикле. Но такой способ получения одной (последней?) строки предыдущего запроса все равно говнокод.

Answer 5

[Николай Чуприк]

Я конечно php-самоучка одиночка в личных целях, не знаю, как делают профессионалы. В таких ситуациях я для отладки в явном виде вывожу сформированную строку запроса на экран, копи-пастом отправляю её в phpmyadmin, и смотрю, что получится. Обычно после этого всё становится ясно.

Comments

[nozzy]

В данном случае запрос может правильным и он отработает в phpmyadmin, а проблема в $link - неправильный connection к базе.

[Николай Чуприк]

nozzy: Это да. Я пользуюсь одной базой в пределах одного скрипта, поэтому $link опускаю.