Discussion :

[gooogl3r]

Bonjour,

Je suis entrain de concevoir un serveur en C qui gère la connexion des joueurs (Jeu en ligne ). Dans mon application, quand les joueurs se connectent, je dois leur envoyer une notification comme quoi la partie peut commencer. Pour cela j'ai utilisé une pile du genre :

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
 
typedef struct Joueur
{
char *nom;
int type ;/*dessinateur ou chercheur */
int score;
int socket_id;
struct Joueur *suivant;
}*Joueur;
 
/*file des joueurs */
typedef struct FileJoueur
{
int taille;
Joueur *premier;
}*FileJoueur;

Par ailleurs, j'ai une fonction EnvoyerRole pas encore complète :

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
18
19
20
21
22
23
 
void EnvoyerRole(FileJoueur *fileJoueur,char role,char* MotADessinser)
{
char* message;
strcat(message,"NEW_ROUND/");
strcat(message,role);
strcat(message,"/");
strcat(message,MotADessiner);
if(fileJoueur==NULL)
exit(EXIT_FAILURE);
Joueur *joueur=(*fileJoueur)->premier;
while(joueur!=NULL)
{
/*un pti teste pour voir que sa marche */
printf(" le descripteur du socket est %d et le nom du client %s \n",(*joueur)->socket_id,(*joueur)->nom);
joueur=(*joueur)->suivant;
}
}
 
 
/*appele de la fonction quelque part dans le code du genre */
char *aDessiner="Mot a dessiner";
EnvoyerRole(&mesJoueurs,'0',aDessiner);

Mais j'ai une erreur de segmentation

PS: mesJoueurs contient des elements

[matafan]

C'est le bon moment pour apprendre à utiliser un debugger.

Quand ton programme plante, il doit produire un core. Si ce n'est pas le cas, fait "ulimit -c unlimited" dans ta console, relance ton programme, et tu dois te retrouver avec un beau fichier nommé "core".

Donc, compiles ton programme avec -g (pour avoir les numéros de ligne), et lance le de façon a produire un core. Ensuite tu fais :

Code :

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

gdb ton_prog core

Et tu te retrouves à l'endroit exact où le programme à planté (tu peux voir où tu es avec "where"). Tu peux inspecter des variables et faire plein de choses intéressantes.

Ensuite il est aussi possible de placer des breakpoints, d'exécuter ton programme dans le debugger, de faire du pas à pas...

[Médinoc]

Il suffit de lire quatre lignes de ton second code pour tomber sur le premier problème:

Code C :

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

1
2
char* message;
strcat(message,"NEW_ROUND/");

PAF! Déréferencement d'un pointeur non-initialisé.

[imperio]

@gooogl3r: Non, ca c'est un segfault bien degueulasse dans les regles de l'art.

[Obsidian]

Merci pour la réponse et c'est intéressant cependant je peut déboguer juste a partir de mon terminal sans utiliser un IDE ?

Oui.

effectivement mais c'est juste un warning pas plus

Un warning, c'est un avertissement qui t'indique qu'il y a quelque chose de louche. Et si ça paraît louche au compilateur, c'est sans doute pour une bonne raison. Efforce-toi de n'en avoir plus aucun dans la version finale de ton exécutable. Tu peux passer à gcc l'option « -Werror » qui lui demande de considérer tous les avertissements comme d'authentiques erreurs, stoppant la compilation.

En l'occurrence, « char * » n'est pas en soi une chaîne de caractères : c'est un pointeur vers le premier d'une suite de caractères, chaîne qui n'existe pas encore. Ton pointeur est non initialisé et sa valeur peut pointer n'importe où. Et dans tous les cas, ça se passera forcément mal puisque tu n'as alloué aucun espace pour cette chaîne.

[gooogl3r]

Merci Les amis pour vos conseils ...maintenant sa marche sans aucun warning et effectivement vous aviez raison parce que quand j’initialise mes pointeurs y a plus de warning et il manqué quelque malloc un peu partout...merci beaucoup

[gooogl3r]

@matafan :
Merci pour la réponse et c'est intéressant cependant je peut déboguer juste a partir de mon terminal sans utiliser un IDE ?

@Médinoc :
effectivement mais c'est juste un warning pas plus