четверг, 13 мая 2010 г.

About code review

Набрел на статью на хабре, рассказывающую о том как в Google организовано владение кодом. Заинтересовала методика в которой доступ на чтение имеют все, а для того чтобы внести изменения в "чужой" код необходимо получить одобрение кода у одного из его собственников,а так же у специалиста по языку на котором он написан.
Идея со специалистом, как и вообще идея code review, последнее время, кажется мне очень здравой. Причем предпочтительно проводить это review в максимально демократичной манере и опираться на некий документ описывающий его принципы. Например в Python это PEP 8:Style Guide for Python Code, в C# - About the C# Coding Style Guide etc.
Пользы от code review больше чем может показаться, это и взаимное обучение приемам программирования, нахождение более оптимальных алгоритмов решений задач, устранение багов и увеличения числа людей "шарящих" в проекте.
Я считаю что code review это более мягкая форма парного программирования, которое при всех своих достоинствах гораздо более трудоемко и спорно в применении.

Так же с интересом прочитал рассказа о группах (grouplet) в Google, но как это работает в реальности пока не понял. Возможно мое непонимание есть следствие того, что очень редко в поле зрения попадают люди, склонные к инновациям и готовые их продвигать. И еще реже таких людей окружающие готовы принимать.

Комментариев нет: