Программное добавление строки в складской журнал

[mazzy]

В совете Сергея Пикина показано как программно добавить строку в складской журнал.

http://axapta.mazzy.ru/lib/journaltrans_insert/

psststInventTransAddJob_RNR.xpo - оригинальная версия job'а
InventTransAddJob_RNR.xpo - после ценных дополнений Максима Горбунова.

[mazzy]

Если вас интересует, как добавить строку в журнал ГК, смотрите класс TutorialJournalCreateExample в Аксапте.

[Maxim Gorbunov]

Вместо

inventJournalTrans.JournalId            = #JournalNum;
inventJournalTrans.JournalType          = inventJournalTable.JournalType;

правильнее было бы использовать

journalTransData.initFromJournalTable();

В этом случае будут так же инициализированы поля EmplId и LedgerAccountIdOffset. Возможно, для данного примера это не принципиально. Тем не менее стоит учитывать, что в дальнейшем возможны изменения приложения, которые затронут как раз процедуру создания строк журналов (например, в строки журнала будет добавлено новое поле, которое будет инициализироваться из заголовка журнала). Тогда после такого обновления Вам придется переписывать Вашу процедуру создания строк, чтобы включить в нее новую логику работы.
Кстати, можно обойтись без всех этих ритуальных танцев в JournalTableData и JournalTransData, а просто использовать

inventJournalTrans.initFromInventJournalTable(inventJournalTable);

Аналогично, вместо

inventJournalTrans.itemId               = inventTable.itemId;
inventJournalTrans.dimension            = inventTable.dimension;
inventJournalTrans.projCategoryId       = inventTable.projCategoryId;

лучше

inventJournalTrans.initFromInventTable(inventTable);

А вот так делать не стоит:

//Инициализируем складскую аналитику
...

Дело в том, что складскую аналитику таким образом Вы полностью затрете своим складом. А как быть с конфигурацией, например? В общем, перед тем, как инициализировать складскую аналитику, неплохо бы добавить еще одну строчку:

inventDim = inventJournalTrans.inventDim();

//Инициализируем активную аналитику из настроке журнала для расчета количества
    inventDimParm = InventJournalTable::journalId2inventDimParm(inventJournalTable.JournalId);

    //*** Специфика заполнения строки для журнала инвентаризации ***
    //Подсчитаем и запишем значение количества запасов в наличии на текущую дату
    inventJournalTrans.inventOnHand = InventSumDatePhysicalDim::onHandQty(
                                          inventJournalTrans.transDate,
                                          inventJournalTrans.itemId,
                                          inventDim,
                                          inventDimParm);

Очень сложно. Можно просто вот так:

inventJournalTrans.inventMovement().journalSetInventDimField();

//Заполним номер документа ГК (если необходимо)
if (inventJournalTable.voucherDraw == JournalVoucherDraw::Entering)
    journalTransData.initVoucher(voucher_last,false);

