添加OnClickListener后Android App崩溃，求排查解决方法

问题原因 #

从Logcat日志可以明确，崩溃是因为调用了null对象的setOnClickListener方法，核心原因有两点：

1. 按钮btnToast属于HomeFragment的布局文件（布局xml中tools:context=".ui.home.HomeFragment"可证明），而非MainActivity的布局，在MainActivity中调用findViewById无法找到该控件。
2. 即使控件属于Activity布局，代码中findViewById的调用顺序在setContentView(binding.getRoot())之前，此时Activity还未加载布局，同样会返回null。

解决方案 #

Fragment中的控件事件应该在对应的Fragment类中处理，而非Activity。以下是Java和Kotlin的实现代码：

Java版 HomeFragment #

package com.example.javaapp.ui.home;

import android.os.Bundle;
import android.view.LayoutInflater;
import android.view.View;
import android.view.ViewGroup;
import android.widget.Button;
import android.widget.Toast;
import androidx.annotation.NonNull;
import androidx.fragment.app.Fragment;
import com.example.javaapp.databinding.FragmentHomeBinding;

public class HomeFragment extends Fragment {

    private FragmentHomeBinding binding;

    public View onCreateView(@NonNull LayoutInflater inflater,
                             ViewGroup container, Bundle savedInstanceState) {
        binding = FragmentHomeBinding.inflate(inflater, container, false);
        View root = binding.getRoot();

        // 通过ViewBinding获取按钮（推荐方式）
        Button btnToast = binding.btnToast;
        btnToast.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                Toast.makeText(getContext(), "hello test 123", Toast.LENGTH_LONG).show();
            }
        });

        return root;
    }

    @Override
    public void onDestroyView() {
        super.onDestroyView();
        binding = null;
    }
}

Kotlin版 HomeFragment #

package com.example.javaapp.ui.home

import android.os.Bundle
import android.view.LayoutInflater
import android.view.View
import android.view.ViewGroup
import android.widget.Toast
import androidx.fragment.app.Fragment
import com.example.javaapp.databinding.FragmentHomeBinding

class HomeFragment : Fragment() {

    private var _binding: FragmentHomeBinding? = null
    private val binding get() = _binding!!

    override fun onCreateView(
        inflater: LayoutInflater,
        container: ViewGroup?,
        savedInstanceState: Bundle?
    ): View {
        _binding = FragmentHomeBinding.inflate(inflater, container, false)
        val root: View = binding.root

        binding.btnToast.setOnClickListener {
            Toast.makeText(context, "hello test 123", Toast.LENGTH_LONG).show()
        }

        return root
    }

    override fun onDestroyView() {
        super.onDestroyView()
        _binding = null
    }
}

额外清理 #

删除MainActivity中原本的按钮相关代码，避免无用逻辑：

// 移除以下代码
Button btnToast;

btnToast = findViewById(R.id.btnToast);
btnToast.setOnClickListener(new View.OnClickListener() {
@Override
public void onClick(View view) {
    Toast.makeText(MainActivity.this,"hello test 123",Toast.LENGTH_LONG).show();
}
});

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