点击AlertDialog确定按钮获取RadioButton文本时应用崩溃原因排查

Let's break down why your app is crashing and how to fix it quickly:

The Root Cause #

The NullPointerException is thrown because your radioGroup variable is null when you call getCheckedRadioButtonId(). Here's why:

• You inflated a custom layout (R.layout.method_pop) for your AlertDialog, which contains the RadioGroup with id method_radio.
• But you're trying to access this RadioGroup using the Activity's findViewById() method, which can only find views that are part of the Activity's main layout—not the Dialog's separate inflated view.

Step-by-Step Fix #

1. Get the RadioGroup from the inflated Dialog view
   Right after inflating the Dialog's layout, use the inflated view (v) to find the RadioGroup instead of the Activity's findViewById().

2. Add null safety checks
   To avoid another crash if the user doesn't select any RadioButton before tapping OK, add a check to ensure a RadioButton is selected.

Modified Working Code #

Here's your updated click listener code with the fixes applied:

Methodlayout.setOnClickListener(new View.OnClickListener() {
    @Override
    public void onClick(View view) {
        AlertDialog.Builder builder = new AlertDialog.Builder(Add_AlarmActivity.this);
        View v = getLayoutInflater().inflate(R.layout.method_pop, null);
        
        // Get RadioGroup from the inflated Dialog view (THIS IS THE KEY FIX!)
        RadioGroup radioGroup = v.findViewById(R.id.method_radio);

        builder.setPositiveButton("Ok", new DialogInterface.OnClickListener() {
            @Override
            public void onClick(DialogInterface dialogInterface, int i) {
                int selectedId = radioGroup.getCheckedRadioButtonId();
                // Check if a RadioButton is selected to avoid another NPE
                if (selectedId != -1) {
                    RadioButton radioButton = v.findViewById(selectedId);
                    String s = radioButton.getText().toString();
                    Toast.makeText(getApplicationContext(), s, Toast.LENGTH_SHORT).show();
                } else {
                    // Optional: Show a message if no option is selected
                    Toast.makeText(getApplicationContext(), "Please select an option", Toast.LENGTH_SHORT).show();
                }
            }
        });

        builder.setNegativeButton("Cancel", new DialogInterface.OnClickListener() {
            @Override
            public void onClick(DialogInterface dialogInterface, int i) {
                dialogInterface.cancel();
            }
        });

        builder.setView(v);
        AlertDialog dialog = builder.create();
        dialog.show();
    }
});

Additional Tips #

• Default selection: To prevent users from leaving no option selected, set one RadioButton as checked by default in your layout (change android:checked="false" to android:checked="true" for one of the RadioButtons).
• Verify IDs: Double-check that the id method_radio in your layout matches the id you use in code—typos here are a common cause of missing views.

内容的提问来源于stack exchange，提问作者Md Faysal Ahmed Akash