[C] Probleme avec socket client-serveur TCP

Bonjour,

j'ai réalisé un serveur et un client en mode TCP, les échanges entre les deux (ou plutôt 3 car il y'a 2 clients) ont pour but de jouer au Morpion.

Mon premier code utilisait la fonction select (cf mon autre topic). Ayant rencontré des difficultés avec cette fonction, j'ai tous recodé, mais cette fois ci je fais 2 accept() dans le serveur au lieu de select.

Lors de l'exécution, je lance mon serveur ainsi que 2 clients, mais les clients renvoient sur la console "Fin de connexion", ce qui veut dire que aucun des 2 n'a reçu de message et que read( ... ) == 0. Ce qui est faux car mon serveur, leur envoie une invite de saisie...

je comprends pas où pourrait se situer le problème, alors si quelqu'un pouvait m'aider ça serait sympa.

Merci .

EDIT: j'ai remis à jour les PJ

Mon probleme, n'a pas l'air de passionner les foules :lol:

Sans chercher à savoir ce que fait le code, quelques erreurs dans My_Server.c
fonction send_msg : fuite de mémoire : malloc sans free
fonction test_jouer

Code:

---

1 2 3 4 5 6 7

int test_jouer(char * pInput) { int x, y; get_x(pInput,x); get_y(pInput,y); return ((x>0 && x<4) && (y>0 && y<4)); }

---

X et y ne sont pas initialisés

Code:

---

1 2 3 4 5

int get_x(char * pInput, int pX) //parse X et Y { pX = pInput[6] - '0'; return pX; }

---

pX n''a pas à être argument de la fonction : sa valeur initiale n'est pas utilisée
fonction get_y : idem get_x pour pY

Code:

---

1 2 3 4 5

void msg_saisi(int pX, int pY) //invite de saisie { printf("\nSaisie case a jouer (X Y) : "); scanf("%d %d", &pX, &pY); }

---

Modifie les variables locales pX et pY (qui sont ensuite détruites) Pas les variables d'appel

MErci pour ton aide...

tu penses que je devrais mettre des pointeurs à la place???

EDIT: si quelqu'un pouvait m'expliquer ce que Diogene a écrit. Parce que là je comprends pas grand chose.

Merci.

Citation:

---

Envoyé par juve1897

MErci pour ton aide...

tu penses que je devrais mettre des pointeurs à la place???

EDIT: si quelqu'un pouvait m'expliquer ce que Diogene a écrit. Parce que là je comprends pas grand chose.

Merci.

---

Pour la fuite de mémoire tu dois faire un free quand tu fais un malloc. Apparemment tu as fais un malloc mais pas de free.

Pour la 2ème truc tu appeles ta fonction get_x et dans cette fonction tu retournes un int que tu ne recuperes pas. A mon avis tu voulais plutôt faire un truc de ce genre

Code:

---

1 2 3 4 5

void get_x(char * pInput, int *pX) //parse X et Y { pX = pInput[6] - '0'; }

---

Avec un appel comme ça

Code:

---

get_x(pInput, &x);

---

Bonjour à tous,

je tente de decortiquer petit à petit mon programme et j'ai une petite question:

lorsque j'écris:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12

int cpt_jouer(char * pTab) //compte le nombre de coup joués dans la partie { int i; int cpt=9;   for (i=0;i<9;i++) //on soustrait au nombre total de case, le nombre de cases vides if (pTab[i]=='-') cpt--;   return cpt; }

---

est ce que la variable "cpt" existe toujours, après l'appel de la fonction???

Si ce n'est pas le cas est ce qu'il serait possible d'avoir une petite explication...

Merci d'avance

j'ai trouvé quelques erreurs,

tout d'abord, lorsque je faisais de write() ou read(), je ne mettais pas une taille assez grande pour le 3em argument.

Et deuxieme erreur, dans la fonction my_client j'ecrivais

Code:

---

sprintf(pData, "JOUER %d %d\n\r\0", &x, &y);

---

au lieu de

Code:

---

sprintf(pData, "JOUER %d %d\n\r\0", x, y);

---

MAis mon morpion ne marche toujours pas...

