Search code examples
androidandroid-radiogroupandroid-radiobutton

How to implement setOnClickListener to get selected value from dynamically created RadioButton Android


I have a dynamically created radio buttons inside a RadioGroup since the values are coming from the database. I want to be able to implement setOnClickListener so I can get the whatever the selected value is. Whenever I try to click on the radio button, nothing shows up.

public void viewAll() {

    Cursor res = myDb.getAllData();

    LinearLayout bg = (LinearLayout) findViewById(R.id.backgroundSM);
    LinearLayout display = (LinearLayout) findViewById(R.id.viewAllMsg);

    final RadioButton[] rb = new RadioButton[5];
    rg = new RadioGroup(this); //create the RadioGroup
    rg.setOrientation(RadioGroup.VERTICAL);//or RadioGroup.VERTICAL
    res.moveToFirst();

    for(int i=0; i<res.getCount(); i++){
        rb[i]  = new RadioButton(this);
        //Toast.makeText(getApplicationContext(),res.getString(1) + " ", Toast.LENGTH_LONG).show();
        rb[i].setText(" " + res.getString(1)
                + "    " + res.getInt(0));
        rb[i].setId(i + 100);
        rg.addView(rb[i]);
        res.moveToNext();
    }

    display.addView(rg); // add the whole RadioGroup to the layout

    rg.setOnClickListener(new View.OnClickListener() {
        public void onClick(View v) {
            for(int i = 0; i < 5; i++) {
                rg.removeView(rb[i]); // now the RadioButtons are in the RadioGroup
            }

            int id = rg.getCheckedRadioButtonId();
            Toast.makeText(getApplicationContext(),id + " worked", Toast.LENGTH_LONG).show();

        }
    });


}

enter image description here

What is wrong with my code? Thank you so much for your help.


Solution

  • Selected position will be saved in position variable

    rg.setOnCheckedChangeListener(new RadioGroup.OnCheckedChangeListener() {
            @Override
            public void onCheckedChanged(RadioGroup radioGroup, int id) {
                for (int i = 0; i < radioGroup.getChildCount(); i++) {
                    if (radioGroup.getChildAt(i).getId() == id) {
                        position = i + 1; //+1 is used to have a start value of 1 like your example
                        break;
                    }
                }
            }
        });