我在使用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()来解决上述问题。效率低而且不是一个好的解决方案，但对用户来说是不可见的。
答案 0 :(得分：15)
感谢@ david.mihola带领我做错了。
这需要很长时间才能弄清楚，因为症状并没有使问题显而易见！
我这样做：
Collections.swap(mProductItems, i, indexOfCorrectItem);
notifyItemMoved(i, indexOfCorrectItem)
但是，我显然没有想到notifyItemMoved()实际上在做什么。它只是通知适配器项i已移至indexOfCorrectItem 它不会告诉适配器indexOfCorrectItem也已移至i
在幕后，它正在执行以下操作：
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);
答案 1 :(得分：1)
我有同样的问题。 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。从现在开始，物品没有腐败。
答案 2 :(得分：0)
要在拖放后获取商品的实际位置，请将此方法添加到适配器：
-F并调用它而不是viewHolder给出的位置。
答案 3 :(得分：0)
好吧，我以稍微不同的方式处理它，可能会帮助别人。
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绑定了项目。它不会杀死动画，并且按预期工作。
答案 4 :(得分：0)
我遇到了同样的问题，实际上我的处理方式也有所不同，按照我的方式，动画将保持不变，您再也不会遇到物品位置错误的麻烦：
@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);
}
有了这个，您可以毫无问题地重新订购商品
答案 5 :(得分：0)
这种方法对我有用，但我看到动画中有非常轻微的抖动。如果有更好的解决方案，请告诉我:)
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());
}