[Résolu]Variable modifié entre deux fonctions

Variable modifié entre deux fonctions [Résolu] - C - Programmation

Marsh Posté le 10-02-2013 à 11:07:44    

Bonjour a tous et a toutes je suis actuellement en train de faire un 'Jeu de la vie' pour un devoir.
je rentre mes fonction notamment une pour la création d'un tableau dynamique mais entre 2 fonctions mes valeurs changent sans que (ré)assigne les variables.
 
exemple ma hauteur je la met a 9 longueur aussi a 9
 et bien quand je l'affiche elle passe a 2686720 pour la hauteur et 2686716 pour la longueur et ce peu importe le nombre que je met dans la variable
 
http://puu.sh/20hjU
 
Avez vous une idée de ce qu'il se passe a mes variable ?
 
Mon Main:

Code :
  1. #include <stdio.h>
  2. #include <stdlib.h>
  3. #include "Autoheader.h"
  5. int main()
  6. {
  7. int **plateau;
  8. int hauteur = 0;
  9. int longueur = 0;
  10. int B[9];
  11. int S[9];
  12. int i;
  13. int j;
  15.      dimensionnement(&hauteur,&longueur);
  17. // Declaration Tableau Dynamique
  18. plateau = malloc(hauteur * sizeof(*plateau));
  19. for (i = 0; i < hauteur; i++)
  20. {
  21.     plateau[i] = malloc(longueur * sizeof(**plateau));
  22.     for(j = 0; j < longueur; j++)
  23.     {
  24.         plateau[i][j] = 0;
  25.     }
  26. }
  28.         printf("\t %d \n",hauteur);
  29.         printf("\t %d \n",longueur);
  31.         initialisation(&plateau,&hauteur,&longueur);
  32.         affiche(&plateau,&hauteur,&longueur);
  33.         affiche_fichier(&plateau,&hauteur,&longueur);
  34.         saisie_B_M(&B[9]);
  35.         saisie_S_M(&S[9]);
  36.         nb_generation ();
  40. return 0;
  41. }


 
le .c de mon header

