为什么Hashtable的equals方法测试值为null的情况

时间:2013-07-24 08:25:28

标签: java hashtable

我正在阅读Hashtable的代码,并且我了解到Hashtable的键和值都不能为null,但是它的equals方法测试值为null的情况。

public synchronized boolean equals(Object o) {
if (o == this)
    return true;
if (!(o instanceof Map))
    return false;
Map<K,V> t = (Map<K,V>) o;
if (t.size() != size())
    return false;

    try {
        Iterator<Map.Entry<K,V>> i = entrySet().iterator();
        while (i.hasNext()) {
            Map.Entry<K,V> e = i.next();
            K key = e.getKey();
            V value = e.getValue();
            if (value == null) { // Can Hashtable's value be null?
                if (!(t.get(key)==null && t.containsKey(key))) 
                    return false;
            } else {
                if (!value.equals(t.get(key)))
                    return false;
            }
        }
    } catch (ClassCastException unused)   {
        return false;
    } catch (NullPointerException unused) {
        return false;
    }

return true;
}

1 个答案:

答案 0 :(得分:1)

这是一种贯穿整个NPE的模式。考虑一个简单的类

public class HelloWorld {
    String data;
}

如果生成hashCode()和equals(),您将看到此常规模式。在这种情况下

@Override
public boolean equals(Object o) {
    if (this == o) return true;
    if (o == null || getClass() != o.getClass()) return false;

    HelloWorld that = (HelloWorld) o;

    if (data != null ? !data.equals(that.data) : that.data != null) return false;

    return true;
}

@Override
public int hashCode() {
    return data != null ? data.hashCode() : 0;
}

如您所见,我们总是检查null。它不是强制性的,而是一种良好的编程习惯。我理解在Hashtable的情况下没有任何意义,但正如我之前提到的,开发人员必须添加此检查以保持统一的模式。

更新:Tim建议Since Hashtable is subclassable, it is possible for a subclass to try to support null keys or values。因此,进行空检查是安全的。