Ответственность за нарушения правил
Пусть те несут, кто их составил
Пусть те несут, кто их составил
Ходил на одну фирму показывать тестовое задание на РоР. Получил массу ц.у.
- Запятая идет сразу после слова.
- Единый стиль выравнивания в коде
- Максимально использовать Рест архитектуру
- Simple responsibility rool -- один контроллер несет минимальное количество действий
- Т.к. действия в контроллерах открыты по умолчанию, то все методы, которые не являются действиями делать приватными
- Названия методов:
- если метод возвращает Boolean => заканчивать название ? (editable?)
- не использовать get_, set_, is_ в названиях
- Контроллер должен быть максимально легким за счет утяжеления модели
- Максимально использовать плагины. Restful_auth— аусентикация, will_paginate- пэйджер
- Постоянно изучать ХОРОШИЙ чужой код. Повторяю: ХОРОШИЙ (http://github.com)
- Любая строка в коде забирает время на ее прочтение, истолкования. Поэтому:
- удалять все комментарии
- удалять все неиспользующиеся строки кода
- удалять все строки, без которых код может работать
- Изменяя вложенные ресурсы используй build и merge (Topic à Post)
- Для Рест использовать сгенерированные урлы
- Не писать слишком длинные методы
- Основная задача— все задачи сводить к использованию Рест архитектуры
- Использовать миксин
- Для вставки тестовых данных на staging использовать rake tasks
- Для salt в хэшировании использовать случайную последовательность и сохранять в базе
- rake db:create – создает базу по конфигурации
- При использовании рест подхода не возникает необходимости использовать одно действие для get и post запросов
- В Руби любой метод по умолчанию начинается с begin. Поэтому если метод бросает в конце исключение, то begin можно не писать
- Использовать save!. Тк бросает исключение. Просто save использовать в if save (те когда происходит обработка результата)
- В Рест подходе не использовать member и тд. Добавлять дополнительные методы как вложенные ресурсы. В результате тонкие контроллеры
No comments:
Post a Comment