Ревью чужого кода
Опирается на главы «Жизненный цикл ПО» и «Требования к коду».
Задание
- Провести ревью предложенного кода, отыскав в нём ошибки, неточности и неэффективные места и дав рекомендации по улучшению.
- Задача не сводится к исправлению ошибок: требуются комментарии и рекомендации.
Требования к сдаче
- Ссылка на форк, в котором замечания оформлены комментариями к строкам кода. Наиболее простой способ: завести в собственном форке ветку с исправлениями и открыть pull request внутри форка, из этой ветки в
main; в диффе GitHub позволяет комментировать каждую строку. Отдельный текстовый файл со списком замечаний не подходит: замечание должно располагаться рядом с кодом. - Замечания разделены на два вида: ошибки, из-за которых программа работает неверно или завершается аварийно, и рекомендации, где код работает, но написан неудачно.
- У каждой ошибки указано, при каких данных она проявится. «Тут может упасть» замечанием не считается.
- У каждой рекомендации указано, что она улучшает: скорость, читаемость или устойчивость к изменениям.