Probleme avec un pointeur

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : Probleme avec un pointeur

lassault1

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 :

void compareNbrG(int nbrMystere)
{
int nbrE, continuer = 0,*ptr = NULL;
do
{
printf("Quel est le nombre Mystere ? " );
scanf("%d", &nbrE);
if (nbrE < nbrMystere)
printf("C'est plus ++ !\n" );
else if (nbrE > nbrMystere)
printf("C'est moins -- !\n" );
else
{
printf("Bravo !! C'est bien le nombre Mystere !!\n\n" );
printf("Voulez vous continuer ?\n1. YES\n0. NO\n\n " );
scanf("%d", &continuer);
if (continuer == 1)
{
ptr = &continuer;
*ptr = 1;
}
else if (continuer == 0)
{
ptr = &continuer;
*ptr = 0;
}
}
}
while (nbrE != nbrMystere);
}

Publicité

Un Programmeur

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)

billgatesanonym

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

gilou

Modérateur
Modzilla

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

lassault1

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 :

#include <stdio.h>
#include <stdlib.h>
#include <time.h>
int main()
{
int choixNiveau,nbrM,nbrM1,nbrJoueurs,continuer = 0,compteur = 1,MIN = 0,MAX = 0;
printf("\t\t\t\tBIENVENU AU PLUS OU MOINS\n\n" );
printf("\t*MENU*\n\n" );
printf("1. Niveau entre 1 et 10\n" );
printf("2. Niveau entre 1 et 20\n" );
printf("3. Niveau entre 1 et 50\n" );
printf("4. Niveau entre 1 et 100\n\n" );
printf("\t*JOUEUR*\n\n" );
printf("1. Jouer seul\n" );
printf("2. Jouer a 2\n\n" );
do
{
srand(time(NULL));
do
{
printf("Quel niveau choisissez vous ? " );
scanf("%d", &choixNiveau);
if (choixNiveau < 1 || choixNiveau > 4)
printf("Choix incorrect !!!" );
}
while (choixNiveau < 1 || choixNiveau > 4);
do
{
printf("\nQuel est le nombre de joueurs ? " );
scanf("%d", &nbrJoueurs);
if (nbrJoueurs < 1 || nbrJoueurs > 2)
printf("Choix incorrect !!!" );
}
while (nbrJoueurs < 1 || nbrJoueurs > 2);
switch (choixNiveau)
{
case 1:
printf("\nVous avez choisi le Niveau 1 : entre 1 et 10 !\n\n" );
MIN = 1;
MAX = 10;
break;
case 2:
printf("\nVous avez choisi le Niveau 2 : entre 1 et 20 !\n\n" );
MIN = 1;
MAX = 20;
break;
case 3:
printf("\nVous avez choisi le Niveau 3 : entre 1 et 50 !\n\n" );
MIN = 1;
MAX = 50;
break;
case 4:
printf("\nVous avez choisi le Niveau 4 : entre 1 et 100 !\n\n" );
MIN = 1;
MAX = 100;
break;
default:
printf("\nChoix incorrect !!!\n\n" );
break;
}
if (choixNiveau == 1)
{
nbrM = (rand() % (MAX - MIN + 1)) + MIN;
compteur = 1;
compareNbrG(nbrM);
}
printf("Continuer vaut : %d" , continuer);
if (choixNiveau ==2)
{
printf("Quel est le nombre Mystere a trouve ? " );
scanf("%d", &nbrM1);
compteur = 1;
system("cls" );
compareNbrT(nbrM1);
}
}
while (continuer);
return 0;
}

voici mes fonctions :

Code :

#include <stdio.h>
#include <stdlib.h>
#include <time.h>
#include "fonction.h"
void compareNbrG(int nbrMystere)
{
int nbrE, continuer = 0,*ptr = NULL;
do
{
printf("Quel est le nombre Mystere ? " );
scanf("%d", &nbrE);
if (nbrE < nbrMystere)
printf("C'est plus ++ !\n" );
else if (nbrE > nbrMystere)
printf("C'est moins -- !\n" );
else
{
printf("Bravo !! C'est bien le nombre Mystere !!\n\n" );
printf("Voulez vous continuer ?\n1. YES\n0. NO\n\n " );
scanf("%d", &continuer);
if (continuer == 1)
{
ptr = &continuer;
*ptr = 1;
}
else if (continuer == 0)
{
ptr = &continuer;
*ptr = 0;
}
}
}
while (nbrE != nbrMystere);
}
void compareNbrT(int nbrMystere1)
{
int nbrE,continuer = 0,*ptr = NULL;
do
{
printf("Quel est le nombre Mystere ? " );
scanf("%d", &nbrE);
if (nbrE < nbrMystere1)
printf("C'est plus ++ !\n" );
else if (nbrE > nbrMystere1)
printf("C'est moins -- !\n" );
else
{
printf("Bravo !! C'est bien le nombre Mystere !!\n\n" );
printf("Voulez vous continuer ?\n1. YES\n2. NO\n\n " );
scanf("%d", &continuer);
if (continuer == 1)
{
ptr = &continuer;
*ptr = 1;
}
else if (continuer == 0)
{
ptr = &continuer;
*ptr = 0;
}
}
}
while (nbrE != nbrMystere1);
}

gilou

Modérateur
Modzilla

Citation :

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

404 Not Found

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]

lassault1

Merci gilou c'est compris...

FORUM HardWare.fr

Programmation

Probleme avec un pointeur

Sujets relatifs
[C] probleme sprintf et pointeur :S	[Résolu]Probleme avec des pointeur de structure
Problème de pointeur	Problème de pointeur objet mère qui pointe sur objet fille
Probleme de malloc sur un tableau de pointeur.	Probleme utilisation pointeur et reference
Opérations matrices problème structure pointeur	problème avec une fonction qui renvoie un pointeur de char
probleme de pointeur	Probleme de pointeur
Plus de sujets relatifs à : Probleme avec un pointeur

Page générée en 0.101 secondes