CODEWITHSUNDEEP
androidandroid-radiogroupandroid-radiobutton
Isabella
Isabella

Reputation: 455

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.

Upvotes: 1

Views: 879

Answers (1)

Tiago Oliveira
Tiago Oliveira

Reputation: 1602

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;
                }
            }
        }
    });

Upvotes: 2

Related Questions