TCP/IP Kommunikation

@Dome204 sagte in TCP/IP Kommunikation:

das a benötige ich

Das war nicht die Frage.

Dome204

@manni66
Um es in der nachfolgenden for-Schleife verwenden zu können.
Ich hoffe das beantwortet die Frage

@Dome204 sagte in TCP/IP Kommunikation:

@manni66
Um es in der nachfolgenden for-Schleife verwenden zu können.
Ich hoffe das beantwortet die Frage

Nein. Es kommt kein String.

Dome204

@manni66 sagte in TCP/IP Kommunikation:

Nein. Es kommt kein String.

Wenn ich das richtig verstehe, habe ich in der folgenden Programmzeile den int nicht korrekt in ein String umgewandelt, weshalb es zu einem falschen Wert bei der Umwandlung kommt.

iRC = send(copieSockets.fd_array[i], (char *)&a, sizeof(int), 0);

@Dome204 aus einem int wird kein String, blos weil man (char*) davor schreibt. Du überträgst das int als int und musst es auch wieder als int lesen.

int a;
recv(clientSocket, (char*)&a, sizeof(a), 0);

Dome204

@manni66 sagte in TCP/IP Kommunikation:

@Dome204 aus einem int wird kein String, blos weil man (char*) davor schreibt. Du überträgst das int als int und musst es auch wieder als int lesen.
int a;
recv(clientSocket, (char*)&a, sizeof(a), 0);

Okay, danke ich verstehe nun den Fehler. Weiß aber leider nicht wie es richtig geht. Könntest du mir einen Tipp geben, wie ich es richtig übertrage?

@Dome204 sagte in TCP/IP Kommunikation:

Weiß aber leider nicht wie es richtig geht

Du hast die Lösung gerade zitiert.

Dome204

@manni66 oh okay, danke
Habe es ausprobiert, aber es wird noch immer nicht der richtige Wert übermittelt

DocShoe

@Dome204
Welchen Typ hat denn nachricht?

Dome204

@DocShoe
nachricht ist ein char Array:

char                nachricht[NACHRICHTENLAENGE];

DocShoe

@Dome204
Ächz....
Und wie groß ist Nachrichtenlänge?

Dome204

@DocShoe 4096

#define NACHRICHTENLAENGE 4096

@Dome204 sagte in TCP/IP Kommunikation:

Habe es ausprobiert, aber es wird noch immer nicht der richtige Wert übermittelt

Soso

DocShoe

Guck dir den Empfangspuffer in deinem Client mal an, was steht denn da drin? Hast du mannis Tipp schon umgesetzt und wie sieht deiner Empfangsfunktion jetzt aus?

Dome204

Ja, ich habe die Tipps schon umgesetzt.
So sehen die Abschnitte aktuell aus:

Client:

else if (nummer == 4)
        {
            printf("\nSie haben Darstellen gewaehlt.");
            int a;
            iRC = recv(clientSocket, (char *) &a, sizeof(a),0); //a, also die Anzahl der in der Liste Personen befindlichen Daten wird übergeben
            printf("-%i-", a);
            if (iRC == -1)
            {
                printf("Verbindung geschlossen\n");
                return 1;
            }
            else
            {
                if (a > 0)
                {
                    printf("\nAnzahl an Personen: %i\n", a);//Die Anzahl der Personen ist wichtig um zu wissen wie oft empfangen werden muss
                    for(int i = 0; i<a; i++)
                    {
                        printf("Person %i:", i+1);

                        memset(nachricht, 0, NACHRICHTENLAENGE);
                        recv(clientSocket, nachricht, sizeof(nachricht), 0);
                        printf(" Name: %s",nachricht);

                        memset(nachricht, 0, sizeof(nachricht));
                        recv(clientSocket, nachricht, sizeof(nachricht), 0);
                        printf(" Alter: %s",nachricht);

                        memset(nachricht, 0, sizeof(nachricht));
                        recv(clientSocket, nachricht, sizeof(nachricht), 0);
                        printf(" Gewicht: %s",nachricht);

                        memset(nachricht, 0, sizeof(nachricht));
                        recv(clientSocket, nachricht, sizeof(nachricht), 0);
                        printf(" Groesse: %s\n",nachricht);
                    }
                }
                else
                {
                    printf("Liste ist leer\n");
                }
            }
        }

Server:

