Threading Question

[NameYou]

Let me give you some psudeo code - because I'm not understanding. Here is what I am doing: public class MainClass { private void GetData() { MyObject mo = new MyObject(); Thread t = new Thread(new ThreadStart(mo.Connect)); t.Start(); MyObject mo2 = new MyObject(); Thread t2 = new Thread(new ThreadStart(mo2.Connect)); t2.Start(); MyObject mo2 = new MyObject(); Thread t3 = new Thread(new ThreadStart(mo3.Connect)); t3.Start(); MyObject mo3 = new MyObject(); Thread t4 = new Thread(new ThreadStart(mo4.Connect)); t4.Start(); //WHAT I WANT TO DO IS BE ABLE TO GRAB THE RESULTS FROM THE THREADS STARTED ABOVE //AND PUSH THEM INTO A FILE. I NEED ALL OF THE DATA FROM ALL FOUR THREADS COMBINED INTO ONE FILE } } Public class MyObject { public MyObject() { } public void Connect() { //Starts all the socket stuff.. } //...OTHER SOCKET RELATED FUNCTIONS GO HERE. } How do I get the data in the Connect function to the calling function in MainClass?

uh... yeah

[Nader Elshehabi]

NameYou wrote:

How do I get the data in the Connect function to the calling function in MainClass?

You don't need to. If there is no good reason not to access the file from that method, you should. To be specific, here is my suggestion of you MyObject.Connect() method:

Public class MyObject
{
public MyObject()
{
}

public void Connect()
{
    //Starts all the socket stuff..
    object LockObject = new object()
    lock(LockObject)
    {
        StreamWriter MyStream = File.AppendText("MyFile.txt");
        MyStream.WriteLine(Input);
        MyStream.Close();
    }
}
//...OTHER SOCKET RELATED FUNCTIONS GO HERE.

}

Of course you got to do some modifications to the code to suite your needs, but I hope this was close enough.

Regards:rose:

[NameYou]

Ah I see what you are saying - my problem is this: then I have multiple disk IO reads, which will slow the system down compared to one. In this project, speed is absolutely necessary. Is there any way that I can write it to memory, so that I can combine the 4 results and write to the file once?

C#... king of languages

[Eric Dahlvang]

I'm no threading expert, but I think you need to consider that NameYou is creating multiple instances of MyObject. When you create LockObject in each instance, that means that each instances Connect code has a lock -- which is different from locking that piece of code for EVERY instance. You can see the difference by comparing the following two classes: 1)This just locks the code in this instances connect method.

public class MyObject
{
    public static string cFieldVal;

    public MyObject()
    {
    }

    public void Connect()
    {
        //Starts all the socket stuff..
        object LockObject = new object();
        lock(LockObject)
        {
            MessageBox.Show(MyObject.cFieldVal);
            MyObject.cFieldVal=MyObject.cFieldVal+="x";
            //StreamWriter MyStream = File.AppendText("MyFile.txt");
            //MyStream.WriteLine(Input);
           // MyStream.Close();
        }
    }
    //...OTHER SOCKET RELATED FUNCTIONS GO HERE.
}

2)This code locks a static field, so it is locked for EACH EVERY instance.

public class MyObject
{

    public static string cFieldVal="";

    public MyObject()
    {
    }

    public void Connect()
    {
        //Starts all the socket stuff..
        
        lock(cFieldVal)
        {
            MessageBox.Show(MyObject.cFieldVal);
            MyObject.cFieldVal=MyObject.cFieldVal+="x";
          //  StreamWriter MyStream = File.AppendText("MyFile.txt");
          //  MyStream.WriteLine(Input);
          //  MyStream.Close();
        }
    }
//...OTHER SOCKET RELATED FUNCTIONS GO HERE.
}

-- modified at 8:57 Wednesday 13th December, 2006

--EricDV Sig--------- Some problems are so complex that you have to be highly intelligent and well informed just to be undecided about them. - Laurence J. Peters

[Nader Elshehabi]

NameYou wrote:

Is there any way that I can write it to memory

Sure! You can make a string object in Connect method, keep concatenating strings to it, then finally at the end of your data reading write the whole string to the file. It's almost the same way. PS. EricDV is right. You should first make the LockObject private static for the lock to work. Another way to do it is to use Mutex class instead.

Regards:rose:

[Andy Brummer]

Actually, the first block doesn't do any locking at all since each method doesn't share the lock object. Your example blocks all instances, it doesn't matter how many MyObjects there are, only one of them can execute the locked code at any time. Removing the static keyword from cFieldVal will protect the value for each instance.

I can imagine the sinking feeling one would have after ordering my book, only to find a laughably ridiculous theory with demented logic once the book arrives - Mark McCutcheon

[Nader Elshehabi]

You are right. I've tested it. I hope I don't miss that next time.:)

Regards:rose:

[karam chandrabose]

I would suggest you to use asynchronous method invocation and callbacks to make this task much much simpler

[NameYou]

I'm not sure I understand Asynchronous method calling? I've never seen that before. Could you point me in the direction of some information on that, or shoot me over some example code?

uh... yeah

[NameYou]

OK, I understand that, but what I am getting at is that if I had the data write to a file at the end of the reading routine, that file write routine would be executed 4 times (since I have four threads) which is slower than dumping all of the read data into a buffer and then writing the buffer outside of the 4 threads - am I correct? Speed is of the essence with this procedure.

C#, king of languages

[Eric Dahlvang]

Andy Brummer wrote:

Actually, the first block doesn't do any locking at all since each method doesn't share the lock object.

That's what i was trying to say.

Andy Brummer wrote:

Your example blocks all instances, it doesn't matter how many MyObjects there are, only one of them can execute the locked code at any time. Removing the static keyword from cFieldVal will protect the value for each instance.

The intention of the example was to have a lock that every instance shares, and I should have said EVERY rather than EACH. I have changed it. Thanks for pointing that out.

--EricDV Sig--------- Some problems are so complex that you have to be highly intelligent and well informed just to be undecided about them. - Laurence J. Peters

[Nader Elshehabi]

Again this won't hurt. It would be a matter of milliseconds. Yet, if this small time really makes a difference with you, try benchmarking your program. For example, I do it by comparing the value of DateTime.Now.Ticks before and after the procedure I'm testing. See which method would give you the most satisfying results. PS. If you really need speed this much, why don't you think of unmanaged code -e.g. C++-?

Regards:rose:

[sno 1]

first sorry if I give'nt you a good answer because I dont undertand you a lot, ok this is fact: if you need know the process data of this 4 thread, you can use many tools, such as delegate, create a delegate and add all this function inside this, and then put the delegate on execution, for more explanation look this example, using a ProgressBar... public delegate void ExceutionAsyncronous(); public void function1(){} public void function2(){} public void function3(){} public void RequestTime(ProgressBar som, int value, int max) { som.Increment(value++,max); } inside your function put this method "RequestTime" in a way that the increment always keeping inside a correct interval, an then ... ExceutionAsyncronous ob = new ExceutionAsyncronous(function1); ob+=new ExceutionAsyncronous(function2); ob+=new ExceutionAsyncronous(function3); ob(); Remember that this explanation only want give you a Idea, ok if you need know anything else, please write me ok :-D:-D:-D