//Присвоим номер строки журнала
inventJournalTrans.lineNum  = InventJournalTrans::lastLineNum(#JournalNum) + 1;

//Добавим запись
// !!! Внимание - по умолчанию при добавлении записи в строку журнала формируется
// складская проводка. Чтобы этого избежать, необходимо вызывать insert c параметром true.
inventJournalTrans.insert();

//Подсчитаем итоги по журналу
journalTransData.addTotal(false,false);

Опять же, все придумано до нас. Используйте

inventJournalTrans.insertFromCode();

Кроме того, не понял, а почему может понадобиться не создавать складскую проводку при создании журнала?

В целом, хотелось бы отметить, что для работы с журналами в Аксапте уже существует очень широкий набор функций. Создание строк журнала из кода - вполне типичная задача, которая решена много раз. Стоит посмотреть, как это делается в стандартном приложении.

[mazzy]

Спасибо, Maxim Gorbunov.

Прежде всего, совет имел сугубо обучающую цель.
Читающий должен понять что именно надо делать для заполнения журнала.
Поэтому было расписано и разжевано до полей.

С чем согласен - конструкция initFrom...() и логически, и технически более правильная. Это я поправлю.

а замена inventOnHand и insertFromCode. Является правильной технически, но полностью затуманивает логику. Думаю, что оставим их в статье как есть. А твой совет выделим в дополнениях.

[mazzy]

Цитата:

Сообщение от Maxim Gorbunov

А вот так делать не стоит:

//Инициализируем складскую аналитику
...

Дело в том, что складскую аналитику таким образом Вы полностью затрете своим складом. А как быть с конфигурацией, например? В общем, перед тем, как инициализировать складскую аналитику, неплохо бы добавить еще одну строчку:

inventDim = inventJournalTrans.inventDim();

<{POST_SNAPBACK}>

Хм... может быть. Мы думали об этом.
Но эта строчка опять же таки сильно затуманивает смысл ОБУЧАЮЩЕГО совета.
Инициализация аналитики сделана нарочито элементарно.
Вообще говоря, корректная установка аналитики здесь должна добавить еще столько же кода... который не относится к журналу, а относится к работе со слакдской аналитикой. Думаю, что работа со складской аналитикой требует отдельного совета.

[mazzy]

Цитата:

Сообщение от Maxim Gorbunov

Кроме того, не понял, а почему может понадобиться не создавать складскую проводку при создании журнала?

<{POST_SNAPBACK}>

Таким образом Сергей дал описание этого параметра
А где ты еще видел его описание?

Пример скорее обучающий. Реальный должен быть раза в три длиннее.
Разве не так?

[Maxim Gorbunov]

Цитата:

Сообщение от mazzy

С чем согласен - конструкция initFrom...() и логически, и технически более правильная. Это я поправлю.

<{POST_SNAPBACK}>

Кроме того, хотелось бы заметить, что в экзамене по программированию есть вопросы на эту тему.

Цитата:

Сообщение от mazzy

а замена inventOnHand и insertFromCode. Является правильной технически, но полностью затуманивает логику.

<{POST_SNAPBACK}>

На счет inventOnHand - как знаешь. Фактически, в статье просто переписан код из метода journalSetInventDim(). И все-таки, ИМХО, даже в рамках обучающего примера стоит пользоваться функциональностью классов InventMovement.
По поводу insertFromCode() - не согласен. Это один из pattern'ов. Похожий метод есть не только у строк журналов, но и у строк заказов и закупок, например. Этот метод надо использовать. Особенно в обучающих примерах.

Цитата:

Сообщение от mazzy

Хм... может быть. Мы думали об этом.
Но эта строчка опять же таки сильно затуманивает смысл ОБУЧАЮЩЕГО совета.
Инициализация аналитики сделана нарочито элементарно.
Вообще говоря, корректная установка аналитики здесь должна добавить еще столько же кода... который не относится к журналу, а относится к работе со слакдской аналитикой. Думаю, что работа со складской аналитикой требует отдельного совета.

<{POST_SNAPBACK}>

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

Цитата:

Сообщение от mazzy

Цитата:

Сообщение от Maxim Gorbunov

Кроме того, не понял, а почему может понадобиться не создавать складскую проводку при создании журнала?

<{POST_SNAPBACK}>

Таким образом Сергей дал описание этого параметра
А где ты еще видел его описание?

<{POST_SNAPBACK}>

Да я вообще видел мало описаний для параметров методов Дело в том, что, не очевидно, что это описание параметра. Скорее, это звучит так, что де бывают ситуации, когда проводку создавать не надо. Вот тогда передавайте в этот метод false и будет Вам счастье. Я вот и поинтересовался, когда бывает нужно не создавать проводку по складскому журналу. Впрочем, ответ здесь такой: этот параметр используется сугубо для повышения быстродействия, когда Вы заранее знаете, что по строке складского журнала никаких проводок не должно быть создано (когда количество в строке равно 0).

Цитата:

Сообщение от mazzy

Пример скорее обучающий. Реальный должен быть раза в три длиннее.
Разве не так?

<{POST_SNAPBACK}>

Ну, на счет "в три раза" я не согласен. Пример-то в статье тоже взят, на самом деле, из реального кода. Фактически, это чуть-чуть дополненный метод InventCountCreate_Base.createInventJournalTrans(). Правда, из него выкинута вся работа со складской аналитикой. Конечно, то, что есть в стандартном методе - избыточно, но вот прямо всю логику выкидывать я бы не стал.

[mazzy]

Цитата:

Сообщение от Maxim Gorbunov

Ну, на счет "в три раза" я не согласен. Пример-то в статье тоже взят, на самом деле, из реального кода. Фактически, это чуть-чуть дополненный метод InventCountCreate_Base.createInventJournalTrans(). Правда, из него выкинута вся работа со складской аналитикой. Конечно, то, что есть в стандартном методе - избыточно, но вот прямо всю логику выкидывать я бы не стал.

<{POST_SNAPBACK}>

Да. Именно этот.
Да. Именно выкинули. Сознательно и намеренно.
Я добавил комментарий.

Я дополнил статью твоим особым мнением.
Не хочешь написать насчет работы со складской аналитикой.

[Maxim Gorbunov]

Цитата:

Сообщение от mazzy

Да. Именно выкинули. Сознательно и намеренно.

<{POST_SNAPBACK}>

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

Цитата:

Сообщение от mazzy

Не хочешь написать насчет работы со складской аналитикой.

<{POST_SNAPBACK}>

Нет смысла писать просто про работу со складской аналитикой. Нельзя рассматривать ее в отрыве от конкретной функции.

[Maxim Gorbunov]

Цитата:

Сообщение от mazzy

Я дополнил статью твоим особым мнением.

<{POST_SNAPBACK}>

Эх... Неточно я процитирован

Во-первых, в исходном примере надо писать не

inventJournalTrans.initFromJournalTable();

а

journalTransData.initFromJournalTable();

У inventJournalTrans такого метода просто нет. А вообще я предлагал вообще удалить все обращения к JournalTableData и JournalTransData и вместо строк:

//Инициализируем класс для операций с записью в таблице журнала
journalTableData    = JournalTableData::newTable(inventJournalTable);

//Очищаем поля записи
inventJournalTrans.clear();
// Инициализируем класс для операций с записью в таблице строк журнала
journalTransData    =
    journalTableData.JournalStatic().newJournalTransData(
                                         inventJournalTrans,
                                         journalTableData);

написать

inventJournalTrans.initFromInventJournalTable(inventJournalTable);

Ну, clear() можно оставить, конечно.

Во-вторых, "сложная конструкция, которая считает остаток", включает в себя так же инициализацию InventDimParm, которая, честно сказать, никому не нужна. В общем, из исходного текста надо убрать

//Инициализируем активную аналитику из настроке журнала для расчета количества
inventDimParm = InventJournalTable::journalId2inventDimParm(inventJournalTable.JournalId);

//*** Специфика заполнения строки для журнала инвентаризации ***
//Подсчитаем и запишем значение количества запасов в наличии на текущую дату
inventJournalTrans.inventOnHand = InventSumDatePhysicalDim::onHandQty(
                                      inventJournalTrans.transDate,
                                      inventJournalTrans.itemId,
                                      inventDim,
                                      inventDimParm);

а добавить

inventJournalTrans.inventMovement().journalSetInventDimField();

В итоге, мой вариант метода следующий:
<div class='XPPtop'>X++</div><div class='XPP'>#Define.JournalNum([color=:red]"000015_061"[/color])
#Define.ItemId([color=:red]"ESB-007"[/color])
#Define.LocationId([color=:red]"MW"[/color])
#Define.QtyCounted(101)

InventJournalTrans      inventJournalTrans;
InventJournalTable      inventJournalTable;
InventTable             inventTable;

InventDim               inventDim;
;

inventTable = inventTable::find(#ItemId);
[color=:blue]if[/color] (!inventTable)
 [color=:blue]throw[/color] error(strfmt(InventTable::txtNotExist(), #ItemId));

[color=:blue]ttsbegin[/color];
 inventJournalTable  = inventJournalTable::find(#JournalNum, [color=:blue]true[/color]);
 [color=:blue]if[/color] (!inventJournalTable)
   [color=:blue]throw[/color] error(strfmt(InventJournalTable::txtNotExist(), #JournalNum));

 inventJournalTrans.clear();
 inventJournalTrans.initFromInventJournalTable(inventJournalTable);
 inventJournalTrans.initFromInventTable(inventTable);
 inventJournalTrans.transDate = systemDateGet();

 inventDim = inventJournalTrans.inventDim();
 inventDim.InventLocationId = #LocationId;
 inventJournalTrans.inventDimId  =  InventDim::findOrCreate(inventDim).inventDimId;
 inventJournalTrans.inventMovement().journalSetInventDimField(inventDim, [color=:blue]fieldNum[/color](InventDim, InventLocationId));

 inventJournalTrans.Counted = #QtyCounted;
 inventJournalTrans.inventMovement().JournalSetCounted();
 inventJournalTrans.insertFromCode();
[color=:blue]ttscommit[/color];</div>

[mazzy]

Хм... В твоем варианте из демопримера полностью скрыт аспект подсчета итогов по журналу...
Однако ж, ты прав.

Хорошо. Убедил, черт языкастый.
Переделаю совет.