I want to delete my record? Messagebox error

Question

0.00/5 (No votes)

See more:

I have made deleting Msgbox...

1. it was deleted successfully
2. But when i give no again it was deleted 

Problem is: if i choose "NO" dont delete the record in datagridview as well as datasource and keep as it is

What I have tried:

<pre>If True Then

                If MessageBox.Show("Do You Want To Remove This Row", "Remove Row", MessageBoxButtons.YesNo, MessageBoxIcon.Stop) = DialogResult.Yes Then
                    DataGridView1.Rows.RemoveAt(DataGridView1.CurrentRow.Index)
                Else
                    MessageBox.Show("Row Not Removed", "Remove Row", MessageBoxButtons.OK, MessageBoxIcon.Information)
                    DataGridView1.Rows.Add(DataGridView1.CurrentRow.Index)
                End If
            End If

Posted 3-Mar-20 4:01am

Member 14621280

Updated 3-Mar-20 5:42am

v3

Add a Solution

2 solutions

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)

OriginalGriff · Answer 1 · 2020-03-03T04:43:00

Firstly, this line:

VB

If True Then

Is irrelevant: the code that follows it will always be executed.

Secondly, why are you adding a row if it's not to be removed?

VB

Else
    MessageBox.Show("Row Not Removed", "Remove Row", MessageBoxButtons.OK, MessageBoxIcon.Information)
    DataGridView1.Rows.Add(DataGridView1.CurrentRow.Index)
End If

Thirdly, you shouldn't really remove rows from the DGV, but from the DataSource it's bound to.

The code you show otherwise looks OK - so I'd start with the debugger and try to find out exactly is happening - put a breakpoint on the first line of the method, and single step through to see exactly where it is going and what variables contain.

CHill60 · Answer 2 · 2020-03-03T05:43:00

Member 14621280 wrote:
Any better commends feedback from anyone....appreciated

To be honest I can't think of a better way of putting that @OriginalGriff already has, but I'll try

However, picking up on Griff's question

Quote:
You know how to use the debugger, don't you?

just in case you don't here is a link to a good article on using the debugger Mastering Debugging in Visual Studio 2010 - A Beginner's Guide[^]

You should change the code in line with what Griff suggested

VB

'Remove this if statement! If True Then

If MessageBox.Show("Do You Want To Remove This Row", "Remove Row", MessageBoxButtons.YesNo, MessageBoxIcon.Stop) = DialogResult.Yes Then
    'Only do this if you are manually updating your datagridview
    DataGridView1.Rows.RemoveAt(DataGridView1.CurrentRow.Index)
    'otherwise update your data source here and rebind the datagridview
Else
    MessageBox.Show("Row Not Removed", "Remove Row", MessageBoxButtons.OK, MessageBoxIcon.Information)
'Remove this line DataGridView1.Rows.Add(DataGridView1.CurrentRow.Index)
End If
'Remove this End If

See solution 1 here for how to rebind how to refresh datagridview in vb.net[^]

I want to delete my record? Messagebox error

2 solutions

Solution 1

Solution 2

Add your solution here

Preview 0