[NEW] Бессмысленные ВЛОЖЕННЫЕ условия

[theshadowco]

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

В процессе рефакторинга либо копипаста появляются участки кода, когда вложенное условие равно вышестоящему. Часто, это условие является ошибки, как следствие копипасты.

Ссылка на источник, подтверждающее нарушение либо обоснование наличия проблемы

Параметры диагностики

Тип Статья на русском

• [ ] :ant: Ошибка
• [ ] :cop: Уязвимость
• [ ] :guardsman: Потенциальная уязвимость
• [x] :poop: Качество кода
• [ ] :trollface: Другое

Важность Статья на русском

• [ ] :broken_heart: Блокирующая / Blocker
• [ ] :heart: Критическая / Critical
• [x] :yellow_heart: Важная / Major
• [ ] :blue_heart: Незначительная / Minor
• [ ] :green_heart: Информационная / Info
• [ ] :revolving_hearts: Другое

Тэги Статья на русском

• [ ] STANDARD - "Нарушение стандартов 1С"
• [ ] LOCKINOS - "Не будет работать в другой ОС"
• [ ] SQL - "Проблема с запросом"
• [ ] PERFORMANCE - "Проблема производительности"
• [x] BRAINOVERLOAD - "Непонятный код"
• [ ] BADPRACTICE - "Плохая практика программирования"
• [x] CLUMSY - "Излишние действия"
• [ ] DESIGN - "Ошибка в проектировании"
• [x] SUSPICIOUS - "Подозрительный код"
• [ ] UNPREDICTABLE - "Непредсказуемо работающий код"
• [ ] DEPRECATED - "Устаревшая функциональность"
• [ ] ERROR - "Ошибочная конструкция"
• [ ] LOCALIZE - "Проблемы локализации"

Время на исправление (минут)

5

Дополнительная информация

[artbear]

@theshadowco Ишуз https://github.com/1c-syntax/bsl-language-server/issues/486 уже имеет название "Бессмысленные условия"

@sfaqer Договоритесь о названии своих ишузов )

[theshadowco]

@artbear переименовал