День добрый.

Хочеться в проекте делать проверку исходного кода. Может есть у кого опыт. Подскажите как быть.

Видеться 2 пути проверки.

Первый. Каждую пятницу собирать собрание разработчиков и обсуждать на них чей-то код. Например
берем последний патч. смотрим кто его написал и обсуждаем ю Обсуждать на предмет:
1) Эффективности
2) Понятности
3) Следования корпоративным стандартам.

Видиться что возникнуть могут такие проблемй
Все должны въехать в этот код. За 15 минут в общей комнате наверное это не сделать. т.е думаеться что этот вариант приведет к бардаку и никакого обсуждения не получитья.
Второй
Создать базу ( в лотусе например) и вести это обсуждения там.
Примерно так
Заводим тему ( говорим какой файл обсуждаем)
Каждый разработчик пишет свои замечания.
Кто-то подводит итог.

Как быть? Какой путь выбрать?

Есть такая штука, как формальные инспекции (formal inspections)
Скорей всего это как раз то, что тебе нужно.
В интернете можно найти рекомендации, как их лучше проводить.
Просто поищи ссылки по словам "formal inspection review".
Например тут.

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

Самое сложное в формальных инспекциях — это аккуратно к ним готовиться
и в ходе проведения аккуратно протоколировать все замечания, не сваливаясь в базар...
Ну и затем, конечно же нужно замечания устранять и проверять,
что они действительно были устранены.

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

Code Review можно делать с помошью автоматизированных
тулзов (для каждого языка — свои). например lint

Здравствуйте, oracle_developer, Вы писали:

OD>Хочеться в проекте делать проверку исходного кода. Может есть у кого опыт. Подскажите как быть.

OD>Видиться что возникнуть могут такие проблемй
OD>Все должны въехать в этот код. За 15 минут в общей комнате наверное это не сделать. т.е думаеться что этот вариант приведет к бардаку и никакого обсуждения не получитья.

Всех нет смысла привлекать, только заинтересованных в конкретном модуле.
Обычно должен быть code owner, который отвечает и ведет определенные модули. Например, один все что касается СНГ, другой все что касается Китая, третий отвечает за GUI. Все это может и пересекаться. Вот эти лица и должны принимать код программера и должны в него "въехать". Если на это не хватает 15 минут, надо потратить столько сколько надо, просто проводить code review можно и при "сдаче" модуля.

Здравствуйте, oracle_developer, Вы писали:

OD>День добрый.

OD>Хочеться в проекте делать проверку исходного кода. Может есть у кого опыт. Подскажите как быть.

Есть сеньор-программеры, которые отвечают за несколько программеров.
Программер написав код. Отдает его на проверку сеньор-программеру.

Здравствуйте, Awaken, Вы писали:

A>Code Review можно делать с помошью автоматизированных
A>тулзов (для каждого языка — свои). например lint

После lint'а встречаются перлы типа

(void)printf("Hello world!\n");

для VB я использую Numega CodeReview.