Probleme avec un pointeur

Probleme avec un pointeur - C - Programmation

Marsh Posté le 25-04-2010 à 17:55:17    

Salut
 
Voila j'ai fais une fonction qui compare le nombre entré avec le nombre mystère en demandant a la fin si on veut continuer, alors pour ça j'ai déclaré un pointeur qui pointe sur la variable continuer qui se trouve dans le main, mais hélas ça fonctionne pas.. pourquoi?
 
La fonction compare :
 

Code :
  1. void compareNbrG(int nbrMystere)
  2. {
  4.     int nbrE, continuer = 0,*ptr = NULL;
  6.     do
  7.     {
  9.         printf("Quel est le nombre Mystere ? " );
  10.         scanf("%d", &nbrE);
  12.         if (nbrE < nbrMystere)
  13.             printf("C'est plus ++ !\n" );
  14.         else if (nbrE > nbrMystere)
  15.             printf("C'est moins -- !\n" );
  16.         else
  17.         {
  18.             printf("Bravo !! C'est bien le nombre Mystere !!\n\n" );
  19.             printf("Voulez vous continuer ?\n1. YES\n0. NO\n\n  " );
  20.             scanf("%d", &continuer);
  21.             if (continuer == 1)
  22.             {
  23.                 ptr = &continuer;
  24.                 *ptr = 1;
  25.             }
  26.             else if (continuer == 0)
  27.             {
  28.                 ptr = &continuer;
  29.                 *ptr = 0;
  30.             }
  31.         }
  32.     }
  33.     while (nbrE != nbrMystere);
  34. }


Reply

Marsh Posté le 25-04-2010 à 17:55:17   

Reply

Marsh Posté le 25-04-2010 à 18:01:07    

1/ Dans le code que tu donnes, ptr pointe vers le continuer de compareNbrG().
2/ Que se passe-t'il si on rentre qqch d'autre que 0 ou 1?


---------------
The truth is rarely pure and never simple (Oscar Wilde)
Reply

Marsh Posté le 25-04-2010 à 18:13:28    

Citation :

ça fonctionne pas

Qu'est-ce qui ne marche pas ? Il y a un plantage ? un message d'erreur à lacompilation ? autre chose ?

if (continuer == 1)
            {
                ptr = &continuer;
                *ptr = 1;
            }

Que fait ce code ? Remplaçons continuer par Sarkozy et ptr par adresse de Fillon. Comme on le sait, Sarkozy habite à l'Elisée, et on va dire qu'il y a un beau vase là bas. Le programme change l'adresse de Fillon. Il n'est plus à Matignon, mais à l'Elysée. Puis à cette nouvelle adresse, on y met un beau vase. Mais il était déjà là. Alors, c'est comme si on n'avait rien fait.
 
N.B. Il faut choisir de meilleurs noms pour ces variables. Généralement, on combine une description du contenu avec un code pour le type de contenu. Par exemple, au lieu de continuer et ptr, on peut chosir ContinuerFlag, ContinuerPtr.


Message édité par billgatesanonym le 25-04-2010 à 18:15:03
Reply

Marsh Posté le 25-04-2010 à 20:41:35    

Citation :

    int nbrE, continuer = 0,*ptr = NULL;


Tu déclares une variable locale (a la procédure) nommée continuer. Cette variable va donc masquer la variable continuer déclarée dans main.

 

Ce que tu aurais du faire:

 

