追加プロセスをリファクタリングする

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


} 

このコードをどのようにリファクタリングしますか？ユーザーarraylistでは、IDでユーザーを重複させるべきではありません。

私はブール変数を使用することは少し不必要だと思いますが、私はよりよい解決策を見つけることができませんでした。

P.Ss：また、これらの種類のコーディングスタイルのための準備がある場合は、名前を付けることができますか？ありがとうございました。

Answer 1

ユーザーが存在する場合は、booleanは不要、方法はreturnとなります。

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

セットを使用

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

    } 


}