RecyclerView 使用 notifyItemMoved() 破坏视图

Question

我在使用 notifyItemMoved() 方法时遇到问题。它似乎错误地显示了未移动的视图。

我的列表中有 4 个元素。我想要做的是为第 1 项和第 3 项之间的交换设置动画。第 1 项和第 3 项正确交换，但第 2 项显示第 3 项的内容！

所以列表开始看起来像这样：

Item 0
Item 1
Item 2
Item 3

然后这样结束：

Item 0
Item 3
Item 3 <-- What the heck has this changed for?
Item 1

我的适配器由 List mProductList 支持。我调用以下代码：

public void sortBackingListUsingSortingList(List<ProductWrapper> newProductItems) {
    Log.e("", "Before:");
    for(ProductWrapper wrapper : mProductItems) wrapper.log();
    for(int i = 0; i < newProductItems.size(); i++) {
        ProductWrapper currentItem   = mProductItems.get(i);
        ProductWrapper correctItem   = newProductItems.get(i);

        if(!currentItem.equals(correctItem)) {
            // Item in wrong place
            int indexOfCorrectItem = getIndexOfItemInList(mProductItems, correctItem);
            Collections.swap(mProductItems, i, indexOfCorrectItem);
            notifyItemMoved(i, indexOfCorrectItem);
            Log.e("", "notifyItemMoved(" + i + ", " + indexOfCorrectItem+")");
            Log.e("", "After:");
            for(ProductWrapper wrapper : mProductItems) wrapper.log();
        }
    }
}

我还向 onBindViewHolder 添加了日志记录以检查是否正在调用我的视图逻辑：

@Override
public void onBindViewHolder(HolderBasic holder, int position) {
    Log.e("", "onBindViewHolder(holder, " + position + ")");
    holder.fill(mProductItems.get(position));
}

我的日志是这样的：

09-02 14:39:17.853: Before:
09-02 14:39:17.853: Item 0
09-02 14:39:17.853: Item 1
09-02 14:39:17.853: Item 2
09-02 14:39:17.853: Item 3

09-02 14:39:17.854: notifyItemMoved(1, 3)

09-02 14:39:17.854: After:
09-02 14:39:17.854: Item 0
09-02 14:39:17.854: Item 3
09-02 14:39:17.854: Item 2
09-02 14:39:17.854: Item 1

09-02 14:39:17.867: onBindViewHolder(holder, 1)
09-02 14:39:17.874: onBindViewHolder(holder, 3)

如您所见，项目 2 根本没有理由改变它的显示 - 然而，它确实改变了。有人知道为什么吗？

编辑

我可以通过遍历整个适配器并在每个项目上调用 notifyItemChanged() 来解决上述问题。效率低下，不是一个好的解决方案，但对用户来说是不可见的。

Answer 1

感谢@david.mihola 引导我做错事。

这花了很长时间才弄清楚，因为症状并没有使问题变得明显！

我是这样做的：

Collections.swap(mProductItems, i, indexOfCorrectItem);
notifyItemMoved(i, indexOfCorrectItem)

但是，我显然没有想清楚 notifyItemMoved() 到底在做什么。它只是通知适配器项目 i 已移动到 indexOfCorrectItem 它并没有告诉适配器 indexOfCorrectItem 也已移动到 i.

在幕后它正在做以下事情：

将项目 1 移动到 3
将 2 变为 1 以填补空白
将 3 处的内容移到 2 处以填补空白
notifyItemChanged(1);
notifyItemChanged(3);

以上当然是将第 3 项下移到第 2 项，而没有刷新视图！是第 4 步和第 5 步通过使 item1 和 item3 正确显示而 item2 不正确来隐藏问题！

意识到这一点后，我尝试了以下代码：

notifyItemMoved(indexOfCorrectItem, i);
notifyItemMoved(i, indexOfCorrectItem);

这使列表按正确的顺序排列，但它使动画短路。

所以，我完全放弃了交换：

mProductItems.remove(indexOfCorrectItem);
mProductItems.add(i, correctItem);
notifyItemMoved(indexOfCorrectItem, i);

Answer 2

我遇到了同样的问题。 RecyclerView - 项目在拖放时损坏。但我找到了一个简单的解决方案：在你的 RecyclerView.Adapter.class 中一定要有以下内容

@Override
public long getItemId(int position) {
    // here code for getting the right itemID, 
    // i.e. return super.getItemId(mPosition);
    // where mPosition ist the Position in the Collection.
}

您必须 return 该职位的正确 itemID。从现在开始，项目不会损坏。

Answer 3

要在拖放后获取项目的实际位置，请将此方法添加到您的适配器中：

private int getItemPosition(Item item){ // (<-- replace with your item)
    int i = 0;
    // (replace with your items and methods here)
    for (Item currentItem : mItems) {
        if (currentItem.getItemId() == item.getItemId()) break;
        i++;
    }
    return i;
}

并调用它而不是 viewHolder 给定的位置。

Answer 4

好吧，我的处理方式略有不同，可能会对其他人有所帮助。

        Collections.swap(mItemList, fromPosition, toPosition);
        // Need to do below, because NotifyItemMove only handle one sided move
        Item fromItem = mItemList.get(fromPosition);
        Item toItem = mItemList.get(toPosition);
        notifyItemChanged(fromPosition, toItem);
        notifyItemChanged(toPosition, fromItem);

我不得不重新排列网格上的项目，并将位置保存到文件中。 @Graeme 是对的，但我不想放弃交换。所以就像@saganaut 一样，我坚持使用 notifyItemChanged。但仅使用 notifyItemChanged 有时会在我的网格上留下相同项目的两个交换项目，因此我将这些项目与 notifyItemChanged 绑定。它没有杀死动画，并且按预期工作。

Answer 5

我遇到了同样的问题，我实际上也以不同的方式处理它，以我的方式，动画将保持不变，你不会再遇到物品位置错误的问题:

@Override
public void onRowMoved(int fromPosition, int toPosition) {
    if (fromPosition < toPosition) {
        for (int i = fromPosition; i < toPosition; i++) {
            Collections.swap(listOfItem, i, i + 1);
        }
    } else {
        for (int i = fromPosition; i > toPosition; i--) {
            Collections.swap(listOfItem, i, i - 1);
        }
    }
    notifyItemMoved(fromPosition, toPosition);
}

有了这个，您可以毫无问题地重新订购商品

Tutorial used to fix my issue

Answer 6

这个方法对我有用，但我发现动画中有非常轻微的抖动。如果有更好的解决方案，请告诉我:)

 public void moveItem(int oldPos, int newPos) {
    Uri item = mDataSet.get(oldPos);
    mDataSet.remove(oldPos);
    mDataSet.add(newPos, item);
    notifyItemMoved(oldPos, newPos);

    notifyItemRangeChanged(0, mDataSet.size());
}

RecyclerView 使用 notifyItemMoved() 破坏视图

RecyclerView corrupts view using notifyItemMoved()

android

android-recyclerview