Код-ревью Junior Функциональные и нефункциональные обновлено

Ревью раздела требований

Вопрос

Это раздел требований к модулю отчётов. Найди проблемы.

FR-1. Система должна быстро формировать отчёты.
FR-2. Пользователь может выгрузить отчёт в Excel, PDF и т. д.
FR-3. Финансист выгружает отчёт «Платежи за период» в XLSX.
FR-4. Система должна быть удобной для бухгалтеров.
FR-5. Отчёт доступен всем пользователям.
FR-6. Отчёт доступен только ролям «Финансист» и «Руководитель».

Коротко

Хорошее требование однозначно, проверяемо и не противоречит соседним. FR-3 и FR-6 близки к хорошим.

Что должен найти ревьюер

  1. Ошибкастрока 1
    Непроверяемо: «быстро» без числа и условий; к тому же это NFR, а не FR
  2. Ошибкастрока 2
    «и т. д.» делает список форматов открытым: непонятно, что реализовать и проверить
  3. Ошибкастрока 4
    «Удобной» не проверить; это NFR, нужен измеримый критерий (время задачи, число ошибок)
  4. Ошибкастрока 5
    Противоречит FR-6: всем пользователям или только двум ролям
  5. Стильстрока 3
    FR-2 и FR-3 частично дублируют друг друга, форматы надо свести в одно требование

Разбор

Хорошее требование однозначно, проверяемо и не противоречит соседним. FR-3 и FR-6 близки к хорошим. Остальное надо переписать числами и закрытыми списками, а NFR вынести в отдельный раздел.

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

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