Создание новых записей и номерные серии

[S.Kuskov]

Цитата:

Сообщение от Владимир Максимов

Если же нажать кнопку "Создать", то будет вызван метод DataSource.create(), а уже из него (внутри super()) будет вызван DataSource.write().

Удивительно.

Я честно говоря, думал что речь про ситуацию, при которой уход с записи DataSource.leaveRecord() не вызывает её сохранение.

По вашим же словам проблема возникает из-за того что leaveRecord первой записи вызывается позже чем Сreate для второй.

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

[Владимир Максимов]

Цитата:

Сообщение от S.Kuskov

По вашим же словам проблема возникает из-за того что leaveRecord первой записи вызывается позже чем Сreate для второй.

Да. Именно так.

Цитата:

Сообщение от S.Kuskov

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

А вот это не так

Внутри метода Create() сначала будут выполнены все методы обработки первой записи и только потом выполняются методы по созданию второй записи. Здесь все корректно и никаких проблем нет.

Проблема в том, что обработчик номерной серии - это некая внешняя "приблуда" не имеющая вообще никакого отношения к стандартным методам ядра. По факту, метод formMethodDataSourceCreatePre() выполняется слишком рано. Он должен был бы быть выполнен после DataSource.write(). Но влезть внутрь super() невозможно. Из-за чего и получаем проблему.

Как другой "костыль" - это "взведение" дополнительного флага внутри Create и его обработка в write

[S.Kuskov]

Цитата:

Сообщение от Владимир Максимов

Внутри метода Create() сначала будут выполнены все методы обработки первой записи и только потом...

Ну как же? В оригинале вызов numberSeqFormHandler().formMethodDataSourceCreatePre() выполняется в самом начале метода Create(), т.е. до данных обработок. Другое дело, что это всего-лишь CreatePre, в котором по идее не должно быть критических операций.

[Владимир Максимов]

Цитата:

Сообщение от S.Kuskov

Ну как же? В оригинале вызов numberSeqFormHandler().formMethodDataSourceCreatePre() выполняется в самом начале метода Create(), т.е. до данных обработок. Другое дело, что это всего-лишь CreatePre, в котором по идее не должно быть критических операций.

По факту, вызов методов leaveRecord(), validateWrite(), write() для старой записи происходит внутри super() в методе create().

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

В результате, метод numberSeqFormHandler().formMethodDataSourceCreatePre() в существующем наполнении - не просто не имеет смысла (занимается не своим делом), но приводит к описанной проблеме.

Другими словами, вызов данного метода следует вообще исключить из процедуры create() на DataSource формы. В идеале, удалить этот метод из класса NumberSeqFormHandler, поскольку в существующей реализации он вводит в заблуждение разработчика. Ведь по факту, этот метод выполняется не перед созданием новой записи, а перед сохранением текущей записи. Что совсем не одно и то же...

Хотя, конечно, есть вероятность того, что новое значение номерной серии было создано вне стандартного функционала создания записей. Например, через программный вызов. Но контролировать эту ситуацию в методе CreatePre() явно избыточно. Избыточный контроль, который добавляет проблем, а не решает их...