ax2012: а зачем табличные переменные инициализируют null'ом?

[mazzy]

Я щас глупость спрошу. Но который раз вижу и который раз удивляюсь.
в ax2012 табличные переменные постоянно инициализируют null'ом.
А зачем?

пример метод в таблице EcoResCategory.getChildren

[AraraT®]

А табличные переменные иерархических таблиц или и обычных?

[mazzy]

хорошая тема. надо подумать.

пока видел обычные таблицы.

[gl00mie]

Я лично инициализирую табличные переменные null'ом, чтоб не было предупреждений при компиляции "переменная не записана, но прочитана", ибо достает.

[Napalm]

Возможно это бездумное копирование кода связанного с "changeCompany".

http://msdn.microsoft.com/en-us/library/aa860755.aspx

[gl00mie]

Вроде бы для кода, связанного с changecompany, сбрасывать табличные переменные актуально только после хотя бы одного select'а - внутри какого-нить цикла перебора компаний. В примере на msdn внутри changeCompany('bbb') {} сбрасывать табличный буфер избыточно.

[ice]

еще есть нововведение, возврат null вместо пустой не найденной строки, такие места дополняют комментарием

static EcoResSize resolveProductVariantSize(
    EcoResSizeName     _sizeName,
    RefRecId            _productMaster,
    RefRecId            _productDimensionAttribute)
{
    EcoResSize              ecoResSize;
    EcoResProductMasterSize ecoResProductMasterSize;

    select ecoResSize
        where ecoResSize.Name == _sizeName
    exists join ecoResProductMasterSize
        where ecoResProductMasterSize.Size                          == ecoResSize.RecId
           && ecoResProductMasterSize.SizeProductMaster             == _productMaster
           && ecoResProductMasterSize.SizeProductDimensionAttribute == _productDimensionAttribute;

    if (!ecoResSize.RecId)
    {
        // Assigning null to selectedRecord does not cause the kernel to receive a true NULL value, but rather
        // just an empty record. Return NULL directly so the kernel knows that the intent is to not clear the
        // selected value.
        return null;
    }

    return ecoResSize;
}

[gl00mie]

Индусский новодел

[S.Kuskov]

Не ответ на вопрос но тоже интересные наблюдения по поводу присваивания null Common.reread() - альтернатива для findRecId?

[kashperuk]

Все, что касается ecoResCategory - "говнокод", но вы от меня это не слышали

Никакого смысла инициализировать так переменные при объявлении нет, imho.

[Ace of Database]

Это андроид-программисты код писали.

PS
Я сейчас интенсивно программирую на Яве, там надо внимательно следить за тем, чтобы движок не посчитал два разных объекта идентичными и наоборот, не посчитал два одинаковых объекта разными. Например выражение "Вася" == "Вася" имеет истину только в том случае, если обе этих строки хранятся по одному физическому адресу. Чтобы все работало правильно, надо писать "Вася".equals("Вася")

[dech]

Как уже говорилось, явное обнуление переменных делается, чтобы не возникало предупреждений ВР. Вообще, для табличных переменных было бы достаточно метода clear(), поскольку они все равно уже имеют инстанс сразу после объявления и присваивание null кажется совсем нелогичным.

[Logger]

Цитата:

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

Как уже говорилось, явное обнуление переменных делается, чтобы не возникало предупреждений ВР. Вообще, для табличных переменных было бы достаточно метода clear(), поскольку они все равно уже имеют инстанс сразу после объявления и присваивание null кажется совсем нелогичным.

Да это же явный косяк реализации проверки BP.
Вот её и надо поправить.

[Evgeniy2020]

я за null в трешке, если используется changeCompany. в 2007 ом году напоролся при построении консолидированных отчетов (в теле которых переключались компании changeCompany()). Тогда в отчетах поплыли суммы. Когда разобрался - добавил = null;
как в Ах 2012 уже не знаю не тестировал. Возможно в АХ 2012 это уже излишне.

[Logger]

Цитата:

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

я за null в трешке, если используется changeCompany. в 2007 ом году напоролся при построении консолидированных отчетов (в теле которых переключались компании changeCompany()). Тогда в отчетах поплыли суммы. Когда разобрался - добавил = null;
как в Ах 2012 уже не знаю не тестировал. Возможно в АХ 2012 это уже излишне.

Да это известный глюк.
Только обнулять надо внутри блока changCompany
А просто присвоить при объявлении как в процитированном примере - не поможет.

Я вообще стараюсь тело changecompany выносить в отдельный метод. Тогда все обнуления автоматом происходят. Так надежнее.