Добрый день.
В одном из отчетов ERP УСО при поиске ошибки столкнулся с такой функцией
ГрафикиРаботы.РазностьДатПоКалендарю(Календарь,ДатаНачала,ДатаОкончания)
если датаНачала =31.12.24 а датаОкончания в 25 году, например 01.01.25 получает почему-то минус 251
стоит поставить дату начала 30.12.24 тогда все нормально

Ну так напишите свою функцию, более правильную.

(0) календарь/график заполнен на 2025 год?

(2) Да заполнен

(0) По-моему, в этой проверке

вторая часть - лишняя.

(4) Вы согласны, что функция работает неправильно?
Проверял и на ERP последней версии. Там такая же ситуация

(4) Происки релиза платформы?
(5) Давайте проверим мысль Доминошника: поставьте остановы и давайте значения для даты начала 31.12.2024
КонецДня(ДатаНачала)
КонецГода(ДатаНачала)

(5) Да, согласен.
Как я понял, ошибка - если дата начала попадает на 31 декабря.

(6) Думаю, "проблемы" зарплатной библиотеки. БСП

(5)

КалендарныеГрафики
-- Календарь
-- Год
-- ДатаГрафика
Ресурсы
-- ДеньВключенВГрафик
-- КоличествоДнейВГрафикеСНачалаГода

Казалось бы простой запрос должен вернуть кол-во дней.
Но  кто же идёт простым путём...

ВЫБРАТЬ 
    КОЛИЧЕСТВО(*) 
ИЗ РегистрСведений.КалендарныеГрафики как т1
ГДЕ 
   т1.ДатаГрафика МЕЖДУ &ДатаНачала и ДатаОкончания
  И т1.ДеньВключенВГрафик = Истина
  И т1.Календарь = &Календарь

"Они там сначала строят ВТ, затем складывают, потом вычитают. 6 запросов не считая легких увлечений

(7) А почему раньше никто не замечал?
Всё-таки надо дождаться проверки (6)

(9) это как проблема 29 февраля. ее замечают раз в 4 года.

(11) Правило високосного года немного сложнее, чем делимость на 4:

1. Если год делится на 4 без остатка, он високосный, но см. также два следующих правила:
2. Если год делится на 100 (годы оканчиваются двумя нулями, например, 1900, 2100), это не високосный год, за исключением случаев из условия 3.
3. Если год делится на 400 — он високосный.

Функция Високосный(Год)
	
	Если Год % 400 = 0 Тогда
		Возврат Истина;
		
	ИначеЕсли Год % 100 = 0 Тогда
		Возврат Ложь;
		
	ИначеЕсли Год % 4 = 0 Тогда
		Возврат Истина;
	Иначе
		Возврат Ложь;
	КонецЕсли; 
		
КонецФункции

(11) Нет. Здесь что-то с платформой. Смотрим:

Мысль логичная: если даты внутри года, то они так и остаются. Если же в разных годах, то между ними добавляют 31 декабря:
ДатаНачала
31.12.2024
ДатаОкончания
Ну а потом идёт запрос к РС КалендарныеГрафики к ресурсу КоличествоДнейВГрафикеСНачалаГода

Сбой может произойти только если неправильно сработают
КонецДня(ДатаНачала)
КонецГода(ДатаНачала)

При этом ошибочно задвоится начало дня 31 декабря:
31.12.2024
31.12.2024
01.01.2025

Тогда далее в запросе получим хрень.

Дубль три: скорее всего платформа. А логика программы безупречна. В силу того, что КоличествоДнейВГрафикеСНачалаГода разбито по годам, то при переходе через НГ надо обязательно вставлять 31.12 Пусть автор проверит у себя
КонецДня(31.12.2024)
КонецГода(31.12.2024)

(7) Думаю, причина в том, что редко кто пытался вычислить разность дат исходя из начальной - последний день года.

