[NEW] Повторное добавление\вставка значений в коллекцию

[artbear]

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

Диагностика находит проблемные места с дублированием кода вставки\добавления значений в различные коллекции.
Подобное часто возникает из-за копипаста.

Подозрительный код

Коллекция = Новый Структура;
Коллекция.Вставить("Ключ1", 1);
Коллекция.Вставить("Ключ1", 2);

// выражения в одном блоке, но не подряд
Массив = Новый Массив;
Массив.Добавить(СтрокаТаблицы);
ДругойКод();
Массив.Добавить(СтрокаТаблицы);

Массив.Вставить(0, ДругаяСтрока);
Массив.Вставить(0, ДругаяСтрока);

Указанные дубли в большинстве случаев являются ошибкой, но иногда их наличие - это особенности алгоритма.
Например, вставка значений в начало массива или повторное добавление значения в массив не всегда является ошибкой..

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

Похожий ишуз

• [NEW] Затираемая\скрываемая переменная - shadow variable #1088

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

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

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

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

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

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

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

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

1

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

Нужно учесть:

• простые блоки кода, без переходов и условий, т.е. без Если, циклов, попыток и т.п.
• учесть, что строки могут идти не подряд друг за другом
• следующие методы объектов:
  • Вставить с 2мя параметрами, если в 2х строках совпадает 1й параметр
  • Добавить с 1м параметром, если в 2х строках совпадает 1й параметр
• можно учесть наличие объектов - Соответствие, Структура, Массив, ТаблицаЗначений, Список и т.п.
• а можно и не учитывать, а ориентироваться только по наименованию вызовов методов объектов Вставить и Добавить
  • подобное может дать больше полезных замечаний

[zeegin]

Кажется будет много FP.

[zeegin]

простые блоки кода, без переходов и условий, т.е. без Если, циклов, попыток и т.п.

Это что именно будет?

[artbear]

@zeegin codeBlock в терминах парсера

Т.е. код в блоке учитывается в правиле

Коллекция = Новый Структура;
Коллекция.Вставить("Ключ1", 1);
Коллекция.Вставить("Ключ1", 2);

а так нет

Коллекция = Новый Структура;
Коллекция.Вставить("Ключ1", 1);
Если Условие Тогда
  Коллекция.Вставить("Ключ1", 2);
КонецЕсли;

[artbear]

Найденные интересные срабатывания из КА 2.5

• 1
  

• 2
  

• 3 веселое

  • 1 я на картинке красным текстом отметил - вычисляют дважды одно и то же
    2 также дважды ставится КПП, но с разными значениями - ошибка!
  • 

• 4 авторы (или авторы) не могут определиться, какой же вариант дублирования вернее )

• 5 еще "замечательный" косяк с копипастом в типовом коде бух.отчетности (КА 2.5) - бедные глаза и руки автора кода )

[EvilPunker]

5 еще "замечательный" косяк с копипастом в типовом коде бух.отчетности (КА 2.5) - бедные глаза и руки автора кода )

Вы там бухгалтерскую отчетность с налоговой не перепутали? Полный путь к модулю какой?

[artbear]

5 еще "замечательный" косяк с копипастом в типовом коде бух.отчетности (КА 2.5) - бедные глаза и руки автора кода )

Вы там бухгалтерскую отчетность с налоговой не перепутали? Полный путь к модулю какой?

Спасибо за быструю реакцию )

2 похожие проблемы с именем НовИмяП009П1М110001 в КА 2.5 в одном отчете

• src\cf\Reports\РегламентированныйОтчетПрибыль\Ext\ObjectModule.bsl
• src\cf\Reports\РегламентированныйОтчетПрибыль\Forms\ФормаОтчета2019Кв4\Ext\Form\Module.bsl

ОтчетПрибыль - это налоговая или бух.отчетность? я уже не помню )

[EvilPunker]

ОтчетПрибыль - это налоговая или бух.отчетность? я уже не помню )

Налоговая. Для удобства лучше использовать обобщающий термин "Регламентированная отчетность"

[artbear]

@EvilPunker вот еще пример ошибки из отчетности КА 2.5 - еще и отступы подозрительные

• src\cf\Reports\КнигаПокупок\Ext\ManagerModule.bsl

хотя в этом же модуле в 2х или 3х местах есть правильный код - найдите отличия (с)

[EvilPunker]

вот еще пример ошибки из отчетности КА 2.5 - еще и отступы подозрительные

Отступы хорошие, сделаны переносы по стандарту https://its.1c.ru/db/v8std#content:444:hdoc

[artbear]

вот еще пример ошибки из отчетности КА 2.5 - еще и отступы подозрительные

Отступы хорошие, сделаны переносы по стандарту https://its.1c.ru/db/v8std#content:444:hdoc

я про отступы на строках, отмеченные красными стрелочками

[artbear]

И еще знаменитая ошибка "последней строки"