Why doesn't DiffUtils update items position in adapter after remove item (JAVA)?

Question

[Dima Dema]

When i remove item in my list and use diffUtils for update my adapter, my adapter doesn't change position for items which are behind him in adapter. How can i do that?

for (AppsItem appsItem : appsItems) {
      if (appsItem.isChecked()) {
                        appsItems.remove(appsItem);
                        break;
                    }
                }

                AppsDiffUtil appsDiffUtil =
                        new AppsDiffUtil(appsAdapter.getAppsList(), appsItems);
                DiffUtil.DiffResult appsDiffResult = DiffUtil.calculateDiff(appsDiffUtil,true);
                appsAdapter.setAppsList(appsItems);
                appsDiffResult.dispatchUpdatesTo(appsAdapter);

Comments

[Денис Загаевский]

Это сайт на русском, вообще.
А ты DiffCallback правильно реализовал?

[Dima Dema]

Да

public class AppsDiffUtil extends DiffUtil.Callback {
        private final List<AppsItem> oldList;
        private final List<AppsItem> newList;

        public AppsDiffUtil(List<AppsItem> oldList, List<AppsItem> newList) {
            this.oldList = oldList;
            this.newList = newList;
        }

        @Override
        public int getOldListSize() {
            return oldList.size();
        }

        @Override
        public int getNewListSize() {
            return newList.size();
        }

        @Override
        public boolean areItemsTheSame(int oldItemPosition, int newItemPosition) {
            AppsItem oldAppsItem = oldList.get(oldItemPosition);
            AppsItem newAppsItem = newList.get(newItemPosition);

            return  oldAppsItem.getPackageName().equals(newAppsItem.getPackageName());
        }

        @Override
        public boolean areContentsTheSame(int oldItemPosition, int newItemPosition) {
            AppsItem oldAppsItem = oldList.get(oldItemPosition);
            AppsItem newAppsItem = newList.get(newItemPosition);

            return oldAppsItem.getName().equals(newAppsItem.getName())
                    && oldAppsItem.getPackageName().equals(newAppsItem.getPackageName())
                    && oldAppsItem.getDate() == newAppsItem.getDate()
                    && oldAppsItem.getSize() == newAppsItem.getSize()
                    && oldAppsItem.isChecked() == newAppsItem.isChecked()
                    && oldAppsItem.getTotalTimeForeground() == newAppsItem.getTotalTimeForeground();
        }

[Dima Dema]

Получается такая ситуация, что я удаляю элемент в списке и даю сразу новый список + беру старый список с адаптера. Diff отрабатывает хорошо и заменяет старый список в адаптере на новый, только проблема в том, что все последующие элементы имеют уже позицию не текущую , а +1 . То есть если я хочу удалить элемент какой-то еще раз, мне нужно отметить предыдущий элемент , а не текущий. И чем больше я удаляю элементов , тем больше не корректная последовательность у списка.

Answer 1

[foonfyrick]

Я не вижу у тебя кода, который список очищает, тебе нужно передавать только обновленные элементы списка, те, что изменились, а ты передаешь вообще всё по новой.
У меня есть такой код, инкапсулированый в адаптер:

//при инициализации, создаю список и добавляю
fun insertData(list:ArrayList<Content>){
        val dfc = MyDiffUril(listOfContent,list)
        val difResult = DiffUtil.calculateDiff(dfc)
        listOfContent.addAll(list)
        difResult.dispatchUpdatesTo(this)
    }
//по нажатию на вью холдер обновляю список
    fun updateData(list:ArrayList<Content>){
        val dfc = MyDiffUril(listOfContent,list)
        val difResult = DiffUtil.calculateDiff(dfc)
        listOfContent.clear()
        listOfContent.addAll(list)
        difResult.dispatchUpdatesTo(this)
    }

listOfContent - список, в адаптере ресайклера
list - список, который я вручную заполняю в мэйн активити
первый аргумент MyDiffUtil - это список в ресайклер адаптере, он же "старый список", второй аргумент - новый список, это мною созданный и заполненный список в мэйн активити.

Comments

[Dima Dema]

Допустим у нас есть список , в нем элементы 1,2,3. Я сейчас удаляю элемент 2 допустим и имею новый список
1,3 . Дальше я беру старый список с адаптера , через метод .getAppsList() - мой кастомный метод, который просто возвращает список с адаптера и передаю новый список ему, получается Diff должен сравнить старый и новый , и заменить все в старом списке на новый. По идее должно получиться 1,3 и смещение позиций. Но по факту я вижу 1,3 , но позиции у них старые остались, не новые .

"Я не вижу у тебя кода, который список очищает, тебе нужно передавать только обновленные элементы списка, те, что изменились, а ты передаешь вообще всё по новой."
Правильно ли я вас понял, что мне нужно согласно верхнего примера поменять все наоборот?
Старый список 1,2,3 , а новый список только 2 и передать это все в Diff и все будет корректно работать?

[foonfyrick]

Dima Dema, вот по этому видео посмотри таймкод 9:42 https://youtu.be/PAOcL1W05Eg?t=582
сделай так же, посмотри как работает, и потом под свой проект подредактируй

[foonfyrick]

Dima Dema, я когда смотрел, как работает DiffUtil, я смотрел по этому примеру, так что он рабочий, от него уже редактируй как тебе угодно для своего проекта. Тяжеловато объяснять просто в переписке, постоянно нужно мотать вверх твой код, потом сравнить с твоими вопросами, которые ты пишешь, потом пытаться объяснить как нужно сделать и т.д. Возня получается)

[Dima Dema]

foonfyrick, Окей, спасибо за видео, как посмотрю и протестирую отпишу тут.
На счет перемотки, то можно либо разделить экран на 2 окна , если у тебя один монитор, либо если есть доп мониторы , то открыть на них)
Еще не забывай есть мобилка)

Еще раз спасибо, все проверю.

[Dima Dema]

Вы еще случайно не знаете ответа на вопрос Как программно узнать, сколько энергии потребляет каждое приложение на Android? ?

[foonfyrick]

Dima Dema,

Вы еще случайно не знаете ответа на вопрос

нет, увы.