- 1
- 2
- 3
- 4
- 5
- 6
- 7
- 8
- 9
$query = "SELECT `parent_id` FROM `items` WHERE `id`='".$_GET['id']."'";
$result = mysql_query($query);
$result = mysql_fetch_assoc($result);
$parent_id = $result[0]['parent_id']
$query = "SELECT `id` FROM `catalog` WHERE `id`='".$parent_id."'";
$result = mysql_query($query);
$result = mysql_fetch_assoc($result);
$catalog_id = $result[0]['id']
Видимо автор читает через строчку, или скорее не дочитывает до конца строки, и ему показалось что тут 2 одинаовых блока по выборке.
Знаешь пословицу?
Не бывает двух одинаковых свиней: где-то жира чуть больше, где-то чуть меньше.
Единственное, это можно было написать одним запросом.
Господа, а вам не приходило в голову, что parent_id и id - это одно и то же?
тут два id: items.id и catalog.id.
какое из них ты имеешь в виду?
Согласен, код просто писал человек, который с SQL почти незнаком. Поэтому особого говнокодинга я лично не заметил. IMHO, напрашивается JOIN:
#8
Второй запрос не нужен. Посмотрите на WHERE и какое значение хотят получить. Т.е., получаемое catalog.id равно переменной $parent_id (если, конечно, id уникальное)
Сори, опечатался. Запрос, а не цикл ))
Всё правильно, проблема в том что последнй запрос возвращает то что итак сидит в $parent_id.
Понял. Возможно, вторым запросом проверяется существование записи в таблице catalog (можно заменить на COUNT)