SysUserInfo find

[wojzeh]

коллеги, может кто-нибудь мне объяснить, зачем в 4-й версии аксапты метод find всеми нами уважаемой таблицы SysUserInfo пытается чего-то внутри себя создавать?

как-то это выглядит против джентльменских правил.

static SysUserInfo find(userId _userId = curuserid(), boolean _forupdate = false)
{
    SysUserInfo parameter;

    parameter.selectForUpdate(_forupdate);

    select firstonly parameter
        index UserIdx
        where parameter.Id == _userId;

    if (!parameter && !isConfigMode())  // Cannot create parameter before all tables are synced.
    {
        Company::createParameter(parameter);
    }

    return parameter;
}

[Mykola Galak]

Цитата:

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

коллеги, может кто-нибудь мне объяснить, зачем в 4-й версии аксапты метод find всеми нами уважаемого класса SysUserInfo пытается чего-то внутри себя создавать?

Так делают в случаях, когда надо заполнить табличку с параметрами дефолтными значениями при первом обращении к параметрам. Поищите по другим таблицам - такой подход используется довольно часто.

[wojzeh]

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

насколько я помню, каждая таблица должна иметь метод find, но нигде не говорилось, что он должен что-то создавать при этом. оправданным был бы подход в создании либо метода findAndCreateIfNotExist или хотя бы параметра наподобие forUpdate.

[gl00mie]

Цитата:

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

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

Пожалуй, код реализован некорректно, потому что пытается всегда создать запись для текущего пользователя, даже если передать ему в качестве _userId код другого пользователя, для которого запись еще не создана. Но одна некорректная реализация не означает, что используемый подход инициализации данных при первом обращении (т.н. ленивая инициализация) в целом не имеет права на жизнь.

Цитата:

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

насколько я помню, каждая таблица должна иметь метод find, но нигде не говорилось, что он должен что-то создавать при этом. оправданным был бы подход в создании либо метода findAndCreateIfNotExist или хотя бы параметра наподобие forUpdate.

Посмотрите по перекрестным ссылкам, в скольки местах вызывается Company::createParameter(), - это более чем 90 параметрических таблиц. Если бы для этой инициализации использовался некий отдельнный метод findAndCreateIfNotExist() либо специальный параметр, то пришлось во всем коде вместо find() для этих таблиц вызывать этот отдельнный метод либо, соответственно, всегда передавать этот параметр, поскольку неизвестно заранее, какой кусок кода первым обратится к той или иной параметрической таблице.