[debutant] Probleme avec le scanf()

Salut a tous,

Je souhaite faire un menu basique dans lequel l'utilisateur devra repondre des parametres et a la fin, un message s'affiche pour lui demander s'il veu quitter ou continuer.
Aussi, je souhaite que si l'utilisateur ne rentre pas le choix demandé, et bien que le menu recommence.

Voila mon code :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30

void Load_3() <div style="margin-left:40px">int sortie = 0 int debut; char choix, fichier   while (!sortie) { <div style="margin-left:40px">system(CMDCLS) // efface l'ecran - pas de prob- printf("n° debut:"); // la je veu un entier long scanf("%d",&debut); printf("nom du fichier:"); //la je veux une chaine de caracter scanf("%s",&fichier); printf("continuer (c) - retour (r) :"); //la je veux c ou r scanf("%c",&choix);   switch (choix) { <div style="margin-left:40px">case 'c': sortir = TRUE; //autre actions... break;   case 'r': sortie=TRUE; //autres actions ... break;   default : break;</div>}</div>}</div>

---

Merci de votre aide... je galere bien pour faire tous ces menus... si quelqu'un peu m'aider !

Un conseil: évite le scanf pour des saisies utilisateurs !
Pourquoi ? Si tu attends un entier et qu'il saisit une chaîne, que va-t-il se passer???

J'utiliserai plutôt gets par exemple, et ensuite une conversion de la chaîne en entier, avec gestion d'erreur.

Toujours intéressant, "Pourquoi scanf doit être interdit" ;) : http://www.developpez.net/forums/sho...obl%E8me+scanf

Qu'est ce qui ne fonctionne pas dans ton programme sinon ?

oui j'ai cru comprendre que le scanf c'est pas terrible... c'est pour cela que je demande conseil en fait.
Alors d'accord pour ton gets, mais esce que tu peux detailler la "conversion de la chaîne en entier, avec gestion d'erreur" pleaze :?
merci

Déjà si c'est pour faire réafficher le menu au cas où l'utilisateur entre de fausse données, l'intérieur de la boucle je le metterais dans une fonction à part qui renverrais 0 ou 1, comme ca il te reste à tester le retour de la fonction pour savoir si tu dois réafficher le menu ou non ;)

Il existe une fonction strtol par exemple pour convertir une chaîne en long. Il y en a d'autres pour les autres types, me semble-t-il.
cf. FAQ

Citation:

---

Envoyé par Ti-R

Qu'est ce qui ne fonctionne pas dans ton programme sinon ?

---

Et bien en fait a la compilation, ya pas de probleme
Le seul probleme, c'est quand je rentre un caractere a la place du numero... le programme accepte cette reponse alors que ca va pas en fait...
Si je demande un N°, je veux verifier en fait que l'utilisateur rentre bien un N°
Si c'est une chaine de caracteres, pareil.

Et de plus, a la fin de la saise, je demande a l'utilisateur s'il veux continuer ou retourné a la page precendente.

Merci

http://c.developpez.com/faq/c/?page=...RINGS_numtostr

La FAQ ne semble pas être à jour

Regarde ICI

Citation:

---

Envoyé par Ti-R

La FAQ ne semble pas être à jour

---

ça sera modifié dans la prochaine version ;)

Citation:

---

Envoyé par mc_fly

Alors d'accord pour ton gets

---

Jamais gets(). Toujours utiliser fgets() + strchr() + la fonction de conversion qui va bien (e.g. strtol() strtod() etc.) pour ce genre d'applications.

Citation:

---

Envoyé par gege2061

ça sera modifié dans la prochaine version ;)

---

Ok merci ;)

Citation:

---

Envoyé par zooro

J'utiliserai plutôt gets par exemple,

---

gets() est pire que scanf() !

Tu veux sans doute dire fgets()...

Oui, c'est exactement ce que je voulais dire !
C'est mon doigt qui a fourché :-)

tu declare un:

Code:

---

char fichier;

---

et tu fai un:

Code:

---

scanf("%s",...);

---

Tu aurais pas un dépassement de memoire?
Utilise fgets sur stdin pour faire une saisie o clavier.
Le second argument est le nombre de caracteres max que l'utilisateur peut rentrer comme ca il n'y a pas d'overflow et le 3e argument si je me souvient bien c'est le nom de ta variable de chaine de caractere.

Citation:

---

Envoyé par drcd

