Покритикуйте пож.
1) Есть мнение, что в ПВР ДопНачисления надо устанавливать зависимость от базы «По периоду РЕГИСТРАЦИИ», т.к. в задании есть строка «…премия процентом от начисленного в том же
расчетном периоде оклада…»
2) В документе рекомендуется делать отдельную ТЧ на каждый регистр
3) не нужны строки:
Движения.ОсновныеНачисления.Записывать = Истина;
Движения.ДополнительныеНачисления.Записывать = Истина;
4) Для оклада указывать базовый период не нужно
5) Для Командировки нет проверки, что ее нельзя вводить задним числом
6) Получение двух предыдущих месяцев для базового периода Командировки выглядит как-то странно, хотя работать будет:
Движение.БазовыйПериодНачало = НачалоМесяца(НачалоМесяца(НачалоМесяца(Дата) - 1)-1);
7) В запросе РасчитатьОсновныеНачисления() слева находится таблица База
Павел Чистов в вебинарах по СПР утверждает, что, при отсутствии данных для расчета, в таблице база не будет записей, поэтому таблицу База слева в запросе располагать нельзя.
Слева должна быть ВТ ДанныеГрафика, либо физическая таблица регистра.
При расчете Премии сделано так:
Запись.Результат = Запись.Размер * Выборка.РезультатБаза/100 ;
Так будет корректнее
Запись.Результат = Запись.Размер /100 * Выборка.РезультатБаза;
9) При расчете ДопНачислений перерасчет периода действия отменять не обязательно
Истина можно убрать:
Движения.ДополнительныеНачисления.Записать(, Истина);
10) Рассчет Основных и ДопНачислений желательно производить именно в общем модуле, несмотря на то, что модуль документа компилируется на сервере.