Объясните пожалуйста, что за ошибка в функции?

Question

[animr]

Объясните пожалуйста, что за ошибка:

mysqli_fetch_assoc() expects parameter 1 to be mysqli_result, boolean given in /db_fns.php on line 226?

вот код самой функции:

function get_products($ids, $start_pos, $perpage){
	global $connection;
	if($ids){
		$query = "SELECT * FROM sdvd_products WHERE parent IN($ids) LIMIT $start_pos, $perpage";
	} else {
		$query = "SELECT * FROM sdvd_products LIMIT $start_pos, $perpage";
	}
	$res = mysqli_query($connection, $query);
	$products = array();
	while($row = mysqli_fetch_assoc($res)){       /*    226 строка это   */
		$products[] = $row;
	}
	return $products;
}

Answer 1

[Александр]

Покажите как вы формируете $ids перед вызовом функции.
Должна быть строка/число.
Поле parent в таблице какого типа? Если int(и его варианты), то строка, где искомые значения разделены запятыми без каких-либо ковычек. Если varchar/text - то строка, где искомые значения разделены запятыми и обрамлены в одинарные кавычки.

Comments

[animr]

я там получаю id дочерных категорий меню, чтоб сформировать хлебные крошки

$ids = cats_id($categories, $id);
$ids = !$ids ? $id : rtrim($ids, ",");

[animr]

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

[Александр]

@animr: var_dump($ids) перед формированием запроса или перед вызовом get_products. Ну и надо знать как формирует возвращаемое значение cats_id()

[Александр]

@animr: ещё попробуйте сделать var_dump($connection); сразу после подключения к базе, дабы исключить проблемы с подключением к БД.

[animr]

я перенес сайт на новый домен, на старом все работало. все проверю напишу о результатах

Answer 2

[litvinoffsky]

WHERE parent IN($ids)
Что это?
ids - это массив или строка?
Скорее всего именно там беда.
var_dump($query) - вам поможет.

Comments

[red_led]

Если это массив, то будет warning "array to string conversion"

[ugodrus]

Абсолютно согласен. Скорее всего MySQL вернул ошибку. Скорее всего как раз в этом месте ошибка. Передавайте в функцию $ids в виде массива и там уже проверяйте на длину массива а для формирования строкового $ids используйте implode().

[animr]

Массив в строку

[animr]

var_dump($query) показывает NUll

[red_led]

@animr: Вы его в начале функции написали или перед mysqli_query?

Answer 3

[Сергей Мельников]

function get_products($ids, $start_pos, $perpage) {
  global $connection;
  $products = array();
  $query = "SELECT * FROM sdvd_products LIMIT $start_pos, $perpage";
  
  if($ids) {
    $query = "SELECT * FROM sdvd_products WHERE parent IN($ids) LIMIT $start_pos, $perpage";
  }

  if ($res = mysqli_query($connection, $query)) {
    while ( $row = mysqli_fetch_assoc($res)) {
      $products[] = $row;
    }
  }	

  return $products;
}

Comments

[FanatPHP]

Абсолютно бессмысленный говнокод, true похапе way. Решаем строго поставленную задачу, "не допустить появления ошибки". Мысль о том, что ошибки надо не прятать под ковер, а ВЫЯВЛЯТЬ и ИСПРАВЛЯТЬ, даже не приходит в похапешную голову.

Answer 4

[Сергей Протько]

При выполнении mysqli_query произошла ошибка. $res содержит не курсор на результат запроса а false.

Comments

[animr]

а как его заставить вернуть результат запроса? в чем ошибка функции?

[animr]

дело в том, что я его с другого домена перенес, на старом все работало

[red_led]

@animr: попробуйте добавить var_dump(mysqli_error($connection)) сразу после mysqli_query(). Увидите полное текстовое описание ошибки от mysql

Answer 5

[red_led]

mysqli_query() возвращает FALSE в случае неудачи. А mysqli_fetch_assoc() хочет объект mysqli_result, а не FALSE.

Comments

[red_led]

Возможно, ошибка в самом запросе, или коннекте к базе. Тут уже надо смотреть в дебаге или vardump-ами

[animr]

так я его перенес с другого домена, на старом домене все работало

[iLexx_13]

@animr: если на старом домене работало - скорее всего не смог подключиться к MySQL. Проверьте login/pass/host/database при подключении

Answer 6

[FanatPHP]

Это означает, что ошибка была в запросе.
По умолчанию mysqli не сигнализирует об ошибках в mysqli_query(). Чтобы заставить её делать это, надо перед коннектом написать волшебное слово:

mysqli_report(MYSQLI_REPORT_ERROR | MYSQLI_REPORT_STRICT);

Это самый простой, но менее удобный способ, поскольку в сообщении об ошибке нам желательно сразу увидеть и запрос с ошибкой. Поэтому лушче проверять результат запроса, и порождать ошибку РНР с ошибкой mysql и запросом.

Ошибка в запросе может быть вызвана чем угодно. В частности - мусором, переданным в запрос. Именно поэтому данные для запроса должны форматироваться только перед непосредственным выполнением запроса, а не где-то ещё!

Поэтому вместо того, чтобы продолжать писать на mysqli ТОТ ЖЕ САМЫЙ УРОДЛИВЫЙ ГОВНОКОДО, что и на mysql, надо использовать более интеллектуальный подход, который будет обеспечивать как форматирование, так и обработку ошибок:

function get_products($db, $ids, $start_pos, $perpage){
    if($ids){
        $query = "SELECT * FROM sdvd_products WHERE parent IN($a) LIMIT ?i, ?i";
    } else {
        $query = "SELECT * FROM sdvd_products LIMIT ?i, ?i";
    }
    return $db->getAll($query, $ids, $start_pos, $perpage);
}