Design Review и его роль

[gaperton]

Про код-ревью слышали многие. Про Design Review - нет. Меж тем, именно присутствие design review в паре с code review  позволяет во многих случаях сделать всю систему review более эффективной, и многократно усилить ее положительный эффект.

management@RSDN: Code Review: а оно надо?

D>>кто-нибудь использует такую процедуру как Code review? Как это

Comments

Дело за малым

[schegloff]

Что такое код-ревью, даже я в общих чертах представляю. А вот как делать дизайн-ревью?

(вопрос не праздный, у самого около 3К строк на стадии этого самого ревью :) )

Re: Дело за малым

[gaperton]

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

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

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

Re: Дело за малым

[gaperton]

Или же, если вам по шагам показывают, каким образом ваша система приходит в очень нехорошее состояние.Тоже ошибка.

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

Короче, заявляя ошибку надо продемонстрировать сценарий, который ее проявляет. При невозможности это сделать - незачот, ошибка непринята. При невозможности внятного ответа на вопрос - тоже низачот, ошибка, и нии... и все. :)

Re: Дело за малым

[gaperton]

Если коллеги не понимают ваших объяснений, то вам также низачот. Надо объяснять лучшею

[alexott]

кстати, а что из code-review утилит вы используете/пользовались. я пытаюсь ввести эту пракитку у себя в компании, но пока безуспешно.
пока что я прикрутил code-review plugin к trac, что дает сразу интеграцию с системой контроля версий, но code-review тут делается post-фактум, а code-review патчей мне делать пока не дают

[kurilka]

а ссылкой не кинешь на плагин?

[alexott]

ссылка на работе, но находил через траковский сайт - их там всего два было

[kurilka]

чтот там 2 не нашёл, нашёл только вот это - http://trac-hacks.org/wiki/PeerReviewPlugin не оно?