RecyclerView适配器弹窗菜单点击不同视频获相同数据问题排查

兄弟，这个问题我之前也踩过坑！核心原因大概率是你没有在菜单点击事件里正确绑定当前点击item的对应数据，而是复用了某个全局变量或者错误的位置值，导致所有菜单都用了最后一个item的数据。下面给你拆解常见问题和解决办法：

最常见的错误写法（你可能中招了） #

比如你在Adapter里定义了一个全局的MyCollection col变量，然后在onBindViewHolder里给它赋值，最后在菜单点击时用这个变量：

// 错误示例：全局变量被最后一个item覆盖
private MyCollection col;

@Override
public void onBindViewHolder(@NonNull RecyclerView.ViewHolder holder, int position) {
    col = data.get(position);
    holder.menuBtn.setOnClickListener(v -> {
        PopupMenu popup = new PopupMenu(mContext, v);
        popup.getMenuInflater().inflate(R.menu.collection_menu, popup.getMenu());
        popup.setOnMenuItemClickListener(item -> {
            if (item.getItemId() == R.id.action_preview) {
                // 这里的col已经被最后一个item的数据覆盖了！
                playVideo(col.getVideoUrl());
                return true;
            }
            return false;
        });
        popup.show();
    });
}

因为RecyclerView会复用ViewHolder，最后这个全局col会被最后一次绑定的item数据覆盖，所以不管点哪个菜单，用的都是最后一条数据。

正确的解决方式 #

方式1：在点击事件中直接使用当前绑定的currentCol #

在onBindViewHolder里先拿到当前item的数据，然后在菜单点击事件里直接用这个局部变量：

@Override
public void onBindViewHolder(@NonNull RecyclerView.ViewHolder holder, int position) {
    // 拿到当前item的对应数据，用局部变量保存
    MyCollection currentCol = data.get(position);
    
    holder.menuBtn.setOnClickListener(v -> {
        PopupMenu popup = new PopupMenu(mContext, v);
        popup.getMenuInflater().inflate(R.menu.collection_menu, popup.getMenu());
        
        popup.setOnMenuItemClickListener(item -> {
            if (item.getItemId() == R.id.action_preview) {
                // 直接用当前的currentCol，不会被复用覆盖
                playVideo(currentCol.getVideoUrl());
                return true;
            }
            // 其他菜单操作同理，都用currentCol
            return false;
        });
        popup.show();
    });
}

方式2：通过holder.getAdapterPosition()获取当前位置再取数据 #

如果担心局部变量的作用域问题，也可以在点击时通过ViewHolder拿到当前在Adapter中的位置，再从数据集中取对应数据：

popup.setOnMenuItemClickListener(item -> {
    if (item.getItemId() == R.id.action_preview) {
        // 获取当前item在Adapter中的正确位置
        int currentPos = holder.getAdapterPosition();
        // 一定要判断位置是否有效，避免RecyclerView复用导致的异常
        if (currentPos != RecyclerView.NO_POSITION) {
            MyCollection col = data.get(currentPos);
            playVideo(col.getVideoUrl());
        }
        return true;
    }
    return false;
});

这里注意要用getAdapterPosition()而不是getLayoutPosition()，后者是item在布局中的位置，可能因为复用导致不准确。

额外排查点 #

1. 检查你的数据集data是否在外部被错误修改，比如是否所有item都引用了同一个MyCollection对象？如果是，那肯定会显示相同数据。
2. 确保每次更新数据集后都调用了notifyDataSetChanged()或者对应的局部更新方法（比如notifyItemChanged()），否则Adapter可能不知道数据变化。

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