Discussion :

[tomasi]

Je voudrais savoir comment je pourrais faire pour savoir si ma variable est bien un chiffre ?

j'ai essayé la fonction isnumeric de la façon suivante

Code :

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

1
2
3
 
if (IsNumeric(ma_variable))
{ faire_ceci; }

mais cela ne fonctionne pas

"request for member `IsNumeric' in something not a structure or union"

[Trap D]

isdigit(var) déclaré dans ctype.h devrait faire l'affaire.

[Emmanuel Delahaye]

Je voudrais savoir comment je pourrais faire pour savoir si ma variable est bien un chiffre ?

j'ai essayé la fonction isnumeric de la façon suivante

Code :

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

1
2
3
 
if (IsNumeric(ma_variable))
{ faire_ceci; }

mais cela ne fonctionne pas

"request for member `IsNumeric' in something not a structure or union"

Déjà, commence par vérifier que tu compiles bien en C, parce que cette erreur est plutôt étrange... L'extension du fichier doit être .c et non .cpp ni .C.

Ensuite, cette fonction IsNumeric() n'est pas standard. Si ma_variable contient la valeur d'un caractère, tu peux utiliser la fonction standard isdigit() de <ctype.h> qui est faite pour ça.

Enfin, en C, IsNumeric est différent de isnumeric. En effet, le C est sensible à la casse.

[tomasi]

merci pour la précision mais cela ne fonctionne pas comme je le souhaite, en effet je dois vérifier dans le code suivant que la variable choix est bien un entier sinon j'obtiens une boucle infinie.

Pour une raison inconnue le default du switch ne permet pas de gérer le cas ou on rentre autre chose qu'un entier au clavier

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
31
32
33
34
35
36
37
38
39
40
 
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <ctype.h>
 
int menu(){ //affiche le menu et retourne l'entier du choix
 
    int choix;
 
     printf("Entrer le numéro de la commande\n");
     printf("1. Fin \n");
     printf("2. Insérer \n");
     printf("3. Rechercher \n");
     printf("4. Supprimer \n");
 
     if (isdigit(choix)){
       scanf("%d", &choix); 
     }
 
     return choix;
     }
 
int main(){
 
    do{
       switch(menu()){
         case 1 : printf("Fin \n"); system("pause"); exit(1); break;
         case 2 : printf("2 \n"); break;
         case 3 : printf("3 \n"); break;
         case 4 : printf("4 \n"); break;
         default : printf("Commande non valide \n"); break;        
       }
     }
  while(1);
 
  system("pause");
 
  return EXIT_SUCCESS;
}

je précise que je suis sous dev c++

[Emmanuel Delahaye]

je précise que je suis sous dev c++

Il suffit de tester la valeur retournée par scanf()

http://xrenault.developpez.com/tutoriels/c/scanf/

et d'agir en conséquence (purge)

Sinon, une saisie avec fgets() évite ces ennuis de mauvaise utilisation de scanf() qui est difficile à utiliser correctement...

Evidemment, c'est un peu plus complexe (saisie, nettoyage et purge, conversion, test des limites) mais une fois qu'on a mis tout ça dans une fonction, c'est fiable et c'est pour la vie...

Et comme il faut faire la même chose avec les saisies de chaines, on peut récupérer une partie du travail pour faire la saisie de nombres...

Bref, c'est l'occasion d'apprendre à coder correctement en se fabriquant ses outils qui seront toujours meilleurs que ceux qu'on achète tout faits... C'est ça l'esprit artisan...

"Le code bio, c'est meilleur que le code industriel"

J'ai dit.

[tomasi]

je ne comprends pas bien comment tester la valeur de retour de scanf()

en faisant

Code :

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

1
2
3
 
if ( scanf("%d", &choix))
       scanf("%d", &choix);

?

et ensuite "purger" ?
là je suis complètement perdu

[Emmanuel Delahaye]

je ne comprends pas bien comment tester la valeur de retour de scanf()

en faisant

Code :

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

1
2
3
 
if ( scanf("%d", &choix))
       scanf("%d", &choix);

?

et ensuite "purger" ?
là je suis complètement perdu

Tu n'as donc pas lu l'article dont je t'ai passé le lien ? Ca ne tombe du ciel tout ça, il faut faire un effort de documentation, de recherche et d'apprentissage...

[tomasi]

le test du retour de scanf :
[*] s'il vaut 1, alors la saisie a réussie : cela veut dire que tab a été correctement rempli. Par contre, on ne sait pas s'il reste des caractères disponibles, à cause du champ taille. En effet, comme on a précisé celle-ci, il se peut que la fonction scanf se soit arrêtée alors que le flux entrant contient encore des caractere différents de '\n'.

On fait un 'getchar' pour récupérer le prochain caractère (il y en a forcement au moins un si scanf a réussi)
-- s'il est égal à '\n', alors c'était le dernier du flux.
-- s'il en est différent, c'est qu'il reste des caractères en trop. On peut choisir de vider le flux de manière standard, en utilisant getchar, ou bien faire un autre traitement, tout dépend de ce que l'on souhaite faire.

[*] s'il est différent de 1, c'est que la saisie a échoué. Dans l'exemple pris plus haut, ce cas peut arriver uniquement en cas de saisie d'une ligne vide (ne contenant qu'un '\n').
Il faut donc vider le flux entrant. En effet, la fonction scanf s'est arrêtée avant le '\n' : ce caractère reste donc présent sur le flux ! (un simple getchar fait l'affaire :-)

si je l'ai lu

mais je ne le comprends pas, je ne vois pas comment cela peut m'aider à résoudre mon problème

[Emmanuel Delahaye]

