Problem overriding hashcode and equals

[Eugenio Flores]

Hello, I got a class named test, and I override both equal and hash code in order to use test objects in hash data structures such as HashSet.
But it seems that both overridings are not working because I'm able to put 2 equals test objects in a HashSet. How can that be?

I have reviewed my code several time just to see if I missing something about the way this overridings are supposed to be written. But I do not see any mistake. I even compare the two test instance to see if the return true from the equal comparison, and also they return the same hash code. So the rule that says that if two objects are equals their hashcode must be equal is accomplished.
Here is my code. Does any body see a mistake?

import java.util.*;
class test
{
   private int Identify;
   private int i;
   public static void main( String args[] )
   {
      Set <test>set = new HashSet ();
    
      test t1 = new test(127,128 );
      test t2 = new test(127,128 );

      System.out.println( set.add(t1) );
      System.out.println( set.add( t2 ) );
           
      System.out.println("equal test: " + t1.equals(t2));
      System.out.println("hashcode test: "+ (t2.hashcode()==t1.hashcode()) + " = " + t2.hashcode() );
        
      System.out.println(set);
        
   }
   test(int ID, int i)
   {
      Identify = ID;
      this.i = i;
   }
   
   public boolean equals (Object obj){
	   
	   return ((test)obj).Identify == this.Identify && ((test)obj).i == this.i;
	   
   }
   
   public int hashcode (){
	   return Identify + i;
	   
   }
   
   public String toString(){
	   return String.valueOf("{"+this.Identify + " "+this.hashcode()+"}");
   }
}

And the output is:

true
true
equal test: true
hashcode test: true = 255
[{127 255}, {127 255}]

[Christophe Verré]

A common error when the hashCode is not spelt properly ;)

(you can make your source more readable by using the "code" button when you write a message)

[Eugenio Flores]

Christophe: You are totally rigth, I what a shame for me. But thanks a lot.


And thanks for the code button tip.

Greetings from Managua.

[Jesper de Jong]

Note that the @Override annotation exists to protect you from this mistake. If you would have used it, like this:

@Override
public int hashcode() {
    // ...
}

then the compiler would have shown you an error message, because hashcode() doesn't override any method from a superclass.

(Annotations are not on the SCJP exam, but it's useful to know this anyway).