|
I have solved the problem, just in case anyone is interested.
private void dgv_CellContentDoubleClick(object sender, DataGridViewCellEventArgs e)
{
SearchIndex search = new SearchIndex();
search.docOpen(this.dgv.CurrentCell.Value.ToString());
}
public void docOpen(string path)
{
Process proc = new Process();
proc.StartInfo.FileName = "iexplore";
proc.StartInfo.Arguments = path;
proc.Start();
}
|
|
|
|
|
This problem has to do with updating a control/dataset based on the DataRecevied event of the serial port. Now I know you are thinking cross threading (which it might be) but let me explain.
I have a dataset in the main UI form that is databound to a number of controls. When I change the values of the dataset the bound controls update. This is fine, but when I update the values of the dataset via the DataReceived event the values DON'T update? To illlustrate the problem simply I have stripped down the code to it's most basic form. I hope someone can shed some light on the problem.
<br />
public Form1()<br />
{<br />
InitializeComponent();<br />
}<br />
<br />
private void Form1_Load(object sender, EventArgs e)<br />
{<br />
UnitDataSet uds = new UnitDataSet();<br />
uds.DIns.ReadXml("Data.xml");<br />
<br />
_comms = new Comms(uds);<br />
<br />
Label lblInputState = new Label();<br />
panel1.Controls.Add(lblInputState);<br />
<br />
lblInputState.DataBindings.Add("Text", uds.Tables["DIns"], "state");<br />
}<br />
<br />
public void button1_Click(object sender, EventArgs e)<br />
{<br />
_comms.ChangeValue();<br />
}<br />
}<br />
<br />
============================================================<br />
<br />
class Comms<br />
{<br />
SerialPort _sp;<br />
UnitDataSet _uds;<br />
<br />
public Comms(UnitDataSet uds)<br />
{<br />
_uds = uds;<br />
_sp = new SerialPort("COM7", 9600, Parity.None, 8, StopBits.One);<br />
_sp.DataReceived += new SerialDataReceivedEventHandler(_sp_DataReceived);<br />
_sp.Open();<br />
}<br />
<br />
private void _sp_DataReceived(object sender, SerialDataReceivedEventArgs e)<br />
{<br />
ChangeValue();<br />
}<br />
<br />
public void ChangeValue()<br />
{<br />
bool tmp = (bool)_uds.Tables["DIns"].Rows[0]["state"];<br />
_uds.Tables["DIns"].Rows[0]["state"] = !tmp;<br />
}<br />
}<br />
Visual C# Express 2005
|
|
|
|
|
If you enable the debugger to break on all CLR exceptions (Tools -> Exceptions -> Check the box) and run the form under debug mode if it's a one of the many cross-thread problems then the debugger should trap the exception.
Try using in the DataReceived handler, this.Invoke(ChangeValue), which will be thread-safe.
|
|
|
|
|
Thanks for you comments Ed. I can't seem to find Tools -> Exceptions. I'm using Visual C# express? I'll try your invoke method now.
|
|
|
|
|
there is no this.invoke() ??!
|
|
|
|
|
The Invoke method is a member of the Form data type, it's used to provide thread-safe updates of the form's controls / components. If you're not using a form you might want to take a look at my article[^] :->, which will allow you to do the thread-safe invocation from anywhere.
The Exceptions box is under Debug | Exceptions sorry, I don't know about the express edition but that's where it is in the Standard edition of VS.
|
|
|
|
|
Thanks Ed! Actually I saw you article today when I was searching for help. You are right it's not a form. I scanned over your article but it seemed quite complex to me (I still haven't really got my head around delegates or generics) but I'm fast running out of options so I'm going to study it and hopefully I'll find the answer.
|
|
|
|
|
The article actually needs an update, which will make it simpler to use (removes generics). I've got the changed code here[^]. Basically from inside your DataReceived handler all you need to do is call:
InvocationHelper.Invoke(this.ChangeValue); All the delegate is is a pointer to a function (if you know C), if you don't it's basically a variable which stores information about a particular function or method, you can then pass this variable around as a normal variable, and you can also invoke the function the delegate points to as well.
The helper routine basically just bubbles up through the class hiearchy to see if the class implements the interface ISynchronizewhatever, this just defines the Invoke method and means that the class can do things in a thread-safe manner.
PS, generics are quite simple to understand and incredibly useful once you know how.
|
|
|
|
|
Thanks again Ed! I've just arrived and work this morning and it's great to see your message. Well, although I'm new to generics, I am actually using them in my project already. I'm sure I'll use them more and more as they do seem well suited to many situations, I'm just getting to the 'once you know how part'. The word delegate makes me nervous, but I'll put it with the 'once you know how' and I'm sure I'll get the hang of it. Hey, your source has been blocked, could you zip it please? Thanks heaps, you've been great.
Oh, I forgot to say that under the Debug -> Exceptions, are you refering to the 'thrown' check box or the 'user-handled' check box associated with the CLR Exceptions tree?
-- modified at 15:05 Wednesday 14th February, 2007
|
|
|
|
|
Ok, zipped, same location[^] but .zip on the end, thanks for finding that out about the .cs extensions, never knew about that.
Best way to think of delegates is to just treat them as variables, they're nothing more anyway.
The checkbox is thrown, it helps a lot because the debugger will break when an exception is thrown rather than when it is not handled, e.g. if you've got a try / catch block normally it'd pass through and execute the catch, but with this it breaks on the exception inside the try allowing you to find the source of the problem.
|
|
|
|
|
Great info about the exception I'll give it a crack now, and check out your helper class. Super! 
|
|
|
|
|
Let me know if it's successful.
|
|
|
|
|
I have an error when I compile. This is the code I updated:
<br />
private void _sp_DataReceived(object sender, SerialDataReceivedEventArgs e)<br />
{<br />
InvocationHelper.Invoke(this.ChangeValue);<br />
}
Argument '1': cannot convert from 'method group' to 'System.Delegate'
P.S If you prefer to have a fiddle yourself I can send you the source. Thanks for being so helpful.
|
|
|
|
|
Oops, forgot the delegate bit 
private delegate void ChangeValueDelegate();
private void _sp_DataReceived(object sender, SerialDataReceivedEventArgs e)
{
InvocationHelper.Invoke(new ChangedValueDelegate(this.ChangeValue));
}
|
|
|
|
|
Dang, didn't work. Does the block:
singleCastDelegate.DynamicInvoke(arguments);
Mean that it's not cross threaded?
Curiously though, after I use your class then click on the button to change the value (just to see if it still works) it doesn't update the value in the label anymore...
P.S I had to put a check in your code as I was getting an error
if (arguments != null) <- here<br />
{<br />
if (requiredParameters != arguments.Length)<br />
....<br />
}
|
|
|
|
|
Yes it's not cross-threading, I had a proper look through your code and now understand exactly what you're trying to do.
Did you change the code in the button's event handler? That can stay the same, it's only the SerialPort stuff because of the way that Windows uses a seperate thread to read data internally.
Re: error, you can actually scrap the first "wrapper" method completely, it's not needed at all (in fact I don't know why it's there at all), if it wasn't there then you wouldn't need your check because of the params declaration the arguments array would be zero length as opposed to null  .
Try setting a breakpoint on the line including the ChangeValue inside the DataReceived handler, if that's hit then all handlers are setup properly for the SerialPort, then try F11 (in standard edition anyway) to do Debug -> Step Into, the debugger should then jump to the ChangeValue routine. Just step through the code and examine the variables to see if anything untoward is happening.
|
|
|
|
|
- I didn't change the code in the buttons event handler.
- I scrapped the wrapper method.
- The handler is firing ok, and I've stepped through the ChangeValue method, it's doing what it should - that is, it's changing the value in the dataset. I know this because whether the DataReceived event handler or the button event calls ChangeValue the value changes and I can see it as I step though that it's been changed from the previous value. What's not updating is the Text property of the Label control that the Dataset is bound to.
P.S If I trigger the ChangeValue method via the button first, the label updates fine. But if I trigger ChangeValue via your delegate method and then try the button the label doesn't update.
|
|
|
|
|
How does it behave without the debugger attached (Ctrl+F5)
|
|
|
|
|
Now I'm confused, I just created a simple program to duplicate what you're doing and got some really bizarre behaviour. If running without the VS debugger attached it works fine, with the debugger attached it won't update the textbox, just as your experiencing. 
If you're interested my code is here[^], all there is is a small dataset (one table, one column), a form which has a combo box for selecting the serial port, a button to open / close the port and a textbox which is bound to the dataset. I didn't create a seperate class like you did but experienced the same problem.
But it works fine without the debugger 
|
|
|
|
|
Holy Bible Batman you're right!!!! Arrgggh! I didn't test your code but just tried it on my own and it works... thanks to you and your tenacious approach. And not one delegate or generic in sight!
My workmate says that if your over our neck of the woods sometime (NZ) we owe you a beer. Thanks very much Ed, you've been a real help on a prob that had me completely stumped. 
|
|
|
|
|
Hmm, might take you up on that offer if I ever go down that way. Confused though, you're in the States (according to bio) and you work in New Zealand?
If it is the States I should hopefully be going from the 19th of April to 7th May, but that'll be Houston (7 days), San Antonio (7 days) and New York (4) days. But most of the time will be spent shooting I think, World Championships being the reason for the trip.
Glad to have been of help.
|
|
|
|
|
Ooops well, I guess I didn't update the default profile. I'm actually from NZ and live in NZ! So, we will have to have that beer another time! Shooting eh??! World champs! I guess you are competing, well if you dead eye with the pistola as you are with code you're bound to do well!
Thanks again for you help, I really appreciated the time you spent. Good luck!
|
|
|
|
|
TheTinSoldier wrote: dead eye with the pistola
Nah, bigger & better :->
The world champs are for Sporting which is a form of Clay Pigeon Shooting == Shotgun
Haven't shot a pistol in years, my other shooting involves Air-Rifling (High-powered[^], i.e. it requires a firearms certificate here in Britain) with a bit of rifling, .22, .243 & .303.
|
|
|
|
|
Now that sounds like fun! I was talking to my brother over the weekend about shooting guns and I must admit that I reakon is would be a real blast - pun intended. I've shot a .22 and a slug gun when I was a kid but that's about it. I've still to check off the shotgun! It's got to be on the list of things to do at least once! Well anyway Ed, good luck and bring back some silverware!
|
|
|
|
|
TheTinSoldier wrote: good luck and bring back some silverware!
Thanks, I've finally been given the go-ahead by the doctors so I'm definitely off to America now , just wish I'd changed my gun before I went (I've been trying out a friend's gun and been shooting better with it)
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
