Осторожно! Грабли!

[Wamr]

Пару раз наступив, хочу предостеречь коллег.

Стандартная схема:
1. Есть форма с гридом и есть кнопочка, вызывающая какую-то обработку текущей строки
2. При нажатии кнопочки запускается процесс, где в классе (форме) обработки выполняется строка
myTable = args().record();
3. В процессе выводится некий диалог с пользователем.
4. Делается окончательная обработка с использованием курсора myTable

Все работает замечательно... но в один "прекрасный" день пользователь на шаге 3 за какой-то надобностью полез обратно в форму с гридом и перешел на другую запись.
В результате на шаге 4 мы используем уже другую(!!!) запись из myTable.

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

Решение:
На шаге 2 использовать строку
myTable.data(args.record());

[andrey_k]

а может ето не грабли а фича

[Wamr]

Есть формочка с табличкой MyTable
Есть кнопка на форме с таким методом:

PHP код:

void clicked()
{;
    super();
    MyTable.changeStatus();
} 


На табличке есть метод:

PHP код:

void changeStatus()
{
    ttsbegin;

    this.Status = MyStatus::Save;

    ...  // (1) куча всяких изменений в других таблицах

    throw error("Делаем ошибку!");

    ... // (2) еще изменения в this
    
    this.update();
   
    ttscommit;
} 


В результате получается, что изменения (1) откатываются, изменения (2) не выполняются, НО статус меняется!!!

[mazzy]

Цитата:

Изначально опубликовано Wamr
...НО статус меняется!!!

Статус меняется ТОЛЬКО в контроле на форме.
В базу данных изменения не пошли. Если нажать F5, то это будет видно.

Решение? Следовать рекомендациям Best Practice и не программировать в формах

[Wamr]

Цитата:

В базу данных изменения не пошли. Если нажать F5, то это будет видно

А вот если перейти на другую запись, то изменения (статус) радостно запишутся в БД.
Большинство пользователей поступает именно так.

Цитата:

Следовать рекомендациям Best Practice и не программировать в формах

А в каком месте были нарушены BPs и как можно еще меньше (1 строка) программировать в формах?
(Кстати, форма создана господами внедренцами)

P.S. Для себя я еще раз убедился, что нужно избегать прямого использования курсора, взятого из DS формы.

[mazzy]

Цитата:

Изначально опубликовано Wamr
А вот если перейти на другую запись, то изменения (статус) радостно запишутся в БД. Большинство пользователей поступает именно так.

Хм... надо подумать. Здесь что-то не так.

Цитата:

Изначально опубликовано Wamr
А в каком месте были нарушены BPs и как можно еще меньше (1 строка) программировать в формах?

Виноват, чушь спорол. Не поглядел внимательно.
Код действительно написан в соответствии с BP.