Donc si quelqu'un détecte d'autres âneries que j'ai pu écrire, qu'il me fasse signe.

Merci

Citation:

---

Envoyé par juve1897

est ce que la variable "cpt" existe toujours, après l'appel de la fonction???

Si ce n'est pas le cas est ce qu'il serait possible d'avoir une petite explication...

Merci d'avance

---

Comment ça existe toujours ?

Tu veux dire si tu rerentres dans la fonction est-ce que la valeur est la même qu'à la dernière sortie ?

Ou bien est-ce à propos de la valeur retournée ?

Citation:

---

Envoyé par Skyrunner

Comment ça existe toujours ?

Tu veux dire si tu rerentres dans la fonction est-ce que la valeur est la même qu'à la dernière sortie ?

Ou bien est-ce à propos de la valeur retournée ?

---

Désolé de mettre mal exprimé, je parle bine de la valeur de retour, celle qui est retournée à la fin de la fonction.

Citation:

---

Envoyé par juve1897

Désolé de mettre mal exprimé, je parle bine de la valeur de retour, celle qui est retournée à la fin de la fonction.

---

Ben si tu la récupère il n'y a pas de soucis. Le retour se fait par valeur.

Ok merci.

Mais j'ai du mal à comprendre à quel niveau mon programme ne fonctionne pas.

c.a.d, est ce du point de vu de l'algo ou bien des erreurs de syntaxes (pointeurs, fonction qui ne retourne pas la bonne valeur ...) que ça ne va pas.

Je vois pas comment debugger mon programme...

j'ai lu le topic qui est consacré, mais même avec des printf() partout, je vois pas.

Je regarderais ton code dès que j'aurais un peu de temps.

Merci c'est cool.

En tous cas moi je continue à traquer les erreurs avec mes petits printf().

fprintf (stderr, " toto"); serait mieux ....

Citation:

---

Envoyé par souviron34

fprintf (stderr, " toto"); serait mieux ....

---

??
Pourquoi donc ?
A partir du moment ou il mets bien un \n à la fin je vois pas de problème.

Citation:

---

Envoyé par souviron34

fprintf (stderr, " toto"); serait mieux ....

---

A quelle ligne j'ai bien pu mettre ça ???

je dis juste que pour tracer une erreur mettre fprintf(stderr..) est mieux que mettre printf(..) car stderr est SYNCHRONE..

Citation:

---

Envoyé par souviron34

je dis juste que pour tracer une erreur mettre fprintf(stderr..) est mieux que mettre printf(..) car stderr est SYNCHRONE..

---

Ah ok, merci beaucoup.

Mais je ne sais pas comment marche cette focntion.

J'utilise printf() pour suivre l'évolution de certaines de mes variables, voir si elles s' incrémentent bien, si les données envoyées sont les bonnes, ect...

man fprintf .. ou voir les cours ou tutoriels ici-même..

fprintf ( stderr, ... )

les ... sont à remplacer par la même chose que ce que tu mets dans le printf..

c'est juste que printf est bufferisé, donc ça ne garanti pas que l'impression se fasse QUAND tu passes sur l'instruction. Donc tu peux faire un printf ("1") printff ("2"), et tu ne verras pas 1 alors qu'en fait ton programme aura déjà dépassé cette instruction avant de crasher. Alors que stderr est SYNCHRONE donc toute écriture sort immédiatement, et donc la traçabilité est instantanée.

Merci beaucoup pour ces explications...

je vais mes documenter un peu sur cette fonction. ;)

Code:

---

1 2 3 4 5 6 7

int cpt_jouer(char * pTab) //compte le nombre de coup joués dans la partie { int i; int cpt=9; .... return cpt; }

---

Citation:

---

est ce que la variable "cpt" existe toujours, après l'appel de la fonction???

---

Non, les variables locales pTab, i et cpt sont détruites en sortie de la fonction. Par contre, le return cpt te permet de récupérer la valeur qu'avait cpt avant sa destruction. Donc le code est Ok.

Code:

---

1 2 3 4 5

int get_x(char * pInput, int pX) //parse X et Y { pX = pInput[6] - '0'; return pX; }

---

