android - RecyclerView 使用 notifyItemMoved() 破坏 View

Question

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

我的列表中有 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 添加了日志记录，以检查是否正在调用我的 View 逻辑:

@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 也已移动到我。

在幕后它做了以下事情:

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

当然，上面的内容将第 3 项向下移动到第 2 项，而没有刷新 View !是第 4 步和第 5 步通过使 item1 和 item3 正确显示而使 item2 不正确来隐藏问题!

一意识到这一点，我就尝试了以下代码:

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

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

因此，我完全放弃了交换:

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