Delete a record from an array VB.NET

Question

I'm trying to delete a record from an array in VB.Net but I can never get it to delete the correct one

In my code, intPosition is the position where the desired record I want to delete is. Customers is the name of the 3D array and NumberOfCustomers can be treated as the size of the array.

  If MsgBox("Are you sure you want to delete this record?", MessageBoxButtons.YesNo) = Windows.Forms.DialogResult.Yes Then
        NumberOfCustomers -= 1
        For i = intPosition + 1 To NumberOfCustomers
            Customers(i - 1) = Customers(i)
        Next
        NumberOfCustomers -= 1
        ReDim Preserve Customers(NumberOfCustomers)
        Call SaveCustomer()
    End If

Please could someone amend or find similar code for this in VB.NET.

Thanks

You are torturing the array. Use a List(Of T) or something better suited to the types of things you are trying to do. — rory.ap
– rory.ap, Commented Jan 12, 2016 at 17:56
rather than an array, if it was a List(Of Customer) (which will take all of 15 mins to learn about) Customers.RemoveAt(indexToRemove) is all you need — Ňɏssa Pøngjǣrdenlarp
– Ňɏssa Pøngjǣrdenlarp, Commented Jan 12, 2016 at 17:58
Also, are you sure this is VB.NET? There is no such enum MessageBoxButtons. It should be MsgBoxStyle. — rory.ap
– rory.ap, Commented Jan 12, 2016 at 17:59
@roryap There's a MessageBoxButtons enum in the System.Windows namespace, although it should be used with the MessageBox class and not with the MsgBox VB function. — D Stanley
– D Stanley, Commented Jan 12, 2016 at 18:39

rory.ap · Accepted Answer · 2016-01-12 18:03:17Z

4

I would propose you do without the array as it's extremely inefficient for operations like this. Instead, you should use one of the built-in classes like List(Of T) which is much better suited for how you're trying to use it.

Dim customers = New List(Of Customer)

'populate your list however you do it.

If MsgBox("Are you sure you want to delete this record?", MsgBoxStyle.YesNo) = MsgBoxResult.Yes Then
    customers.RemoveAt(position)
End If

answered Jan 12, 2016 at 18:03

rory.ap

35.7k15 gold badges104 silver badges199 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

Delete a record from an array VB.NET

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related