В этой главе происходит разбор кода для класса SerialDate. Заметки:
- Проводить Code Review мы должны спокойно и с уверенностью. И нужно приветствовать, если кто-то это делает для нас. Тут прям полностью согласна. Я очень люблю CR и много комментариев, потому что всегда узнаю что-то новое.
- Есть некое приложение Clover, которое показывает покрытие тестами. Видимо, вот это — https://openclover.org/documentation
- История изменений — пережиток прошлого и её надо удалять.
- Выбирайте содержательные названия.
- Не оставляйте избыточные комментарии.
- Если сложные вычисления, то лучше использовать в методе временные переменные.
- Если что-то зависит от реализации на логическом уровне, то что-то должно зависеть и на физическом уровне.
- Выносите перечисления в отдельные файлы.
