해시 코드 메서드를 재정의하는 동안 hashmap에서 올바른 값을 가져올 수 없습니다.

Question

개체 (BookMe)의 hashcode 메서드를 재정의 한 후 아래 코드를 실행하고 있습니다. 목표는 내지도 (해시 맵)에서 키로 사용할 객체의 해시 코드를 재정의하는 것입니다. 하지만, 실행 후 null 값을 참조하십시오. 지도의 실제 크기에는 아무런 문제가 없습니다. 아래는 코드입니다. 해시 코드 메서드를 재정의하지 않으면 올바른 출력을 얻습니다. (나는 세 가지 값 모두를 의미합니다). `

class BookMe{ 
private String isbn ; 
static int i = 0; 
public BookMe(String isbn) 
{ 
    this.isbn = isbn;  
} 
public String getIsbnValue() 
{ 
    return this.isbn; 
} 
@Override 
public boolean equals(Object o) 
{ 
    if(o instanceof BookMe && ((BookMe)o).getIsbnValue() == this.getIsbnValue()) 
    { 
     return true; 
    } 
    else{ 
     return false; 
    } 
} 
@Override 
public int hashCode() 
{ 
    return this.isbn.toString().length() + (++i); 
} 

}

공용 클래스 HashMapTest는 {

public static void main(String[] args) { 

    Map<BookMe, Integer> map = new HashMap<BookMe, Integer>(); 

    BookMe b1 = new BookMe("Graham"); 
    BookMe b2 = new BookMe("Graham"); 
    BookMe b3 = new BookMe("Graham"); 
    map.put(b1, 19); 
    map.put(b2, 33); 
    map.put(b3, 22); 

    System.out.println("----444444--------"); 
    System.out.println(map.size()); 

    Set <BookMe> set = map.keySet(); 
    System.out.println("------*****------"); 


    for(BookMe bk : set) 
    { 
     System.out.println("bk : "+ bk); 
     System.out.println(map.get(bk)); 
    }  
} 

} `

Answer 1

귀하의 hashCodeObject::hashCode 계약을 위반하는 것입니다. 동일한 객체에 대해 동일한 값을 반환해야합니다. Javadoc의에서

: 같은 예를 들어 hashCode를 호출 할 때마다 다른 값을 반환하면

* <li>Whenever it is invoked on the same object more than once during 
*  an execution of a Java application, the <tt>hashCode</tt> method 
*  must consistently return the same integer, provided no information 
*  used in <tt>equals</tt> comparisons on the object is modified. 
*  This integer need not remain consistent from one execution of an 
*  application to another execution of the same application. 

, 당신은 당신의 HashMap이 키의 위치를 ​​기대할 수 없다.

당신이 ISBN에 따라 해시 코드를 원한다면, 단지 ISBN의 해시 코드를 반환 :

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

당신은 또한 당신의 equals 방법을 수정해야합니다, 둘다 말한 외에도

public boolean equals(Object o) 
{ 
    if(o instanceof BookMe && ((BookMe)o).getIsbnValue().equals(this.getIsbnValue())) 
    { 
     return true; 
    } 
    else{ 
     return false; 
    } 
} 

Answer 2

해시 코드는 동일한 객체에 대해서조차도 새로운 해시 코드를 반환합니다. 요소를지도에 올릴 때와 반복 할 때 equals 메서드를 사용하여 두 문자열을 비교해야합니다. 에

((BookMe)o).getIsbnValue() == this.getIsbnValue() 

변경을 :

((BookMe)o).getIsbnValue().equals(this.getIsbnValue()) 

Answer 3

귀하의 hashcode 방법은 고장도 equals 방법 당신은 참조하지 값을 기준으로 문자열을 비교한다. 당신은 해시 코드를 생성 할 수 Objects 방법을 사용할 수 있습니다,이

@Override 
public boolean equals(Object o) { 
    return o instanceof BookMe && ((BookMe) o).getIsbnValue().equals(this.getIsbnValue()); 
} 

같은 equals 메소드를 해결할 수 있습니다 그리고 hashcode를 해결하기

@Override 
    public int hashCode() { 
     return Objects.hash(isbn); 
    } 

이 메소드의 오버라이드 (override) 구현을위한 일반적인 계약은 입니다 그들은 동일한 객체의 equals() 메서드와 일치하는 방식으로 동작합니다. 메서드 : 지정된 객체가 동일한 해시를 일관되게보고해야합니다 ( 값이 변경되지 않는 한 새 버전은 더 이상 이 이전 버전과 동일한 것으로 간주되지 않음), 이()와 동일한 두 객체가 동일한 해시 값을보고해야 함을 의미합니다. 해시 값이 서로 다른 Java 구현간에 일관성을 유지할 필요가 없습니다 ( 또는 심지어 동일한 프로그램의 실행 실행간에 차이가 있음). 다른 해시를 갖는 두 개의 동일하지 않은 객체가 매우 바람직하지만 이것은 필수가 아닙니다 (즉 해시 구현 된 함수가 일 필요는 없습니다.