pX est un argument de la fonction : Au moment de l'appel de cette fonction, une variable locale pX est créée et initialisée avec la valeur placée dans l'appel. De cette valeur, tu ne fais rien puisque la première chose que tu écris est px =.... Ce n'est donc pas un argument. On devrait avoir :

Code:

---

1 2 3 4 5

int get_x(char * pInput) //parse X et Y { int pX = pInput[6] - '0'; return pX; }

---

ou simplement :

Code:

---

1 2 3 4

int get_x(char * pInput) //parse X et Y { return pInput[6] - '0'; }

---

avec en appel

Code:

---

1 2 3 4 5 6 7

int test_jouer(char * pInput) { int x, y; x = get_x(pInput); y = get_y(pInput); return ((x>0 && x<4) && (y>0 && y<4)); }

---

Code:

---

1 2 3 4 5

void msg_saisi(int pX, int pY) //invite de saisie { printf("\nSaisie case a jouer (X Y) : "); scanf("%d %d", &pX, &pY); }

---

Pour modifier une variable du code appelant une fonction, en dehors de l'utilisation du return, il faut connaître son adresse. Ici, tu ne transmet pas l'adresse de pX et pY mais leur valeur.

Code:

---

1 2 3 4 5

void msg_saisi(int * ppX, int* ppY) //invite de saisie { printf("\nSaisie case a jouer (X Y) : "); scanf("%d %d", ppX, ppY); }

---

avec un appel du genre

Code:

---

msg_saisi(&pX,&pY);

---

Merci pour ces explication, j'avais deja effectué les changements (j'ai mis à jour les P.J).

En tous cas si tu captes d'autres erreurs n'hesites pas à me les signaler. ;)

Merci encore.

Bonjour,

je voulais savoir, s'il n'y aurait pas une personne sur le forum qui puisse m'aider.

Car mon projet est à rendre pour la fin de la semaine, et pour ceux qui ont suivit mes posts depuis le debut savent que je planche dessus depuis plus d'un mois...

Donc je ne demande aucunement, de faire mon projet à ma place, mais simplement de m'aider à trouver ce qui ne va pas dans mon programme.

C'est un appel de detresse que je lance...

Merci à tous ceux qui voudront bien m'aider, car seul je n'y arriverai jamais.

De plus il serait vraiment dommage que je me ramasse une mauvaise juste à cause d'un detail que je ne vois pas.

Merci

PS: le truc qui ne fonctionne pas bien, c'est que le client a toujours un message de retard ...


Algo:
Serveur:

• envoie une invite au client avec l'etat du jeu
• receptionne les coordonnées envoyées par le premier cleint
• test si les coordonnées sont bonnes (case vide, si le joueur gagne...)
• si c'est ok, il envoie au client OK et l'etat du jeu
• sinon il renvoie INVALIDE et l'eat du jeu
• Si le compteur de tour arrive à 8 et qu'il n'y a pas de gagnant il envoie NUL et etat du jeu au 2 joueurs

Client:

• reception (OK ou INVALIDE) + l'etat du jeu
• envoie les coordonnées avec l'entete JOUER
• et il boucle dessus, jusque reception du message de victoire ou de nul

Voila en gros mon programme

Code:

---

1 2 3 4 5 6 7 8 9 10

