If, else funkcyjnie :: 4programmers.net

Pasjonat

2018-09-12 09:43

Pasjonat2018-09-12 09:43

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);
    }

Java

jarekr000000 2018-09-12 09:47

To nie ma nic wspólnego z modą, ani funkcyjnym. Ta metoda jest po prostu dość średnia.

pavarotti

2018-09-12 09:56

~~pavarotti~~2018-09-12 09:56

Rejestracja:ponad 6 lat
Ostatnio:prawie 3 lata
Postów:288

Może użyć switch?

workout

2018-09-12 10:00

workout2018-09-12 10:00

Rejestracja:ponad 6 lat
Ostatnio:prawie 6 lat
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);
    }

edytowany 1x, ostatnio: workout 2018-09-12 10:00

yarel

2018-09-12 10:39

yarel2018-09-12 10:39

Rejestracja:prawie 10 lat
Ostatnio:13 dni
Postów:2370

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ć...