Thread Problem

[supercracker13]

Hallo ich habe ein kleines Problem mit den Threads ^^

Ich habe mal das Programm einfach zusammen gefasst und den Code geschrieben

PHP Code:

        public void Test()
        {
            string[] teststring = new string[] { "Nummer1", "Nummer2", "Nummer3", "Nummer4", "Nummer5" };
            List<Thread> tdlist = new List<Thread>();
            for (int i = 0; i < 3; i++)
            {
                tdlist.Add(new Thread(() => dowork(teststring[i])));
                Console.WriteLine("Sollte ankommen: " + teststring[i]);
                tdlist[i].Start();
            }
        }

        void dowork(string val)
        {
            Thread.Sleep(new Random().Next(2000, 10000));
            Console.WriteLine("Kommt an: " + val);
        } 


Es gibt ein Array, hier "teststring" und 3 Threads werden erstellt.
Auf diese 3 Threads sollen die Strings aufgeteilt werden.
Also festgelegt ist das der erste Thread den ersten string bekomt und so weiter. Aber danach soll der Thread der als erstes fertig ist den nächsten String bekommen. Ich habe es auch schon mit Thread.Join() probiert aber das Problem ist immer das damit nur ein Thread überwacht wird.

Zusätztlich kam das Problem daszu das bei den Code daoben die Ausgabe jetzt immer das selbe ist also bei dowork wird immer das selbe ausgegeben.
Wisst ihr zufällig was ich falsch mache ?

[Tasiro]

Originally Posted by supercracker13

Zusätztlich kam das Problem daszu das bei den Code daoben die Ausgabe jetzt immer das selbe ist also bei dowork wird immer das selbe ausgegeben.

Es wird keine (lokale) Kopie von i im Lambda referenziert, sondern direkt i. Das heißt, dass bis dahin das Programm z. B. folgendes tun könnte:
1. Das Erstellen von Thread eins, zwei und drei an Windows wird an Windows gemeldet, derweil wird i inkrementiert.
2. Thread zwei wurde gestartet und greift auf i zu. Das Auswerten von i ergibt den Wert 4. Dieser wird dann verwendet.
Du kannst direkt vor der Zeile mit dem Lambda ein var copy = i; einfügen und dann die Kopie verwenden, wie du es erwartest.
Wenn du eine foreach-Schleife verwendest, erhälst du ebenfalls das erwartete Verhalten.


Was du scheinbar zu erstellen versuchst, ist eine Art Thread Pool. Sofern nicht zu Lehrzwecken, solltest du versuchen, das bereits Vorhandene zu verwenden, wie z. B. die TPL.
Intern wird in der Regel eine Liste von durchzuführenden Aufgaben unterhalten, von welcher sich jeder Arbeits-Thread bedient.

[Onkelmat]

mach das ganze doch mit backgroundworker, der hat ein event completed

[G0dLesZ]

oder für die Aufgaben (in deinem Fall Liste von Zeichenketten) verwenden.

Pseudo Beispiel:

var queue = new Queue<string>{ "s1", "s2", "s3" };
while (queue.Count > 0) 
{
    var item = queue.Pop();
}

Für die Verwaltung der Threads verwendest Du dann einen Pool von Threads.
Hier bietet sich der an.


PS: Ab .NET 4.0 empfehle ich für Threading die (Task Parallel Library) zu verwenden ()

[supercracker13]

Hatte es schon gelöst. Auch mit einem ThreadPool.
Mache es zwar ein bisschen anders aber es funktioniert ^^

[KraHen]

TBH I don`t know any german and I don`t know what you`re asking but if you`re using threads like this it`s generally a good idea to join them. So after your first loop you`d do

PHP Code:

for (int i = 0; i < tdlist.Length; i++) 
{ 
     tdlist[i].Join();
} 


[supercracker13]

Your answer is one of the baddest i ever got.

1: I solved the problem allready.
2: If you dont know the question, dont answer
3: you code does nearly the same thing like my code, there is just the lenght of the list instead a static number

[Mostey]

Originally Posted by supercracker13

1: I solved the problem allready.

How is he supposed to know that if he doesn't speak German at all?

Originally Posted by supercracker13

2: If you dont know the question, dont answer

While this is partially true, you should appreciate his answer since he took time to help you and not to disturb you.

Originally Posted by supercracker13

3: you code does nearly the same thing like my code, there is just the lenght of the list instead a static number

That's wrong, his code is joining the threads instead of starting them, as your code does.

[Schlüsselbein]

And last but not least:

Originally Posted by supercracker13

baddest

*scnr*