[debutant] Probleme avec le scanf()

**mc_fly** · 04/09/2006, 16h42

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 !

**zooro** · 04/09/2006, 16h46

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

**mc_fly** · 04/09/2006, 16h50

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

**zooro** · 04/09/2006, 16h53

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

**Ti-R** · 04/09/2006, 16h54

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

La FAQ ne semble pas être à jour

Regarde ICI

**gege2061** · 04/09/2006, 17h02

Envoyé par Ti-R

La FAQ ne semble pas être à jour

ça sera modifié dans la prochaine version

**Ti-R** · 04/09/2006, 17h22

Envoyé par gege2061

ça sera modifié dans la prochaine version

Ok merci

**DaZumba** · 04/09/2006, 17h03

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.

**Emmanuel Delahaye** · 04/09/2006, 18h43

Envoyé par zooro

J'utiliserai plutôt gets par exemple,

gets() est pire que scanf() !

Tu veux sans doute dire fgets()...

**zooro** · 05/09/2006, 09h25

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

**drcd** · 05/09/2006, 13h30

tu declare un:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

char fichier;

et tu fai un:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

**Emmanuel Delahaye** · 05/09/2006, 13h35

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...

**Ti-R** · 04/09/2006, 16h48

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

**Franck.H** · 04/09/2006, 16h51

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

**mc_fly** · 04/09/2006, 16h54

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

**skip78** · 06/09/2006, 10h37

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

**homeostasie** · 06/09/2006, 10h58

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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 */
    }
  }

**Emmanuel Delahaye** · 06/09/2006, 11h23

Envoyé par skip78

Oui bah il suffit de blinder :

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

**skip78** · 07/09/2006, 09h44

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

scanf("%d",num);

non plus

[debutant] Probleme avec le scanf()

C

Vue hybride

Discussions similaires

Partager

Partager