Почему mysql_query отдаёт только одно значение?

Question

[jenya7771]

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

$resultVendor = array_unique($arrayRow);

$resultVendor = array_values($resultVendor);

$countRes = count($resultVendor);

echo $countRes;

for ($i = 0; $i < $countRes; $i++) {
	$randVendor = $resultVendor[$i];
	$queryRandom = "SELECT id FROM `pudra` WHERE `vendor` = $randVendor ORDER BY RAND() LIMIT 1";
	$resRandom = mysql_query($queryRandom);

	$arrayRowrand = array();

	while($rowRandom = mysql_fetch_array($resRandom))
	{
		array_push($arrayRowrand, $rowRandom['id']);
		echo $rowRandom['id']."<br>";
		$liknId = NAMELINK."/?id=".$rowRandom['id'];
	};

	echo "<a href=".$liknId.">".$resultVendor[$i]."</a><br>";
};

Кроме того что отдаётся только одна Id-шка, ещё и предупреждение вылазит на каждый запрос я так понимаю.
Warning: mysql_fetch_array() expects parameter 1 to be resource, boolean given in /home/sanerix/domains/sanerix.tk/public_html/serverhome.php on line 49
49 строка это
while($rowRandom = mysql_fetch_array($resRandom))

Answer 1

[Ne-Lexa]

Тот код что ниже отдаёт только один id

LIMIT 1 как бы намекает, что вернётся только 1 запись.

Warning: mysql_fetch_array() expects parameter 1 to be resource, boolean given in /home/sanerix/domains/sanerix.tk/public_html/serverhome.php on line 49

Значит $resRandom возвращает false.
Проверьте на ошибку в запросе, при помощи mysql_error().

P.S.: Перейдите на расширение mysqli или PDO для работы с БД. Расширение mysql является устаревшим.

Comments

[jenya7771]

Там же цикл, каждый раз должен быть ответ с разным id для определённого производите, я так думал, я не прав?

[Ne-Lexa]

Убирайте цикл, делайте всём 1 запросом в базу.

$sql = "
SELECT id, vendor 
FROM `pudra` 
GROUP BY vendor
ORDER BY RAND()
";
$res = mysql_query($sql);
if($res){
    while($data = mysql_fetch_assoc($res)){
         echo $data['id'] . "<br>".PHP_EOL;
         echo "<a href=".NAMELINK."/?id=".$data['id'].">".htmlspecialchars($data['vendor'])."</a><br>".PHP_EOL;
    }
    mysql_free_result($res);
}

[jenya7771]

NeLexa: Спасибо, очень помогло. Для меня удивительно как мало код и всё отлично работает. Можете объяснить как оно работает?

[jenya7771]

Можно ли это записать в переменную что бы вывести в html

[Ne-Lexa]

> Можно ли это записать в переменную что бы вывести в html
Не просто можно, а нужно! Пользуйтесь шаблонизаторами, например Smarty или Twig и НИКОГДА не мешайте бизнес-логику с выводом. Прочитайте про паттерн MVC, он поможет решить вам эту кашу в коде.

> Можете объяснить как оно работает?
Мы посылаем запрос в БД на выборку случайного id и группируем записи по vendor, поэтому он будет уникальным.

[jenya7771]

NeLexa: Только я сейчас заметил что меняется местами vendor, а id остаётся той же, нужна случайная id

[Ne-Lexa]

Тогда попробуйте так:

SELECT vendor,
(SELECT
id
FROM pudra t2
WHERE t1.vendor = t2.vendor
ORDER BY RAND()
LIMIT 1
) AS id
FROM pudra t1
GROUP BY vendor

[jenya7771]

NeLexa: Спасибо всё работает

Answer 2

[romy4]

как бэ
> LIMIT 1

> Warning: mysql_fetch_array() expects parameter 1 to be resource, boolean given in /home/sanerix/domains/sanerix.tk/public_html/serverhome.php on line 49
всегда проверяйте ресурс не равен ли он !== false перед тем как отдать fetch_*

Answer 3

[Дмитрий]

Добрый день.
Возможно надо $rowRandom = mysql_fetch_array($resRandom) убрать из цикла, а в цикле обрабатывать просто $rowRandom

$rowRandom = mysql_fetch_array($resRandom);
while($rowRandom as $value){
   // остальной код
}

Comments

[jenya7771]

Ошибка в на этой строчкеwhile($rowRandom as $value){