Пишу такой код


Должен выводить так:

Категория 1
Запись 1
Запись 2
Категория 2
Запись 1
Запись 2

и т.п., а выводит так:

Категория 1
Запись 1
Запись 1
Категория 2
Запись 1
Запись 1

В чём ошибка?

$row_in наверное надо.

А вообще, я сторонник того, чтобы не выполнять каждый раз запрос, а сначала все загрузить в единый массив, а потом его мучить.

Mal Hack
Точно, спасибо.

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

МНе мож удобно, как в Асме с каждым регистром работать...

Твой подход не рационален с точки зрения времени работы и ресурсоемкости.

По-моему, здесь лучше и гораздо рациональнее будет пробежаться рекурсивно.

Mal Hack

Глупости. Наоборот такой подход будет работать быстрее, если база (как в 99% случаев) расположена на том же хосте, что и скрипты.

Строка 14:
хотя по логике должно быть

ИМХО
такой код будет работать побыстрее, чем твой при большом количестве записей...

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

Bikutoru
Неверное понимание. Эти несколько тысяч запросов будут работать во много раз быстрее чем несколько тысяч array_filter-ов, которые придется применять при нормальной организации базы при выводе дерева элементов. П.С. Фраза моя была ответом на фразу

А организация базы, как у товарища Куксикус, где каждой у записи в поле razdel вместо id раздела написано его имя(!!) это дурдом! О чем ему и стоит задуматься.

Да? Ты бы для начала хоть тестирование провел...
Нет, я не спорю, что на каких-то задачах выгоднее сделать много маленьких запросов, чем отом шерстить один большой. Но таких задач меньше, точнее они редко ставятся.

А где ты там видел, чтобы я упоминал про фильтры?

Запросы к базе это внешние операции, которые в большинстве случаев будут выполняться намного дольше, чем внутрескриптовые конструкции.

Модератор: за хамский тон по отношению к участникам в следующий раз буду кирпич вешать.

С чего ты взял что у меня нет id ?

Mal Hack

Чтобы сначала всё засунуть в переменную а потом это выводить за циклом нужен ещё один цикл для вывода или я неправильно тебя понял? Привиди пожалуйста пример
Добавлено @ 12:52
Кстати насчёт кирпичей... а куда мой пропал? нет я конечно не против но просто интересно

Ты должен делать что-то вроде:

Там где .... Надо сделать условие, для вывода инфы от нужной категории.. Тут ИМХО if надо будет еще юзать.

Так исходя из
Который должен отображать

"Категория 1" на id не очень похожа Если бы был id выводилось бы соответственно просто "1", верно?



О, поверь. Я с этим намахался много в свое время. И тестирование проводил еще как.


Про большинство случаев - совершенно верно.
Но при случае фильтрования (выбрать записи которые входят в раздел) то обращение к базе ("select * ... where razdel_id=....") будет куда быстрее чем выбирать эти записи "внутрескриптовыми конструкциями" из массива всех записей, считаных предварительно. Именно то что предложил ты.


Твой код + как ты говришь если if заюзать - вот это и будет работать медленее. И чем больше будет записей тем больше будет разница. (Повторюсь - при условии того что mysql база находится на localhost по отношению к скриптам)

smartov спорить не буду, у меня тесты показывали другое.
Да и один запрос к БД всегда лучше, чем много маленьких (в большинстве, опять же)

smartov
А ты не думал что поле в котором хранятся id может называться id а не раздел и переменная $row['razdel'] выводит название раздела а не id или ты во всех своих скриптах вместо текста предпочитаеш одни id'шники выводить?

Mal Hack
Т.е. я должен всё забить в массив а потом этот массив вывести, так?

Ну тогда советую тебе подумать зачем в записях хранить еще и имя раздела если там уже хранится id раздела к которому эта запись принадлежит. Толку от такого дублирования информации?


Расскажу заодно как у меня было.
Надо было мне значит вывести дерево. Вот почти как у гр. Куксикус только 4-й вложенности. Пункстов 4-го уровня было около 4000 и увеличивалось с каждым днём. Когда я делал так
1) выбирал все пункты 4 уровня в массив в начале скрипта
2) выводил в цкиле, опираясь на if, только те, которые сейчас надо (не запутал надеюсь %))
Т.е. дерево было:

Так вот как я при варианте, как я написал, скрипт выполнялся около 10(!) секунд.
Когда же я сменил все циклы с if-ами на mysql_query(".... where parent_id=...") скрипт стал выполняться 2.5 секунды. Вот такие показатели.

Это сообщение отредактировал(а) smartov - 7.2.2006, 12:26