¿Por qué los objetos personalizados no son claves equivalentes para un HashMap?

Question

estoy teniendo problemas para usar mi propia clase como una clave para un HashMap

public class ActorId { 
    private final int playerId; 
    private final int id; 

    ActorId(int playerId, int id) { 
     this.playerId = playerId; 
     this.id = id; 
    } 

    public boolean equals(ActorId other) { 
     return this.id == other.id && this.playerId == other.playerId; 
    } 

    public int hashCode() { 
     int hash = 1; 
     hash = hash * 31 + playerId; 
     hash = hash * 31 + id; 
     return hash; 
    } 

    public String toString() { 
     return "#" + playerId + "." + id; 
    } 

    public int getPlayerId() { 
     return playerId; 
    } 
} 

Aquí está una prueba unitaria no

import static org.junit.Assert.*; 
import java.util.Map; 
import org.junit.Test; 

public class ActorIdTest { 
    @Test 
    public final void testAsMapKey() { 
     ActorId a = new ActorId(123, 345); 
     ActorId b = new ActorId(123, 345); 

     assertTrue(a.equals(b)); 
     assertEquals(a.hashCode(), b.hashCode()); 

     // Works with strings as keys 
     Map<String, String> map1 = new java.util.HashMap<String, String>(); 

     map1.put(a.toString(), "test"); 
     assertEquals("test", map1.get(a.toString())); 
     assertEquals("test", map1.get(b.toString())); 
     assertEquals(1, map1.size()); 

     // But not with ActorIds 
     Map<ActorId, String> map2 = new java.util.HashMap<ActorId, String>(); 

     map2.put(a, "test"); 
     assertEquals("test", map2.get(a)); 
     assertEquals("test", map2.get(b)); // FAILS here 
     assertEquals(1, map2.size()); 

     map2.put(b, "test2"); 
     assertEquals(1, map2.size()); 
     assertEquals("test2", map2.get(a)); 
     assertEquals("test2", map2.get(b)); 
    } 
} 

Answer 1

necesita cambiar

public boolean equals(ActorId other) { 
    .... 
} 

a

public boolean equals(Object other) { 
    .... 
} 

---

Consejo del día: Siempre use la anotación @Override.

Si se hubiera usado la anotación @Override, el compilador habría cogido el error y dijo:

El método es igual a (actorId) de tipo actorId debe anular o poner en práctica un método supertipo

Answer 2

Su código es correcto, pero también debe anular el método equals heredado de Object.

Agregue esto a su clase ActorId:

@Override 
public boolean equals(Object other) { 
    if(other == null || other.getClass() != getClass()) 
     return false; 
    return equals((ActorId)other); 
} 

Answer 3

Definitivamente debe reemplazar el método equals (Object), y por cierto implementación de un mapa (HashMap) también es NECESARIO que overrdide el método hashCode ()

Tuve el mismo problema, y ​​sin la implementación personalizada de hashCode, nunca se llamó al método igual de la clase "ActorId".

Answer 4

Por defecto Java invoca booleano equals(Object obj); Por lo tanto, de hacer login es correcto, pero si desea sobrescribir equals() utiliza objetos como parámetro y comprueba la clase y por instanceOf o getClass() y hacer un casting clase.

if (obj instanceOf ActorId) { 
    ActorId other = (ActorId)obj; 
    ... compare fields 
}