Refactoring der Addiervorgang

Question

public static void addToUserList(User newUser){ 

    boolean hasFound = false; 

    for (User user : users) { 
     if(user.getUserID() == newUser.getUserID()){ 
      System.out.println("DUPLICATED USER ID: " + user.getUserID() + "ALREADY EXISTS"); 
      hasFound = true; 
      break; 
     } 

    } 

    if(hasFound = false){ 
     users.add(newUser); 
    } 


} 

Wie Refactoring diesen Code? In der Benutzer-Arraylist sollte es keine doppelten Benutzer nach ID geben.

Ich denke, mit booleschen Variablen ist ein bisschen unnötig, aber ich konnte keine bessere Lösung finden.

P.SS: Auch wenn es einen Convention für diese Art von Codierungsstilen gibt, können Sie einen Namen angeben? Vielen Dank.

Answer 1

Sie benötigen nicht die boolean, nur return von der Methode, wenn der Benutzer existiert.

public static void addToUserList(User newUser){ 
    for (User user : users) { 
     if(user.getUserID() == newUser.getUserID()){ 
      System.out.println("DUPLICATED USER ID: " + user.getUserID() + "ALREADY EXISTS"); 
      return; 
     } 
    } 

    users.add(newUser); 
} 

Answer 2

bitte versuchen Sie es mit this.Change der == in ersten, wenn auf! = Wie unten, und entfernen Sie den letzten Teil

public static void addToUserList(User newUser){ 

    boolean hasFound = false; 

    for (User user : users) { 
     if(user.getUserID() != newUser.getUserID()){ 

      users.add(newUser); 
     } 

    else{ 
      System.out.println("DUPLICATED USER ID: " + user.getUserID() + "ALREADY EXISTS"); 
     } 

    } 


}