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-Items在拖放时损坏。但我找到了一个简单的解决方案：在您的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.
}

您必须为该职位返回正确的itemID。从现在开始，物品没有腐败。

Answer 3

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

-F

并调用它而不是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（）破坏视图

6 个答案: