使用PagedList+MVVM的RecyclerView复选框乱选及空指针异常求助

你遇到的空指针问题，核心原因是PagedList的异步加载特性加上RecyclerView的ViewHolder复用机制——哪怕你在onBindViewHolder开头判断了item != null，后续操作（比如设置checkbox选中状态、处理监听回调）时，item引用仍可能因为数据源更新或ViewHolder复用变为null。结合你的代码，我整理了针对性的修复方案：

1. 修复Adapter的onBindViewHolder逻辑 #

你代码里用了成员变量item，这很容易在ViewHolder复用的时候被覆盖或置空。我们改用局部变量持有当前item，并且在所有使用item的地方都做安全检查，尤其是checkbox的回调逻辑：

@Override 
public void onBindViewHolder(PageViewAdapter.PageViewHolder holder, int position) { 
    // 用局部变量持有当前item，避免成员变量被复用干扰
    Item currentItem = getItem(position); 

    if (currentItem != null) { 
        holder.lineTextView.setText(String.valueOf(currentItem)); 
        
        holder.checkBox.setTag(position); 
        // 先移除之前的监听，防止复用旧回调逻辑
        holder.checkBox.setOnCheckedChangeListener(null); 
        // 仅当item非空时设置选中状态
        holder.checkBox.setChecked(currentItem.getSelected()); 
        
        holder.checkBox.setOnCheckedChangeListener((compoundButton, isChecked) -> { 
            if (compoundButton.isPressed()) { 
                // 回调触发时，重新通过position获取最新的item，避免旧引用为空
                Item updatedItem = getItem(position);
                if (updatedItem != null) {
                    updatedItem.setSelected(isChecked);
                    // 若需要立即更新UI，可调用notifyItemChanged(position);
                }
            } 
        }); 
    } else {
        // 处理item为空的情况，设置默认UI状态，避免残留旧数据
        holder.lineTextView.setText("");
        holder.checkBox.setChecked(false);
        // 移除监听，防止触发时出现空引用
        holder.checkBox.setOnCheckedChangeListener(null);
    }
}

关键优化点： #

• 用局部变量currentItem替代成员变量item，避免ViewHolder复用导致的引用混乱
• 在checkbox回调中重新通过position获取item，而非直接持有旧引用——回调触发时，原item可能已被数据源回收或更新
• 增加item == null分支处理，清空UI状态并移除监听，防止残留旧数据或触发空指针

2. 修复Item类的潜在问题 #

你提供的Item类代码里，构造函数使用了this.value = value;但未定义value成员变量，这会导致编译错误，同时setText(String.valueOf(item))会显示对象地址而非实际值。我们补上成员变量并重写toString：

public class Item { 
    private boolean selected = true;
    private String value; // 补上缺失的value成员变量
    public Item(String value) { 
        this.value = value; 
    } 
    public void setSelected(boolean selected) { 
        this.selected = selected; 
    } 
    public boolean getSelected() { 
        return selected; 
    }
    // 重写toString，让setText显示实际的value值
    @Override
    public String toString() {
        return value;
    }
}

额外建议 #

如果你使用PagedList，要确保数据源（PositionalDataSource）的加载逻辑线程安全，且数据更新时正确通知PagedList刷新。另外，MVVM模式下建议通过ViewModel中的LiveData管理选中状态，而非直接修改Item对象，这样数据与UI的同步会更可靠。

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