Исходное сообщение
"Проект по автоматическому анализу кода в пакетной базе Debia..." Отправлено klalafuda, 17-Дек-10 21:54
> Ну почему же, для черновой чистки кода, особенно после рефакторинга, когда классы обменивались членами и методами, вполне годится. Тут переменная уже не используется, тут - можно уменьшить ее область видимости, свичи без дефолтов лишний раз посмотреть - может, и нужно сделать... В общем, для "полировки" и в качестве имитации свежего взгляда - нормально. Как альтернатива ключам компилятора, в результате которых имеешь ворох замечаний, из которых 90% - вообще не к твоему коду, а к библиотечному. Неиспользуемые переменные прекрасно отсекает gcc. И в этом он совершенно прав. Если, допустим, неиспользуемые аргументы метода вполне имеют право на жизнь (соответственно -Wno-unused-parameter валиден и правомерен) то неиспользуемые переменные - это откровенный мусор. Естественно, если это простой тип. Свичи без дефолтов - тоже вполне валидны. При условии, что это перечисление и в свиче by design полный набор элементов. Это позволяет ещё на уровне компиляции обнаружить, что оказывается, перечисление внезапно! кто-то расширил новыми элементами а вот в селекторы тут там и сям их не добавил (забыл). Если бы стоял дефолт то компилятор бы это проглотил на ура. С самым разными последствиями. Ворох же замечаний - он в подавляющем большинстве случаев не просто так. И не суть важно откуда они идут - из кода A или из стороннего кода B. Какая разница где именно грязь === риски багов? Если оно есть - с этим нужно разбираться, без вариантов.

Ваше сообщение
Имя*:
EMail:	Для отправки ответов на email укажите знак ! перед адресом, например, !user@host.ru (!! - не показывать email). Более тонкая настройка отправки ответов производится в профиле зарегистрированного участника форума.
Заголовок*:
Сообщение*:
	Введите код, изображенный на картинке: