[ken2010]

int count = 0;
                RadioButton[] personRadio = new RadioButton[rowsPerson];
                foreach (DataRow dataRow in dataTable.Rows)
                {

                    string strFirtsname = dataRow["firstname"].ToString();
                    string strLastname = dataRow["lastname"].ToString();

                    personRadio[count] = new RadioButton();
                    personRadio[count].Name = personRadio[count].ToString();
                    personRadio[count].Text = strFirtsname + " " + strLastname;
                    personRadio[count].TabIndex = count;
                    personRadio[count].Location = new Point(20, count * 20);
                    personRadio[count].AutoCheck = true;
                    personPanel.Controls.Add(personRadio[count]);
                    personPanel.Visible = true;
                    count++;
                }
            }
            catch (Exception ex)
            {
                MessageBox.Show(ex.ToString());
            }
            finally
            {
                sqlConnection.Close();
            }
        }

Radio buttons are dynamically created based on the number of items that are in the database table(Person) which is working fine, my question is how do I get the selected value once i click any of this on the panel since they are dynamically created? Please help me out. Thanks. God bless in Jesus name.

[Toadill]

Most radio button used the .Checked keyword, not sure for C# let me check....

example

//return
boolResult = radio1.checked;

//conditional
if radio1.checked
{

}

//assignment
radio1.checked = True;

This post has been edited by Toadill: 06 March 2012 - 06:42 AM

[Toadill]

This seems to be the correct way, I would not see it being any different from a constant control.

[ken2010]

.....but it dos not respond when i try to troubleshoot it with a messagebox. What else do you think I need to do?

[Toadill]

What do you mean it does not respond
What error message are you getting copy and paste it.
also you could try setting break points in your code to find out where it is throwing errors.

[eclipsed4utoo]

You should handle the CheckedChanged event for the radiobutton.

http://msdn.microsof...kedchanged.aspx

Then you could show the MessageBox for the checked radiobutton's Text.

private void RadioButton_CheckedChanged(object sender, EventArgs e)
{
     RadioButton rb = sender as RadioButton;
     if (rb.Checked)
     {
          MessageBox.Show(rb.Text);
     }
}

You should also look into using a generic list instead of an array.

[Toadill]

I am a little confused by this

//This could be causing as error since you can not have spaces, could maybe use Trim
personRadio[count].Name = personRadio[count].ToString()

Great idea eclipsed4utoo

This post has been edited by Toadill: 06 March 2012 - 07:08 AM

[ken2010]

Basically that is what I did, and i went on and click one of the dynamically created radio button and nothing happened. I'd expected the messagebox to display the selected radiobutton text. Thanks what other suggestion do you think I should use?

Thanks you talked about the Trim can you discuss more on this?

[Toadill]

Set a break point on line 10 and 11 of your initial post. Tell me what is being stored under name. I am just wondering if your radio buttons are being assigned valid names.

I am not sure what this would return.
personRadio[count].ToString()

[Toadill]

If you could just tell me what is being stored in the controls name property, this could be causing problems if the data is the same more then once, or if it is just junk it may include invalid characters I would suggest just naming the control the persons name.

[Toadill]

Spoiler

personRadio[count].Name = "opt" + strFirtsname + strLastname;
//optJohnDoe

If you wanted the control name to just be a number

Spoiler

personRadio[count].Name = "opt" + count.ToString();
//opt1

This post has been edited by Toadill: 06 March 2012 - 07:45 AM

[ken2010]

the messagebox shows this when i try to get the name by doing this: MessageBox.show(personRadio[count].Name) that "System.Windows.Forms.RadioButton, Checked:false". does this mean it can not read the values attac to it?

[Toadill]

Are you saying that once you set values they are not printing correctly if so this is because you do not have a event handler like eclipsed4utoo said earlier

Quote

You should handle the CheckedChanged event for the radiobutton.

http://msdn.microsof...kedchanged.aspx

Then you could show the MessageBox for the checked radiobutton's Text.

1	private void RadioButton_CheckedChanged(object sender, EventArgs e)
2	{
3	     RadioButton rb = sender as RadioButton;
4	     if (rb.Checked)
5	     {
6	          MessageBox.Show(rb.Text);
7	     }
8	}

Okay try this once take a real control and drag it to a form in a new project then click events in the properties area. The events and properties can all be set through code as suggested.

Btw just double click an event to get the code needed. Such as RadioButton_CheckedChanged event

This post has been edited by Toadill: 06 March 2012 - 08:15 AM

[eclipsed4utoo]

You need to subscribe to the event...

personRadio[count].CheckedChanged += new EventHandler(RadioButton_CheckedChanged);

Then use my code from earlier.

[eclipsed4utoo]

Toadill, on 06 March 2012 - 10:15 AM, said:

Set a break point on line 10 and 11 of your initial post. Tell me what is being stored under name. I am just wondering if your radio buttons are being assigned valid names.

I am not sure what this would return.
personRadio[count].ToString()

It would return System.Windows.Forms.RadioButton,Checked:{Checked Value}.