le test du retour de scanf :
<...>
mais je ne le comprends pas, je ne vois pas comment cela peut m'aider à résoudre mon problème

Ton problème, c'est de savoir si

Code :

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

   scanf("%d", &choix);

a fonctionné correctement. Pour ça, il suffit de vérifier que la valeur retournée par la fonction est conforme. Quelle est la valeur conforme ?

Pour le savoir, il faut lire la doc complète de scanf, par exemple ici :

http://man.developpez.com/man3/scanf.3.php

[tomasi]

j'ai compris que si il y a une erreur dans le scanf il retourne la valeur EOF mais si je fais

Code :

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

    if (scanf("%d", &choix)!=EOF)

cela ne fonctionne pas

[Emmanuel Delahaye]

j'ai compris que si il y a une erreur dans le scanf il retourne la valeur EOF mais si je fais

Code :

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

    if (scanf("%d", &choix)!=EOF)

cela ne fonctionne pas

Non. Ce qui est important de comprendre, c'est que quand tout se passe bien, scanf() retourne le nombre de conversions réussies. Ici, 1. Donc :

Code :

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

1
2
3
4
5
 
   if (scanf ("%d", &choix) != 1)
   {
      /* L'operateur c'est trompe. Il va falloir purger, sinon c'est le drame... */
   }

[tomasi]

Code :

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

if (scanf ("%d", &choix) != 1)

ne fonctionne toujours pas, j'ai toujours une boucle infinie si je rentre autre chose qu'un entier

je ne vois pas ce que veut dire le terme "purger" dans ce contexte...

[kamouminator]

tu devrais montrer ton code "corrigé" !

[tomasi]

j'ai pris l'exemple de la documentation, dans ma hâte je ne l'avais pas appliqué correctement

je vais essayer de comprendre tout ça et de l'appliquer à la saisie d'une suite de caractères en minuscules

un grand merci pour l'aide et la patience

voici le code en question

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
31
32
33
34
35
36
int menu(){ //affiche le menu et retourne l'entier du choix - l'entrée doit être un nombre sinon on redemande la saisie
 
  int choix;
  int nombre = 0;
  int ok = 0;
  int retour;
 
  printf("Entrer la commande\n");
  printf("1. Fin \n");
  printf("2. Insérer \n");
  printf("3. Rechercher \n");
  printf("4. Supprimer \n");
 
  while ( !ok ){
 
    retour = scanf("%d%*[^\n]", &choix);
    //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 */
    }
  }
 
     return choix;
}

si je ne m'abuse, pour tester avec une chaine de caractères minuscules il suffit de changer l'expression régulière...

[kamouminator]

ce que je ferai moi:

Code :

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

1
2
3
if (scanf ("%d", &choix) != 1) return 0;
 
return choix;

[tomasi]

ce que je ferai moi:

Code :

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

1
2
3
if (scanf ("%d", &choix) != 1) return 0;
 
return choix;

hélas, cela ne fonctionne pas

[kamouminator]

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
31
32
33
34
35
36
37
38
39
40
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <ctype.h>
 
int menu(){ //affiche le menu et retourne l'entier du choix
    
    int choix;
  
     printf("Entrer le numéro de la commande\n");
     printf("1. Fin \n");
     printf("2. Insérer \n");
     printf("3. Rechercher \n");
     printf("4. Supprimer \n");
   
     if (scanf ("%d", &choix) != 1) return 5;
 
return choix;
     }
 
int main(){
	int recommencer=0;
    
    do{
		fflush(stdin); <-----correction
       switch(menu()){
         case 1 : printf("Fin \n"); system("pause"); exit(1); break;
         case 2 : printf("2 \n");recommencer=0; break;
         case 3 : printf("3 \n");recommencer=0; break;
         case 4 : printf("4 \n");recommencer=0; break;
         default : printf("Commande non valide \n");recommencer=1;break;      
       }
	   ;
     }
  while(recommencer);
	
  system("pause");
	
  return EXIT_SUCCESS;
}

[kamouminator]

heu....en fait je crois que c'est deconceillé (histoire de normes)...meme si ca marche...
essaye plutot ca:

Code :

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

1
2
int c;
while ((c = getchar()) != '\n' && c != EOF);

[Emmanuel Delahaye]

Code :

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

if (scanf ("%d", &choix) != 1)

ne fonctionne toujours pas, j'ai toujours une boucle infinie si je rentre autre chose qu'un entier

Ben oui, c'est normal :

Code :

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

/* L'operateur c'est trompe. Il va falloir purger, sinon c'est le drame... */

je ne vois pas ce que veut dire le terme "purger" dans ce contexte...

Lorsque l'opérateur entre autre chose que ce qui est attendu, la conversion échoue et la fonction ne retourne pas 1. De plus, comme la conversion s'est arrêtée à la première non conformité, les caractères suivants ne sont pas lus et sont toujours dans le flux stdin. Mais comme un <enter> a été tapé, aucun des appels suivants ne sera bloquant et c'est la boucle infernale.

C'est pourquoi dès que l'erreur a été détectée, il faut purger le flux, c'est à dire lire les caractères qui restent dans le flux jusqu'à trouver un '\n' (matérialisation du <enter>) ou une indication de fin de lecture (EOF).

Code :

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

1
2
3
4
5
6
7
 
   int c;
   do
   {
      c = fgetc (stdin);
   }
   while (c  != '\n' && c != EOF);

ou, si on est un peu plus habitué au coté 'compact' du langage C :

Code :

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

1
2
3
4
 
   while ((c = fgetc (stdin)) != '\n' && c != EOF)
   {
   }

donc, voilà ce qu'il faut mettre dans la branche 'erreur'.

[tomasi]

merci pour ces explications