Допустимы ли во VIEW условия?

Question

[Nick Murzin]

Во вью получается довольно большое количество условий(if-else). Особенно при выводе постраничной навигации.
Допустимо ли это?

Answer 1

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

А почему нет? Много if/else это в любом месте не сильно хорошо и стоит подумать о том как бы избавиться от лишних условий и вынести эту логику в компонент хелпер (судя по всему у вас так и есть).

VIEW в контексте MVC это слой конвертирующий данные из формата приложения в формат требуемый клиентом на выходе, так что без условий сложно. То есть в вашем случае трансформация доменных моделей в HTML. То есть вообще весь код который занимается такими трансформациями это view. Шаблонизатор, хелперы и т.д. Внутри этого слоя все те же правила что и для других слоев. DRY и все такое.

p.s. Приведите пример "много условий".

Comments

[FanatPHP]

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

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

@FanatPHP: я понимаю что такое пагинация и не припомню что бы мне приходилось делать массу условий для пагинации. Массив со ссылками готовить не нужно. Достаточно простых операций деления и всякие min/max что бы упростить логику.

[FanatPHP]

Сергей Протько: простая пагинация действительно выводится несложно. Но надо ведь ограничивать список выводимых страниц. А это куча проверок.

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

Nick Murzin: да... я забыл что еще так можно писать на PHP... у меня пропало желание разбираться, извините.

А по поводу ошибок из контроллера. что вы имеете в виду? Крутые ребята выкидывают из контроллера исключения которые обрабатываются во фронт-контроллере и вызывается отдельный метод который рендрит страничку с ошибкой. Или опять же какие-то хелперы типа flashMessages и т.д.

[Nick Murzin]

Сергей Протько: вот ещё про хелперы. Можете привести пример их применения и суть их как такова. Можно линком

[Nick Murzin]

Сергей Протько: Ну например: пришла из контроллера ошибка или нет, а в pagination вообще такая петрушка
pastie.org/private/yg42fpeo1qpsacgjeddl4a

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

@FanatPHP: да нет, max/min и чуть чуть упороться по математике + range для генерации массивов и у нас ни одного условия для простой пагинации) Ну как, одно условие будет - текущая ли страница или нет. Опять же хелпер для этого можно сделать. В любом случае это не проблема, в контексте вопроса ничего не поможет.

[FanatPHP]

Nick Murzin: Ну, входящие данные должен в любом случае проверять контроллер, и передавать во вью уже строго отобранные и проверенные. То есть ошибка пришедшая из контроллера - это ошибка в контроллере.

[FanatPHP]

Сергей Протько: математики много получается. И от условий она не позволяет полностью избавиться. Полоса со ссылками не имеет фиксированной длины. Чтобы определить начальное и конечное значение для range, нужны проверки.

Answer 2

[FanatPHP]

Вью - это логика отображения. Условный переход - это, несомненно, логическая конструкция. То есть, утверждение сводится к "допустимо ли использовать логику для логики?" ответ - TRUE.

Больше того, избавиться от условных переходов в шаблоне в принципе не получится. Любая попытка сведется всего лишь к переименовыванию условного перехода в анонимную конструкцию, которая будет выполнять те же функции, но при этом запутает читателя.

А вот сократить количество логики, как ваыше написали - можно. К примеру, подготовить все нужные данные заранее - либо в контроллере, либо в хелпере, а в шаблоне заниматься только выводом этих данных. Втом числе данных для пагинации. То есть, условных переходов не станет меньше, но они перейдут в тот слой, где более уместны.

Comments

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

Подготовку данных в контроллере выполнять пожалуй не стоит... иначе мы перекладываем обязанность по формированию представления частично на контроллеры. Так что мне больше импонирует вариант с хелперами.

[FanatPHP]

Сергей Протько: В принципе я согласен, что так правильнее. Вид полосы со ссылками на страницы, их количество, наличие элементов управления и так далее - это всё логика отображения, за которую отвечает дизайнер-верстальщик.