Случайно наткнулся на код

[Garic]

Axapta 3.0 SP3 HF1

Класс InventDimCtrl_Frm_Mov_Transfer слой (sys)

PHP код:

    tmpFieldId = inventDimControls.fieldId(_fc);
    if (curuserid() == 'fwt' && tmpFieldId == 7)
        tmpFieldId = tmpFieldId; 


интересно много ещё такого?

[Oz]

Ну что вы так уж привередничаете?
Этот код по крайней мере работает...

[George Nordic]

Это называется КОК Константно - Ориентированный Код
Много ли там такого, можно узнать, запустив поиск по "fwt"

С Уважением,
Георгий

[Alks]

Не знаю какой глубинный смысл стоит за словосочетанием "константно-ориентированный код", но смысл сего куска кода очевиден поставить на строчку tmpFieldId = tmpFieldId; брякпойнт и отлаживать случай когда он равен 7...


Мне гораздо более интересным показался с этой точки зрения метод decodeItemId класса Barcode. Смысл метода - найти в базе ItemId товара по отсканированному штрих-коду:

PHP код:

ItemId decodeItemId(str _barcodeStr = barcodeStr)
{
    ItemId              itemId = _barcodeStr;
    InventItemBarcode   inventItemBarcode = InventItemBarcode::findInputBarcode(_barcodeStr);
    InventTable         inventTable;
    ;
    if (inventItemBarcode)
        return inventItemBarcode.itemId;

    select firstonly inventTable
        index hint ItemIdx
        where inventTable.itemId == itemId;

    if (inventTable)
        itemId = inventTable.itemId;

    return itemId;
} 


Приглядитесь повнимательнее - кто увидит одну очень большую странность сего метода, тому будет счастье!

P.S.
AX 3.0 SP2

P.P.S
Хотя и тут я подозреваю как именно эта странность поимела место быть.

[demID]

Наверное, метод decodeItemId() писали какие-нибудь аутсорсеры, у которых оплата идет по количеству написанных строк кода

[George Nordic]

Возможено, речь сначала шла о
InventTable decodeItemId(str _barcodeStr = barcodeStr)

метод findInputBarcode возвращал ItemId, по которому происходил поиск в InventTable и возвращалась целая запись. Потом была оптимизация, и люди поняли, что нет надобности тянуть всю запись. У меня есть подозрения, что данный код - результат "оптимизации"..

2Alks Это код, завязанный на константы, прошитые в коде, в данном случае "fwt".

С Уважением,
Георгий.

[Alks]

Цитата:

Изначально опубликовано George Nordic
2Alks Это код, завязанный на константы, прошитые в коде, в данном случае "fwt".

Понятно.
Моя версия по поводу decodeItemId() исходит из того что у InventTable есть устаревшее поле DEL_BarCode. Соответственно ляп сей появился когда это поле устраняли, а до этого устранения участок кода видимо выглядел так:


select firstonly inventTable
index hint ItemIdx
where inventTable.BarCode == itemId;


Мораль: все мы люди.

[xonix]

2 Alks
Ну, положим, так то он точно не выглядел...
Минимум -не было hint ItemIdx

[Alks]

Цитата:

Изначально опубликовано xonix
2 Alks
Ну, положим, так то он точно не выглядел...
Минимум -не было hint ItemIdx

Да, точно - проглядел. См. мораль выше.

[Maxim Gorbunov]

Цитата:

Изначально опубликовано George Nordic
Много ли там такого, можно узнать, запустив поиск по "fwt"

Лучше использовать Add-Ins - Проверка Best Practices

Хотя в случае с Barcode никакие Best Practices не помогут. Как говорится, ошибка в ДНК.