Мой хэш-код не будет работать - что-то не так с моим equals () ?;

Question

Мой хэш-код не будет работать - что-то не так с моим equals () ?;

1

Всякий раз, когда я пытаюсь запустить эту программу, я получаю следующий вывод:

6150

5000

1015612567

Люди в списке:

{Лицо A = 100, лицо B = 50}

String name;
int day;

static Map<String, Integer> people = new HashMap<>();

public PeopleHash(String name, int day) {
    this.name = name;
    this.day = day;

}

public String getName() {
    return name;
}

public int getDay() {
    return day;
}

public boolean equals(PeopleHash other) {
    if (other instanceof PeopleHash) {
        PeopleHash d = (PeopleHash)other;
        return name == d.name && day == d.day;
    } else {
        return false;
    }
}

@Override
public int hashCode() {
    return 50 * day;
}

public PeopleHash peopleEntry(){
    String namn = "A";
    int schiffre = 123;
    return null;
}


public static void main(String[] args) {


    PeopleHash p1 = new PeopleHash("Isolde", 123);
    PeopleHash p2 = new PeopleHash("Jean", 100);

    people.put("Person A", 100);
    people.put("Person B", 50);



    System.out.println(p1.hashCode());
    System.out.println(p2.hashCode());



    System.out.println("People in list: \n" + people);

    for (Entry<String, Integer> entry : people.entrySet()) {
        String a = entry.getKey();
        int b = entry.getValue();

        System.out.println("Next:");
        System.out.println("Person: " + a + " Hashcode: " + entry.hashCode() + " Value: " + b);

    }

}

}

Что я делаю не так?

Jeramo 25 март 2015, в 18:19

Источник

1

Ой, извини. Я не получаю вывод «1015612567».
Jeramo 25 март 2015, в 16:28
0

ничего. в чем проблема?
blackbird014 13 дек. 2017, в 15:29

Теги:

java

dictionary

hashcode

hashmap

3 ответа

Ещё вопросы

Ой, извини. Я не получаю вывод «1015612567».

Kiskae · Answer 1 · 2015-03-25T15-48-00.000Z

Для сравнения объектов используется метод Object.equals(Object). Вы определили новый метод с подписью PeopleHash.equals(PeopleHash).

Измените подпись метода на

@Override
public boolean equals(Object other) {

Всегда рекомендуется добавлять аннотацию @Override, поскольку компилятор предупредит вас, если выбранный вами метод не переопределяет другой метод.

Adeel Ahmad · Answer 2 · 2015-03-25T14-33-00.000Z

Возможно, что-то неправильно с методом equals. Попробуй это -

public boolean equals(Object other) {

            if (other instanceof PeopleHash) {
           PeopleHash d = (PeopleHash)other;
           return name.equals(d.name) && day == d.day;
  }
   return false;
}

Я не проверял весь ваш код, поэтому, если у вас все еще возникают проблемы, поднимите вопрос, я рассмотрю это.

user2864740 · Answer 3 · 2015-03-25T14-09-00.000Z

Хеш/равен себе. Проблема связана с использованием (или отсутствием).

"Лицо:" Выход показывает хэш - код для Map.Entry объекта (entry.hashCode()), и не имеет никакого отношения P1/P2. Записи на карте добавляются String Keys, а объекты p1/p2 тривиально отбрасываются.

Карта, вероятно, должна быть объявлена как Map<HashPerson, Integer>, для которой будет использоваться equals/hashCode HashPerson. Объекты отдельных лиц должны использоваться непосредственно в качестве ключей:

people.add(p1, 1234);

Тогда цикл можно записать так:

for (Entry<HashPerson, Integer> entry : people.entrySet()) {
   HashPerson person = entry.getKey();
   Integer value = entry.getValue();

   System.out.println(
     String.format("Person %s HashCode: %d Value: %d",
       person.getName(), person.hashCode(), value));
}