If, else funkcyjnie :: 4programmers.net

2018-09-12T11:43:28+02:00

Rejestracja: dni
Ostatnio: dni

Witam wszystkich mam oto taką metodę która przelicza kursy walut czy opłaca się ją przerobić jakoś funkcyjnie teraz jest to takie modne xd . Głównie o to chodzi czy wydajnościowo da się to zrobić lepiej o to fragment kodu

Kopiuj


BigDecimal getAccountAmountInPLN(final Account account) {
        if (account.getCurrency() == EUR)
            return account.getAmount().multiply(new BigDecimal(Currency.EUR.rate)).setScale(3, 1);
        else if (account.getCurrency() == USD)
            return account.getAmount().multiply(new BigDecimal(Currency.USD.rate)).setScale(3, 1);
        else if (account.getCurrency() == CHF)
            return account.getAmount().multiply(new BigDecimal(Currency.CHF.rate)).setScale(3, 1);
        else
            return account.getAmount().setScale(2);
    }

pavarotti

2018-09-12T11:56:59+02:00

Rejestracja: dni
Ostatnio: dni
Postów: 288

Może użyć switch?

workout

2018-09-12T12:00:00+02:00

Rejestracja: dni
Ostatnio: dni
Postów: 34

Czemu nie zrefaktorujesz tego zeby to wygladalo mniej wiecej tak?

Kopiuj

BigDecimal getAccountAmountInPLN(final Account account) {
        if ( account.getCurrency() == PLN ){
            return account.getAmount().setScale(2);
        }
        return account.getAmount().multiply(new BigDecimal(getCurrency().rate)).setScale(3, 1);
    }

yarel

2018-09-12T12:39:34+02:00

Rejestracja: dni
Ostatnio: dni
Postów: 2384

A co u Ciebie zwraca getAmount()? JSR-354 nie dostarcza wszystkiego co potrzebujesz do takich konwersji?

Kopiuj

    MonetaryAmount amount = ...;
    CurrencyConversion conversion = MonetaryConversions.getConversion("CHF");
    MonetaryAmount amountInCHF = amount.with(conversion);

Zaokrąglenia pewnie też można robić...