为什么我的哈希码得到一个空指针？

Question

我正在尝试创建自己的哈希码，但出现了空指针错误。 是因为我的对象Suit和Face没有自己的哈希码吗？我一直在网上查找哈希码示例，因为这是我第一次使用它们，因此我尝试从网上看到的示例中进行合并。

import java.util.HashMap;

enum Suit {Diamonds, Hearts, Spades, Clubs};
enum Face {
            King, Queen, Jack, Ten, Nine, Eight, Seven,
            Six, Five, Four, Three, Two, Ace, Joker
        };

public class SuitAndFace {

    Suit suit;
    Face face;

    SuitAndFace(Suit s, Face f){
        suit = s;
        face = f;
    }

    public String toString(){
        if(!face.toString().equals("Joker"))
            return face + " of " + suit;
        else//joker has no suit
            return face.toString();
    }

@Override
    public boolean equals(Object o){

    System.out.println(" in equals");
    if(o instanceof SuitAndFace){
        SuitAndFace s = (SuitAndFace) o; 
        if(this.face.equals(s.face) && this.suit.equals(s.face))
            return true;    
    }
    return false;
}

@Override
public int hashCode() {
    int hash = 3;
    hash = 7 * hash + this.suit.hashCode();
    hash = 7 * hash + this.face.hashCode();
    return hash;
}

}

更新的方法：

@Override
    public boolean equals(Object o){

    System.out.println(" in equals");
    if(o instanceof SuitAndFace){
        SuitAndFace s = (SuitAndFace) o; 
        if(face != null && suit != null && this.face.equals(s.face) && this.suit.equals(s.face))
            return true;    
    }
    System.out.println("no match!");
    return false;
}

@Override
public int hashCode() {
    int hash = 3;
    if(suit != null && face != null){
        hash = 7 * hash + this.suit.hashCode();
        hash = 7 * hash + this.face.hashCode();
    }
    return hash;
}

Answer 1

我刚刚意识到，您似乎在将小丑表示为suit == null ， face == Joker 。 只要您提出suit.anything就会收到NullPointerException 。

该行可能是：

hash = 7 * hash + this.suit == null ? 0 : this.suit.hashCode();

另外，您也可以在Suit枚举中添加一个Joker常量，这样就不必担心null 。

编辑

使用您编辑的代码，我可以看到这些行

if(face != null && suit != null && this.face.equals(s.face) && this.suit.equals(s.face))
        return true;

不太正确。 首先，您要比较suit和face （显然是错字）。 我也在猜测，你想让小丑平等吗？ 目前，它们不会是因为如果suit == null该方法将返回false 。 线条应该简单

if (face == s.face && suit == s.suit)
    return true;

您无需对枚举常量使用.equals 。 相反，使用==好处在于，因为您没有使用方法，所以无需担心null。