Soit:
Déclaration
void compareNbrG(int nbrMystere, int *cont)
{
   int nbrE, *ptr = NULL;
   *cont = 0;
......................
Appel:
void compareNbrG(xxx, &continuer);

 

Ou bien (ce me semblerait bien plus logique):
Déclaration
int compareNbrG(int nbrMystere)
{
  int nbrE, continuer = 0,*ptr = NULL;
.................
return continuer;
}
..........................
Appel:
continuer = compareNbrG(xxx);

 

A+,

 



Message édité par gilou le 25-04-2010 à 20:42:08

---------------
There's more than what can be linked! --    Iyashikei Anime Forever!    --  AngularJS c'est un framework d'engulé!  --
Reply

Marsh Posté le 26-04-2010 à 11:03:50    

Merci, mais je crois que vous avez pas compris ce que je voulais dire, car a la fin de la fonction compare je veux retourné dans le main avec la valeur de "continuer changer" pour pouvoir faire la boucle jusqu'au menu..
 
Voici mon main :
 

Code :
  1. #include <stdio.h>
  2. #include <stdlib.h>
  3. #include <time.h>
  6. int main()
  7. {
  9.     int choixNiveau,nbrM,nbrM1,nbrJoueurs,continuer = 0,compteur = 1,MIN = 0,MAX = 0;
  11.     printf("\t\t\t\tBIENVENU AU PLUS OU MOINS\n\n" );
  12.     printf("\t*MENU*\n\n" );
  13.     printf("1. Niveau entre 1 et 10\n" );
  14.     printf("2. Niveau entre 1 et 20\n" );
  15.     printf("3. Niveau entre 1 et 50\n" );
  16.     printf("4. Niveau entre 1 et 100\n\n" );
  18.     printf("\t*JOUEUR*\n\n" );
  19.     printf("1. Jouer seul\n" );
  20.     printf("2. Jouer a 2\n\n" );
  22.     do
  23.     {
  24.         srand(time(NULL));
  25.         do
  26.         {
  27.             printf("Quel niveau choisissez vous ? " );
  28.             scanf("%d", &choixNiveau);
  29.             if (choixNiveau < 1 || choixNiveau > 4)
  30.                 printf("Choix incorrect !!!" );
  31.         }
  32.         while (choixNiveau < 1 || choixNiveau > 4);
  34.         do
  35.         {
  36.             printf("\nQuel est le nombre de joueurs ? " );
  37.             scanf("%d", &nbrJoueurs);
  38.             if (nbrJoueurs < 1 || nbrJoueurs > 2)
  39.                 printf("Choix incorrect !!!" );
  40.         }
  41.         while (nbrJoueurs < 1 || nbrJoueurs > 2);
  43.         switch (choixNiveau)
  44.         {
  46.         case 1:
  47.             printf("\nVous avez choisi le Niveau 1 : entre 1 et 10 !\n\n" );
  48.             MIN = 1;
  49.             MAX = 10;
  50.             break;
  51.         case 2:
  52.             printf("\nVous avez choisi le Niveau 2 : entre 1 et 20 !\n\n" );
  53.             MIN = 1;
  54.             MAX = 20;
  55.             break;
  56.         case 3:
  57.             printf("\nVous avez choisi le Niveau 3 : entre 1 et 50 !\n\n" );
  58.             MIN = 1;
  59.             MAX = 50;
  60.             break;
  61.         case 4:
  62.             printf("\nVous avez choisi le Niveau 4 : entre 1 et 100 !\n\n" );
  63.             MIN = 1;
  64.             MAX = 100;
  65.             break;
  66.         default:
  67.             printf("\nChoix incorrect !!!\n\n" );
  68.             break;
  69.         }
  71.         if (choixNiveau == 1)
  72.         {
  74.             nbrM = (rand() % (MAX - MIN + 1)) + MIN;
  75.             compteur = 1;
  76.             compareNbrG(nbrM);
  77.         }
  78.         printf("Continuer vaut : %d" , continuer);
  80.         if (choixNiveau ==2)
  81.         {
  82.             printf("Quel est le nombre Mystere a trouve ? " );
  83.             scanf("%d", &nbrM1);
  84.             compteur = 1;
  85.             system("cls" );
  86.             compareNbrT(nbrM1);
  87.         }
  89.     }
  90.     while (continuer);
  92.     return 0;
  93. }


 
voici mes fonctions :
 

Code :
  1. #include <stdio.h>
  2. #include <stdlib.h>
  3. #include <time.h>
  4. #include "fonction.h"
  6. void compareNbrG(int nbrMystere)
  7. {
  9.     int nbrE, continuer = 0,*ptr = NULL;
  11.     do
  12.     {
  14.         printf("Quel est le nombre Mystere ? " );
  15.         scanf("%d", &nbrE);
  17.         if (nbrE < nbrMystere)
  18.             printf("C'est plus ++ !\n" );
  19.         else if (nbrE > nbrMystere)
  20.             printf("C'est moins -- !\n" );
  21.         else
  22.         {
  23.             printf("Bravo !! C'est bien le nombre Mystere !!\n\n" );
  24.             printf("Voulez vous continuer ?\n1. YES\n0. NO\n\n  " );
  25.             scanf("%d", &continuer);
  26.             if (continuer == 1)
  27.             {
  28.                 ptr = &continuer;
  29.                 *ptr = 1;
  30.             }
  31.             else if (continuer == 0)
  32.             {
  33.                 ptr = &continuer;
  34.                 *ptr = 0;
  35.             }
  36.         }
  37.     }
  38.     while (nbrE != nbrMystere);
  39. }
  43. void compareNbrT(int nbrMystere1)
  44. {
  46.     int nbrE,continuer = 0,*ptr = NULL;
  48.     do
  49.     {
  51.         printf("Quel est le nombre Mystere ? " );
  52.         scanf("%d", &nbrE);
  54.         if (nbrE < nbrMystere1)
  55.             printf("C'est plus ++ !\n" );
  56.         else if (nbrE > nbrMystere1)
  57.             printf("C'est moins -- !\n" );
  58.         else
  59.         {
  60.             printf("Bravo !! C'est bien le nombre Mystere !!\n\n" );
  61.             printf("Voulez vous continuer ?\n1. YES\n2. NO\n\n  " );
  62.             scanf("%d", &continuer);
  63.             if (continuer == 1)
  64.             {
  65.                 ptr = &continuer;
  66.                 *ptr = 1;
  67.             }
  68.             else if (continuer == 0)
  69.             {
  70.                 ptr = &continuer;
  71.                 *ptr = 0;
  72.             }
  73.         }
  74.     }
  75.     while (nbrE != nbrMystere1);
  76. }

Reply

Marsh Posté le 26-04-2010 à 13:03:33    

Citation :

Merci, mais je crois que vous avez pas compris ce que je voulais dire, car a la fin de la fonction compare je veux retourné dans le main avec la valeur de "continuer changer" pour pouvoir faire la boucle jusqu'au menu..

Si, j'ai parfaitement compris ce que tu dis, la c'est toi qui n'a pas compris les explications.
La variable continuer de main ne sera pas modifiée par les appels a compareNbrG et compareNbrT parce que dans compareNbrG  la variable continuer de main est masquée par la variable continuer de compareNbrG et que dans compareNbrT  la variable continuer de main est masquée par la variable continuer de compareNbrT.
 
C'est pas comme tu as voulu faire qu'on fait (tu as essayé d'utiliser continuer comme variable globale, c'est pas une mauvaise idée [si tout était dans un seul fichier, et que tu supprimais   les définitions de continuer dans compareNbrG et compareNbrT, ça pourrait marcher], sauf que ça cause beaucoup trop de problèmes a la longue, les variables globales, et c'est une habitude qu'il ne faut pas prendre). Je t'ai indiqué dans mon post précédent comment remédier a ton pb.
 
A+,

Message cité 1 fois
Message édité par gilou le 26-04-2010 à 13:04:43

---------------
There's more than what can be linked! --    Iyashikei Anime Forever!    --  AngularJS c'est un framework d'engulé!  --
Reply

Marsh Posté le 26-04-2010 à 15:24:44    

gilou a écrit :

Je t'ai indiqué dans mon post précédent comment remédier a ton pb.


Faudrait déjà qu'il les lise (et pas en diagonale) [:ocube]

Reply

Marsh Posté le 26-04-2010 à 15:39:49    

Merci gilou c'est compris...

Reply

Sujets relatifs:

Leave a Replay

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