Petros Vel
Petros Vel

Reputation: 23

VB.net dynamic radiobuttons add event onclick

Hello i have problem with this code... i take from my database id and name and i add every row in a new RadioButton but how i can take the id with msgbox onclick? the code is this:

Imports MySql.Data.MySqlClient

Public Class Order_info
 Dim conn As New MySqlConnection
 Dim sqlcommand As New MySqlCommand
 Dim regDate As Date = Date.Now()
 Dim strDate As String = regDate.ToString("dd MMMM yyyy")
 Dim dbdate As String = regDate.ToString("yyyy-M-dd")
 Dim RButton As RadioButton

 Private Sub Order_info_Load(sender As Object, e As EventArgs) Handles MyBase.Load
    Label1.Text = strDate
    connect()
 End Sub

 Private Sub connect()
    If Not conn Is Nothing Then conn.Close()
    conn.ConnectionString = String.Format("server={0};user id={1};
                                           password={2};database={3}; pooling=false",
                                          My.Resources.DatabaseIP,
                                          My.Resources.DatabaseUsername,
                                          My.Resources.DatabasePassword,
                                          My.Resources.DatabaseName)
    Try
        conn.Open()

        sqlcommand.Connection = conn
        Dim stm As String = "SELECT o.id,s.name,o.status FROM orders o INNER JOIN supplier s ON s.id = o.id_supplier where datetim = '" + dbdate + "'"
        Dim cmd As MySqlCommand = New MySqlCommand(stm, conn)
        Dim reader As MySqlDataReader = cmd.ExecuteReader()
        Dim x As Integer = 25
        Dim y, p As Integer
        Do While reader.Read()
            RButton = New RadioButton
            RButton.Name = "RadioButton" + x.ToString


            RButton.Text = reader.GetString(1)
            RButton.Top = y
            y += x
            p += 1
            If (reader.GetInt32(2) = 1) Then
                RButton.BackColor = Color.LightGreen
            End If
            Panel1.Controls.Add(RButton)
        Loop

        reader.Close()

    Catch ex As Exception
        MsgBox(ex.Message)
    End Try
    conn.Close()
 End Sub
End Class

Upvotes: 0

Views: 1637

Answers (1)

OneFineDay
OneFineDay

Reputation: 9024

You make an event handler for them - they all use the same one. You just need to cast the sender object from the signature to tell which one was selected. I would also just make a new RadioButton object for each iteration. A FlowLayoutPanel would make adding the RBs easier - you won't have to set the location manually.

  Do While reader.Read()
        Dim RButton As New RadioButton
        RButton.Name = "RadioButton" + x.ToString
        RButton.Text = reader.GetString(1)
        RButton.Top = y
        y += x
        p += 1
        If (reader.GetInt32(2) = 1) Then
            RButton.BackColor = Color.LightGreen
        End If
        Addhandler RButton.Click, AddressOf RB_Clicked 
        'or use the CheckChanged event
        Panel1.Controls.Add(RButton)
    Loop
    '... rest of your code

The event handler:

Private Sub RB_Clicked(sender As Object, e As EventArgs)
  Dim rb As RadioButton = DirectCast(sender, RadioButton)
  'now rb is the one that was clicked
End Sub

Upvotes: 3

Related Questions