C# Chatserver

[RedJ0hn]

Hallo Leute,

ich habe folgende Frage, da ich für die Schule einen Chat Server in C# schreiben muss.

Und zwar geht es um folgendes ich habe einen TCP Server der auf einem Port lauscht, und bereits die erste Anfrage annimmt und bearbeitet. Er akzeptiert neue Clients und diese können sich schon einen Namen zu weisen.

Jetzt ist das Problem, dass jeder Client welcher auf dem Server verbunden ist eine Nachricht bekommen soll. Ich habe ein Klasse und darin die Methode für die Verbindung, welche folgendermaßen aussieht:

Aus einer meiner Klassen:

 Spoiler

        private void Run()
        {
            sw.AutoFlush = true;
            sw.WriteLine("Hallo lieber Client!");
            if (name == "unbenannt") // Ist der Standard Wert der Variable
            {
                sw.WriteLine("Gebe zunächst deinen Namen ein:");
            }

            sw.WriteLine("Wie heißt Du?");
            name = sr.ReadLine();
            if (name == "")
            {

                sw.WriteLine("Wenn du weiter möchtest gib deinen Namen ein:");

            }
            else
            {
                sw.WriteLine("Hallo " + name);
 


                    foreach (Verbindung v in verbindungen) // Dort liegt das Problem
                    {
                        string nachricht = sr.ReadLine();
                        sw.WriteLine(nachricht);


                    }

            }

 Spoiler

    private void Run() {


            Listener = new TcpListener(IPAddress.Any, PORT);
            Listener.Start();

            while (true) {


               var client = Listener.AcceptTcpClient();
               //verbindungen.Add(client);
               var verbindung = new Verbindung(client, verbindungen);
               counter++; // Bitte nicht beachten spielt keine Rolle
            }

Ich denke ich muss, meine Clients einer Liste hinzufügen, und dann sagen jeder in der Liste soll die Nachricht empfangen.

Jedoch habe ich da anscheinend noch irgendwo ein Fehler, da mir gesagt wird, dass Verbindungen keinen Wert enthält und leider weiß ich nicht, wie man einen client dieser Liste hinzufügen soll
Die Liste:

 Spoiler

List<Verbindung> verbindungen = new List<Verbindung>();

Vielen Dank.

MfG

[#SoNiice]

Wo ist das Problem deiner Liste etwas hinzuzufügen? Die Liste ist vom Typ "Verbindung".
Also wärs 'verbindungen.Add(verbindung);' - Oder wo hapert es bei dir?

[RedJ0hn]

Ich glaube das Problem ist, dass ich die Klasse und Liste doof benannt habe, aber davon abgesehen kann ich Verbindung nicht einsetzen, weil das der Name der Klasse ist und nicht als Variable verwendet werden kann.

[#SoNiice]

Mir ist gerade erst aufgefallen, dass da ja gar nicht die selbe Methode ist in den beiden Spoilern

Habe dir mal eine PN geschickt, schau es dir am besten einfach mal an ;-)

[RedJ0hn]

^Erstmal Vielen Dank für die Mühe.

Jedoch würde ich es gerne ohne Hashtable machen, gibt es dann noch eine andere Alternative?

MfG

[#SoNiice]

Warum möchtest du die Nachrichten überhaupt vom Clienten aus an alle senden?

Client -> Server -> All Clients

[RedJ0hn]

Originally Posted by SoNiice

Warum möchtest du die Nachrichten überhaupt vom Clienten aus an alle senden?

Client -> Server -> All Clients

Naja,

ich wollte den Server so aufbauen, dass es einen Channel gibt, in dem alle schreiben und die Nachrichten die ohne jegliche Befehle geschrieben werden, kann jeder lesen, also ein Broadcast innerhalb des Servers an die Clients.

Dann wollte ich das es eine /flüster Funktion gibt für einzel Gespräche.

Also ein Client --sendet Nachricht --> Server --> X
--> Y
--> Z

Ich hoffe das Problem was ich habe ist soweit klar, also wenn ich die Methode aufrufe und etwas schreibe dann bekommt nur der Client die Nachricht der diese auch geschrieben hat. Darum brauche ich eine Möglichkeit jeden Clienten der auf den Server ist anzusprechen.

Desweiteren wäre es ja schön eine Benachrichtigung zu haben für jeden neuen User Zugang. Also XYZ hat den Server betreten / verlassen.

MfG

[#SoNiice]

Du erstellst auf deinem Server eine Liste mit den Verbindungen, bei jeder eingehenden Verbindung fügst du eine neue in die Liste ein. Wenn der Server eine Nachricht vom Clienten empfängt überprüft dieser diese Nachricht auf einen bestimmten Parameter, wenn der nicht gesetzt ist, geht er die Liste durch und sendet an jeder User die Nachricht vom anderen User. Der Client empfängt diese und verarbeitet sie.

Ich verstehe nicht so ganz, was bei dir daran hapert.

[RedJ0hn]

Originally Posted by SoNiice

Du erstellst auf deinem Server eine Liste mit den Verbindungen, bei jeder eingehenden Verbindung fügst du eine neue in die Liste ein. Wenn der Server eine Nachricht vom Clienten empfängt überprüft dieser diese Nachricht auf einen bestimmten Parameter, wenn der nicht gesetzt ist, geht er die Liste durch und sendet an jeder User die Nachricht vom anderen User. Der Client empfängt diese und verarbeitet sie.

Ich verstehe nicht so ganz, was bei dir daran hapert.

Soo, erstmal vielen Dank. Ich habe es jetzt soweit geschafft, das Einzige Problem, was ich jetzt habe ist, dass nur jede Zweite Nachricht ankommt <.<

MfG

[dready]

Der Debugger ist dein Freund +g+
Entweder es ist ein Bug oder ein Logikfehler.
Ohne den Code können wir dir da schlecht weiterhelfen

[RedJ0hn]

Originally Posted by dready

Der Debugger ist dein Freund +g+
Entweder es ist ein Bug oder ein Logikfehler.
Ohne den Code können wir dir da schlecht weiterhelfen

 Spoiler

        private void name() {
            string name;
            name = clientname; // Prüfen, ob Name bereits vergeben ist. Muss noch gemacht werden.
            if (clientname == "unbenannt")
            {
                sw.WriteLine("Gebe zunächst deinen Namen ein:");
            }

            sw.WriteLine("Wie heißt Du?");
            clientname = sr.ReadLine();
            user(clientname);
            if (clientname == "")
            {

                sw.WriteLine("Wenn du weiter möchtest gib deinen Namen ein:");

            }
            else
            {
                sw.WriteLine("Hallo " + clientname);
                verbunden = true;
                StreamWriter info;
                foreach (TcpClient c in Liste)
                {
                    for (int i = 0; i < Liste.Count; i++) // TCP CLient einen eindeutigen Name zuweisen
                    {
                        info = new StreamWriter(Liste[i].GetStream());

                        info.WriteLine(clientname + " hat den Raum betreten");
                        info.Flush();
                        info = null;
                    }
                }
               
            }          
            

        }

Dürfte vermutlich ein Logik Fehler sein.

Die Kommentare könnt ihr Ignorieren sind nur Infos für mich bzw. Sachen die noch zu machen sind.

Ich bin derzeit noch am überlegen, wie ich meiner TCP Liste eindeutig die Clientnamen zuweisen kann.

MfG

[#SoNiice]

Entweder bin ich blind oder du hast uns nicht die Funktion für's senden der Nachrichten gegeben.

[RedJ0hn]

Originally Posted by SoNiice

Entweder bin ich blind oder du hast uns nicht die Funktion für's senden der Nachrichten gegeben.

                foreach (TcpClient c in Liste)
                {
                    for (int i = 0; i < Liste.Count; i++) // TCP CLient einen eindeutigen Name zuweisen
                    {
                        info = new StreamWriter(Liste[i].GetStream());

                        info.WriteLine(clientname + " hat den Raum betreten");
                        info.Flush();
                        info = null;
                    }
                }

Habe das nur etwas umgeschrieben, bis jetzt kriegt man also selbst auch noch die Nachricht, wie gesagt, wenn ich den TCP Client einen Namen zugewiesen habe, kann ich dies ja beheben und eine Flüster Funktion noch extra einbauen.
Also bis jetzt kriegt jeder im Chat die Nachricht.