Задолбался чистить код

Aug 10, 2007 16:59

Решил почистить код, а то поднадоело листать портянку предупреждений которую выдает Visual Studio. Этих Warnings она выдает в количестве аж 912 штук, включая установленных вручную используя #warning и несколько Maximum number of warnings has been exceeded. (Максимальное количество предупреждений для одного проекта - 100). Так что предупреждений ( Read more... )

vb, работа, зло

Leave a comment

Comments 3

xoposhiy August 10 2007, 17:16:56 UTC
Как же так случилось, что столько предупреждений набралось?!

Хозяйке на заметку (это вроде тривиально, но мало ли):
Case Else не стоит использовать для одного из обычных значений enum-а. Когда у енума появится новое значение эта практика выйдет боком. Поэтому полезно писать Case Else и кидать там исключение "Не известное значение енума". Такие исключения сразу же напомнят о необходимости дополнить свитч.

Reply

eye_ru August 10 2007, 19:11:09 UTC
Предыдущие разработчики не придавали этому большого значения, проект компилится продукт с горем пополам работал...

Даже сейчас чувствую скептическое отношение к этому со стороны некоторых.

Большое спасибо за совет, учту.
;-)

Reply


xoposhiy August 10 2007, 17:20:19 UTC
> Больше половины уже заборол, но вот что делать с не CLS-compliant именами ума не приложу, на них уже ссылаются из других (прикладных) сборок.

1. Делать членов двойников с правильными именами
2. Помечать старые имена как Obsolete (правда не знаю можно ли это в VB) и окружать прагмами, подавляющими ворнинги (опять же не знаю, можно ли это в VB)
3. Бороть соответствующие ворнинги в прикладных сборках
4. Через некоторое время убивать члены, помеченные Obsolete

Reply


Leave a comment

Up