Блог: ревью кода

✖ Фильтр по слову «ревью кода»

Командное правило может мне не нравиться

Разные бумажные инструменты лежат у общей линии шва, закреплённой розовым зажимом.

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

Читать дальше →

Что я хочу узнать на ревью своего кода

Лупа выделяет тугой узел в бумажной конструкции, рядом лежат ещё не использованные ножницы.

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

Читать дальше →

Кто должен принимать работу, которую команда делала вместе?

Бумажную арку проверяют одной меркой с двух сторон; розовая отметка показывает конкретное несовпадение.

Приёмка учебного проекта другим наставником помогает обнаружить то, что пропустили во время совместной работы. При этом ученик получает вторую оценку, которая может противоречить первой. Ему нужно понимать, какое требование задания нарушено и как это проверить. Независимость проверяющего сама по себе ещё не делает его замечание верным.

Читать дальше →

Зачем показывать незавершённую работу

Незаконченный бумажный макет стоит на открытом основании, важное соединение помечено розовым зажимом.

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

Читать дальше →