streamwriter eller reader fryser

allHandlers var for at serveren kunne holde styr på sine forbindelser og lukke dem hvis det blev nødvendigt.

Hmm det virker fint her.

Serveren står blot for at acceptere forbindelserne, den delegerer kommunikationen til ConnectionHandlers.

Server.cs:

 using System.IO;
 using System.Net;
 using System.Net.Sockets;
 using System.Threading;
 
 namespace TestServer
 {
     class Server
     {
         private string ipAddress = "127.0.0.1";
         private int port = 8085;
 
         NetworkStream networkStream;
         StreamWriter streamWriter;
         StreamReader streamReader;
 
         int clientsHaveConnected = 0;
 
         public void start()
         {
             TcpListener tcpListener = new TcpListener(IPAddress.Parse(ipAddress), port);
             tcpListener.Start();
 
             while (true)
             {
                 Socket socketForClient = tcpListener.AcceptSocket();
                 ConnectionHandler handler = new ConnectionHandler(socketForClient, ++clientsHaveConnected);
 
                 Thread newThread = new Thread(new ThreadStart(handler.HandleClient));
                 newThread.Start();
             }
         }
 
     }
 }

ConnectionHandler får en socket som er klienten den skal snakke med og så begynder at kommunikere:

ConnectionHandler.cs:

 using System;
 using System.Net.Sockets;
 using System.IO;
 
 namespace TestServer
 {
     class ConnectionHandler
     {
         Socket socket;
         StreamReader reader;
         StreamWriter writer;
         int client;
         public ConnectionHandler(Socket socket, int clientNumber)
         {
             this.socket = socket;
             NetworkStream stream = new NetworkStream(socket);
             reader = new StreamReader(stream);
             writer = new StreamWriter(stream);
             client = clientNumber;
         }
 
         public void HandleClient()
         {
             if (socket.Connected)
             {
                 Console.WriteLine("Thred startede ");
 
                 while (true)
                 {
                     sendAndReseaveMessage("Hello Client" + client);
                 }
             }
             else
             {
                 Console.WriteLine("Not Connected");
             }
         }
 
         private void sendAndReseaveMessage(string message)
         {
             writer.Flush();
 
             string messeageFromClient = reader.ReadLine();
             Console.WriteLine(messeageFromClient);
 
             writer.WriteLine(message);
             writer.Flush();
         }
     }
 }

Der er nok et par ændringer i IP og namespaces som du lige skal rette tilbage, men ellers er der ikke meget andet. I stedet for at opretty nye stream objects gemmes de på ConnectionHandleren.

Med dette var jeg i stand til at starte serveren, forbinde 3 klienter, og de kunne alle skriv og modtage hver for sig.

Når din applikation bliver mere udviklet bør du tilføje fejlhåndtering. F.eks. går det ikke godt hvis en klient lige pludselig lukker forbindelsen.

For hver klient der forbinder starter du en ny tråd op som udfører metoden socketServer, som skal håndtere kommunikation med den nyligt forbundne klient. Men du har kun en enkelt socketForClient til alle trådene så de andre socket forbindelser går tabt fordi du overskriver socketForClient og derfor virker kun en klient.

Du er nødt til at gemme din socket for hver enkelt forbindelse. Nu kan jeg ikke se hvad din nuværende klasse hedder, men lad os kalde den 'Server'. En mulighed er at du så laver en ny instans af en klasse f.eks. kaldet 'ConnectionHandler' som står for at håndtere forbindelsen:

 while (true)
 {
     socketForClient = tcpListener.AcceptSocket();
     ConnectionHandler handler = new ConnectionHandler(socketForClient);
     allHandlers.add(handler); //en collection af handlers i server klassen.
     Thread newThread = new Thread(new ThreadStart(handler.Start));
     newThread.Start();
 }

Og din ConnectionHandler vil så indeholde det der var din socketServer og sendAndReseaveMessage. Groft udkast:

 class ConnectionHandler
 {
     private Socket socket;
     public ConnectionHandler(Socket socket)
     {
         this.socket = socket;
     }
 
     public void Start() {
         //Same like socketServer func.
     }
 
     public void SendAndReceive() {
         //Like sendAndReseaveMessage
     }
 }

Hejsa igen..

Jeg er i tvivl om hvad linien

 allHandlers.add(handler); //en collection af handlers i server klassen.

er til for?

Jeg har lavet ConnectionHandler klassen med det indhold du har vist. men synes ikke at kunne få det til at virke /:

Hejsa igen...

Inden vi så dit svar havde vi løst problemet ved at lægge networkstrem ned i get og set metoder, men har valgt updatere det med din løsning, da den ser pænere ud..

Vi takker for din hjælp