1

Есть такой код

    class User{
       private String name;
       public User(String name){this.name = name;}
       public String getName() {
           return this.name;
       }
       @Override
       public boolean equals(Object obj) {
           if(obj instanceof String){
               return obj.equals(this.getName());
           }
           return super.equals(obj);
       }
    }

Я пытаюсь провернуть такую операцию.

ArrayList<User>.contains(String)

Но результат выполнения никакой. Даже при наличии заведомо верных данных не находит совпадений.

Как корректно переопределить метод equals()?

4
  • вы также должны hashCode определить Commented 19 февр. 2018 в 15:24
  • 1
    @SeniorPomidor, а зачем массиву хэш? Он же не словарь.
    – Qwertiy
    Commented 19 февр. 2018 в 16:14
  • Тут несколько вариантов... 1,унаследоваться от ArrayList и переписать метод contains так, как вам надо. 2. проходить циклом по вашей коллекции и сравнивать соответсвующие поля с помощью equals. 3.если у вас нет повторяющихся элементов, то переопределить методы hashCode и equals, заменить ArrayList на Set, какой из них уже зависит от потребностей. 3 вариант предпочтительней , разумеется, нужны уникальные элементы. цикл сделать можно, если размер коллекции заведомо невелик. первый способ сложно оценить, тут все в ваших руках Commented 19 февр. 2018 в 16:54
  • Спасибо за совет. Как раз с помощью contains я хотел избежать foreach, ради производительности. Повторы есть. Так что сделаю всё-таки с помощью цикла. Commented 19 февр. 2018 в 18:38

1 ответ 1

2

Use the Source, Luke - смотрите исходники ArrayList

public class java.util.ArrayList<E> extends AbstractList<E>
     implements List<E>, RandomAccess, Cloneable, java.io.Serializable {

//blah-blah

public boolean contains(Object o) {
    return indexOf(o) >= 0;
}

public int indexOf(Object o) {
    if (o == null) {
        for (int i = 0; i < size; i++)
            if (elementData[i]==null)
                return i;
    } else {
        for (int i = 0; i < size; i++)
            if (o.equals(elementData[i]))
                return i;
    }
    return -1;
}

Отсюда следует, что идет сравнение не User.equals(String), а String.equals(User)...

То есть вам надо создать свой class MyArrayList<> extends ArrayList<>, в котором операция contains() переопределена так как надо вам.

Ваш ответ

Нажимая «Отправить ответ», вы соглашаетесь с условиями пользования и подтверждаете, что прочитали политику конфиденциальности.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.