void traiterClient(Joueur j, char * pMorpion) { char *pData=malloc(sizeof(char)*1024); char buf[1024];//=malloc(sizeof(char)*1024); int x, y, rval;   do { bzero(buf, sizeof(buf)); bzero(pData,sizeof(pData));

---

La dernière ligne ne remet le buffer pData à zéro : sizeof(pData) = taille d'un pointeur (typiquement 4), pas la taille du tableau pointé

MErci pour ton aide.
Meme si je dois avoué que je n'ai pas trop compris le probleme qui est engendré par cette erreur...


j'ai aussi capté une erreur, j'envoie deux fois le message "MSG + etat du jeu " au client alors que je devrais le faire qu'une fois...

je sens que je m'approche petit à petit du but...

PS: j'ai actualisé les codes donnés.

Autre chose, est ce que quelqu'un aurait une idée de comment coder, une fonction qui demande au le joueur gagnant si il souhaite refaire une partie, si il dit oui on recommence...

j'avais pensé à faire un genre de

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23

sprintf(pData,"Voulez vous jouer une nouvelle partie ??? O/N \n"); write(j.sock,pData,1024); read(j.sock,buf,1024); oui=strcmp("O",buf); non=strcmp("N",buf);   while(oui!=0 || non !=0) { write(j.sock,pData,1024); read(j.sock,buf,1024); oui=strcmp("O",buf); non=strcmp("N",buf);   } if(oui==0) { //recommencer le jeu } else { //quitter le jeu }

---

le hic c'est que je vois pas trop comment je pourrais coder le "recommencer jeu" et le "quitter jeu"...

A mon avis avant de rajouter des fonctionnalités, tu devrais plutôt faire en sorte que les actuelles fonctionnent (ce n'est pas méchant hein).

:D

pour quitter, c'est facile :

Code:

---

1 2	if ( oui ) exit (EXIT_SUCCESS);

---

Maintenant pour le recommencer, il faudrait inclure tout ton code jusquelà y compris dans un

Code:

---

while (!oui)

---

initialisé à oui=0.

Et du coup tu n'as plus besoin du exit...

A la sortie du while tu es à la sortie du main..

Citation:

---

Envoyé par Skyrunner

A mon avis avant de rajouter des fonctionnalités, tu devrais plutôt faire en sorte que les actuelles fonctionnent (ce n'est pas méchant hein).

:D

---

Lol, je sais bien. Tu as parlé comme mon prof... :mouarf:

Citation:

---

Envoyé par souviron34

pour quitter, c'est facile :

Code:

---

1 2	if ( oui ) exit (EXIT_SUCCESS);

---

Maintenant pour le recommencer, il faudrait inclure tout ton code jusquelà y compris dans un

Code:

---

while (!oui)

---

initialisé à oui=0.

Et du coup tu n'as plus besoin du exit...

A la sortie du while tu es à la sortie du main..

---

Merci beaucoup, pour ton aide, je garde cette idée (très bonne) de coté pour quand j'aurai fini de corriger ce qui ne va pas encore.

:aie:

Bonsoir,

j'ai résolu mon problème... :yaisse2:

Je mettrai à disposition mon code source une fois que j'aurai un peu de temps.

Cependant j'aimerai ajouter des fonctionnalités à mon jeu mais je sais pas lesquelles...

J'avais pensé à demander au vainqueur s'il souhaite refaire une partie, mais il me reste 3 jours pour le faire...

Alors si vous avez quelques idées à me soumettre (qui ne prendront pas beaucoup de temps à la realisation), je suis preneur.

Merci.

Citation:

---

Envoyé par juve1897

Bonsoir,

j'ai résolu mon problème... :yaisse2:

Je mettrai à disposition mon code source une fois que j'aurai un peu de temps.

Cependant j'aimerai ajouter des fonctionnalités à mon jeu mais je sais pas lesquelles...

J'avais pensé à demander au vainqueur s'il souhaite refaire une partie, mais il me reste 3 jours pour le faire...

Alors si vous avez quelques idées à me soumettre (qui ne prendront pas beaucoup de temps à la realisation), je suis preneur.

Merci.

---

Tu sauvegardes ton code. Et tu modifies une copie pour essayer d'ajouter des trucs comme refaire jouer le joueur qui a gagné.

Merci, du conseil, mais je suis pas fou au point de faire des modif' dans un code qui marche à merveille. ;)


Cependant, je pensais aussi faire rejouer le joueur "gagnant" de la partie, mais je vois pas comment faire...

De plus j'ai peur que ça me prenne trop de temps et que je fasse un début de travail pour rien. :aie:

Citation:

---

Envoyé par juve1897

Merci, du conseil, mais je suis pas fou au point de faire des modif' dans un code qui marche à merveille. ;)


Cependant, je pensais aussi faire rejouer le joueur "gagnant" de la partie, mais je vois pas comment faire...

De plus j'ai peur que ça me prenne trop de temps et que je fasse un début de travail pour rien. :aie:

---

A toi de voir. souviron34 t'as montré la voie à suivre.