Чистые контроллеры: подготовка данных для просмотров

Question

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

1. Использование переменных экземпляра

   class CitiesController < ApplicationController 
       def index 
       @cities = Cities.order(:name).limit(10) 
       end 
   end 
   

   Это подход по умолчанию, который можно найти в Rails documentation, но она имеет некоторые недостатки :

   1. Он делает код действия жирным, который становится ответственным не только за логику контроллера, но и за подготовку данных.
   2. Мнения требуют доступа к этим данным через переменные экземпляра - эти @ -переменные нарушают принцип наименьшего удивления.

2. Использование методов хелперов

   class CitiesController < ApplicationController 
       helper_method :cities 
   
       def index 
       end 
   
       def cities 
       @cities ||= Cities.order(:name).limit(10) 
       end 
   end 
   

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

   1. Подготовка данных по-прежнему находится в контроллере. Он становится нечитаемым, когда существует много этих вспомогательных методов, особенно когда они относятся к различным действиям/представлениям.
   2. Необходимо наличие уникального имени для каждого вспомогательного метода. Скажем, у меня не может быть метода с именем products, который будет возвращать разные данные для разных действий (конечно, я могу сделать это одним способом, но это выглядело бы уродливо).

3. Использование шаблона фасада

   Частично проблема решается в этой статье: https://medium.com/p/d65b86cdb5b1 Но я не нравится этот подход, поскольку он вводит @magic_facade_object в представлениях.

4. Использование унаследованных ресурсов

   Это может выглядеть красиво в примерах, но на мой взгляд, когда дело доходит до реального кода, код контроллера становится спагетти-монстра очень быстро. Другое дело, что для просмотра страниц обычно требуется не только ресурс, но и другие данные для рендеринга (блоки боковых панелей и т. Д.), И мне все еще нужно использовать другой способ его подготовки. Объединение различных подходов делает код еще более нечитаемым. Наконец, я не люблю использовать переменную resource, потому что не совсем понятно, что такое представление.

Итак, вот в чем вопрос. Как вы держите свои контроллеры в чистоте?

Answer 1

How do you keep your controllers clean? 

Путем написания СУХОГО кода и посыпания магией драгоценных камней.

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

1. @cities = Cities.order(:name).limit(10) именно то, что я думаю, что принадлежит в контроллере рельсы, и это не нарушает принцип наименьшего удивления, это своего рода противоположность. переменные экземпляра являются стандартным способом передачи переменных из контроллеров в представления, хотя это довольно уродливая вещь. это «путь рельсов» (TM)!

2. decent_exposure занимает большую часть этих проблем

3. пожалуйста, прекратить применение старой школы шаблона рельсов или код рубина прочь. это действительно полезно в больших приложениях, где вы изо всех сил пытаетесь сохранить смысл кода, который находится в рамках одного метода контроллера. напишите чистый код, тщательно протестируйте его, и вы будете в порядке 80% времени.

4. Не используйте инструменты «один размер подходит всем». чаще всего вам нужно написать больше настроек, чем вам понадобится код, чтобы он работал. это также становится намного более сложным в таких вещах.