Solved

500pts: Not all events are arriving in my thread, please help

Posted on 2006-10-31
8
155 Views
Last Modified: 2010-04-16
Hi there,

I wonder if anyone can help..

I have a thread which i start like this..

      _workerThread = new Thread(new ThreadStart(threadMarket.ProcessMarket));
                        _workerThread.Name = market;
                        _threads.Add( market, _workerThread );
                        _workerThread.IsBackground = true;
                        _workerThread.Start();

But once in my thread i subscribe to some events on an object, generally what happens is the event arrives with some arguments saying "LOADING" and then later "LOADED" but i am finding that the event "LOADED" never arrives unless i do a Thread.Sleep (500) and even then not all the events are arriving... This doesn't happen if i subscribe to the events in the gui....

I can't do a InvokeRequired because ThreadMarket is a class...

I am really stuck ... does _workerThread not have enough threads or something....

I look forward to any inspiration or help

Thanks in advance

Ian
0
Comment
Question by:ianinspain
[X]
Welcome to Experts Exchange

Add your voice to the tech community where 5M+ people just like you are talking about what matters.

  • Help others & share knowledge
  • Earn cash & points
  • Learn & ask questions
  • 4
  • 4
8 Comments
 
LVL 48

Expert Comment

by:AlexFM
ID: 17840598
Please show your code.
0
 

Author Comment

by:ianinspain
ID: 17840656
Hi alexfm,

Thanks... Ok yes of course... its quite simple.. the thing to remember is that it does work as far as the events do arrive with statusCode LOADING but never loaded unless i put the thread.sleep .... i think its something to do with threads... but a little unsure.

This is from the main GUI form ... i start the thread passing in

// This is just a class which i use to pass in a variable to a thread..
ThreadMarket threadMarket = new ThreadMarket(market);

_workerThread = new Thread(new ThreadStart(threadMarket.ProcessMarket));
                        _workerThread.Name = market;
                        _threads.Add( market, _workerThread );
                        _workerThread.IsBackground = true;
                        _workerThread.Start();

Then in the thread itself U(the class)... i have the following ....... _stcs is a hashtable which holds a number of stock items.

            private void SubscribeToMarkets(string market)
            {
                  // Iterate through all of the stocks
                  for (int j = 0; j < _stcs.Count; j++)
                  {
                        if ( (_stcs[j] as Stock).Market == market) // Do we have an stock in the current market?
                        {
                              // Yes - then lets process it
                              string stockInstance = (_stcs[j] as Stock).STC;
                              string marketInstance = (_stcs[j] as Stock).Market;
                              User localmyUser = MonitorForm.MainForm.GetUser;
                              
                              Console.WriteLine("sub to : " + marketInstance + "/" + stockInstance );
                              
                        
                              CliCommon.Stock stock= new Stock (localmyUser,stockInstance , marketInstance );
                              _myStocks.Add(stockInstance ,stock);
                              stock.view.xml = MonitorForm.MainForm.GetXml;
                              stock.view.np = 1;


                              // Subscribe to the event for receiving
                              stock.statusChanged +=new StatusChanged(stock_statusChanged);
                              Thread.Sleep(5000);
                              stock.load();
                        }
                  }
            }


and the event


            private void stock_statusChanged(object sender)
            {
                  //System.Windows.Forms.Application.DoEvents();
                  Stock stock= sender as Stock;
                  if(stock.status == Stock.statusCode.LOADED)
                  {
                        stock.statusChanged -=new StatusChanged(stock_statusChanged);
                        

                              ///////////// NEVER ARRIVES HERE UNLESS I PUT A THREAD.SLEEP but even then some things are missing..

                  }
                  else
                  {
                        //Thread.Sleep(5000);
                  }
            }
0
 
LVL 48

Expert Comment

by:AlexFM
ID: 17840703
stock.statusChanged -=new StatusChanged(stock_statusChanged);

This line unsubscribes from statusChanged event. After this line stock_statusChanged function is not called. Is this your intention?
0
Independent Software Vendors: We Want Your Opinion

We value your feedback.

Take our survey and automatically be enter to win anyone of the following:
Yeti Cooler, Amazon eGift Card, and Movie eGift Card!

 

Author Comment

by:ianinspain
ID: 17840721
Yes ... because ... once its status is loaded.. i unsubscribe so that i don't receive more evetns for that stock..

It is in the IF statement when its loaded but without the Thread.sleep ... it never arrives...

Ian
0
 
LVL 48

Expert Comment

by:AlexFM
ID: 17840857
What happens in the Stock class? Does it raise this event, what is thread context where this event is raised? Event is always raised in the context of thread where raising code is executed.

What is the purpose of this line:
Thread.Sleep(5000);        // SubscribeToMarkets

0
 

Author Comment

by:ianinspain
ID: 17840923
Hi AlexFM... The stock component belongs to somebody else... i am only using it... but I have just been in contact with him... and he has informed me that we might be running out of threads in thread pool....

Be in contact shortly

Ian
0
 
LVL 48

Accepted Solution

by:
AlexFM earned 500 total points
ID: 17840989
BTW, does your class exist when event is raised? Maybe it is already released at this time. Adding Sleep can explain this - class remains alive for some time and its event handler is working.
Other version: adding Sleep to event handler actually keeps caller's raising event call. This can change Stock class behavior if Stock is some complicated class which has different threads, synchronization etc. Sleep changes timing of this class, this can be question to class developer.
0
 

Author Comment

by:ianinspain
ID: 17864876
Thanks AlexFM,

its sorted now.... it appears that the class (not mine) was using too many threads... it is now resolved...

thanks

ian
0

Featured Post

Free Tool: Subnet Calculator

The subnet calculator helps you design networks by taking an IP address and network mask and returning information such as network, broadcast address, and host range.

One of a set of tools we're offering as a way of saying thank you for being a part of the community.

Question has a verified solution.

If you are experiencing a similar issue, please ask a related question

Extention Methods in C# 3.0 by Ivo Stoykov C# 3.0 offers extension methods. They allow extending existing classes without changing the class's source code or relying on inheritance. These are static methods invoked as instance method. This…
This article aims to explain the working of CircularLogArchiver. This tool was designed to solve the buildup of log file in cases where systems do not support circular logging or where circular logging is not enabled
The Email Laundry PDF encryption service allows companies to send confidential encrypted  emails to anybody. The PDF document can also contain attachments that are embedded in the encrypted PDF. The password is randomly generated by The Email Laundr…

733 members asked questions and received personalized solutions in the past 7 days.

Join the community of 500,000 technology professionals and ask your questions.

Join & Ask a Question