if (a == true)

[SRF]

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

Но к примеру на JavaScript, можно просто завести переменную, не инициализировать ее и фактически будет три значения true, false, null поэтому для упрощения можно использовать сравнение с какой либо константой, отметая два других значения, например, != true.

var flagTest;
//..
flagTest=$("CheckBoxTest").selected;
//...

if(flagTest !=true) {

}

И еще в C# существует хитрый тип bool? - логический с null(bool + null), так вот такая рода конструкция, в C# просто не откомпилируется, потребует явного приведения типа

bool? test = null;

if (test)
{ 
            
}

тогда как

bool? test = null;

if (test == true)
{ 
            
}

не вызовет никаких нареканий, примечательно, что такое неявное преобразование работает насколько я понимаю только для типа bool?, например если во втором случае написать int, то ничего не прокатит

[kornix]

В PriceDisc.findPrice():

return ((findAll     && inventDimAllActivated.RecId     && this.findPriceAgreement(_priceGroupId, inventDimAllActivated.inventDimId))       ||
            (findItemDim && inventDimItemDimActivated.recId && this.findPriceAgreement(_priceGroupId, inventDimItemDimActivated.inventDimId))   ||
                                                               this.findPriceAgreement(_priceGroupId, InventDim::inventDimIdBlank())            ||
             this.findItemPrice());

На мой взгляд это еще хуже чем написал fed.
В return неудобно анализировать такой код.

В классе sysDatabaseList метод new:

...
while (1 == 1 && listOfNames)
{
}
...

[Napalm]

Общеизвестно, что для работы разработчиком Dynamics AX не обязательно знать X++ или уметь программировать.

Пример из LedgerJournalEngine\projQtyModified (AX 2012):

// This switch handles the normal(default) case and 2 other special cases
switch(true)
{
    case _ledgerJournalTrans_Project.Qty == 0:
        _ledgerJournalTrans.AmountCurCredit = 0;
        _ledgerJournalTrans.AmountCurDebit = 0;
        break;

    // If Credit/Debit Amounts are both 0, the Cost Price would be incorrectly set to 0 in the normal case
    // Handle this case separately
    case !_ledgerJournalTrans.AmountCurDebit && !_ledgerJournalTrans.AmountCurCredit && _ledgerJournalTrans_Project.CostPrice:
        this.projRecalcAmountCurDebitCredit(_ledgerJournalTrans, _ledgerJournalTrans_Project);
        break;

    default:
        // this.amount gets the debit amount or the negative credit amount
        offsetFactor = this.projOffsetFactor(_ledgerJournalTrans, _ledgerJournalTrans_Project);
        _ledgerJournalTrans_Project.CostPrice = Currency::price((offsetFactor * _ledgerJournalTrans.amount())/_ledgerJournalTrans_Project.Qty);
        // Cost Price should always be positive
        if (_ledgerJournalTrans_Project.CostPrice < 0)
            _ledgerJournalTrans_Project.CostPrice = - _ledgerJournalTrans_Project.CostPrice;
        // Cost Price have changed, recalc debit and credit amounts to avoid rounding diff
        this.projRecalcAmountCurDebitCredit(_ledgerJournalTrans, _ledgerJournalTrans_Project);
        break;
}

PS: Рекомендовано Microsoft! http://msdn.microsoft.com/en-us/libr...(v=ax.50).aspx

[fed]

Цитата:

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

Общеизвестно, что для работы разработчиком Dynamics AX не обязательно знать X++ или уметь программировать.

Пример из LedgerJournalEngine\projQtyModified (AX 2012):

// This switch handles the normal(default) case and 2 other special cases
switch(true)
{
    case _ledgerJournalTrans_Project.Qty == 0:
        _ledgerJournalTrans.AmountCurCredit = 0;
        _ledgerJournalTrans.AmountCurDebit = 0;
        break;

    // If Credit/Debit Amounts are both 0, the Cost Price would be incorrectly set to 0 in the normal case
    // Handle this case separately
    case !_ledgerJournalTrans.AmountCurDebit && !_ledgerJournalTrans.AmountCurCredit && _ledgerJournalTrans_Project.CostPrice:
        this.projRecalcAmountCurDebitCredit(_ledgerJournalTrans, _ledgerJournalTrans_Project);
        break;

    default:
        // this.amount gets the debit amount or the negative credit amount
        offsetFactor = this.projOffsetFactor(_ledgerJournalTrans, _ledgerJournalTrans_Project);
        _ledgerJournalTrans_Project.CostPrice = Currency::price((offsetFactor * _ledgerJournalTrans.amount())/_ledgerJournalTrans_Project.Qty);
        // Cost Price should always be positive
        if (_ledgerJournalTrans_Project.CostPrice < 0)
            _ledgerJournalTrans_Project.CostPrice = - _ledgerJournalTrans_Project.CostPrice;
        // Cost Price have changed, recalc debit and credit amounts to avoid rounding diff
        this.projRecalcAmountCurDebitCredit(_ledgerJournalTrans, _ledgerJournalTrans_Project);
        break;
}

PS: Рекомендовано Microsoft! http://msdn.microsoft.com/en-us/libr...(v=ax.50).aspx

Ну это такой любопытный побочный эффект от использования оператора switch(). Сама по себе конструкция абсолютно легальна - пробегает по веткам case и заходит в первую ветку с выполненным условием. Теоретически может быть заменено на серию вложенных if() {} else{ if() {} else {if () {}}, однако вариант с switch() превосходит их по читаемости. На мой взгляд все легально.
Для раздумья о нелегкой судьбе оператора switch() в различных языках программирования, рекомендую прочитать: Duff's device

[Napalm]

Цитата:

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

Теоретически может быть заменено на серию вложенных if() {} else{ if() {} else {if () {}}, однако вариант с switch() превосходит их по читаемости. На мой взгляд все легально.

"if (a == true)" - тоже легально.