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:同样,如果这些编码样式有约定,您能提供一个名字吗?谢谢。
最佳答案
您不需要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);
}
关于java - 重构添加过程,我们在Stack Overflow上找到一个类似的问题:https://stackoverflow.com/questions/37013555/