3

Есть ArrayList значений. При добавлении новых данных вся новая коллекция всегда попадает в исходный список. Как от этого избавиться?

Кейс: на экране есть список, по пул ту рефреш новые данные добавляются. Они могут быть точно такими же, что мы уже загрузили. Как сделать так, чтобы повторений не было?

for (Items item : response.body().getItems()) {
    if (!mItems.contains(i)) {
        mItems.add(i);
    }
}

Переопределил эти методы. Не помогает.

@Override
public boolean equals(Object o) {
    if (this == o)
        return true;
    if (o == null || getClass() != o.getClass())
        return false;

    Ad ad = (Ad) o;

    if (getSlug() != null ? !getSlug().equals(ad.getSlug()) : ad.getSlug() != null)
        return false;
    if (getLat() != null ? !getLat().equals(ad.getLat()) : ad.getLat() != null)
        return false;
    if (getLng() != null ? !getLng().equals(ad.getLng()) : ad.getLng() != null)
        return false;
    if (getKind() != null ? !getKind().equals(ad.getKind()) : ad.getKind() != null)
        return false;
    if (getTitle() != null ? !getTitle().equals(ad.getTitle()) : ad.getTitle() != null)

    ....

@Override
public int hashCode() {
    int result = getId();
    result = 31 * result + (getSlug() != null ? getSlug().hashCode() : 0);
    result = 31 * result + (getLat() != null ? getLat().hashCode() : 0);
    result = 31 * result + (getLng() != null ? getLng().hashCode() : 0);
    result = 31 * result + (getKind() != null ? getKind().hashCode() : 0);
    result = 31 * result + (isPremium() ? 1 : 0);
    result = 31 * result + getRoomsAmount();
    result = 31 * result + (getTitle() != null ? getTitle().hashCode() : 0);
    result = 31 * result + (getSpace() != null ? getSpace().hashCode() : 0);
    result = 31 * result + getImagesAmount();

Пробовал решения наподобие:

List<String> al = new ArrayList<>();
// add elements to al, including duplicates
Set<String> hs = new HashSet<>();
hs.addAll(al);
al.clear();
al.addAll(hs);

Не работает.

5
  • 1
    если hashSet содержит дубликаты, значит у вас неправильный hashCode и equals(). Оставьте в этих методах только те поля, равенство которых для вас означало бы равенство объектов.
    – Artem
    14 сен 2016 в 17:29
  • 1
    Оно у вас в многопоточной среде работает?
    – Adeptius
    14 сен 2016 в 17:44
  • Да, в многопоточной. В активити делаю запрос ретрофитом, в колбеке вызываю runUiThread и туда элементы записываю
    – Garf1eld
    15 сен 2016 в 6:55
  • @Adeptius пробовал так же и без вызова runUiThread
    – Garf1eld
    15 сен 2016 в 7:16
  • Если HashSet содержит "дублирующиеся" элементы, значит вы неправильно определили метод equals() Для этого класса. Именно он определяет в HashSet уникальность элемента 15 сен 2016 в 7:32

2 ответа 2

2

Значит обьекты, которые вы ложите в лист имеют разный хэшкод и виртуальной машиной читаются как разные, хотя данные обьекта, как вы считаете - одинаковы. Тогда вы должны написать свой алгоритм по которому у вас будут сравниватся объекты. Например вы можете реализовать у вашего обьекта Item() интерфейс Comparable<Item>. И в реализованном методе compareTo(AnotherObject) напишите свою логику сравнения. В сигнатуру метода будет попадать второй обьект для сравнения. Он должен возвращать отрицательное число(если обьект меньше), 0(если оба идентичны), положительное число(если обьект больше другого). А потом ложите его в структуру, которая поддерживает работу с компаратором. Например TreeSet<>(). Вот вам пример класса Woman:

public class Woman implements Comparable<Woman>
{
 public int age;

 public Woman(int age) {
  this.age = age;
 }

 public int compareTo(Woman o) {
  return this.age - o.age;
 }
}

TreeSet<> будет вызывать этот метод каждый раз при вставке нового элемента. Если вы правильно напишите алгоритм сравнения - дубликаты исключены.

Например вы можете у вашего Item() сравнивать ключевые поля. Если это стринг - можете использовать equals(). если он возвращает true - возвращайте 0 из метода compareTo().

0

При добавлении элемента в ArrayList никакой встроенной проверки на уникальность нет. Поэтому возникают дубликаты.

Для хранения уникальных элементов используйте HashSet. Для этого нужно чтобы equals и hashcode были определены и использовали одинаковые поля (это похоже у вас сделано).

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

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