遍历 hashmap 并创建唯一对象 - 尝试防止重复

Question

我在方法中的部分上方解释了我在 cmets 中尝试做的事情：

public int addPatron(String name) throws PatronException {
    int i = 0;
    //1. Iterate through a hashmap, and confirm the new name I am trying to add to the     record doesn't already exist in the hashmap
    for (Map.Entry<Integer, Patron> entry : patrons.entrySet()) {
        Patron nameTest = entry.getValue();
        //2. If the name I am trying to add already exists, we want to throw an exception saying as much.
        if (nameTest.getName() == name) {
            throw new PatronException ("This patron already exists");
            //3. If the name is unique, we want to get the largest key value (customer number) already in the hash, an increment by one.
        } else if (nameTest.getName() != name) {
            Map.Entry<Integer,Patron> maxEntry = null;
            for(Map.Entry<Integer, Patron> entryCheck : patrons.entrySet()) {
                if (maxEntry == null || entryCheck.getKey() > maxEntry.getKey()) {
                    maxEntry = entryCheck;
                    i = maxEntry.getKey();
                    i++;
                }
            }

        } else {
            throw new PatronException("Something's not working!");
        }
        //4. If everything is ok up to this point, we want to us the name and the new customer id number, and use those to create a new Patron object, which then gets added to a hashmap for this class which contains all the patrons.
        Patron newPatron = new Patron(name, i);
        patrons.put(i, newPatron);
    }
    return i;
}

当我尝试运行一个简单的单元测试时，如果我连续两次成功地为 addPatron 添加相同的名称，该单元测试将失败，测试失败。

try {
    testLibrary.addPatron("Dude");
    testLibrary.addPatron("Dude");
    fail("This shouldn't have worked");

测试失败，告诉我 addPatron 方法可以使用相同的名称两次。

@乔恩斯基特：

我的 Patron 类如下所示：

public class Patron {

//attributes
private String name = null;
private int cardNumber = 0;

//operations
public Patron (String name, int cardNumber){
    this.name = name;
    this.cardNumber = cardNumber;
}

public String getName(){
    return name;

}

public int getCardNumber(){
    return cardNumber;
}

}

Answer 1

正如其他人所说，使用== 比较字符串几乎肯定是不合适的。但是，它实际上不应该在您的测试用例中引起问题，因为您两次使用相同的常量字符串，所以 == 应该可以工作。当然，您仍然应该修复代码以使用equals。

也不清楚Patron 构造函数或getName 方法的作用——其中任何一个都可能导致问题（例如，如果它们创建字符串的新副本——这会导致你的测试失败，但通常也没有必要）。

让我稍微担心的是这条评论：

// 3. If the name is unique, we want to get the largest key value (customer number) 
// already in the hash, an increment by one.

这条评论在主循环中。所以到那时我们不知道这个名字是唯一的 - 我们只知道它与此迭代中赞助人的名字不匹配。

更令人担忧的是——我刚刚注意到这一点——你也在迭代块中执行了添加。在我看来，你应该有更多这样的东西：

public int addPatron(String name) throws PatronException {
    int maxKey = -1;

    for (Map.Entry<Integer, Patron> entry : patrons.entrySet()) {
        if (entry.getValue().getName().equals(name)) {
            // TODO: Consider using IllegalArgumentException
            throw new PatronException("This patron already exists");
        }
        maxKey = Math.max(maxKey, entry.getKey());
    }
    int newKey = maxKey + 1;
    Patron newPatron = new Patron(name, newKey);
    patrons.put(newKey, newPatron);
    return newKey;
}

此外，听起来真的您想要一张从姓名到赞助人的地图，可能还有从 id 到赞助人的地图。

Answer 2

你需要使用equals来比较java中的String对象，而不是==。所以替换：

if (nameTest.getName() == name) {

与：

if (nameTest.getName().equals(name)) {

Answer 3

尝试使用

nameTest.getName().equals(name)

而不是

nameTest.getName() == name

因为现在您要比较的是引用而不是字符串的值。 it's explained here

重新审视您的代码

好吧，我再次查看了您的代码，问题是，您的 HashMap 在测试开始时为空。所以循环永远不会运行 ==> 永远不会添加赞助人或抛出异常。

Answer 4

问题的原因是您如何使用比较运算符==。

当你对两个对象使用这个操作符时，你测试的是那个变量指向同一个reference。

要测试两个对象的值是否相等，您应该使用equals() 方法或compareTo（如果有）。

对于 String 类，调用 equals 足以检查存储相同的字符 more。

What is equals method ?

比较对象的值 问题是你如何比较名字。