Android RadioGroup getCheckedRadioButtonId()

Question

I am trying to check if at least a radio button is checked inside a radiogroup. Obviously i can choose to check each one at a time, But according to Android Developers Reference Site, public int getCheckedRadioButtonId () is a public method of radiogroup which returns -1 upon empty election. so this is my codes

if(radioGroup.getCheckedRadioButtonId() == -1){
        Toast.makeText(getApplicationContext(),"Please make a selection",Toast.LENGTH_SHORT);
        return;
    }
 <RadioGroup
        android:layout_width="match_parent"
        android:layout_height="wrap_content"
        android:id="@+id/radioGroup"
        android:layout_below="@id/loanPurposeTV">

        <RadioButton
            android:id="@+id/radioButton"
            android:layout_width="wrap_content"
            android:layout_height="wrap_content"
            android:layout_alignParentLeft="true"
            android:layout_alignParentStart="true"
            android:layout_alignParentTop="true"
            android:text="Personal"
            android:checked="false" />

        <RadioButton
            android:id="@+id/radioButton2"
            android:layout_width="wrap_content"
            android:layout_height="wrap_content"
            android:layout_alignParentTop="true"
            android:layout_toEndOf="@+id/radioButton"
            android:layout_toRightOf="@+id/radioButton"
            android:text="Bussiness" />

        <RadioButton
            android:id="@+id/radioButton3"
            android:layout_width="wrap_content"
            android:layout_height="wrap_content"
            android:layout_alignBottom="@+id/radioButton2"
            android:layout_toEndOf="@+id/radioButton2"
            android:layout_toRightOf="@+id/radioButton2"
            android:text="Option Three" />

        <RadioButton
            android:id="@+id/radioButton4"
            android:layout_width="wrap_content"
            android:layout_height="wrap_content"
            android:layout_alignParentTop="true"
            android:layout_toEndOf="@+id/radioButton3"
            android:layout_toRightOf="@+id/radioButton3"
            android:text="Option Four" />

        <RadioButton
            android:id="@+id/radioButton5"
            android:layout_width="wrap_content"
            android:layout_height="wrap_content"
            android:layout_alignParentTop="true"
            android:layout_toEndOf="@+id/radioButton4"
            android:layout_toRightOf="@+id/radioButton4"
            android:text="Other" />
    </RadioGroup>

But its not working as intended. Does anybody know why its not working or a much simpler way to implement my idea

Answer 1

First of all, you forgot ".show()" when you call your Toast.

Personnally, I encountered the same problem (it returned always -1, even when a radiobutton is selected), in spite of there are answers that prove it works, like this one

You can try them, but honnestly, when I read this, I tried everything but I never made it work. Maybe I was too in a hurry when I had this problem.

Otherwise, a very simple workaround would be to set selected a default radiobutton when the layout loads, so, you're sure there will be one radiobutton well selected (and need no longer check before validation)

Or you can create a bool var initialized to false which will be set to true as soon as any radiobutton is selected (use a OnCheckChangeListener).