Utilise fgets sur stdin pour faire une saisie o clavier.
Le second argument est le nombre de caracteres max que l'utilisateur peut rentrer comme ca il n'y a pas d'overflow et le 3e argument si je me souvient bien c'est le nom de ta variable de chaine de caractere.

---

Non. Relis ta doc...

Citation:

---

Envoyé par Emmanuel Delahaye

Non. Relis ta doc...

---

Je la relirai le jour ou j'en aurai besoin. La c'est pas moi ki en ai besoin lol.

Citation:

---

Envoyé par drcd

Je la relirai le jour ou j'en aurai besoin. La c'est pas moi ki en ai besoin lol.

---

OK. En attendant, abstient toi de parler de ce que tu ne connais pas. On est pas là pour ramasser le crottin...

Citation:

---

Envoyé par mc_fly

Le seul probleme, c'est quand je rentre un caractere a la place du numero... le programme accepte cette reponse alors que ca va pas en fait...

---

Oui bah il suffit de blinder :

Tu fait ton scanf et tu verifie que c'est un int ...

Code:

---

1 2 3 4 5 6 7 8 9 10 11

// Du genre (si le num doit être entre 1 et 5 (au passage) : int test = 1; do{ printf("Merci de saisir le numero\n>"); scanf("%d",&num); if(num>=1 && num <=5)test = 0; //blindage else("Erreur de saisie (le numero doit etre compris entre 1 et 5)\n\n"); }while(test); //blindage   //suite

---

Citation:

---

mc_fly a écrit :
Le seul probleme, c'est quand je rentre un caractere a la place du numero... le programme accepte cette reponse alors que ca va pas en fait...

---

Voici un exemple d'utilisation de scanf avec les précautions nécessaires (saisie d'un entier, purge du flux stdin, ignorer les caractères qui suivent la saisie d'un nombre entier...):

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25

int nombre = 0; int ok = 0; int retour;   printf("saisissez un nombre :\n"); while (!ok){ retour = scanf("%d%*[^\n]", &nombre); printf("retour : %d\n", retour); if ( !retour ){ /* erreur de saisie, on vide le flux */ int c; while ( ((c = getchar()) != '\n') && c != EOF);   printf("on vous a demande de saisir un nombre\n"); printf("veuillez recommencer :\n"); } else { /* reussite de la saisie */ getchar(); /* on enleve le '\n' restant */   printf("saisie acceptee\n"); ok = 1; /* sort de la boucle */ } }

---

Citation:

---

Envoyé par skip78

Oui bah il suffit de blinder :

Tu fait ton scanf et tu verifie que c'est un int ...

Code:

---

1 2 3 4 5 6 7 8 9 10 11

// Du genre (si le num doit être entre 1 et 5 (au passage) : int test = 1; do{ printf("Merci de saisir le numero\n>"); scanf("%d",&num); if(num>=1 && num <=5)test = 0; //blindage else("Erreur de saisie (le numero doit etre compris entre 1 et 5)\n\n"); }while(test); //blindage   //suite

---

---

Marche pas si tu saisies une lettre... Il faut d'abord tester la valeur retournée par scanf()...

Si tu y tiens, apprends à utiliser scanf() correctement : http://xrenault.developpez.com/tutoriels/c/scanf/ , sinon, évite de parler de ce que tu ne connais pas.

Citation:

---

Envoyé par Emmanuel Delahaye

Marche pas si tu saisies une lettre... Il faut d'abord tester la valeur retournée par scanf()...

Si tu y tiens, apprends à utiliser scanf() correctement : http://xrenault.developpez.com/tutoriels/c/scanf/ , sinon, évite de parler de ce que tu ne connais pas.

---

... c'était pour montrer le chemin à suivre, je n'ai jamais dit que mon code était complet :/ Ici le blindage s'effectue uniquement pour verifier que le nombre saisi est compris entre 1 et 5.

De plus je ne fais ca que pour apporter une aide, alors même si je ne donne pas l'intégralité de la solution, je vois pas sur quoi tu peut t'appuyer pour dire que je ne sais pas me servir de scanf ... C'est pas comme si j'avais mis

Code:

---

scanf("%d",num);

---

non plus :roll:

Citation:

---

Envoyé par skip78

je vois pas sur quoi tu peut t'appuyer pour dire que je ne sais pas me servir de scanf

---

• Pas de test de la valeur retournée
• Pas de purge en cas d'erreur.