Code :
  1. #include <stdio.h>
  2. #include <stdlib.h>
  3. #include "Autoheader.h"
  6. void dimensionnement(int *hauteur, int *longueur)
  7. {
  9.     printf("Entrez la resolution du plateau de jeu\n" );
  10.     printf("Suivez bien cette regle, il ne peut y avoir une resolution negative\n\n" );
  12.     //boucle pour avoir seulement une valeur sup a 1
  13.     do
  14.     {
  15.         printf("Inscrivez la hauteur du tableau (nombres de Lignes) :\t" );
  16.         scanf("%d", hauteur);
  17.     }
  18.     while (0 > hauteur);
  20.     do
  21.     {
  22.     printf("Inscrivez la longueur du tableau (nombres de Colones) :\t" );
  23.     scanf("%d", longueur);
  24.     }
  25.     while (0 > longueur);
  26.     printf("\n\n" );
  27. }
  29. void initialisation(int **plateau,int hauteur,int longueur)
  30. {
  31. int h = hauteur;
  32. int l = longueur;
  33. int celluleactive;
  34. int comp_cellule_for;
  35. int buffer_activation_h;
  36. int buffer_activation_l;
  38.         printf("\t %d \n",hauteur);
  39.         printf("\t %d \n",longueur);
  40.         printf("\t %d \n",h);
  41.         printf("\t %d \n",l);
  42.     printf("Veuillez entrer le nombre de celules vivantes dans la 1ere generation :\t" );
  43.     scanf("%d", &celluleactive);
  44.     for(comp_cellule_for = 0; comp_cellule_for < celluleactive; comp_cellule_for++)
  45.     {
  46.         printf("%d", h);
  47.         do // coordonées de la cellule en hauteur
  48.         {
  49.             do
  50.             {
  51.                 printf("A quelle ligne se trouve la cellule a activer ? :\t" );
  52.                 scanf("%d",&buffer_activation_h);
  53.             }
  54.             while (buffer_activation_h < 0);
  55.         }
  56.         while (buffer_activation_h > hauteur);
  60.         do // coordonées de la cellule en longueur
  61.         {
  62.             do
  63.             {
  64.                 printf("A quelle colonne se trouve la cellule a activer ? :\t" );
  65.                 scanf("%d", &buffer_activation_l);
  66.             }
  67.             while (buffer_activation_l < 0);
  68.         }
  69.         while (buffer_activation_l > longueur);
  71.     plateau[--buffer_activation_h][--buffer_activation_l] = 1;
  72.     }
  74. }
  76. int nb_generation(nb_gene)
  77. {
  79.         do
  80.         {
  81.         printf("Veuillez entrez le nombre de generation desire \n" );
  82.         printf("il ne peut etre inferieur a 0\n" );
  83.         scanf("%d", &nb_gene);
  84.         }
  85.         while (nb_gene <= 0 );
  86. printf("\n\n" );
  87. }
  89. void affiche(int**plateau,int hauteur, int longueur)
  90. {
  91. int ia;
  92. int ja;
  94.     for(ia=0;ia < hauteur;ia++)
  95.     {
  96.         for(ja=0; ja < longueur;ja++)
  97.         {
  98.             if (plateau[ia][ja]==1)
  99.             {
  100.                 printf("o" ); //affiche des 'o' a la place des '1' du plateau
  101.             }
  102.             else
  103.             {
  104.                 printf("." ); //affiche des '.' a la place des '0' du plateau
  105.             }
  106.         }
  107.     }
  108. printf("\n\n" );
  109. }
  111. void affiche_fichier(int** plateau, int hauteur, int longueur)
  112. {
  113.     int i,j;
  114.     FILE * resultat = NULL;
  116.     resultat = fopen("F:\\AutoMath\\Automath.txt","w+" );
  118.     if (resultat != NULL) //Verification de l'ouverture
  119.     {
  120.         printf ("Le fichier Automath.txt a reussi son ouverture\n" );//Succes
  121.         for(j=0;j<hauteur;j++)
  122.         {
  123.             for (i=0;i<longueur;i++)
  124.             {
  125.                 if(plateau[i][j]==1)
  126.                     fputc('o', resultat);
  127.                 else
  128.                     fputc('.', resultat);
  129.             }
  130.             printf("\n" );
  131.         }
  133.         fclose(resultat);
  134.     }
  135.     else// Echec
  136.     {
  137.         printf("Erreur: Le fichier Automath.txt ne peut s'ouvrir" );
  138.         system("PAUSE" );
  139.     }
  140. printf("\n\n" );
  141. }
  143. void saisie_B_M(int B[9])
  144. {
  145.     int b_moore[9];
  146.     int compteur_moore= 0;
  147.     int choix_moore;
  149.     printf("Veuillez entrez les regles de naissances B \n" );
  150.     printf("N'entrez que des 0 ou des 1 \n" );
  151.     printf("0 = Non\n" );
  152.     printf("1 = Oui\n\n" );
  154.     for(compteur_moore= 0 ;compteur_moore<9; compteur_moore++)
  155.     {
  156.         do
  157.         {
  158.             printf("Une cellule morte ayant %d voisins dois t'elle vivre ? :\t", compteur_moore);
  159.             scanf("%d", &choix_moore);
  160.             b_moore[compteur_moore]=choix_moore;
  161.         }
  162.         while (choix_moore != 0 && choix_moore != 1);
  163.     }
  164. printf("\n\n" );
  165. }
  167. void saisie_S_M(int S[9])
  168. {
  169. int s_moore[9];
  170. int compteur_moore_s= 0;
  171. int choix_moore_s;
  173.     printf("Veuillez entrez les regles de survie S \n" );
  174.     printf("N'entrez que des 0 ou des 1 \n" );
  175.     printf("0 = Non\n" );
  176.     printf("1 = Oui\n\n" );
  178.     for(compteur_moore_s= 0 ;compteur_moore_s<9; compteur_moore_s++)
  179.     {
  180.         do
  181.         {
  182.             printf("Une cellule vivante ayant %d voisins dois t'elle survivre ? :\t", compteur_moore_s);
  183.             scanf("%d", &choix_moore_s);
  184.             s_moore[compteur_moore_s]=choix_moore_s;
  185.         }
  186.         while (choix_moore_s != 0 && choix_moore_s != 1);
  187.     }
  188. printf("\n\n" );
  189. }


 
Mon header

Code :
  1. #ifndef AUTOMATH_H_INCLUDED
  2. #define AUTOMATH_H_INCLUDED
  4. void dimensionnement(int *hauteur, int *longueur);
  5. void initialisation(int **plateau,int hauteur,int longueur);
  6. void affiche(int **plateau,int hauteur,int longueur);
  7. void affiche_fichier(int **plateau, int hauteur, int longueur);
  8. void saisie_B_M(int B[9]);
  9. void saisie_S_M(int S[9]);
  10. int nb_generation ();
  12. #endif // AUTOMATH_H_INCLUDED


 
le compilateur me dit :

