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

Возможные сценарии:

• 0 простая повторная переустановка значения - реализовано в ПР

Значение = 1;
Значение = 2;

или похожий кейс из #178

МояПеременная = СтрЗаменить(КакаятоПеременная);
МояПеременная = СтрЗаменить(КакаятоДругаяПеременная);

• 1 во вложенных циклах используется одна и та же переменная цикла

Для Счетчик = 0 По 10 Цикл
	Для Счетчик = 2 По 10 Цикл
		Для Счетчик = 0 По 10 Цикл
		
		КонецЦикла; 
	КонецЦикла; 
КонецЦикла; 

или

Для Каждого Элемент Из Коллекция1  Цикл
	Для Каждого Элемент Из Коллекция2  Цикл
		Для Каждого Элемент Из Коллекция3  Цикл
		
		КонецЦикла; 
	КонецЦикла; 
КонецЦикла; 

• 2 из [FP] The iterator is not used in the For Each loop body #940
  • неожиданно счетчик в цикле затирает переменную модуля
    [ ] нужно учесть циклы Для по; Для Каждого

var Field;

procedure proc1 ()
    set = new Array ();
    for each Field in set do
        Message ( Field );
        proc2 ();
    enddo;
endprocedure

procedure proc2 ()
    Message ( Field );
endprocedure

• 3 кейс из [NEW] Бесполезное присваивание #945
  • во всех ветвлениях затирается значение переменной
    [ ] нужно учесть Если, ИначеЕсли; Попытка-Исключение;

новЗначение = 5;
Если НекотороеУсловие Тогда
    новЗначение = 10;
Иначе
    новЗначение = 20;
КонецЕсли;

• 4 переменные, которые использовались, установлены снова, но далее не используются

Для Каждого Элем Из Коллекция Цикл
  ИД = 10;
  Элем.Реквизит = ИД;
  ИД = 20; // не используется или потеряна !
КонецЦикла;

• Есть небольшое пересечение с "Неиспользуемая локальная переменная" Неиспользуемая локальная переменная #920

• также нужно учесть не только переменные, но и изменение реквизитов объектов - но это сложно и могут быть ФП, т.к. реквизит может быть переиспользован в методах, куда передается сам объект

ЗаказКлиента.ДатаНачалаДействияАбонемента =  ПривестиСтрокуКТипу(ДанныеЗаказа.АбонементС,  "Дата");        
ЗаказКлиента.ДатаНачалаДействияАбонемента =  ПривестиСтрокуКТипу(ДанныеЗаказа.АбонементПО, "Дата");

• в последнем примере сработала ошибка последней строки

• Еще интересный кейс затирания\скрытия возможен в циклах в [NEW] Неверное изменение переменной\реквизита внутри цикла - имеет значение только последняя итерация цикла #1522

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

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

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

• 🐜 Ошибка
• 👮 Уязвимость
• 💂‍♂️ Потенциальная уязвимость
• 💩 Качество кода
• Другое

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

• 💔 Блокирующая / Blocker
• ❤️ Критическая / Critical
• 💛 Важная / Major
• 💙 Незначительная / Minor
• 💚 Информационная / Info
• 💞 Другое

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

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

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

2 минуты

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