Android RecyclerView调用notifyItemMoved时崩溃问题求助

核心问题分析 #

notifyDataSetChanged()是全量刷新，会重新绑定所有ViewHolder，掩盖了ViewHolder状态处理的问题；而notifyItemMoved()是局部刷新，只处理受影响的条目，对ViewHolder的状态一致性要求更高，你的怀疑方向（clearView方法）大概率是对的。

具体排查与修复步骤 #

• 检查clearView方法的操作边界
  不要在clearView()中做不可逆的资源释放或状态销毁操作，比如不要调用holder.imageView.setImageDrawable(null)、移除必要的点击/触摸监听、或者清空ViewHolder内的关键变量。这些操作应该放在onBindViewHolder()中处理，因为notifyItemMoved后ViewHolder可能会被直接复用，若clearView做了破坏性操作，会导致复用时报空指针或状态异常。
  正确的clearView应该只做临时状态恢复，比如拖动时的高亮背景、位移重置：

  @Override
  public void clearView(@NonNull RecyclerView recyclerView, @NonNull ViewHolder viewHolder) {
      super.clearView(recyclerView, viewHolder);
      // 仅恢复默认视觉状态
      viewHolder.itemView.setBackgroundColor(ContextCompat.getColor(recyclerView.getContext(), R.color.item_default_bg));
      // 不要做资源释放类操作
  }
  

• 确保数据集合与notify操作同步
  调用notifyItemMoved(fromPosition, toPosition)前，必须先正确修改数据集合的元素顺序，比如使用Collections.swap(dataList, fromPosition, toPosition)或者手动移除再插入。如果数据集合未同步修改，RecyclerView的布局状态和数据会出现不一致，直接引发崩溃。

• 检查ItemTouchHelper的onMove实现
  如果你用ItemTouchHelper实现拖动，在onMove()方法中：

  1. 必须先完成数据集合的修改，再调用notifyItemMoved；
  2. 方法末尾返回true表示拖动操作已处理；
  3. 不要在该方法中调用其他刷新方法（比如notifyDataSetChanged()），避免冲突。

• 排查ViewHolder的状态一致性
  确保ViewHolder内的变量（比如位置标记、临时状态）在onBindViewHolder()中会被重新赋值，而不是依赖拖动过程中的临时修改。比如不要在拖动时直接修改ViewHolder内的position变量，应该每次绑定都从数据源获取最新位置。

内容的提问来源于stack exchange，提问作者user19516326