Discussion :

[ZouBi]

Bonjour,
voici les deux prototypes :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[Noxen]

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.

[ram-0000]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

et à l'autre bout

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[Alp]

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.

[Laurent Gomila]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Personnage p();

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Personnage p;

[ram-0000]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

Personnage p;

mais peut être moins facile à lire (quoique)

[JolyLoic]

Et non, c'est un piège.

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

Personnage p();

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[ram-0000]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
void main()
{
   A tst(10);
}

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

Avec ce code:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[loufoque]

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 ?