Code :
  1. F:\Auto\automath\main.c||In function 'main':|
  2. F:\Auto\automath\main.c|31|warning: passing argument 1 of 'initialisation' from incompatible pointer type [enabled by default]|
  3. F:\Auto\automath\Autoheader.h|5|note: expected 'int **' but argument is of type 'int ***'|
  4. F:\Auto\automath\main.c|31|warning: passing argument 2 of 'initialisation' makes integer from pointer without a cast [enabled by default]|
  5. F:\Auto\automath\Autoheader.h|5|note: expected 'int' but argument is of type 'int *'|
  6. F:\Auto\automath\main.c|31|warning: passing argument 3 of 'initialisation' makes integer from pointer without a cast [enabled by default]|
  7. F:\Auto\automath\Autoheader.h|5|note: expected 'int' but argument is of type 'int *'|
  8. F:\Auto\automath\main.c|32|warning: passing argument 1 of 'affiche' from incompatible pointer type [enabled by default]|
  9. F:\Auto\automath\Autoheader.h|6|note: expected 'int **' but argument is of type 'int ***'|
  10. F:\Auto\automath\main.c|32|warning: passing argument 2 of 'affiche' makes integer from pointer without a cast [enabled by default]|
  11. F:\Auto\automath\Autoheader.h|6|note: expected 'int' but argument is of type 'int *'|
  12. F:\Auto\automath\main.c|32|warning: passing argument 3 of 'affiche' makes integer from pointer without a cast [enabled by default]|
  13. F:\Auto\automath\Autoheader.h|6|note: expected 'int' but argument is of type 'int *'|
  14. F:\Auto\automath\main.c|33|warning: passing argument 1 of 'affiche_fichier' from incompatible pointer type [enabled by default]|
  15. F:\Auto\automath\Autoheader.h|7|note: expected 'int **' but argument is of type 'int ***'|
  16. F:\Auto\automath\main.c|33|warning: passing argument 2 of 'affiche_fichier' makes integer from pointer without a cast [enabled by default]|
  17. F:\Auto\automath\Autoheader.h|7|note: expected 'int' but argument is of type 'int *'|
  18. F:\Auto\automath\main.c|33|warning: passing argument 3 of 'affiche_fichier' makes integer from pointer without a cast [enabled by default]|
  19. F:\Auto\automath\Autoheader.h|7|note: expected 'int' but argument is of type 'int *'|
  20. ||=== Build finished: 0 errors, 9 warnings (0 minutes, 1 seconds) ===|


 
D'avance merci Mimic²


Message édité par Mimic² le 10-02-2013 à 16:16:14
Reply

Marsh Posté le 10-02-2013 à 11:07:44   

Reply

Marsh Posté le 10-02-2013 à 11:25:13    

Code :
  1. do
  2.     {
  3.         printf("Inscrivez la hauteur du tableau (nombres de Lignes) :\t" );
  4.         scanf("%d", hauteur);
  5.     }
  6.     while (0 > hauteur);

Ceci ne fonctionne pas car hauteur est une adresse (pointeur sur un entier). il faut mettre

Code :
  1. do
  2.     {
  3.         printf("Inscrivez la hauteur du tableau (nombres de Lignes) :\t" );
  4.         scanf("%d", hauteur);
  5.     }
  6.     while (0 > *hauteur);

De même pour la longueur.

Reply

Marsh Posté le 10-02-2013 à 11:44:17    

Merci pour ton aide mais j'avais essayé, ça me retourne la même chose
une fois que je met un chiffre même sans le *hauteur il me retourne le bon chiffre.
Une fois que je sort de la fonction je le teste avec un printf et le chiffre est bon je rentre dans la fonction initialisation, la le chiffre a changé

Reply

Marsh Posté le 10-02-2013 à 12:09:45    

Les incohérences sont les suivantes :  
 
Déclaration des variables :

Code :
  1. int **plateau;
  2. int hauteur;
  3. int longueur;


 
Déclaration de la fonction d'initialisation :

Code :
  1. void initialisation(int **plateau,int hauteur,int longueur)


 
Utilisation des variables :  

Code :
  1. initialisation(&plateau,&hauteur,&longueur);


 
Vous passez un triple pointeur au lieu d'un double (pour plateau) et des pointeurs au lieu de valeurs (hauteur, longueur)
 
Note : Les "warnings" du compilateur auraient dû vous mettre la puce à l'oreille :)
 
Edit : En lisant la fin des "warnings", pas besoin de boule de cristal pour prédire que vous allez avoir le même problème avec la fonction "affiche_fichier"


Message édité par Farian le 10-02-2013 à 12:12:14
Reply

Marsh Posté le 10-02-2013 à 12:31:24    

ces fonction sont dans l'énoncé éventuellement ce qui pourrais être faux est :
initialisation(&plateau,&hauteur,&longueur);

Reply

Marsh Posté le 10-02-2013 à 12:52:56    

L'idée est le manque de cohérence entre la déclaration et l'appel.
 
Si la déclaration est fixée, c'est l'appel qui est incorrect, en effet.
 
Globalement, comprendre et, sauf cas particulier, corriger tous les warnings de compilation est nécessaire, le compilateur ne les émet pas par plaisir mais pour indiquer que quelque chose ne va pas.

Reply

Marsh Posté le 10-02-2013 à 16:15:15    

Et bien merci j'ai réussi grâce a vous a trouver le problème en effet il faillait appeler avec :
 
initialisation(plateau,hauteur,longueur);
idem pour les affichages

Reply

Sujets relatifs:

Leave a Replay

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