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/

10-11 22:20