The daily Axapta WTF

[dech]

Цитата:

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

Опять разбираюсь с SSRS
класс VendInvoiceDocumentDP

Там полно таких перлов. Вот еще пример похлеще:

private boolean printTaxTrans()
{
    if (taxInvoiceItemType == ItemType::Item)
    {
        return TaxTable::find(taxSpec.taxCode()).UnrealizedTax == NoYes::No;
    }
    else
    {
        return TaxTable::find(taxSpec.taxCode()).UnrealizedTax == NoYes::Yes;
    }
}

А здесь название говорит, что нужно что-то вернуть, однако...

private void inventDimLine()
{
    insertInventDim = true;
}

[George Nordic]

Цитата:

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

А здесь название говорит, что нужно что-то вернуть, однако...
[XPP]private void inventDimLine()

Что должен вернуть void???

[dech]

Цитата:

Сообщение от George Nordic

Что должен вернуть void???

Я не про тип, а про название. В названии должен быть хотя бы префикс set, иначе метод выглядит как возвращающий нечто (в данном случае похожее на запись InventDim).
Второе, зачем в private-метод пихать единственную операцию присваивания? В контексте вызов подобного метода только дизориентирует. Другое дело, если бы метод вызывался наследниками, хотя все равно над названием надо было лучше думать.
Еще нарыл в VendInvoiceDocument.main() 2 условия по Voucher

            select firstOnly TableId, RecId from taxEngineLedgerJournalTransHeader
                join RecId from ledgerJournalTransLoc
                    where ledgerJournalTransLoc.AccountType    == LedgerJournalACType::Vend
                        && ledgerJournalTransLoc.JournalNum    == taxEngineLedgerJournalTransHeader.JournalNum
                        && ledgerJournalTransLoc.Voucher       == taxEngineLedgerJournalTransHeader.Voucher
                        && ledgerJournalTransLoc.Invoice       == vendInvoiceJournal.InvoiceId
                        && ledgerJournalTransLoc.Voucher       == vendInvoiceJournal.LedgerVoucher;

Кроме того, TableId не нужно указывать в списке полей селекта.