Ho il seguente codice in Java;
BigDecimal price; // assigned elsewhere
if (price.compareTo(new BigDecimal("0.00")) == 0) {
return true;
}
Qual è il modo migliore per scrivere la condizione if?
Ho il seguente codice in Java;
BigDecimal price; // assigned elsewhere
if (price.compareTo(new BigDecimal("0.00")) == 0) {
return true;
}
Qual è il modo migliore per scrivere la condizione if?
Risposte:
Usa compareTo(BigDecimal.ZERO)
invece di equals()
:
if (price.compareTo(BigDecimal.ZERO) == 0) // see below
Il confronto con la BigDecimal
costante BigDecimal.ZERO
evita di dover costruire new BigDecimal(0)
ogni esecuzione.
Cordiali saluti, BigDecimal
ha anche costanti BigDecimal.ONE
e BigDecimal.TEN
per vostra comodità.
Il motivo per cui non BigDecimal#equals()
è possibile utilizzare è che prende in considerazione la scala :
new BigDecimal("0").equals(BigDecimal.ZERO) // true
new BigDecimal("0.00").equals(BigDecimal.ZERO) // false!
quindi non è adatto per un confronto puramente numerico. Tuttavia, BigDecimal.compareTo()
non considera la scala quando si confronta:
new BigDecimal("0").compareTo(BigDecimal.ZERO) == 0 // true
new BigDecimal("0.00").compareTo(BigDecimal.ZERO) == 0 // true
In alternativa, è possibile utilizzare signum () :
if (price.signum() == 0) {
return true;
}
BigDecimal.ZERO.compareTo(null)
lanci NPE
C'è una costante che puoi verificare contro:
someBigDecimal.compareTo(BigDecimal.ZERO) == 0
equals
e compareTo
non è come pensi. docs.oracle.com/javase/1.5.0/docs/api/java/math/…
In alternativa, penso che valga la pena ricordare che il comportamento di uguali e di confronto con i metodi della classe BigDecimal non sono coerenti tra loro .
Ciò significa sostanzialmente che:
BigDecimal someValue = new BigDecimal("0.00");
System.out.println(someValue.compareTo(BigDecimal.ZERO)==0); //true
System.out.println(someValue.equals(BigDecimal.ZERO)); //false
Pertanto, devi stare molto attento con la scala nella tua someValue
variabile, altrimenti otterrai risultati inaspettati.
Si desidera utilizzare equals () poiché sono oggetti e utilizzare l'istanza ZERO integrata:
if(selectPrice.equals(BigDecimal.ZERO))
Nota che .equals()
tiene conto della scala, quindi a meno che selectPrice sia la stessa scala (0) di .ZERO
allora questo restituirà false.
Per togliere la scala dall'equazione per così dire:
if(selectPrice.compareTo(BigDecimal.ZERO) == 0)
Dovrei notare che per alcune situazioni matematiche, 0.00 != 0
motivo per cui immagino che .equals()
tenga conto della scala. 0.00
dà precisione al centesimo posto, mentre 0
non è così preciso. A seconda della situazione che potresti voler rispettare .equals()
.
equals
e compareTo
non è come pensi. docs.oracle.com/javase/1.5.0/docs/api/java/math/…
equals
tiene conto della scala, che non è ciò che vogliamo qui.
equals
dovrebbero essere utilizzate al posto di compareTo()
. L'OP non specifica quale tipo di matematica sta usando, quindi hai ragione è meglio dargli entrambe le opzioni.
GriffeyDog è decisamente corretto:
Codice:
BigDecimal myBigDecimal = new BigDecimal("00000000.000000");
System.out.println("bestPriceBigDecimal=" + myBigDecimal);
System.out.println("BigDecimal.valueOf(0.000000)=" + BigDecimal.valueOf(0.000000));
System.out.println(" equals=" + myBigDecimal.equals(BigDecimal.ZERO));
System.out.println("compare=" + (0 == myBigDecimal.compareTo(BigDecimal.ZERO)));
risultati:
myBigDecimal=0.000000
BigDecimal.valueOf(0.000000)=0.0
equals=false
compare=true
Mentre comprendo i vantaggi del confronto BigDecimal, non lo considero un costrutto intuitivo (come gli operatori ==, <,>, <=,> =). Quando hai in testa un milione di cose (ok, sette cose) nella tua testa, allora tutto ciò che puoi ridurre il tuo carico cognitivo è una buona cosa. Quindi ho creato alcune utili funzioni utili:
public static boolean equalsZero(BigDecimal x) {
return (0 == x.compareTo(BigDecimal.ZERO));
}
public static boolean equals(BigDecimal x, BigDecimal y) {
return (0 == x.compareTo(y));
}
public static boolean lessThan(BigDecimal x, BigDecimal y) {
return (-1 == x.compareTo(y));
}
public static boolean lessThanOrEquals(BigDecimal x, BigDecimal y) {
return (x.compareTo(y) <= 0);
}
public static boolean greaterThan(BigDecimal x, BigDecimal y) {
return (1 == x.compareTo(y));
}
public static boolean greaterThanOrEquals(BigDecimal x, BigDecimal y) {
return (x.compareTo(y) >= 0);
}
Ecco come usarli:
System.out.println("Starting main Utils");
BigDecimal bigDecimal0 = new BigDecimal(00000.00);
BigDecimal bigDecimal2 = new BigDecimal(2);
BigDecimal bigDecimal4 = new BigDecimal(4);
BigDecimal bigDecimal20 = new BigDecimal(2.000);
System.out.println("Positive cases:");
System.out.println("bigDecimal0=" + bigDecimal0 + " == zero is " + Utils.equalsZero(bigDecimal0));
System.out.println("bigDecimal2=" + bigDecimal2 + " < bigDecimal4=" + bigDecimal4 + " is " + Utils.lessThan(bigDecimal2, bigDecimal4));
System.out.println("bigDecimal2=" + bigDecimal2 + " == bigDecimal20=" + bigDecimal20 + " is " + Utils.equals(bigDecimal2, bigDecimal20));
System.out.println("bigDecimal2=" + bigDecimal2 + " <= bigDecimal20=" + bigDecimal20 + " is " + Utils.equals(bigDecimal2, bigDecimal20));
System.out.println("bigDecimal2=" + bigDecimal2 + " <= bigDecimal4=" + bigDecimal4 + " is " + Utils.lessThanOrEquals(bigDecimal2, bigDecimal4));
System.out.println("bigDecimal4=" + bigDecimal4 + " > bigDecimal2=" + bigDecimal2 + " is " + Utils.greaterThan(bigDecimal4, bigDecimal2));
System.out.println("bigDecimal4=" + bigDecimal4 + " >= bigDecimal2=" + bigDecimal2 + " is " + Utils.greaterThanOrEquals(bigDecimal4, bigDecimal2));
System.out.println("bigDecimal2=" + bigDecimal2 + " >= bigDecimal20=" + bigDecimal20 + " is " + Utils.greaterThanOrEquals(bigDecimal2, bigDecimal20));
System.out.println("Negative cases:");
System.out.println("bigDecimal2=" + bigDecimal2 + " == zero is " + Utils.equalsZero(bigDecimal2));
System.out.println("bigDecimal2=" + bigDecimal2 + " == bigDecimal4=" + bigDecimal4 + " is " + Utils.equals(bigDecimal2, bigDecimal4));
System.out.println("bigDecimal4=" + bigDecimal4 + " < bigDecimal2=" + bigDecimal2 + " is " + Utils.lessThan(bigDecimal4, bigDecimal2));
System.out.println("bigDecimal4=" + bigDecimal4 + " <= bigDecimal2=" + bigDecimal2 + " is " + Utils.lessThanOrEquals(bigDecimal4, bigDecimal2));
System.out.println("bigDecimal2=" + bigDecimal2 + " > bigDecimal4=" + bigDecimal4 + " is " + Utils.greaterThan(bigDecimal2, bigDecimal4));
System.out.println("bigDecimal2=" + bigDecimal2 + " >= bigDecimal4=" + bigDecimal4 + " is " + Utils.greaterThanOrEquals(bigDecimal2, bigDecimal4));
I risultati si presentano così:
Positive cases:
bigDecimal0=0 == zero is true
bigDecimal2=2 < bigDecimal4=4 is true
bigDecimal2=2 == bigDecimal20=2 is true
bigDecimal2=2 <= bigDecimal20=2 is true
bigDecimal2=2 <= bigDecimal4=4 is true
bigDecimal4=4 > bigDecimal2=2 is true
bigDecimal4=4 >= bigDecimal2=2 is true
bigDecimal2=2 >= bigDecimal20=2 is true
Negative cases:
bigDecimal2=2 == zero is false
bigDecimal2=2 == bigDecimal4=4 is false
bigDecimal4=4 < bigDecimal2=2 is false
bigDecimal4=4 <= bigDecimal2=2 is false
bigDecimal2=2 > bigDecimal4=4 is false
bigDecimal2=2 >= bigDecimal4=4 is false
BigDecimal.ZERO.setScale(2).equals(new BigDecimal("0.00"));
C'è una costante statica che rappresenta 0 :
BigDecimal.ZERO.equals(selectPrice)
Dovresti farlo invece di:
selectPrice.equals(BigDecimal.ZERO)
al fine di evitare il caso in cui si selectPrice
trova null
.
equals
e compareTo
non è come pensi. docs.oracle.com/javase/1.5.0/docs/api/java/math/…