16 июля 2009

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

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

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

3 комментария:

  1. у тебя перевод названия много лучше, чем в том хабрапереводе.

    ОтветитьУдалить
  2. Да я вообще неплохие переводы делаю (/me скромно водит носком ботинка)...

    ОтветитьУдалить
  3. Проклятый JSKit Comments устроил мне разговор с самим собой. На всякий случай: шизофрении у меня пока что нет, первый комментарий был написан моим коллегой :)

    ОтветитьУдалить