Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате
Форум программистов > Общие вопросы по .NET и C# > Программа FXCop


Автор: ДобренькийПапаша 24.7.2009, 19:06
Программа анализирует сборку и выдаёт рекомендации по "улучшению" кода. В книжке Троелсена эта программа дюже советуется)))
Кто-нибудь пользовался, что можете сказать по поводу этой программы?

Автор: Rififi 24.7.2009, 22:12
в двух словах - must have.

Автор: jonie 24.7.2009, 22:21
добавлю : "не забудьте конфигурирование её сделать" - не все советы полезны, не все применимы ко всем приложениям.... поиграться стоит, возможно даже встроить в сборочный конвеер её тоже стоит.. а может и нет  - все зависит от сложности проекта и внутренних гайдлайнов имхо.

Автор: ДобренькийПапаша 25.7.2009, 18:38
Я её попробовал вчера, мне показалось, что для неопытного программиста, (такого как я) программа немного помогает (хотя всё, что она предложила исправить, я бы исправил и без неё). Для профи, как мне показалось, программа абсолютно бесполезна. Затестил я её на сборке, около 2000 строк кода. Все рекомендации были связаны с нормальным оформлением MessageBox.Show(), переименуйте такую-то переменную в соответствии со стандартом, в парсе задайте IFormatProvider, не используйте Exception (конкретизируйте исключение) ну и всякое такое.
Я уж молчу по поводу того, что прога не даёт никаких рекомендаций относительно перформанса где бы то ни было (лично я рассчитывал на это).

Цитата

не забудьте конфигурирование её сделать


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

Автор: jonie 25.7.2009, 21:31
Цитата
Ничего не конфигурировал, так что извиняйте... Если можно, скажите пару слов, что даст конфигурирование... Есть ли реальный профит?

например то над чем я работаю имеет вполне осмысленые двухбуквенные принятые всеми (не только в нашем ПО) идентификаторы (например AI (accruid Interest) или MC (Marging Call) и т.д.) и FxCop ругается на них, т.е. я к тому что нужно настраивать такие моменты.

Цитата
Все рекомендации были связаны с нормальным оформлением MessageBox.Show(), переименуйте такую-то переменную в соответствии со стандартом, в парсе задайте IFormatProvider, не используйте Exception (конкретизируйте исключение) ну и всякое такое.
Я уж молчу по поводу того, что прога не даёт никаких рекомендаций относительно перформанса где бы то ни было (лично я рассчитывал на это).

она не создана для перфоманса - для перфомонса есть тесты и профилировщики (даже в той же студии есть).

Powered by Invision Power Board (http://www.invisionboard.com)
© Invision Power Services (http://www.invisionpower.com)