Discussion :

[diogene]

Code :

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

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;
}

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 :

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

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 :

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

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

ou simplement :

Code :

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

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

avec en appel

Code :

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

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 :

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

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 :

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

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 :

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

 msg_saisi(&pX,&pY);

[LinuxUser]

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.

[LinuxUser]

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

[diogene]

Code :

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

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é

[LinuxUser]

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.

[LinuxUser]

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 :

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
 
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"...

[Skyounet]

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).

[souviron34]

pour quitter, c'est facile :

Code :

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

1
2
if ( oui )
  exit (EXIT_SUCCESS);

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

Code :

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

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..

[LinuxUser]

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).

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

[LinuxUser]

pour quitter, c'est facile :

Code :

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

1
2
if ( oui )
  exit (EXIT_SUCCESS);

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

Code :

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

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.

[LinuxUser]

Bonsoir,

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

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.

[Skyounet]

Bonsoir,

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

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é.

[LinuxUser]

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.

[Skyounet]

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.

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