How do I update multiple selected row in datagridview to another in VB.NET

Question

0.00/5 (No votes)

See more:

I have two datagridview: dgServer and dgBranch, i am using checkbox in selecting the row and i want to update the data of the dgbranch from the selected row in dgServer i have selected multiple rows but the last selected row is the only updated.

What I have tried:

I have two datagridview: dgServer and dgBranch,
i am using checkbox in selecting the row and i want to
update the data of the dgbranch from the selected row in dgServer.
i have selected multiple rows but the last selected row is the only updated.

i have tried this code but this code only update a single row

VB

Private Sub btnUpdateS_Click(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles btnUpdateS.Click

        Try
            Dim dr As New DialogResult
            dr = MessageBox.Show("Are you sure you want to Update?", "Transfer", MessageBoxButtons.OKCancel, MessageBoxIcon.Question)
            If dr = Windows.Forms.DialogResult.OK Then
                If conn1.State = ConnectionState.Closed Then
                    conn1.ConnectionString = "Server= localhost;Database=jlt_2;username=root;password=SystemDeveloper2011;"
                    conn1.Open()
                End If
                Dim st As String = "UPDATE prcpricing SET deal_price = '" & DataGridView1.CurrentRow.Cells(3).Value & _
                                        "', unit_price = '" & DataGridView1.CurrentRow.Cells(4).Value & "', unit_amt = '" & DataGridView1.CurrentRow.Cells(5).Value & _
                                        "'  WHERE p_code = '" & DataGridView1.CurrentRow.Cells(1).Value & "' AND b_code = '" & dgBranch.CurrentRow.Cells(2).Value & "'"
                MessageBox.Show((DataGridView1.CurrentRow.Cells(1).Value) & (DataGridView1.CurrentRow.Cells(2).Value) & (DataGridView1.CurrentRow.Cells(3).Value) & (DataGridView1.CurrentRow.Cells(4).Value) & (DataGridView1.CurrentRow.Cells(5).Value))
                Dim cmd As New MySqlCommand(st, conn1)
                cmd.ExecuteNonQuery()
                MessageBox.Show("Process Successful!", "Transfer", MessageBoxButtons.OK, MessageBoxIcon.Information)

                conn1.Close()
            End If
        Catch err As Exception
            MessageBox.Show(err.ToString)
        End Try
      
    End Sub

Posted 30-Jun-17 17:06pm

Member 12374765

Updated 30-Jun-17 19:50pm

Michael_Davies

v3

Add a Solution

Comments

Richard Deeming 4-Jul-17 13:08pm

Your code is vulnerable to SQL Injection[^]. NEVER use string concatenation to build a SQL query. ALWAYS use a parameterized query.

Everything you wanted to know about SQL injection (but were afraid to ask) | Troy Hunt[^]
How can I explain SQL injection without technical jargon? | Information Security Stack Exchange[^]
Query Parameterization Cheat Sheet | OWASP[^]

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Michael_Davies · Answer 1 · 2017-06-30T19:50:00

You only get the last row because that is all your code updates, CurrentRow is the last row selected.

Use a For Each loop to walk the selected rows for example;

For Each dgr as DataGridViewRow in DataGridView1.SelectedRows
Dim st As String = "UPDATE prcpricing SET deal_price = '" & dgr.Cells(3).Value & _

...etc.
Next

If you are using checked rows you may have to walk the entire set of rows checking for checked = true, for an example see
[^]

Also never concatenate a string to construct an SQL statement, always use parameterised queries.