Problème de coordonnées...

Problème de coordonnées... - C - Programmation

Marsh Posté le 23-05-2007 à 15:43:09    

Tout d'abord les structures :

Code :
  1. #include <stdio.h>
  2. #include <stdlib.h>
  3. #include <string.h>
  5. /* Plateau */
  6. struct plateau_s {
  7.   char** bille; //pointeur sur le plateau de jeu
  9.   int plateau_taille; //taille du plateau
  10. };
  12. /* ColorKiller */
  13. struct colorkiller_s {
  15.   struct liste_s** bille; //pointeur sur les billes en jeu
  16.   struct plateau_s* plateau; //pointeur sur le plateau de jeu
  17.   int** score; //pointeur sur le tablmeau des scores
  19.   /* Parametres de jeu : */
  20.   int nombre_ajoute; //nombre de billes ajoutee a  chaque tour
  21.   int nombre_couleur; //nombre de couleurs
  22.   int nombre_deplacement; //nombre de deplacements autorises
  23.   int nombre_groupe; //nombre de billes qu'un groupe devra comprendre pour etre supprime
  25.   /* Statut de la partie : */
  26.   char* tour_couleur; //couleur dont c'est le tour de jouer
  27.   int tour_nombre; //nombre de tours depuis le debut de la partie
  28.   int tour_joueur; //numeros du joueur en cours
  29. };
  31. /* Coordonnees */
  32. struct coordonnees_s {
  33.   int* x;
  34.   int* y;
  35. };


 
Voila le code de la fonction qui merdouille :  

Code :
  1. struct coordonnees_s bille_selection_nom (struct colorkiller_s* colorkiller) {
  2.   struct coordonnees_s tmp_coordonnees;
  3.   struct liste_s* bille;
  4.   char tmp_nom[20];
  5.   char* tmp_couleur;
  7.   printf("bille : selection : nom = " );
  8.   fgets (tmp_nom, sizeof (tmp_nom), stdin);
  10.   bille = liste_recherche_element_nom (colorkiller->bille, tmp_nom);
  12.   printf("%d\n",&(bille->position_x));
  13.   printf("%d\n",&(bille->position_y));
  15.   tmp_coordonnees.x = &(bille->position_x);
  16.   tmp_coordonnees.y = &(bille->position_y);
  18.   printf("%d\n",tmp_coordonnees.x);
  19.   printf("%d\n",tmp_coordonnees.y);
  21.   tmp_couleur = case_etat (colorkiller, tmp_coordonnees);
  23.   if (tmp_couleur == NULL) tmp_couleur = "rien";
  24.   if (strcmp (tmp_couleur, colorkiller->tour_couleur) != 0) printf ("bille : selection : cette bille ne vous appartient pas\n" );
  26.   while (strcmp (tmp_couleur, colorkiller->tour_couleur) != 0) {
  27.     printf("bille : selection : nom = " );
  28.     fgets (tmp_nom, sizeof (tmp_nom), stdin);
  30.     bille = liste_recherche_element_nom (colorkiller->bille, tmp_nom);
  31.     tmp_coordonnees.x = &(bille->position_x);
  32.     tmp_coordonnees.y = &(bille->position_y);
  33.     tmp_couleur = case_etat (colorkiller, tmp_coordonnees);
  35.     if (tmp_couleur == NULL) tmp_couleur = "rien";
  36.     if (strcmp (tmp_couleur, colorkiller->tour_couleur) != 0) printf ("bille : selection : cette bille ne vous appartient pas\n" );
  37.   }
  38.   printf("bille : selection : OK\n" );
  39.   return tmp_coordonnees;
  40. }


 
Ma fonction liste recherche element nom :  

Code :
  1. struct liste_s* liste_recherche_element_nom (struct liste_s** liste, char* nom) {
  3.   if ((*liste)->nom == NULL && (*liste)->couleur == NULL && (*liste)->position_x == NULL && (*liste)->position_y == NULL) return NULL;
  4.   if (strcmp((*liste)->nom, nom) == 0)
  5.     return *liste;
  6.   else
  7.     return liste_recherche_element_nom (&(*liste)->suivant, nom);
  8. }


 