else if (nummer == 4)
                    {
                        fnListeDrucken(&Personen);
                        int a = Personen.anz;
                        printf("-%i-",a);
                        send(copieSockets.fd_array[i], (char *) &a, sizeof(int), 0);//Die Anzahl benötigt der Client um zu wissen wie viele Personen ankommen

                        if (!a)
                        {
                            printf("Liste ist leer");
                        }
                        else if (a>0)
                        {
                            ListenElem *pE; //pE wird die Adresse von der ersten Person und läuft durch bis die nächste Person ein NULL ist
                            pE = Personen.root;
                            if (!pE)
                            {
                               printf("\nListe ist leer");
                            }

                            while (pE)
                            {
                                memset(nachricht, 0, sizeof(nachricht));
                                strcpy(nachricht, pE->Listenperson.Name);
                                send(copieSockets.fd_array[i], nachricht, sizeof(nachricht), 0);

                                memset(nachricht, 0, sizeof(nachricht));
                                strcpy(nachricht, (char*) &pE->Listenperson.Alter);
                                send(copieSockets.fd_array[i], nachricht, sizeof(nachricht), 0);

                                memset(nachricht, 0, sizeof(nachricht));
                                strcpy(nachricht, (char *) &pE->Listenperson.Gewicht);
                                send(copieSockets.fd_array[i], nachricht, sizeof(nachricht), 0);

                                memset(nachricht, 0, sizeof(nachricht));
                                strcpy(nachricht, (char *) &pE->Listenperson.Groesse);
                                send(copieSockets.fd_array[i], nachricht, sizeof(nachricht), 0);

                                pE = pE->next;
                            }
                        }
                    }

SeppJ

Du hast also exakt gar nichts von den gegebenen Antworten umgesetzt. PS: Vielleicht war ich zu harsch. Du hast es an einer Stelle umgesetzt. Aber es ist enttäuschend, dass du die anderen Stellen nicht mit korrigiert hast. Hier rächt sich erneut, dass du Abstraktion meidest, und in deinem Programm 10x den fast gleichen Code für den gleichen Zweck wiederholst, mit jeweils allen Fehlern. Anstatt nur einmal, so dass man den Fehler an allen Stellen gleichzeitig korrigieren könnte.

Du musst erst einmal grundlegend mit C-Datenstrukturen umgehen können, bevor du anfangen kannst, Daten zu versenden. Sowohl beim Einpacken der Nachrichten, als auch das Auspacken ist ungefähr jede Zeile falsch. Da du bisher aber noch nie auf eine der dir gegebenen Erklärungen genauer eingegangen bist, mag ich keine Komplettliste aller Fehler und ihrer Korrekturen geben, und verweise erst einmal auf ein Grundlagenbuch.

DocShoe

Beschränk dich erst mal auf das Versenden der Anzahl, den Rest kannst du ja später ergänzen. Und gewöhn´ dir bitte an, ausreichende Informationen zu geben. So Sachen wie "ich verschicke was, aber es kommen komische Daten an" helfen keinem weiter. Also bitte "ich verschicken x als 4 Byte integer mit dem Wert 4711, beim Client bekomme ich beim Zurücklesen aber 1147 raus."

@Dome204 du zeigst das Ergebnis nicht. Muss man dir alles einzeln aus der Nase ziehen?

Wenn du vorher nicht haarklein die Bytes so gelesen hast, wie sie geschrieben wurden, fällst du hier natürlich immer noch auf die Schnauze. Hier sei

@DocShoe sagte in TCP/IP Kommunikation:

TCP/IP kann Pakete fragmentieren, d.h. deine Nachricht kommt möglicherweise nicht in einem Block an, sondern muss aus mehreren Telegrammen zusammengebaut werden

guck dir die Rückgabewerte von send und recv an!

noch einmal genannt.

wob

@Dome204 sagte in TCP/IP Kommunikation:

strcpy(nachricht, (char*) &pE->Listenperson.Alter);

Was hat "Alter" für einen Typ? int? Was macht das strcpy dann da?

Dome204

@SeppJ Ich habe die nachfolgenden Programmzeilen, nicht korrigiert, weil ich mich vorerst auf das Übertragen von "a" beschränkt habe.
Ich bin leider noch sehr ungeübt und Anfänger im Programmieren, das ist mir klar. Vermutlich ist der Fehler trivial, leider ist er für mich nicht so einfach nachvollziehbar.
Der Rückgabewert von send ist 0
Beim Server ist a = 1
Der Rückgabewert von recv ist 4
Beim Client ist a = 943128576

Ich bin euch sehr dankbar für eure Hilfen!