tags:

views:

158

answers:

7

I have a class which I want to set up as keys in HashMap. I already have implemented the compareTo method for that class. But still when I do:

map.put(new MyKey(dummyArguements) , dummyValue );
System.out.println(map.get( new MyKey(dummyArguements) ) );

I get null. So that means hashmap is not able to identify that the two keys (for get & put call) are same.

Could someone help me here please ?

+13  A: 

You need to implement hashCode() and equals(). compareTo() is additionally required for sorted map/set.

See this question for details.

Mike Q
+3  A: 

1) In general for collections, what you want to override is the equals() method (and also the hashcode() method) for your class. compareTo()/Comparable and Comparator are typically used for sorting and only take the place of using the equals() method for object equivalance in some cases - examples are implementers of SortedSet such as TreeSet.

2) Please conform to Java naming standards in your code. Your class names should be capitalized... e.g new MyKey(dummyArguments). See http://www.oracle.com/technetwork/java/codeconventions-135099.html#367 (and http://www.oracle.com/technetwork/java/codeconvtoc-136057.html) for more detail.

whaley
Class renamed. Thanks for pointing out. :)
Nikhil Garg
A: 

Do you have the hashCode() defined? compareTo is needed for sorting.

fastcodejava
A: 

HashMap doesn't check compareTo();

HashMap checks hashCode() and equals().

卢声远 Shengyuan Lu
+5  A: 

You should implement equals() and hashCode(). Your class should also be immutable. If it is mutable, it's hash code can change after adding it to map. Then the map can have problems finding it.

amorfis
+1 for the inmutable
helios
He is doing new before `put` and `add`. So immutable does not matter. Still a good point, +1.
fastcodejava
A: 

When using Collections that rely on hashing like Map and Set you have to implement the equals() and hashCode() to guarantee correct functionality. If you don't a new myKey will always be different from the key stored in the map because it uses the default implementations of equals() and hashCode().

kostja
A: 

You should implement equals() and hashCode(). Your class should also be immutable. If it is mutable, it's hash code can change after adding it to map. Then the map can have problems finding it.

vikash bhorale