[NEW] Многократный вызов функций ТекущаяДатаСеанса (ТекущаяДата) в рамках одного метода

artbear commented 3 years ago

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

Следует избегать в коде одной процедуры (функции) многократного обращения к функции ТекущаяДатаСеанса (ТекущаяДата), так как возвращаемые значения будут отличаться друг от друга.

Неправильно

ДатаПоследнегоОповещения = ТекущаяДатаСеанса();
ДатаСледующегоОповещения = РассчитатьДату() + ТекущаяДатаСеанса();

Правильно использовать ранее рассчитанные дату и время:

ДатаПоследнегоОповещения = ТекущаяДатаСеанса();
ДатаСледующегоОповещения = РассчитатьДату() + ДатаПоследнегоОповещения;

связанные ишузы по методам ТекущаяДата, ТекущаяДатаСеанса

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

https://its.1c.ru/db/v8std/content/643/hdoc

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

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

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

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

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

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

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

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

1 минута

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

asosnoviy commented 3 years ago

НачалоЗамера = ТекущаяДатаСеанса();
ОкончаниеЗамера = ТекущаяДатаСеанса();

НачалоЗамера = ТекущаяДатаСеанса();
ВремяВыполнения = ТекущаяДатаСеанса() - НачалоЗамера;

artbear commented 3 years ago

Да, в случае замеров нужно исключение, но подобных замеров все-таки не так много. И в стандарте 1С об этом ни слова (

zeegin commented 3 years ago

НачалоЗамера = ТекущаяДатаСеанса();
ОкончаниеЗамера = ТекущаяДатаСеанса();

НачалоЗамера = ТекущаяДатаСеанса();
ВремяВыполнения = ТекущаяДатаСеанса() - НачалоЗамера;

Пример знатный, но плохо когда в одном методе делается несколько замеров. Ведь замеряют обычно ключевые операции, а вызов нескольких ключевых операций в одном методе противоречит определению ключевой операции.

Либо это какие-то частные замеры, которые тоже стоит разделить на пару методов.

1c-syntax / bsl-language-server