2016-05-20 11 views

Antwort

2

Sonar ist darauf hindeutet, das Ergebnis der compareTo gegen 0 zu überprüfen, nicht, wenn es direkt 1, -1 zurückgibt.

if (recBalanceAmt.compareTo(recRolloverEligibility) > 0) { 

Sie können den Grund für diesen Vorschlag finden in der compareTo() Javadoc

Rückkehr: eine negative ganze Zahl, Null oder eine positive ganze Zahl, wie dies Objekt kleiner als, gleich oder größer als das angegebene Objekt

+0

Warum? mit 1 vergleichen ist nicht in Ordnung? oder gibt es ein Problem mit dem Vergleich mit 1? –

+3

vertraglich kann eine compareTo-Methode jeden positiven Wert zurückgeben, nicht unbedingt 1. Wenn also die Methode 2 zurückgibt und Sie das Ergebnis mit 1 vergleichen, wird Ihre Bedingung nicht erfüllt. – benzonico