|
30.08.2019, 09:10 | #1 |
Участник
|
Там полно таких перлов. Вот еще пример похлеще:
X++: private boolean printTaxTrans() { if (taxInvoiceItemType == ItemType::Item) { return TaxTable::find(taxSpec.taxCode()).UnrealizedTax == NoYes::No; } else { return TaxTable::find(taxSpec.taxCode()).UnrealizedTax == NoYes::Yes; } } X++: private void inventDimLine() { insertInventDim = true; }
__________________
// no comments Последний раз редактировалось dech; 30.08.2019 в 09:13. |
|
30.08.2019, 09:28 | #2 |
Модератор
|
|
|
30.08.2019, 11:06 | #3 |
Участник
|
Я не про тип, а про название. В названии должен быть хотя бы префикс set, иначе метод выглядит как возвращающий нечто (в данном случае похожее на запись InventDim).
Второе, зачем в private-метод пихать единственную операцию присваивания? В контексте вызов подобного метода только дизориентирует. Другое дело, если бы метод вызывался наследниками, хотя все равно над названием надо было лучше думать. Еще нарыл в VendInvoiceDocument.main() 2 условия по Voucher X++: 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;
__________________
// no comments Последний раз редактировалось dech; 30.08.2019 в 11:10. |
|