Название переменной $tmp, как по мне, не очень. Я бы рекомендовал все переменные изначально именовать по их прямому смыслу, это позволит избежать их некореектного предназначения. Иногода даже правилньно бывает потратить минут 5-10 на обоснованное именование переменной, понять, что она не нужна и затем использовать другое решение!
Отлично, Степан. По поводу закомментированного кода: большое-большое спасибо, разработка его нужна. Ну а по использованию: здесь уже смотря у кого какие цели. Если есть смысла при пустом фильтре выдать на-гора всё, что есть в базе, то решение как на уроке. Если показать, что при пустом фильтре пустая выборка - то надо использовать Ваше решение (Кстати, а Вы его проверяли? При bulletins==null в представлении ошибки не выпадает?)
Вячеслав, здравствуйте, я Валерий Жданов, автор курса "Доска объявлений". Такое некорректное наименование переменных - это скорее моя вина (Степан сделал как было в моем видео). В общем-то Вы правы, переменные лучше именовать по прямому назначению. Но в данном случае почему я отказался от этой практики, - чтобы не увязнуть в "классификации" функций, методов, свойств и предназначений, из которых будет название функции (какое-нибудь bulletins_themes_add_index_view_bullselect_tmp... и попробуй ошибись в названии). Вы говорите, что даже правильно бывает потратить минут 5-10 на обоснованное именование переменной. В уроках и учебных проектах я считаю это даже вредным, так как отвлекает от запоминания темы и отработки навыков, которым данный урок и посвящен.
Да и понимать, что она не нужна, и использовать другое решение... на уроках в большинстве случаев нам надо не искать обходные пути решений (пусть даже и оптимальные), а рассмотреть стандартные (чтобы изучить базу, и обучающемуся было от чего оттолкнуться).
Вот из расчёта этих моментов в данном курсе я отказался от траты времени на "именование по прямому смыслу". А по поводу реальных проектов Вы правы, там это необходимо
Да, Валерий, всё работает и ошибок не выдает, я просто закоментировал код потому, что так было удобнее выполнять задание за Вами, а не скакать между вкладками, теперь даже не знаю что лучше... Пока так оставлю