कस्टम ऑब्जेक्ट्स हैश मैप के लिए समतुल्य कुंजी क्यों नहीं हैं?

Question

मैं एक 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; 
    } 
} 

यहाँ के लिए एक प्रमुख के रूप में अपने ही वर्ग का उपयोग कर समस्या हो रही है एक असफल JUnit परीक्षण

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

आप

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

को बदलने की जरूरत है है

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

---

दिन की युक्ति: हमेशा @Override एनोटेशन का उपयोग करें।

आप @Override एनोटेशन का इस्तेमाल किया था, तो संकलक त्रुटि पकड़ लिया और कहा है |:

विधि के बराबर होती है प्रकार ActorId की (ActorId) ओवरराइड या लागू करना चाहिए एक महाप्रकार विधि

Answer 2

आपका कोड सही है, लेकिन आपको Object से प्राप्त equals विधि को ओवरराइड करने की भी आवश्यकता है।

अपने ActorId वर्ग को यह करें:

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

Answer 3

आप निश्चित रूप से ओवरराइड चाहिए विधि के बराबर होती है (वस्तु), और एक मानचित्र (HashMap) के कुछ कार्यान्वयन के लिए यह भी आवश्यक है कि आप विधि hashCode overrdide ()।

मुझे एक ही समस्या थी, और कस्टम हैशकोड कार्यान्वयन के बिना कक्षा "अभिनेता आईडी" के बराबर विधि कभी नहीं बुलाया गया था।

Answer 4

डिफ़ॉल्ट रूप से जावा बूलियन invokes equals(Object obj); तो, आप के लिए लॉग इन सही है, लेकिन यदि आप बराबर ओवरराइड करने के लिए() चाहते हैं एक पैरामीटर और के रूप में वस्तु का उपयोग करें और instanceOf या getClass() द्वारा वर्ग की जाँच करें और एक वर्ग के कास्टिंग है।

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