Threading Issue : "app.thread Event handler not working properly"

Expand|Select|Wrap|Line Numbers

1.               [STAThread]
2.         public static void main(string[] args)
3.         {
4.             Random rnd = new Random(); 
5.  
6.             for (int j = 0; j < 5; j++)
7.             {
8.                 App.Threads.Add(new Convertor());
9.             }
10.  
11.             Init();
12.  
13.             App.Start();
14.  
15.             Thread.Sleep(600);
16.  
17.             App.Stop();
18.         }
19.  

Maybe I'm blind before my first coffee in the morning but are you trying to use an object called App without having first defining/creating it?

The part which you are looking at is not mine... That was the part which was given to me written by some other guy... And Because I dont know what he did. SO I just used it :D .... The class convertor is written by me ... :( ..

I have an eventhandler called app.finished, but unfortunately, it is never called. Can any one tell me why?

Maybe I'm blind before my first coffee in the morning but are you trying to use an object called App without having first defining/creating it?

The part which you are looking at is not mine... That was the part which was given to me written by some other guy... And Because I dont know what he did. SO I just used it :D .... The class convertor is written by me ... :( ..

I think this answers your original question of "why doesn't it work?" Because you got a chunk of code that you don't understand, that really doesn't relate to your project, using objects you don't create and are just hoping that somehow it's all going to magically fit together and work.

There is no reason to have any kind of overrides in what you are doing. You really are doing anything that needs to be a class inheriting from another class, which is where you most commonly use an override.

I would suggest you start smaller. Before you worry about getting the readings from your weather sensor, just work on threading. Make a tiny application that does threading for frivolous things like adding two numbers.

Suggestion: Instead of having all of your sensor readings as one long method called "start"... have a method for each sensor, then you can call them easily from anywhere in your application. Like this you have to read all the sensors together. What if you want to read the windspeed twice as often as the temperature?

I think this answers your original question of "why doesn't it work?" Because you got a chunk of code that you don't understand, that really doesn't relate to your project, using objects you don't create and are just hoping that somehow it's all going to magically fit together and work.

Ouch that hurts!!!!
Buddy the only problem is that I am working under somebody else who is asking me to use this logic instead of creating the whole thing from scratch...

I would suggest you start smaller. Before you worry about getting the readings from your weather sensor, just work on threading. Make a tiny application that does threading for frivolous things like adding two numbers.

This hurts even more :@
I totally agree with you that I dont have knowledge for thread based programming, because I am doing it for the first time in my entire life, but with the exception of I already had done what you are telling me (Making of these tiny applications) but again its not me who's the boss of this all..

I can do two things, not post this code any place and bang my head on the wall or else I should use help of some one who is well educated like you ...

I actually wanted to write an article based on S/W house and R&D departments of different Regions .... but for right now thanks for the advice :)

Because you got a chunk of code that you don't understand, that really doesn't relate to your project, using objects you don't create and are just hoping that somehow it's all going to magically fit together and work.

Tell me if the code is written all wrong or it is wrong.... Because then I will start writing it again from the scratch and also do some sample programs for learning thread based programming as well..

Terribly sorry if some of those comments hurt your feelings.
If you have no choice but to use the code your boss provided then I suggest using it as it is nice to stay employed and buy food.

Since you have no choice but to use it, you have no choice but to understand it and clean it up.

Expand|Select|Wrap|Line Numbers

1.               [STAThread]
2.         public static void main(string[] args)
3.         {
4.             Random rnd = new Random(); 
5.  
6.             for (int j = 0; j < 5; j++)
7.             {
8.                 App.Threads.Add(new Convertor());
9.             }
10.  
11.             Init(); 
12.             App.Start(); 
13.             Thread.Sleep(600);
14.             App.Stop();
15.         }
16.  

Line 4: Makes a new Random. Why? It is never used.
Line 13: Sleeps the current thread for 600 milliseconds. Why?
Lines 8, 12, 14 all make use of an App ... What is it? Where does it get initialized?
These are probably all things you need to make sure you have a complete understanding of. Since they came from the boss: Ask him.

Expand|Select|Wrap|Line Numbers

1.     class Convertor : GThread
2.  

Your converter is inheriting from class GThread which appears nowhere in the code you posted, so there is nothing anyone here can tell about that. I can point out that your override of Start() never calls base.Start. So if there is some initialization that needs to happen there it doesn't happen.

I have an eventhandler called app.finished, but unfortunately, it is never called. Can any one tell me why?

Expand|Select|Wrap|Line Numbers

1.             // subscribe to ThreadFinish event
2.             App.ThreadFinish += new GThreadFinish(App_ThreadFinish);
3.  

This is the line at attaches the handler to the App.ThreadFinish event. If the handler doesn't get called, then one of two things is to blame:
1) The App.ThreadFinish event doesn't fire.
2) This line that does the attaching is never called, so you don't attach to the event. This line is part of the Init() rountine called from ThreadStarter.Main

Other than this I alse need to retrieve the value of the converted data and save it into the database.
Can any one provide me some professional assistance?

Please read both parts 1 & 2 of these excellent articles on using a database in your applications.
http://bytes.com/topic/net/insights/...rogram-part-ii

Its OK dear .... I dont have any problems with what you have said.,... :)

Well actually I am using Alchemi a software for enabling grid computing. To use Alchemi for grid based computing in your code you have to use a Alchemi.core.dll. This code actually refers to the Alchemi's Code. I tried to find it on many sites, but for alchemi I only got the same result that without creating an object of app writing codes.

Please read both parts 1 & 2 of these excellent articles on using a database in your applications.
How To Use A Database In Your Program Part II

And about the DB I know how to CRUD.. What I meant was taking back the values from the threads direct into the DB.. Is it possible?