Сделал блог с учетом всех требований из 1 части. Имеется еще ряд ц.у.
- Для форматирования использовать 2 пробела, а не табуляцию
- Максимально ни использовать наследование. У меня в примере 3 контроллера вызывали один набор фильтров. Для избавления от дублирования, я создал для них базовый контроллер, в котором вызывал общие фильтры. Вместо этого следовало отказаться от наследования, вынести общие фильтры в один фильтр более высокого уровня и его вызывать во всех контроллерах.
- Вместо методов ресурса использовать параметры запроса. Поясню. Есть Топик с стандартным набором КРУД. Требуется добавить возможность закрыть\открыть топик. Я добавил вложенный ресурс Состояние к Топику и два метода create( открыть топик) и destroy(закрыть топик). Нужно делать по другому: добавить один метод update и передавать параметром запроса открыть топик или закрыть
- Использовать для методов ЧЕЛОВЕЧЕСКИЕ имена
- Вызов пагинатора в конторллере вынести в метод модели
- Если в вызове метода один аргумент- скобки не пишуться
- Если в строке нет подстановки- использовать одинарные ковычки
- В файлах разбивать группы- асоциации и валидации
- Использовать partial
- Добавить тесты
No comments:
Post a Comment