Выбор ответа Middle Код-ревью и правила команды обновлено

Свой хелпер вместо существующего

Вопрос

В пулреквесте ассистент написал функцию formatDate, хотя в проекте есть общий модуль форматирования дат. Что это за проблема?

Попробуйте сами

Выберите вариант, и откроется разбор.

Показать ответ и разбор

Коротко

Правильный ответ: Дублирование: два источника правды, которые разойдутся. Ассистент видит только часть проекта и часто пишет своё вместо существующего.

Варианты ответа

  1. Дублирование: два источника правды, которые разойдутсяправильный ответ
  2. Только стилистика, на работу кода это никак не влияет
  3. Проблема производительности: функция медленнее общей
  4. Ошибка безопасности, потому что даты приходят от пользователя

Разбор

Ассистент видит только часть проекта и часто пишет своё вместо существующего. Два форматтера со временем начнут вести себя по-разному, баг исправят в одном месте. Это вопрос архитектуры и поддержки, а не скорости или безопасности.

Ответьте сами, а не прочитайте

В тренажёре вопрос попадёт в повторения и вернётся, когда начнёт забываться. Плюс билеты с зачётом: 20 вопросов, не больше 2 ошибок.