Occasional error when updating DataGridView via DataTable

Question

4.50/5 (2 votes)

See more:

I'm re-posting this in QA from the C# forum to see if I can get a wider response.

I have a datagridview bound to a datatable. I update the datatable quickly and constantly (52 rows, each row updated multiple times per second). The datatable, datagridview, and updates are all done on the same thread and no updates are concurrent, although I feel my issue is caused by updates to the DataTable while the GridView is updating itself from a previous update to the Datatable.

Every so often, usually after a few minutes, I get an exception, here are the details:
Message = "Index was outside the bounds of the array."
Source = "System.Data"
Stack Trace:
at System.Data.Common.StringStorage.Get(Int32 recordNo)
at System.Data.DataColumnPropertyDescriptor.GetValue(Object component) at System.Windows.Forms.DataGridView.DataGridViewDataConnection.GetValue(Int32 boundColumnIndex, Int32 columnIndex, Int32 rowIndex)

The program does appear to continue functioning normally regardless of the error. It's driving me mad, has anyone ever encountered this before or have any advice? My only guess is that I think I may be updating the data source so quickly that the control is not finished drawing before I update the data source again. If this is the case, I have a couple of things to try:

I don't really want to throttle the updates because the GUI looks really good right now, but I can if needed.

I could try to un-bind and re-bind the datasource every time I update the datatable, too.

Update 1, I have tried putting a lock on the datagridview and the datatable before updating the datatable. However this had no positive effect.

Update 2, I can add my own handler to the DataError event of the GridView, but it doesn't help with debugging. If I leave out code for the event subscriber it still gets the exception, but the user would never know (I know this is not a solution).

Update 3, Addressing John's questions. My datatable is updated every time I receive a new price that I crack via a socket connection. Here is what the code looks like. Symbol_Feed is a Dictionary<string,>

MIDL

this.Symbol_Feed[symbol].SetField("BankVar0", quote.BankVar0);
this.Symbol_Feed[symbol].SetField("BankVar1", quote.BankVar1);
this.Symbol_Feed[symbol].SetField("BankVar2", quote.BankVar2);
this.Symbol_Feed[symbol].SetField("BankVar3", quote.BankVar3);
this.Symbol_Feed[symbol].SetField("BankVar4", quote.BankVar4);
this.Symbol_Feed[symbol].SetField("BankVar5", quote.BankVar5);

I had not given the ObservableCollection object any thought yet, but I will now.

I am not adding or deleting rows, only updating existing.

Posted 17-Dec-10 8:14am

wizardzz

Updated 17-Dec-10 10:05am

v7

Add a Solution

Comments

Slacker007 17-Dec-10 14:16pm

You may get some heat from the community for reposting a question. :)

wizardzz 17-Dec-10 14:18pm

I plan to delete the forum post, hope that's enough though.

wizardzz 17-Dec-10 14:21pm

Or does that make it worse? I know everyone loves seeing the "Message Removed" subject.

Pete O'Hanlon 17-Dec-10 15:20pm

You would have been better updating the forum message to point to this question.

wizardzz 17-Dec-10 15:22pm

You mean 3 wrongs don't make a right? I keep effing it up.

4 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)

#realJSOP · Answer 1 · 2010-12-17T08:20:00

Solution 1

Do you update it by re-retrieving the data, or by manually setting properties in the contained items?

Would it help to make the collection an ObservableCollection and binding to that?

Are you adding/deleting entire rows?

Posted 17-Dec-10 8:20am

#realJSOP

Comments

wizardzz 17-Dec-10 14:26pm

Since posting code in here is not going to format good, do you mind if I respond with an update to the original?

Slacker007 17-Dec-10 14:35pm

wizardzz: putting your code in the <pre> tags won't work?

wizardzz 17-Dec-10 14:37pm

yeah, but I hate not being able to see a big code block without scrolling.

#realJSOP · Answer 2 · 2010-12-17T09:42:00

Solution 3

Try putting a try/catch block around the code, and then just eating the exception, and see if that at least makes the code run without interruption.

If it's an index out of range exception, find out what index it's talking about and try to resolve it.

Posted 17-Dec-10 9:42am

#realJSOP

Comments

wizardzz 17-Dec-10 16:02pm

My update is wrapped in try / catch which never gets triggered. It's the GridView code that triggers the Exception. As mentioned in Update #2, I am able to add a handler to the DataError Event, and subsequently do nothing with it. This lets the program continue to run without interruption.

wizardzz 17-Dec-10 16:05pm

Oh, also, the index appears to be either the row or column the GridView is attempting to fetch data from, I do not call any of the code that generates this error, it just appears under load the GridView is corrupting itself.

paulesc · Answer 3 · 2012-08-23T04:58:00

Solution 4

I had a similar problem and I just handled the data error event.
I know it is crude but it seemed to do the trick.

Private Sub dgv_DataError(ByVal sender As Object, ByVal e As System.Windows.Forms.DataGridViewDataErrorEventArgs) Handles dgv.DataError
'No code here
End Sub

Posted 23-Aug-12 4:58am

paulesc

wizardzz · Answer 4 · 2010-12-17T09:14:00

I'm not going to mark it as solved because what I am about to say is not, in my opinion, a great solution. After discussing this with an architect here he told me he had a similar issue when doing something similar. He speculates that GridView just can't keep up, so he uses ListBox or ListView for this situation, especially since users will not be entering or editing any of the data. I guess it can handle the load.

So the question remains, is there any way to use gridview for this much data?

Occasional error when updating DataGridView via DataTable

4 solutions

Solution 1

Solution 3

Solution 4

Solution 2

Add your solution here

Preview 0

Existing Members

...or Join us