Выбор ответа Middle Код-ревью и правила команды обновлено
Свой хелпер вместо существующего
Вопрос
В пулреквесте ассистент написал функцию formatDate, хотя в проекте есть общий модуль форматирования дат. Что это за проблема?
Попробуйте сами
Выберите вариант, и откроется разбор.
Показать ответ и разбор
Коротко
Правильный ответ: Дублирование: два источника правды, которые разойдутся. Ассистент видит только часть проекта и часто пишет своё вместо существующего.
Варианты ответа
- Дублирование: два источника правды, которые разойдутсяправильный ответ
- Только стилистика, на работу кода это никак не влияет
- Проблема производительности: функция медленнее общей
- Ошибка безопасности, потому что даты приходят от пользователя
Разбор
Ассистент видит только часть проекта и часто пишет своё вместо существующего. Два форматтера со временем начнут вести себя по-разному, баг исправят в одном месте. Это вопрос архитектуры и поддержки, а не скорости или безопасности.
Ответьте сами, а не прочитайте
В тренажёре вопрос попадёт в повторения и вернётся, когда начнёт забываться. Плюс билеты с зачётом: 20 вопросов, не больше 2 ошибок.
Ещё по теме «Код-ревью и правила команды»
- Место ИИ-ревьюера в процессе
- Пулреквест на 1500 строк
- Характерные ошибки ИИ-кода на ревью
- Плюсы и минусы ИИ-ревьюера
- Новая зависимость от ассистента
- Код-ревью: эндпоинт из пулреквеста, написанный с ИИ
- Файл правил проекта для ассистента
- Перед отправкой на ревью