Подходы обзора кода

Question

Просто выходите на конечность здесь. Выполняет ли анализ кода путем отбора нескольких классов и единиц кода разработчиком, достаточным для того, чтобы судить о том, что он сделал неправильно, чтобы обеспечить входные данные? Это на всякий случай, если вы нажали на время и не можете просмотреть каждую строчку, написанную ими. Я работаю над тем, что разработчики могут дублировать ошибки одного и того же типа во всем мире. Это эффективный подход? или частично эффективны или вообще не эффективны?

Какие другие подходы лучше? Сидели с каждым разработчиком, который написал строки кода и спрашивал их, почему они сделали это определенным образом? или сделать это самостоятельно, а затем отправить краткий обзор ваших результатов по электронной почте?

Как вы относитесь к рассмотрению кода, написанного младшими разработчиками/сверстниками, когда вам нужно просмотреть всю базу кода в типичном 15-дневном спринте?

Answer 1

Я не думаю, что для выбора нескольких классов и единиц достаточно знать, что код является приемлемым. Возможно, вам удастся выбрать несколько частей кода, которые написаны отлично и пропустить большие проблемы.

Я думаю, что если вы ограничены во времени, вам нужно взглянуть на оптимизацию того, как вы просматриваете. Во-первых, есть инструменты (StyleCop, FxCop и многие другие), которые вы можете запустить, чтобы дать автоматический обзор общих ошибок или стандартных отклонений кодирования.

После этого я думаю, вам нужно будет разобраться, какие важные функции и просмотреть их по строкам. Отправка синопсиса, а затем сидение с разработчиками, если нужно, будет разумным подходом.

Это, вероятно, только первый набор кода, который нуждается в полном обзоре, если вы можете найти время, чтобы сделать это после этого, просмотр того, что изменилось, обычно не так уж плох, но все зависит от размера вашей команды.

Answer 2

Как младший разработчик, я считаю, что сидение лучше, если у вас есть время. Легче объяснить, что я пытаюсь сделать лично, когда что-то неясно или неправильно.

Прохождение каждой отдельной строки написанного кода является излишним - QA существует по определенной причине. Вы ищете, чтобы убедиться, что мы следим за кодовыми соглашениями компании и не допускаем глупого материала, которого у нас недостаточно, чтобы признать, что это глупо.