RecyclerView滚动时Item ID异常变更的问题排查与解决

从你描述的情况来看，这明显是踩了RecyclerView ViewHolder复用的坑——这是RecyclerView提升性能的核心特性，但稍不注意就会引发数据错乱。大概率是数据绑定没处理彻底，或者Item ID的实现出了问题。下面是几个针对性的排查和解决步骤：

1. 先把getItemId()改对 #

RecyclerView默认用position作为Item ID，但滚动时position会随着ViewHolder复用动态变化，如果你靠这个ID关联本地文件或数据库记录，肯定会出现串数据的情况。

你得让getItemId()返回数据模型中固定的唯一ID（比如你存在SQLite里的文件主键），而不是position：

@Override
public long getItemId(int position) {
    // 假设你的数据模型是PDFModel，里面有个id字段是SQLite的主键
    return pdfList.get(position).getId();
}

别忘了在Adapter的构造方法里加一句关键代码：

setHasStableIds(true);

这会告诉RecyclerView：“每个Item的ID是固定不变的，别乱关联”，从根源上避免ID混乱。

2. 绑定数据时别留“尾巴” #

ViewHolder被复用时，之前绑定的旧数据不会自动清空，所以你在onBindViewHolder()里必须把当前Item的所有相关内容都重新绑定一遍，不能偷懒。尤其是按钮的点击事件，一定要用当前position对应的最新数据：

@Override
public void onBindViewHolder(@NonNull PDFListAdapter.MViewHolder holder, int position) {
    PDFModel currentPDF = pdfList.get(position);
    
    // 先更新显示内容
    holder.tvFileName.setText(currentPDF.getFileName());
    holder.tvFileId.setText(String.valueOf(currentPDF.getId()));
    
    // 重点！点击事件里直接用currentPDF，别依赖ViewHolder存储的旧数据
    holder.btnOpen.setOnClickListener(v -> {
        // 用当前Item的本地路径打开，而非复用的旧路径
        openPDF(currentPDF.getLocalPath());
    });
    
    holder.btnSave.setOnClickListener(v -> {
        // 用当前Item的ID操作数据库，而非旧ID
        savePDF(currentPDF.getId());
    });
}

很多人踩坑就是因为点击事件里用了ViewHolder成员变量存的旧数据，结果滚动后复用了ViewHolder，点按钮触发的是之前Item的操作。

3. 数据源要保持稳定 #

确保你的pdfList数据源是稳定的，更新数据时一定要通知Adapter。比如从SQLite加载数据后，建议直接赋值新的List，而非在旧List上增删：

// 推荐的做法
pdfList = dbHelper.getAllPDFs();
notifyDataSetChanged();

// 尽量避免这种写法（容易引发线程安全或数据不一致问题）
pdfList.clear();
pdfList.addAll(dbHelper.getAllPDFs());
notifyDataSetChanged();

如果数据源在滚动时被偷偷修改，Item对应的数据自然会错乱。

4. 检查ViewHolder的成员引用 #

如果你的ViewHolder里存储了PDFModel的引用，一定要在绑定数据时更新它：

public class MViewHolder extends RecyclerView.ViewHolder {
    TextView tvFileName;
    PDFModel currentPDF; // 存储当前Item的数据引用

    public MViewHolder(@NonNull View itemView) {
        super(itemView);
        tvFileName = itemView.findViewById(R.id.tv_file_name);
    }
}

// 在onBindViewHolder中更新这个引用
@Override
public void onBindViewHolder(@NonNull MViewHolder holder, int position) {
    holder.currentPDF = pdfList.get(position);
    holder.tvFileName.setText(holder.currentPDF.getFileName());
}

这样就算ViewHolder被复用，currentPDF也会指向当前正确的Item数据，不会拿旧数据去操作文件。

总结 #

最可能的问题就是ViewHolder复用导致数据绑定不彻底和Item ID未返回稳定唯一值。按上面的步骤逐一排查，应该能解决滚动时保存/打开错误文件的问题。

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