Ma fonction case etat :  

Code :
  1. char* case_etat (struct colorkiller_s* colorkiller, struct coordonnees_s coordonnees) {
  2.   struct liste_s* bille;
  3.   if (liste_recherche_element (colorkiller->bille, coordonnees.x, coordonnees.y) != NULL) {
  4.     bille = liste_recherche_element (colorkiller->bille, coordonnees.x, coordonnees.y);
  5.     printf("[%d][%d] : %s\n",coordonnees.x,coordonnees.y,bille->couleur);
  6.     return bille->couleur;
  7.   }
  8.   printf("[%d][%d] : vide\n",coordonnees.x,coordonnees.y);
  9.   return NULL;
  10. }


 
Pour finir la sortie terminal :  

Code :
  1. nom : Rémi
  2. couleur : Orange
  3. X = 2
  4. Y = 0
  5. ñ
  6. nom : Alexandre
  7. couleur : Rouge
  8. X = 2
  9. Y = 2
  10. ñ
  11. [2][0] : Orange
  12. bille : selection : nom = Rémi
  13. 8
  14. 12
  15. 8
  16. 12
  17. [8][12] : vide
  18. bille : selection : cette bille ne vous appartient pas
  19. bille : selection : nom = Rémi
  20. [8][12] : vide
  21. bille : selection : cette bille ne vous appartient pas
  22. bille : selection : nom = Alexandre
  23. [8][12] : vide
  24. bille : selection : cette bille ne vous appartient pas
  25. bille : selection : nom = rien
  26. [8][12] : vide
  27. bille : selection : cette bille ne vous appartient pas
  28. bille : selection : nom =


 
Le probleme c'est que les coordonnees des elements restent constantes...
Je ne vois pas comment m'en sortir...

Reply

Marsh Posté le 23-05-2007 à 15:43:09   

Reply

Marsh Posté le 23-05-2007 à 18:54:20    

Une idée... ???

Message cité 1 fois
Reply

Marsh Posté le 23-05-2007 à 19:32:33    

Toegarf a écrit :

Une idée... ???


 
Le problème, c'est que ton code a vraiment beaucoup trop de commentaires. Tu devrais en élaguer un peu.
 
En dehors de ça, je pense que tu dois mélanger les pointeurs.
Par exemple: pourquoi ta structure "coordonnée_s" stocke des pointeurs et non des valeurs
Autre chose: pourquoi le pointeur déclaré comme "pointeur sur les billes en jeu" est en réalité un pointeur sur un pointeur ?
 
Et surtout: pourquoi ta fonction qui merdouille renvoie l'intégralité d'une structure ? A mon avis, en dehors du fait que renvoyer une structure est toujours plus gourmand que renvoyer un simple pointeur sur la structure, le problème vient que quand tu renvoies ta structure (que tu recopies donc forcément dans la fonction appelante) et que ta structure en question contient des pointeurs ben tu ne recopies que les pointeurs et non les pointés (shallow copy au lieu de deep copy).
 
Je ne sais pas si je suis bien clair. Mais si tu arrives à comprendre pourquoi le code suivant donne deux fois le contenu de la variable HOME (alors qu'il est sensé afficher le PATH et le HOME) tu auras une idée de ce qui se passe dans tes structures

Code :
  1. int main()
  2. {
  3.     char *path;
  4.     char *home;
  6.     path=getenv("PATH" );
  7.     home=getenv("HOME" );
  9.     printf("path=[%s] - home=[%s]\n", path, home);
  10. }


Message édité par Sve@r le 23-05-2007 à 19:37:24

---------------
Vous ne pouvez pas apporter la prospérité au pauvre en la retirant au riche.
Reply

Sujets relatifs:

Leave a Replay

Make sure you enter the(*)required information where indicate.HTML code is not allowed