Как вылечить mysql_fetch_array() expects parameter 1 to be resource, boolean given?

Question

[Крил]

подскажите как вылечить mysql_fetch_array() expects parameter 1 to be resource, boolean given
функция проверяет наличие поля и возвращает его

function get_post_metas($id,$key){
	$sql = @mysql_query("SELECT * FROM _postmeta WHERE post_id =". $id ." AND meta_key = '". $key ."'");
	$val = @mysql_fetch_array($sql);	
	return $val['meta_value'];
}

верно ли я понимаю что можно вылечить так

function get_post_metas($id,$key){
	$sql = @mysql_query("SELECT * FROM _postmeta WHERE post_id =". $id ." AND meta_key = '". $key ."'");
if($sql){
	$val = @mysql_fetch_array($sql);	
}
	return $val['meta_value'];
}

Просто уверен что есть другой путь решения

Answer 1

[VisualIdeas]

Коллега, а покажите кусок файла где у вас есть ранее в программе mysql_connect?
Уберите собачку дабы видеть реальные ошибки.
Ну и я бы сделал так:

function get_post_metas($id,$key){
  $res = mysql_query("SELECT * FROM _postmeta WHERE post_id =". $id ." AND meta_key = '". $key ."'");
  if (mysql_num_rows($res)!=0) {
    $val = mysql_fetch_array($res);	
    return $val['meta_value'];
  } else {
    return false;
  }
}

Или правильнее даже так так

function get_post_metas($id,$key){
  $res = mysql_query("SELECT * FROM _postmeta WHERE post_id =". $id ." AND meta_key = '". $key ."'");
  if ($val = mysql_fetch_array($res)) {
    return $val['meta_value'];
  } else {
    return false;
  }
}

Comments

[VisualIdeas]

А вообще https://habrahabr.ru/post/137664/

[Крил]

VisualIdeas: mysql_connect всё на месте и выше выполняются другие действия
тет ошибка из-за того что скорей всего SELECT * FROM _postmeta WHERE post_id =". $id ." AND meta_key = '". $key ."' возвращает 0 строк, а собачками я пытался избавиться от ошибки - в коде собачки естественно нет

[Крил]

VisualIdeas: теперь ругается на if ($val = mysql_fetch_array($res)) {

[Крил]

точно так же ругается на while($rown = mysql_fetch_array($cat)){ когда в $cat 0 строк

[Крил]

и даже в первом вашем примере ругается на if (mysql_num_rows($res)!=0) {

[VisualIdeas]

var_dump подробный после запроса (до if) что покажет? ПО идее должен быть resourse

[Крил]

сам дурак - в $id передавалось битое значение или null
ругалось на синтаксис, хоть я и не понял что было с синтаксисом не так, но теперь всё ок

Answer 2

[Александр]

Для начала неплохо бы убрать @, чтобы видеть ошибки, возможно запрос некорректный и не возвращает ресурс
mysql_fetch_array должен использоваться внутри цикла while

Comments

[VisualIdeas]

Если надо получить лишь 1 запись из БД то никакого while не нужно)

[Крил]

без собачки выдаёт тоже самое - собачкой я пытался избавиться от ошибки