Envoyer des instances de classes via TCP

Bonjour,
voici les deux prototypes :

Code:

---

1 2	int SDLNet_TCP_Send(TCPsocket sock, void *data, int len); int SDLNet_TCP_Recv(TCPsocket sock, void *data, int maxlen);

---

Je possède une classe "Personnage", dont j'en fais une instance, et que je souhaite envoyer au serveur.

Code:

---

1 2	Personnage p(); SDLNet_TCP_Send( sock, &p, sizeof( Personnage );

---

Jusqu'ici, aucun soucis, mais c'est lors de la réception au serveur, du personnage que celui-ci me fait une erreur de segmentation.

Code:

---

1 2	Personnage p(); SDLNet_TCP_Recv( sock, &p, sizeof( Personnage );

---

voilà, et là une erreur de segmentation.
La classe Personnage possède un constructeur par copie, ainsi qu'une surcharge de l'opérateur d'affectation.

Je pensais au début qu'on pouvait pas envoyer d'instance via TCP, mais j'ai pourtant essayer avec des entiers ( en remplacant les types Personnage par des int ) et j'ai toujours des erreurs de segmentation, donc je sais pas comment faire, si vous pouvez m'aider.

On me détrompera peut-être, mais il me semble qu'au lieu d'envoyer une classe sur le réseau, on envoie plutôt ses données. Essaie plutôt de faire une méthode dans ta classe (std::string Personnage::ToMessage()) qui renvoie une chaîne de caractère contenant les données de ta classe de manière formatée (des mots ou chiffres séparés par des ';', par exemple, à toi de voir ce qui est nécessaire pour instancier un Personnage), et à l'autre bout fait une fonction (ou une méthode de classe) qui permet d'analyser la chaîne reçue, d'en extraire les données nécessaires, et de construire un Personnage à partir de ça.

Ton problème s'appelle la serialisation ou aussi le marshalling.

Comment transmettre une classe ou des données d'un PC à l'autre sachant qu'ils ne partagent pas de mémoire commune. Vu que la mémoire n'est pas partagée, un pointeur ne se transmet pas, il faut absolument le déréférencer et sérialiser.

Vu que tu es dans un forum c++, je peux continuer sur les autres classes standard, comment transmettre un vector :question:

Code:

---

send(socket, &vector, sizeof(vector));

---

et à l'autre bout

Code:

---

recv(socket, &vector, sizeof(vector));

---

C'est le crash assuré.

Il va donc falloir que tu parcours chacuns des membres de ta classe, que tu les envoies sur le socket et qu'à l'autre bout, tu les recoives et reconstruises ta classe de toute pièce.

ram_0000 a raison... Et c'est pourquoi je te conseille de lire la sérialisation avec Boost. Ensuite, tu as donc à sérialiser, envoyer la chaine à l'autre ordi, et dé-sérialiser.

Et accessoirement, ton autre problème c'est que ceci déclare une fonction :

Code:

---

Personnage p();

---

Ceci par contre déclare bien une instance de Personnage :

Code:

---

Personnage p;

---

Citation:

---

Envoyé par Laurent Gomila

Et accessoirement, ton autre problème c'est que ceci déclare une fonction :

Code:

---

Personnage p();

---

---

Heu non, ceci déclare bien une instance de Personnage nommée "p" et dont ont ne passe aucun paramètre au constructeur. C'est strictement equivalent à

Code:

---

Personnage p;

---

mais peut être moins facile à lire (quoique)

Et non, c'est un piège.

Code:

---

Personnage p();

---

Est bien une déclaration de fonction. A comparer avec :

Code:

---

int p();

---

Quand il y a ambiguïté, le compilateur doit préférer la déclaration de fonction. Il y a des cas encore pires...

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17

struct A { A() {} };   struct B { B(A a1, A a2) {} void f() {} };   int main() { int i = 12; B b1 (A(i), A()); // déclare une fonction b1 B b2 (A(12), A()); // déclare une variable b2 }

---

b1 est une fonction prenant deux paramètres, et retournant un objet de type B.
Le premier paramètre, nommé i (avec des parenthèses en trop, mais c'est pas grave), de type A.
Le second paramètre, non nommé de type : pointeur sur une fonction ne prenant pas d'argument et retournant un A.

Oups, j'aurai dit une grosse connerie !!! Et aussitôt, je pris mon compilateur...

Code:

---

1 2 3 4 5 6 7 8

class A { public: A(int param = 12) { printf("A constructor(%d)\n", param); } };

---

Avec ce code:

Code:

---

1 2 3 4

void main() { A tst(); }

---

J'ai ce warning : warning C4930: 'A tst(void)'*: fonction prototypée non appelée (était-ce la définition de variable souhaitée*?)

Avec ce code:

Code:

---

1 2 3 4

void main() { A tst(10); }

---

Pas de warning et j'ai le comportement souhaité.

Avec ce code:

Code:

---

1 2 3 4

void main() { A *tst = new A(); }

---

Pas de warning non plus et j'ai le comportement souhaité.

Donc mes excuses Laurent Gomila, tu avais raison et merci JolyLoic d'avoir rectifié mon erreur, j'aurai au moins appris une chose aujourd'hui.

Pourquoi utilises tu des pointeurs et des new alors qu'il n'y en a absolument pas besoin, et qu'au contraire ça apporte rien que des problèmes ?