ON ProjectCarrier.ProjectId = ProjectCarrier.ProjectId

[elseiver]

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

Посему вопрос: не встречались ли уважаемые колеги с автоматическими тулзами, умеющими такие косяки отлавливать, по схеме базы данных, например?
Хочется запустить что-то такое и ужаснуться. Самому писать не хочется:)

Comments

[evg25.livejournal.com]

Переходи на питон, он такое определяет как синтаксическую ошибку :-)

[torrio.livejournal.com]

У меня десятки мегабайт исходников на дотнете, ошибки, которые я хочу найти - вообще в T-SQL, а ты мне предлагаешь перейти на питон?! Это, конечно, идеальное решение:)

[evg25.livejournal.com]

Я знаю, что идеальное, потому и предлагаю. :-)

(мне смайлик побольше поставить?) :-)
Просто, когда я обнаружил это в питоне, то очень порадовался :-)

[vgramagin.livejournal.com]

Для Java мы используем Findbugs - он такое, безусловно, отлавливает.

[vgramagin.livejournal.com]

http://stackoverflow.com/questions/38635/what-static-analysis-tools-are-available-for-c

[torrio.livejournal.com]

Он и в T-SQL отлавливает?!

[torrio.livejournal.com]

В целом, для C# у нас есть Re-Sharper, он со многим справляется и подсказывает всякое полезное, но вот оказалось, что есть еще непаханное поле в SQL... По идее, у нас куплен Red Gate SQL Tolbelt, но не факт, что он умеет:(

[vgramagin.livejournal.com]

Умммм, не уверен что в таком виде сработает. Но наверняка есть статиканалайзеры и для SQL тоже.

[antomus.livejournal.com]

Если нужно только такие вот повторы найти, то можно использовать регулярные выражения например. Они так могут. Правда это не совсем подходит под определение "автоматической тулзы"...

[torrio.livejournal.com]

Тут немного сложнее. Тот пример, который я привел - такое тоже, конечно, случается, но в реальности, например, было написано вот так:

ON ProjectCarrier.ProjectId = ProjectId

Имелось в виду поле ProjectId из другой таблицы, но у парсера SQL свой взгляд на это:) И его нужно учитывать. Под "простой повтор" такая штука уже не подпадает...

[5-sb-werewolf93.livejournal.com]

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

[elder-george.livejournal.com]

Хммм, мне SSMS выдал ошибку Msg 209, Level 16, State 1, Line 10
Ambiguous column name 'projectid'..

Пробовал вот так:


declare @ProjectCarrier table(
ProjectId INT NOT NULL
)

declare @Project table(
ProjectId INT NOT NULL
)

select * from @ProjectCarrier pc
inner join @project on pc.ProjectId = ProjectId


Ты что-то скрываешь!

[torrio.livejournal.com]

Честное слово не вру! Ambiguous column name не возникло потому, что этот запрос был вложенный, и, видимо, ambiguity разрешилась за счёт более низкого приоритета таблицы из внешнего запроса...

[torrio.livejournal.com]

Решарпер вроде как с T-SQL не работает... В C#, да, помогает!

[elder-george.livejournal.com]

Коллеги подсказали, что есть, например, http://www.sqlcodeguard.com/ (плагин к SSMS) и http://www.ubitsoft.com/products/t-sql-analyzer/index.php (+ платная версия к нему). Может, ещё что-то подскажут…


Сам не пробовал. Ну и у нас SQL код отличный же =)

[torrio.livejournal.com]

Спасибо!

[elder-george.livejournal.com]

О, вот тут рассказывают, как гонять static analysis на SQL проектах. Правда, там тоже набор правил фиксированный, не факт, что покрывает твои сценарии.