Jump to content
xisto Community
turbopowerdmaxsteel

Help Needed @ C#: Cross-thread Operation Not Valid

Recommended Posts

I have a class named as test which triggers an event using the Timer class, the code for which is given below.

using System;using System.Timers;namespace Test{	class test	{		public delegate void TestEventHandler();		public event TestEventHandler TestEvent;		protected Timer TestTimer = new Timer();		public test()		{			TestTimer.Elapsed += new ElapsedEventHandler(Tick);			TestTimer.Interval = 1000;		}		private void Tick(object source, ElapsedEventArgs e)		{			TestEvent();		}		public void Go()		{			TestTimer.Enabled = true;		}	}}


The problem that I am facing with, is that the Event runs in a seperate thread and doesn't allow any kind of operation on the Form's controls. The code for the form is given below.

using System;using System.Windows.Forms;namespace Test{	public partial class Form1 : Form	{		test A = new test();		public Form1()		{			InitializeComponent();			A.TestEvent += new test.TestEventHandler(A_TestEvent);		}		void A_TestEvent()		{			this.Text = "Test Text";		// This is where the Exception "Cross Threaded Operation not valid" is generated		}		private void Form1_Load(object sender, EventArgs e)		{			A.Go();		}	}}How do I invoke the event in the same thread as that of the form's?

Share this post


Link to post
Share on other sites

It's a new design change either with VS2005 or with the .NET Framework 2.0 to eliminate alot of concurrency bugs. When the exception handler pops up in VS, click the "more info" link...it will take you to an MSDN article explaining how to properly use delegates to do what you want to do in a thread-safe manner.

Share this post


Link to post
Share on other sites

Cross Thread Operations

Help Needed @ C#: Cross-thread Operation Not Valid

 

Yes , you cannot access a windows forms control or a windows forms component from a different thread. What you need to do in order to access the form controls is to invoke the control using a delegate with the help of control.BeginInvoke(). I have posted an article on the issue with sample code on my blog at https://asadsiddiqi.wordpress.com/2007/12/24/responsive-user-interfaces-with-ui-threads/

Hope this helps .

 

Thanks

 

-reply by Muhammad Asad Siddiqi

Share this post


Link to post
Share on other sites

Here I am posting the full solution to my own problem. Maybe this will help out some beginners. And no, it didn't take me a year and a half to figure this one out, thanks to MSDN. Its just that I didn't bother to look into this once I had the answer. Anyways, here goes it.

 

As said by Muhammad, the trick is in using the Control.BeginInvoke (or Control.Invoke) method to force the method to be executed in the same thread as that of the control. The difference between Invoke and BeginInvoke is that the former is synchronous while the latter is asynchronous. Using BeginInvoke makes more sense as the whole idea is to create efficient multi-threaded application. BeginInvoke starts a new thread passing along a delegate object of the method to be invoked and the parameters to be passed to it.

 

In this case, the BeginInvoke method of the owner Form has to be called from the test class. So, we include an ISynchronizeInvoke parameter in the constructor of the class which is then stored in a member variable. The ISynchronizeInvoke Interface must be implemented in-order to execute a delegate asynchronously. Thankfully, this has been done in the System.Windows.Forms.Control class, so it doesn't require any work on our part.

 

protected ISynchronizeInvoke SyncObject;public test(ISynchronizeInvoke SyncObject){	this.SyncObject = SyncObject;	TestTimer.Elapsed += new ElapsedEventHandler(Tick);	TestTimer.Interval = 1000;}

We add a new delegate with the same signature as that of the method Tick (the event handler for the System.Timers.Timer's elapsed event).

 

protected delegate void TickDelegate(object source, ElapsedEventArgs e);

The Tick method makes sure that it is running in the same thread as that of the Synchronizing object passed to the class's constructor using the Control.InvokeRequired property. If it is not, it calls the BeginInvoke method of the SyncObject passing along a delegate to itself and an object array of it's parameters. No further work is done in this call to the Tick method. After a while, the Tick method is invoked again, only this time on the owner form's thread. The InvokeRequired property returns false and the actual work of the method is done.

 

private void Tick(object source, ElapsedEventArgs e){	if (SyncObject.InvokeRequired)		SyncObject.BeginInvoke(new TickDelegate(Tick), new object[] { source, e });	else		TestEvent();}

Only one change needs to be made in the Form class - a reference to itself must be passed to the contstructor of the test class. For this, the instantiation of the object is now done inside the constructor (this keyword is only valid inside a non-static property, method, or constructor).

 

test A;public Form1(){	InitializeComponent();	A = new test(this);	A.TestEvent += new test.TestEventHandler(A_TestEvent);}

Full code for the two classes:-

 

Test class

 

using System;using System.Timers;using System.ComponentModel;namespace Test{	class test	{		public delegate void TestEventHandler();		public event TestEventHandler TestEvent;		protected Timer TestTimer = new Timer();		protected ISynchronizeInvoke SyncObject;		public test(ISynchronizeInvoke SyncObject)		{			this.SyncObject = SyncObject;			TestTimer.Elapsed += new ElapsedEventHandler(Tick);			TestTimer.Interval = 1000;		}		protected delegate void TickDelegate(object source, ElapsedEventArgs e);		private void Tick(object source, ElapsedEventArgs e)		{			if (SyncObject.InvokeRequired)				SyncObject.BeginInvoke(new TickDelegate(Tick), new object[] { source, e });			else				TestEvent();		}		public void Go()		{			TestTimer.Enabled = true;		}	}}

Form Class

 

using System;using System.Windows.Forms;namespace Test{	public partial class Form1 : Form	{		test A;		public Form1()		{			InitializeComponent();			A = new test(this);			A.TestEvent += new test.TestEventHandler(A_TestEvent);		}		void A_TestEvent()		{			this.Text = "Test Text";		// This is where the Exception "Cross Threaded Operation not valid" used to be generated		}		private void Form1_Load(object sender, EventArgs e)		{			A.Go();		}	}}

Edited by turbopowerdmaxsteel (see edit history)

Share this post


Link to post
Share on other sites
Another wayHelp Needed @ C#: Cross-thread Operation Not Valid

Hi all!

Just wondering why you wouldnt just use the timer thats found in the System.Windows.Forms.Timer class? The difference is that this "Forms" timer is always calling the Tick event handler on the same thread as the UI. So it seems to be an easier solution to your problem above. However, your solution is still valid when its not the UI thread you are trying to execute commands on. :-)

Kyryll 

-reply by Kyryll

Share this post


Link to post
Share on other sites

Create an account or sign in to comment

You need to be a member in order to leave a comment

Create an account

Sign up for a new account in our community. It's easy!

Register a new account

Sign in

Already have an account? Sign in here.

Sign In Now

×
×
  • Create New...

Important Information

Terms of Use | Privacy Policy | Guidelines | We have placed cookies on your device to help make this website better. You can adjust your cookie settings, otherwise we'll assume you're okay to continue.