Savoir si une variable est bien un chiffre

Version imprimable

13/11/2006, 22h25
tomasi

Savoir si une variable est bien un chiffre
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:

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"
13/11/2006, 22h43
Trap D

isdigit(var) déclaré dans ctype.h devrait faire l'affaire.
13/11/2006, 22h50
Emmanuel Delahaye
Citation:
Envoyé par 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:

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.

Citation:

Envoyé par 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:

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

13/11/2006, 23h03
Emmanuel Delahaye

Citation:

Envoyé par tomasi

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.
13/11/2006, 23h11
tomasi
je ne comprends pas bien comment tester la valeur de retour de scanf()

en faisant
Code:

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

et ensuite "purger" ?
là je suis complètement perdu
13/11/2006, 23h16
Emmanuel Delahaye
Citation:
Envoyé par tomasi

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

en faisant

Code:

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...
13/11/2006, 23h19
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
13/11/2006, 23h29
Emmanuel Delahaye

Citation:

Envoyé par tomasi

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:

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
13/11/2006, 23h47
tomasi

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

Code:

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

cela ne fonctionne pas
13/11/2006, 23h55
Emmanuel Delahaye
Citation:

Envoyé par tomasi

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

Code:

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:

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

Code:

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...
14/11/2006, 00h22
kamouminator

tu devrais montrer ton code "corrigé" !

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:

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

ce que je ferai moi:

Code:

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

14/11/2006, 00h34
tomasi
Citation:
Envoyé par kamouminator

ce que je ferai moi:

Code:

1 2 3 if (scanf ("%d", &choix) != 1) return 0; return choix;
hélas, cela ne fonctionne pas

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
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;
}

14/11/2006, 00h54
kamouminator
heu....en fait je crois que c'est deconceillé (histoire de normes)...meme si ca marche...
essaye plutot ca:
Code:

1 2 int c; while ((c = getchar()) != '\n' && c != EOF);
14/11/2006, 08h51
Emmanuel Delahaye
Citation:

Envoyé par tomasi

Code:

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:

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

Citation:

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:

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:

1 2 3 4 while ((c = fgetc (stdin)) != '\n' && c != EOF) { }
donc, voilà ce qu'il faut mettre dans la branche 'erreur'.
14/11/2006, 08h57
tomasi

merci pour ces explications
14/11/2006, 08h58
Emmanuel Delahaye
Citation:
Envoyé par kamouminator

ce que je ferai moi:

Code:

1 2 3 if (scanf ("%d", &choix) != 1) return 0; return choix;
Ben non. Tu n'as rien purgé du tout. Si on ne sait pas utiliser scanf(), le mieux est d'apprendre à le faire ou de ne pas l'utiliser.
14/11/2006, 09h08
Emmanuel Delahaye

Citation:

Envoyé par kamouminator

Code:

fflush(stdin); <-----correction

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

La norme dit que le comportement de fflush() n'est défini que pour les flux sortants. Or stdin est à l'évidence un flux entrant. Le comportement n'est donc pas défini. C'est tout. Le code est donc incorrect.

Le fait que 'ça marche' est une des possibilité d'un comportement indéfini. Ca ne signifie pas que le code est correct.

(En fait, sous Linux, "ça ne marche pas" et des cas de plantages ont été rapportés).

j'essaye donc de la même façon de ne prendre qu'une chaine de caractères minuscules dans la variable chaine

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
int saisie(){
  char chaine[30];
  int ok = 0;
  int retour;
 
  printf("Chaine en minuscules à inserer ? \n");
 
  while ( !ok ){
 
    retour = scanf("%s%[a-z][^\n]", chaine);
    printf("retour : %d\n", retour);
    if ( !retour ){
      /* erreur de saisie, on vide le flux */
      int c;
      while ( ((c = getchar()) != '\n') && c != EOF);
      printf("Perdu, il faut rentrer uniquement des minuscules ! \n");
      printf("Une autre chance ? \n");
    }
    else {
      /* reussite de la saisie */
      getchar(); /* on enleve le '\n' restant */
      printf("saisie acceptee\n");
      ok = 1;  /* sort de la boucle */
      printf("%s", chaine);
    }
  }
 
     return 1;
}

mais il me retourne une erreur de segmentation et il accepte n'importe quelle chaîne...

14/11/2006, 09h49
Médinoc

C'est normal que tu ne limites pas la taille ?
14/11/2006, 09h54
tomasi

la taille de la chaine ?
14/11/2006, 09h57
Médinoc

Oui. Je ne vois rien pour la limiter dans le scanf()...
PS: Tu as sûr qu'il faut deux "%" dans le format ?
14/11/2006, 10h02
Emmanuel Delahaye

Citation:

Envoyé par tomasi

j'essaye donc de la même façon de ne prendre qu'une chaine de caractères minuscules dans la variable chaine

Code:

retour = scanf("%s%[a-z][^\n]", chaine);

Je vois 2 "%" et 1 variable... C'est là que ça dépasse ma compréhension et que j'utilise fgets(), islower() etc.
15/11/2006, 00h28
tomasi

en testant la valeur de sortie ?

je peut utiliser islower pour convertir des majuscules si jamais l'utilisateur en rentre mais que faire pour les autres caractères ? (entiers, ect...)
15/11/2006, 00h39
Emmanuel Delahaye

Citation:

Envoyé par tomasi

en testant la valeur de sortie ?

kékidi ?

Citation:

je peut utiliser islower pour convertir des majuscules

8O 8O 8O islower() ne convertit rien. Il teste. Pour convertir, c'est tolower()...

Citation:

si jamais l'utilisateur en rentre mais que faire pour les autres caractères ? (entiers, ect...)

C'est trop pour mon neurone, là. Ca part dans tout les sens...

Alors 1 problème à la fois. Que veux-tu faire exactement ?
15/11/2006, 10h35
tomasi

tester si la chaine rentrée par l'utilisateur est bien une chaine de caractères minuscules

tester si on a aucune majuscule (on considere qu'on peut avoir des caracteres speciaux):

Code:

1
2
3
4
5
6
7
8
9
10
 
int IsMaj(char *str)
{
    for (int i = 0; str[i]; i++)
    {
         if (str[i]>= 'A' && str[i] <= 'Z')
              return (-1);
    }
     return (1);
}

Bonjour,

je vais rajouter mon grain de sel

Code:

1
2
3
4
5
6
7
8
9
 
int isLowerStr(const char *s) {
 
  while (*s && islower(*s)) {
     s++;
  }
 
  return (*s == 0);
}

Pour les fainéant de mon genre:

Code:

1
2
3
4
5
6
7
8
 
int checkCharTypeStr(const char *s, int (*ctype)(char)) {
 
   while (*s && ctype(*s)) {
      s++;
   }
   return (*s == 0);
}

on utilise de cette manière:

Code:

1
2
3
4
5
 
  char *text = "le papa de marie fume la pipe";
 
  checkCharTypeStr(text, islower);
  checkCharTypeStr(text, isalpha);

15/11/2006, 11h43
Emmanuel Delahaye
Citation:
Envoyé par toxycyty

Code:

1 2 if (str[i]>= 'A' && str[i] <= 'Z')
Pas portable. Il y a isupper() pour ça...
15/11/2006, 11h44
Emmanuel Delahaye

Citation:

Envoyé par tomasi

tester si la chaine rentrée par l'utilisateur est bien une chaine de caractères minuscules

Une boucle avec islower() de <ctype.h>
15/11/2006, 13h15
tomasi

merci la vérification fonctionne bien

Fuseau horaire GMT +2. Il est actuellement 06h35.