(14) Здрасьте. Как раз условие по "И" именно для 31 декабря как даты начала. Эрго разработчики этот момент как раз и ПРОДУМАЛИ давным-давно.

(13) Вот результат запроса "по конфигурации"
31.12.2024 = 366
01.01.2025 = 1
(в графике все дни рабочие)

А дальше - начинается "чудо вычисления"

Подробности

    КоличествоДнейВГрафике = Неопределено;
    ДобавлятьПервыйДень = Ложь;

    Пока Выборка.Следующий() Цикл
        Если КоличествоДнейВГрафике = Неопределено Тогда
            КоличествоДнейВГрафике = Выборка.КоличествоДнейВГрафикеСНачалаГода;
            ДобавлятьПервыйДень = Выборка.ДеньВключенВГрафик;
        Иначе
            КоличествоДнейВГрафике = КоличествоДнейВГрафике - Выборка.КоличествоДнейВГрафикеСНачалаГода;
        КонецЕсли;
    КонецЦикла;

    Возврат - КоличествоДнейВГрафике + ?(ДобавлятьПервыйДень, 1, 0);

И, соответственно, получаем:
КоличествоДнейВГрафике = 366 (1-й шаг)
КоличествоДнейВГрафике = 365 (2-й шаг)

Возврат = (- КоличествоДнейВГрафике) = - 365

Если бы в результате запроса было
31.12.2024 = 366
31.12.2024 = 366
01.01.2025 = 1

то получили бы
КоличествоДнейВГрафике = 366 (1-й шаг)
КоличествоДнейВГрафике = 0 (2-й шаг)
КоличествоДнейВГрафике = 1 (3-й шаг)

Возврат = (- КоличествоДнейВГрафике) = - 1


Для простоты опустил "ДобавлятьПервыйДень"

(16) Давайте проверим. Пусть автор уберёт этот кусок с "И". Тогда мы получим:
31.12.2024
31.12.2024
01.01.2025

Кстати, откуда тогда у автора 251, если перед запросом имеем:
248
248
1
?

а зачем такое уродство? нельзя просто запросом собрать количество ненулевых записей по регистру МЕЖДУ &ДатаНачала и &ДатаОкончания?

(19) Скорее всего для быстроты. Ведь Вы правильно заметили, что в РС уже прописано количество рабочих дней с начала года. Что быстрее: собирать за 20 лет все ненулевые записи или просто арифметически вычесть/сложить уже имеющиеся количества?

(19) Вы только гляньте-гляньте!
https://www.turboconf.ru/Templates/BspDetails/obshhijmodul-kalendarnyegrafiki-raznostdatpokalendaryu/raznost-dat-po-kalendaryu

А мы ещё боремся за почётное звание дома высокой культуры и быта!

(20) 20 лет - это 7300 записей с отбором по определенному календарю (если это реально нужно). Запрос это прожует моментально.
При этом я сомневаюсь что кто-то держит записи за 20 лет назад. ЗУП и ERP вышли позже.

(21) вообще не понимаю людей, которые ЗУП любят. это же кошмар.

(21) Странно. У меня в последнем релизе БП в этой функции стоит
2020,01,01
вместо
%1

(24) Судя по копирайту в ссылке (2019 год) - это довольно старая версия БСП

(24) Наверное, на турбоконфе старая версия этой функции

(22) Тогда согласен. Переделывайте эту функцию.
Это дробление по годам некрасиво )

(9) В ЗУпе та же проблем, я давно заметил. Проще свое написать.

(28) А чего молчали как партизан? Этой функции хрен знает сколько лет )

(29) Никто не спрашивал. Да и не такая серьезная проблема, пишешь расчёт под свои задачи и забываешь об этом.

(30) Я так сразу и сказал (1)

(31) "человеку не важен сыр, человеку важно быть правым", что и доказывает в (5)

(31) Да я и не спорю. Но нельзя же игнорировать вопрос человека, который в эту тему серьезно закопался и понял, что там не работает конкретно.