Если не знаешь, куда плывёшь, никакой ветер не будет попутным (с) либо Спиноза, либо Сенека, либо Конфуций

Либо все вместе взятые, но толку от этого никакого...

batigoal, цель этого топика и выяснить по каким критериям оценивать исходники. На данном этапе меня вполне устраивают правила типа "Microsoft .net best practices" которые и заявляются в большинстве таких тулзовин.

+1 за FxCop, только относись к его предложениям с известной долей скептицизма. 

А критерии оценки тебе что, не сообщили? Задача вида "пойди куда-нибудь, сделай что-нибудь и напиши отчет"?   

arilou, ситуация следующая: один постоянный клиент попросил проревьювать исходный код (хз где он его взял), мы думаем брать/не брать (так как опыта анализа кода у нас нет), попросили меня прозондировать этот вопрос, вот я и зондирую (паралельно участвуя в основном проекте, где сейчас просто засада...)...

ivashkanet, дык спосите у него, что ему нужно в результате. Ревьюить можно по-разному, например на секьюрность, или соответствие стандарту кодирования (какому?), или на расширяемость, или под конкретную задачу, и т.п.

arilou, да понимаю я все это.
Дело в том, что он хочет чтобы мы проверили на "хорошо ли написан код". А если мы ему начнем перечислять 

то он скажет: ДА, Да, ДА!!!, и это заверните.

Добавлено через 1 минуту и 30 секунд
Тем более сейчас все это затихло, так я держу крестики, чтобы не выползло опять   

Это сообщение отредактировал(а) ivashkanet - 24.1.2008, 17:40

ivashkanet, ну смари сам