RecyclerView列表项点击弹出Dialog时触发空指针异常求助

看起来你的问题出在Dialog初始化时使用的上下文对象已经失效，导致调用show()时触发了NullPointerException。我们来一步步拆解问题并解决：

问题根源 #

从错误栈和你的代码来看，有两个核心问题：

• ChooserAdapter直接持有ChooserActivity的强引用，如果Activity已经进入销毁状态（比如用户返回但RecyclerView还存在），这个引用会变成无效对象，同时还会导致内存泄漏。
• 你在创建MusclePopup时传入的是Adapter持有的context（即Activity），当Activity不再活跃时，用它初始化Dialog并调用show()，就会触发与ActivityThread相关的空指针错误——因为Dialog需要依附一个活跃的Activity上下文才能正常显示。

修复步骤 #

1. 修改Adapter，避免持有Activity强引用 #

把Adapter里的context类型从ChooserActivity改为通用Context，并且在点击事件中使用View的上下文（更安全，因为View的上下文会绑定当前活跃的组件）：

class ChooserAdapter(
    private val context: Context, // 改为通用Context类型
    private val chooseList: ArrayList<ChoosersModel>
) : RecyclerView.Adapter<ChooserAdapter.ViewHolder>() {
    class ViewHolder(view: View) : RecyclerView.ViewHolder(view) {
        var chooserNumber: TextView = view.findViewById(R.id.item_number)
        var chooserDescription: TextView = view.findViewById(R.id.item_description)
    }

    override fun onCreateViewHolder(parent: ViewGroup, viewType: Int): ViewHolder {
        val view = LayoutInflater.from(parent.context)
            .inflate(R.layout.item_chooser, parent, false)
        return ViewHolder(view)
    }

    override fun onBindViewHolder(holder: ViewHolder, position: Int) {
        val currentItem = chooseList[position]
        holder.chooserNumber.text = currentItem.number
        holder.chooserDescription.text = currentItem.description
        
        // 使用itemView的上下文，而不是Adapter持有的context
        holder.itemView.setOnClickListener {
            // 先检查上下文是否是活跃的Activity
            if (context is Activity && !context.isFinishing && !context.isDestroyed) {
                MusclePopup(context, currentItem).show()
            }
        }
    }

    override fun getItemCount(): Int = chooseList.size
}

2. 优化MusclePopup的构造与初始化 #

确保Dialog使用的是有效的Activity上下文，同时添加空指针防护：

class MusclePopup(
    private val activity: Activity, 
    private val currentItem: ChoosersModel
) : Dialog(activity) {
    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        requestWindowFeature(Window.FEATURE_NO_TITLE)
        setContentView(R.layout.fragment_popup)
        setupComponents()
    }

    private fun setupComponents() {
        // 避免findViewById返回null导致的异常
        findViewById<TextView>(R.id.input_chooser)?.text = currentItem.description
    }
}

3. 额外的内存泄漏防护（可选但推荐） #

如果你的Adapter生命周期可能比Activity长，建议用WeakReference包裹上下文，彻底避免内存泄漏：

class ChooserAdapter(
    context: Context,
    private val chooseList: ArrayList<ChoosersModel>
) : RecyclerView.Adapter<ChooserAdapter.ViewHolder>() {
    // 用WeakReference包裹Context，避免强引用导致内存泄漏
    private val contextRef = WeakReference(context)

    // ... ViewHolder和onCreateViewHolder代码保持不变 ...

    override fun onBindViewHolder(holder: ViewHolder, position: Int) {
        val currentItem = chooseList[position]
        holder.chooserNumber.text = currentItem.number
        holder.chooserDescription.text = currentItem.description
        
        holder.itemView.setOnClickListener {
            val context = contextRef.get()
            // 确认上下文有效且是活跃的Activity
            if (context is Activity && !context.isFinishing && !context.isDestroyed) {
                MusclePopup(context, currentItem).show()
            }
        }
    }

    override fun getItemCount(): Int = chooseList.size
}

为什么这样能解决问题？ #

• 使用itemView.context或弱引用的上下文，能确保我们只在Activity活跃的时候创建并显示Dialog，避免使用已经销毁的Activity对象。
• 去掉Adapter对Activity的强引用，既能避免内存泄漏，又能减少无效上下文的使用场景。

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