Rails: оставляя много кода на виду

Question

Я пишу представление, в котором присутствует некоторая логика. Я мог бы поместить их в модель или контроллер. Но мой вопрос: это плохая практика, оставляющая столько логики в поле зрения? (код рубина) Пожалуйста, смотрите мой код здесь

http://pastie.org/3440454

Answer 1

ДА, вы должны указать минимальный код для просмотра и максимальный код для модели

Мне нравится эта философия тощих контроллеров и толстых моделей: http://weblog.jamisbuck.org/2006/10/18/skinny-controller-fat-model

Answer 2

Да, это плохая практика.Нечто вроде

TestFieldDefinition.find_all_by_company_id(...)

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