kamalblogs: Using while select firstonly to avoid validations in Dynamics Ax

[Blog bot]

Источник: http://kamalblogs.wordpress.com/2010...n-dynamics-ax/
==============

When you execute a select statement to retrieve a single record most of us right the query first. This is followed by a validation to ensure that the query has returned a record. we make it simpler If we right it the way mentioned below…

static void WhileSelectInsteadoFSelect(Args _args)
{
    InventTable inventTable;
    ;
-----General way---------
    //fetch the record first
    select firstonly forupdate inventtable;

    //add an additional validation
    if (inventTable.recid)
    {
        inventTable.itemName = 'newname';
        inventTable.update();
    }

------Alternate--------------
    //prevents the extra 'if' chek
    while select firstonly forupdate inventTable
    {
        inventTable.itemName = 'newname';
        inventTable.update();
    }

    //can also be used for simple readonly
    while select firstonly inventTable
    {
        Info(inventTable.ItemId);
    }
}

A few of you might already follow it but for the rest it could be a learning





Источник: http://kamalblogs.wordpress.com/2010...n-dynamics-ax/

[gl00mie]

По-моему, не все так однозначно. Эта тема обсуждалась также в блоге Эрика Липперта, в заметке Style follows semantics:

Цитата:

The stylistic choices should typically be driven by a desire to clearly communicate the semantics of the program fragment.

Мне кажется, в данном случае использование while может при беглом просмотре кода сбить с толку, поэтому вариант с простым select firstonly и проверкой результата все же предпочтительнее за счет большей выразительности. Кроме того, с точки зрения последующей возможной модификации код лучше сразу писать так, чтобы для его изменения нужно было править как можно меньше строк исходного кода Х++; совсем хорошо, если модификации можно просто дописать в нужном месте, не изменяя строки исходного код Х++ вовсе. Так вот, если понадобится, к примеру, добавить ветвление для случая, когда запись не найдена, то в варианте с if можно будет просто дописать блок else {}, а для while select придется изголяться - к примеру, вводить какую-то переменную-флаг, изменять ее внутри цикла и затем анализировать - вся экономия строк кода моментально улетучится, а читабельности это все совсем не прибавит.

[petr]

Я использую вариант, упомянутый в блоге в Job'ax и стандарный акспатовский подход (с отдельным селектом без while и последующей проверкой выбранной записи) в бизнес-логике.

[TasmanianDevil]

Бирюльки дурацкие ...

[petr]

Цитата:

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

Бирюльки дурацкие ...

Нет, просто понимание того, как работает аксапта и когда какой инструмент надо использовать.

[kashperuk]

Я бы не рекомендовал использовать такой синтаксис.
Кто его знает, может в АХ следующих версий компилятор будет запрещать использование firstonly в while select

[TasmanianDevil]

Цитата:

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

Нет, просто понимание того, как работает аксапта и когда какой инструмент надо использовать.

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