Анализатор кода

[deas]

Здравствуйте.

Первое впечатление о анализаторе кода:
1. В списке ошибок не нашел как можно перейти на подробное объяснение. Добавьте там ссылку на сайт или лучше на локальный раздел помощи.
2. Уведомление PF005 и BP002 сделайте отдельные правила для временных объектов и реальных.
3. BP005 что бы не реагировал на записи типа

Код: Выделить всё

WHERE EXISTS (SELECT 1 FROM

а так же когда это встречается в подзапросе в части SELECT

Код: Выделить всё

SELECT (SELECT count(*)  FROM ..) AS cnt

или второе выделить в отдельное правило
4. Правила PF011 и подобные я бы тоже как то разделил, с одной стороны они полезны, но по факту почти в каждой процедуре есть или WITH(NOLOCK) или WITH(TABLOCK), без которых процедура просто не будет правильно работать. В результате проще отключить правило, чем проверять все ситуации. И уж точно это не Warning.

А так инструмент безусловно полезный и желательно его дальнейшее развитие, а особенно с привязкой к системе контроля версий, что бы проверялись правила при checkin.

[Alexander]

Приносим извинение за задержку с ответом.

1. В списке ошибок не нашел как можно перейти на подробное объяснение. Добавьте там ссылку на сайт или лучше на локальный раздел помощи.

Данная функциональность реализована в версии 5.0.

2. Уведомление PF005 и BP002 сделайте отдельные правила для временных объектов и реальных.
3. BP005 что бы не реагировал на записи типа
4. Правила PF011 и подобные я бы тоже как то разделил, с одной стороны они полезны, но по факту почти в каждой процедуре есть или WITH(NOLOCK) или WITH(TABLOCK), без которых процедура просто не будет правильно работать. В результате проще отключить правило, чем проверять все ситуации. И уж точно это не Warning.

Мы рассмотрим данные предложения при разработке будущих версий продукта.