16 июля 2009

Я ненавижу твой говнокод и тебя лично.

Инспекции кода (code reviews) -- отличный способ повысить качество разрабатываемого кода. В оригинальной статье "Your code sucks and I hate you" и в её переводе можно почитать о некоторых тонкостях проведения инспекций.

В Google инспектируется любое изменение до попадания в систему контроля версий, неважно, написал ли его интерн или Jeff Dean. Это работает. На моей памяти несколько неприятных багов было отловлено во время code review, не говоря уж о многочисленнх исправлениях стиля кодирования и полезных улучшений, не влияющих на логику программы. Разумеется, code review -- не панацея. Многие ошибки прорываются сквозь code review, и нужны и другие средства по борьбе с ними (в первую очередь unit тесты).