Доступ к DataSource формы из класса ... (+)

[topgun]

Прошу сильно не ругать за вероятно простой вопрос.
Однако только начал разбираться с Axapta.

На форме по кнопке вызываю класс.
В нем добавляю строки в таблицу на форме.
Как после добавления, перечитать данные в таблице?

Спасибо.

[macklakov]

Объекту знать, кто его вызвал, такой же плохой стиль, как отношение многие ко многим в базе. На кнопке, переопредели clicked, и обновляй ds, после super.
P.S. Более того, по идее, класс должен исполняться на сервере, а форма на клиенте и передача лишних данных между ними не желательна.

[topgun]

Цитата:

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

Объекту знать, кто его вызвал, такой же плохой стиль, как отношение многие ко многим в базе.

Т.е. вот такой код, это моветон?

PHP код:

static void main(Args _args)
{
    LedgerJournalTransCopy ledgerJournalTransCopy;
    LedgerJournalEngine     ledgerJournalEngine = _args.caller().ledgerJournalEngine();
    ;
 
    ledgerJournalTransCopy = new LedgerJournalTransCopy(_args);
    if (ledgerJournalTransCopy.prompt())
        ledgerJournalTransCopy.run();
 
    ledgerJournalEngine.refreshWhenPosted(); -- <-- ВОТ ЭТО
} 


[macklakov]

Цитата:

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

Т.е. вот такой код, это моветон?

PHP код:

static void main(Args _args)
{
    LedgerJournalTransCopy ledgerJournalTransCopy;
    LedgerJournalEngine     ledgerJournalEngine = _args.caller().ledgerJournalEngine();
    ;
 
    ledgerJournalTransCopy = new LedgerJournalTransCopy(_args);
    if (ledgerJournalTransCopy.prompt())
        ledgerJournalTransCopy.run();
 
    ledgerJournalEngine.refreshWhenPosted(); -- <-- ВОТ ЭТО
} 


У меня нет класса LedgerJournalTransCopy, но насколько я понимаю, этот метод вызывается не из ledgerJournalEngine, а из другого объекта.

[st_msav]

Цитата:

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

Объекту знать, кто его вызвал, такой же плохой стиль, как отношение многие ко многим в базе. На кнопке, переопредели clicked, и обновляй ds, после super.
P.S. Более того, по идее, класс должен исполняться на сервере, а форма на клиенте и передача лишних данных между ними не желательна.

А если пунктов меню, вызавающих класс будет много, то для каждого из них надо переопределять. По-моему проще это сделать в классе.

[sukhanchik]

2topgun:
Есть понятие Best Practice. Это рекомендации по стилю программирования. Им не всегда следуют, не все следуют, однако следует учитывать:
1. Сам факт наличия Best Practice означает что Аксапта заточена под определенный механизм решения проблем. И отклонение от Best Practice может приводить к выявлению ряда случаев когда Аксапта "глючит". При максимальном следовании Best Practice - ни пользователь ни программист не будут сталкиваться (ну или максимально редко) с неожиданными никому неизвестными невоспроизводимым поведением.
2. Ваш код могут читать впоследствии другие люди. Которым привычен Best Practice. И Ваше отклонение - только затруднит чтение.

Резюме:
Что предлагал Юра (macklakov):
Идем в форму - доходим до контрольки (MenuItemButton), перекрываем метод clicked, пишем код:

        super();
        MyDataSource_ds.executeQuery(); // Или reread, refresh

При этом к MenuItemButton прилеплена менюшка, вызывающая класс. Обычно (согласно Best Practice) - этот класс должен вызываться на сервере - тк подразумевает себе некую обработку, а если есть обработка - значит она д.б. на сервере. И с сервера обращаться к форме (к клиенту) - не есть правильно.

Что имелось ввиду под фразой "это используется сплошь и рядом":
Аксапту писали люди. Не везде следуя единому стилю. Местами даже может быть криво. Местами - нормально. Поэтому:
Если изначально создается новая форма, к ней новый класс и т.д. - то нужно следовать рекомендациям Best Practice.
Если приходится править уже написанный код (как в этом случае) - то в каждом случае надо разбираться отдельно и стараться сделать в стиле уже написанного кода не забывая про производительность (клиент-сервер). Но по возможности следует вспоминать про Best Practice

[macklakov]

Да, выложи, пожалуйста :-)

[Wamr]

BP - не догма
Примеров, когда в main вытаскивается DS переданого курсора и после обработки обновляется его содержимое в коде Аксапты много (SalesFormLetter, PurchFormLetter)
Сделайте поиск по методам main содержащим FormDataSource

[belugin]

static void main(Args _args)
{
    MyClass action=new MyClass();
    FormDataSource ds;
    ;
    if(action.prompt())
    {
        action.run();
        if(_args && _args.dataset()==tableNum(TheTable))
        {
            ds=_args.record().dataSource();
            ds.research();
        }
    }
}

[macklakov]

Цитата:

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

        if(_args && _args.dataset()==tableNum(TheTable))

А если на форме несколько dataSources, на основании таблицы TheTable? К примеру, форма InventTable, таблица InventTableModule
Никто не говорит, что это невозможно, но в данном случае, это не желательно. Ведь этот класс может быть вызыван не из формы, либо из совсем другой формы и мы получим неприятную runtime ошибку. Либо мы получим монструозный код, типа SalesFormLetter::main, большей частью написанный с помощью copy/paste и опять таки он остается опасным, т.к. на форме может оказаться несколько dataSources на основании одной и той же таблицы.
IMHO: Подобные конструкции оправданны лишь в классах, которые отвечают за внешний вид формы, к примеру, InventDimCtrl_Frm

[belugin]

мне кажется что существует распространенный частный случай, когда в форме только один датасурс такого типа.

Как альтернатива, предлагаю реализовать небольшой фреймворк на основе паттерна Observer который бы позволял подписываться на изменения такого рода и подписывать все формы из
SysSetupFormRun.run

[macklakov]

Цитата:

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

Как альтернатива, предлагаю реализовать небольшой фреймворк на основе паттерна Observer который бы позволял подписываться на изменения такого рода и подписывать все формы из
SysSetupFormRun.run

Интересная идея. Но по какому событию мы будем обновлять подписанные формы? Наверное, по изменению данных в таблице. Таблица может обновляться на сервере, а подписанные формы на клиентах. Теперь предположим, что два пользователя находятся в одной форме и один из них редактирует/просматривает данные, а другой запускает обработчик, котрый по исполнении обновит подписанные формы, при этом первый пользователь ругается.
Поправьте, если я ошибся.

[belugin]

я пошутил

[macklakov]

Цитата:

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

2 macklacov

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

в том то и дело, что приходится дополнительные сущности добавлять, чтоб их разрулить

Цитата:

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

А то, что класс ДОЛЖЕН исполняться на сервере, будь любезен, расскажи по-подробнее, пож. Откуда такие сведения?

Цитата:

Сообщение от Axapta Application Development Best Practices

Where to place the Code

Object Server
This is the home of the business application logic. Transaction oriented database updating jobs should be paced to run here; close to the database.

[macklakov]

Цитата:

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

Как раз тот случай, не находишь?

Не нахожу, т.к. в изначальном посте указывается, что класс создает несколько записей в базе

[macklakov]

Цитата:

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

Вполне возможно, что create-ом и сохранением строки или нескольких строк формы траффик будет намного ниже.

Необычная техника. Можешь привести мне примеры в коде, где так делают?