Chat Programm, Thread wird nicht beendet [C#]

Werbeanzeige

Pixma

Frischling

Beiträge: 35

Wohnort: Mainz

Beruf: Fachinformatiker für Anwendungsentwicklung

08.11.2011, 14:15

Chat Programm, Thread wird nicht beendet [C#]

Hallo,
diese Funktion macht mir den Ärger und zwar führe ich die Funktion in einem Thread aus.

C#-Quelltext

 private void ListenForClients()
        {
            this.tcpListener.Start();
            while (runsApplication)
            {
                //blocks until a client has connected to the server
                TcpClient client = this.tcpListener.AcceptTcpClient();

                //create a thread to handle communication
                //with connected client
                Thread clientThread = new Thread(new ParameterizedThreadStart(HandleClientComm));
                clientThread.Start(client);
            }
        }

Wenn ich das Programm nun beendet, läuft dieser Thread immernoch, weil der nicht aus der Schleife herausgeht.
Ich hoffe ihr könnt mir ein paar Hilfestellungen geben, wie ich dieses Problem lösen kann.

Zum Seitenanfang

dot

Supermoderator

Beiträge: 9 757

Wohnort: Graz

08.11.2011, 14:22

Ich versteh das Problem nicht ganz, wie sieht denn die Funktion aus, die in den gestarteten Threads ausgeführt wird? Wie genau sorgst du dafür, dass diese bei Programmende returned?

m4studios

Zum Seitenanfang

Sacaldur

Community-Fossil

Beiträge: 2 301

Wohnort: Berlin

Beruf: FIAE

08.11.2011, 14:25

this.tcpListener.AcceptTcpClient() blockiert den aktuellen Thread bis eine Verbindung aufgebaut wird
sobald ein Client die Verbindung aufbaut wird ein weiterer Thread gesatertet (Schleife zu Ende) und solange die Anwendung noch läuft, wird die Schleife neu begonnen und wieder auf einen Client gewartet
an und für sich ist das richtig und nicht dsa Problem
sollte das Ergebnis von runsApplication später den Wert ändern, dann kann das erst ausgewertet werden, wenn sich ein weiterer Client verbindet
das heißt, du musst den Thread auch manuell beenden

Spieleentwickler in Berlin? (Thema in diesem Forum)
---

Zitat von »Jonathan_Klein«

Es ist ja keine Schande etwas falsch zu machen, als Programmierer tu ich das täglich, [...].

Zum Seitenanfang

DerMark

Treue Seele

Beiträge: 324

Wohnort: Emsdetten

Beruf: Softwareentwickler

08.11.2011, 14:31

Aus der MSDN:

Zitat

Use either AcceptSocket or AcceptTcpClient to pull a connection from the incoming connection request queue. These two methods will block. If you want to avoid blocking, you can use the Pending method first to determine if connection requests are available in the queue.

Prüfe also regelmäßig ob überhaupt jemand sich verbinden möchte und Accepte erst dann, dadurch wird erstmal der Thread nicht blockiert. Noch besser wäre es aber wenn du

http://msdn.microsoft.com/en-us/library/…ttcpclient.aspx

verwenden würdest, ist sicher komplexer zu benutzen, aber nimmt dir die Arbeit mit dem Thread ab.

Zum Seitenanfang

Pixma

Frischling

Beiträge: 35

Wohnort: Mainz

Beruf: Fachinformatiker für Anwendungsentwicklung

08.11.2011, 14:32

selbst das manuelle beenden, des Threads funktioniert nicht.
private void Form1_FormClosing(object sender, FormClosingEventArgs e)
{
runsApplication = false;
listenThread.Abort()
}

Der mark:
Ok probier ich grade aus

Zum Seitenanfang

dot

Supermoderator

Beiträge: 9 757

Wohnort: Graz

08.11.2011, 14:35

Thread.Abort() ist keine Lösung. Der einzige richtige Weg um einen Thread zu beenden, ist ein return aus der Thread-Funktion.

m4studios

Zum Seitenanfang

Pixma

Frischling

Beiträge: 35

Wohnort: Mainz

Beruf: Fachinformatiker für Anwendungsentwicklung

08.11.2011, 14:40

danke, der Mark.
Ich konnte das Problem lösen

C#-Quelltext

 private void ListenForClients()
        {
            this.tcpListener.Start();
            while (runsApplication)
            {
                //blocks until a client has connected to the server
                //TcpClient client = this.tcpListener.AcceptTcpClient();
                tcpListener.BeginAcceptTcpClient(
                new AsyncCallback(HandleClientComm),
                tcpListener);
                //create a thread to handle communication
                //with connected client
                //Thread clientThread = new Thread(new ParameterizedThreadStart(HandleClientComm));
                //clientThread.Start(client);
            }
        }

Dot, danke ebenfalls, nun bin ich das nächste mal Gerüstet, wenn ich wieder einmal mit Threads arbeiten muss

Zum Seitenanfang

BlueCobold

Community-Fossil

Beiträge: 10 738

Beruf: Teamleiter Mobile Applikationen & Senior Software Engineer

08.11.2011, 19:41

DAS ist eine prima Lösung, wenn Du den gesamten Speicher und die gesamte CPU-Last Deines Rechners auf einmal verbraten willst.
Das Ding heißt ja nicht umsonst AsyncCallback!

Teamleiter von Rickety Racquet (ehemals das "Foren-Projekt") und von Marble Theory

Willkommen auf SPPRO, auch dir wird man zu Unity oder zur Unreal-Engine raten, ganz bestimmt.[/Sarkasmus]