Показать сообщение отдельно
Старый 06.10.2022, 11:58   #5  
dech is offline
dech
Участник
Аватар для dech
Самостоятельные клиенты AX
 
642 / 347 (13) ++++++
Регистрация: 25.06.2009
Адрес: Омск
Записей в блоге: 3
Я бы обратил внимание на переменную doSplit. Мы здесь видим её изменение на true пока только в одном месте, однако, если бы она менялась только там, то уже на втором витке цикла было бы добавление. Я бы также изменил порядок if на противоположный для более естественной проверки, убрав отрицание.
И самое главное, найдите максимальное значение LineNum поиском по JournalId и прибавьте к нему 1.
Ну и еще я бы выделил весь этот кастомный кусок в отдельный метод. Как минимум это облегчает понимание.
__________________
// no comments