-
Notifications
You must be signed in to change notification settings - Fork 5
Требования к коду
Alex Anakin edited this page Mar 11, 2016
·
17 revisions
Данные требования согласованы с Виктором Кучиным и Александром Анакиным. Они не отменяют правил на других страницах вики.
- Использовать в рест-запросах under_score стиль.
- Роллбек если есть, то должен приводить базу в то состояние, которое было до запуска ченчсета.
- Тесты не должны использовать интернет.
- Соблюдаем принцип YAGNI: нужно реализовывать минимум методов, который полностью удовлетворяет постановке задачи. Особенно это касается дао и сервисов.
- Группировать методы в дао и сервисах по выходным параметрам. Если метод возвращает void, то по логике применения метода. Иллюстрация
- Запрещена операция
DELETE
в БД, так как информация может быть полезной в будущем. Нужно добавить в таблицы полеactive
. - Перед созданием нового changeset, зарезервировать номер в Liquibase changeset LOG
- Все объекты должны быть имьютебл (immutable). Узнать, почему это полезно, можно в этой статье. Узнать, как сделать объект имьютебл - в документации.
- Запрещено создание статических утилитных методов; [25:00]. Почитайте статью о том, почему это плохо и чем нужно их заменить.
###Дополнительно