Using objects as keys in Java HashMap

package test;

import java.util.HashMap;

public class test {
    public static void main(String args[]) {
        HashMap<ID, String> h = new HashMap<ID, String> ();
        String b;

        ID id1 = new ID(100);
        ID id2 = new ID(200);
        ID id3 = new ID(200);

        h.put(id1, "apple");
        h.put(id2, "pear");

        **System.out.println(h.containsKey(id3));**
    }
}

class ID {
    Integer code;

    public ID(Integer id) {
        this.code = id;
    }

    @Override
    public int hashCode()
    {
        return code.hashCode();
    }

    @Override
    public boolean equals(Object o)
    {
        return this.code.equals(o);
    }
}

      

the output of this program is "false". I cannot understand this result. I have implemented hashCode and equals which are overridden. I have no idea how to solve. How to use objects as keys in java HashMap?

+3


source to share


2 answers


Your implementation of equal values ​​is wrong:

public boolean equals(Object o)
{
    return this.code.equals(o); // this will never return true,
                                // since you are comparing an Integer instance
                                // to an ID instance
}

      



You should compare this code with other code:

public boolean equals(Object o)
{
    if (!(o instanceof ID))
        return false;
    ID oid = (ID) o;
    return this.code.equals(oid.code);
}

      

+7


source


Incorrect method implementation equals

. As per your implementation, it does the equals method of the class Integer

. We have to handle logic without library class methods equals

.



@Override 
public boolean equals(Object o) {
    if (o == null || ((o instanceof ID) && o.hashCode() != this.hashCode())) {
        return false;
    }
    return true;
}

      

0


source







All Articles