SONAR beklagen, ändern Sie die Bedingung so, dass es nicht immer auswerten "auf " false"

public String generateURLSafeToken(String username, char[] password) throws CredentialTokenException {
    this.tokenValid = false;

    String token = null;
    if ((username.length() < 1) || (username == null)) {
        throw new CredentialTokenException("Username cannot be an empty string or null.");
    }
    if ((password.length < 1) || (password == null)) {
        throw new CredentialTokenException("Password cannot be an empty or null.");
    }

Ich mich vor diesem Fehler in Zeile 4 und Zeile 7 (username == null und password == null)

Und ich brauche dieses Teil in meinem code. Ich versuche isEmpty() anstelle von null, aber mit Problemen konfrontiert, in das auch . Was ist eine Alternative oder die Lösung um dies zu beheben SONAR Fehler

  • Tauschen Sie die Bedingungen rund um. Sie sollten überprüfen, für null vor der Prüfung der Länge.
InformationsquelleAutor Awa | 2016-09-14
  1. 11

    Den Bedingungen, die immer wahr false sind username == null und password == null.

    Nehmen wir das Beispiel von username. Der Betreiber || ist Kurzschluss Bedeutung, die er nicht auswerten der rechten Seite, wenn die linke Seite ist true. Grundsätzlich gibt es 2 Fälle:

    • Die username nicht null. Der Zustand username.length() < 1 bewertet
      • Wenn das Ergebnis true kehren wir direkt und geben Sie die if Zweig
      • Wenn das Ergebnis false versuchen wir zu bewerten username == null. Aber da die username nicht null diese immer bewerten zu false.
    • Die username gegeben ist null. Der Zustand username.length() < 1 ausgewertet wird. Dies tatsächlich hält direkt gibt es: es wirft ein NullPointerException - und nicht auswerten der rechten Seite.

    Daher können Sie sehen, Wann immer der username == null Zustand war eigentlich ausgewertet, das Ergebnis war immer false. Dies ist, was die SonarQube Warnung sagt Sie.

    Hier die Lösung ist, drehen Sie die 2 Bedingungen. Sollten Sie mit

    if (username == null || username.length() < 1)

    statt. Wenn Sie beginnen, und gehen Sie durch jeden Fall, werden Sie feststellen, dass keiner der Ausdrücke wird immer das gleiche Ergebnis:

    • Die username nicht null. Erste Bedingung eindeutig ausgewertet false und der zweite wird ausgewertet, welche möglicherweise zurück true oder false.
    • Die username gegeben ist null. Die erste Bedingung eindeutig ausgewertet, um true und Kurzschlüsse.
    • Vielen Dank für die Zeit nehmen, zu erklären, mir meine Fehler.
    InformationsquelleAutor Tunaki
Schreibe einen Kommentar