Problème de lecture d'un entier au clavier

Bonsoir,
J'ai remarqué que la lecture au clavier, en langage C, pose souvent des problèmes, surtout dans le cas de la lecture d'entiers : quand on doit entrer un entier au clavier, et que celui qui execute le programme tape quelque chose avec au moins une lettre,l'entier prend une valeur bizarre(toujours la même : 1075028800).
Mais c'est encore pire quand la lecture de l'entier se situe dans une boucle : dans ce cas,il suffit de lire une fois une mauvaise valeur au clavier pour le programme boucle ensuite indéfiniment sans jamais rendre la main à l'utilisateur.

Le programme ci-dessous est censé lire un entier au clavier jusqu'à ce que cette valeur lue soit égale à 0.

Code:

---

1 2 3 4 5 6 7 8 9

#include<stdio.h>   main() { int entier; float reel; char caractere;char* chaine; do { puts("Entrez un entier : 0 pour arreter"); scanf("%d",&entier); printf("entier = %d\n",entier); } while(entier); exit(0);

---

Lorsque je l'execute et que j'entre,par exemple, abc, voici les resultats de l'execution :

Code:

---

1 2 3 4 5 6 7 8 9 10 11

Entrez un entier : 0 pour arreter entier = 1075028800 Entrez un entier : 0 pour arreter entier = 1075028800 Entrez un entier : 0 pour arreter entier = 1075028800 Entrez un entier : 0 pour arreter entier = 1075028800 Entrez un entier : 0 pour arreter entier = 1075028800 ............................................. (et ceci des centaines de fois)

---

En Java, ce problème est evitable grace à l'exception declenchée lors de l'appel de la méthode parseInt,mais en C, je ne vois pas du tout comment résoudre le problème. Si qqn pouvait me dépanner...
Merci d'avance.

Tu devrais fouiller un peu sur ce forum, celà a déjà été évoqué mille fois : il ne faut jamais utiliser scanf !!!
La lecture ce fais de cette façon :

Code:

---

1 2 3 4 5 6

char buflec[256]; // ou autre longueur long int val; // on lit les caractères dans un buffer fgets(buflec, sizeof(buflec), stdin); // on étudie la saisie avec strtol, seule fonction adéquate pour transformer une chaîne en nombre val = strtol(buflec, NULL, 10);

---

Tu peux approfondir l'étude de la chaîne en passant un deuxième paramètre à strtol, vois les détails sur l'aide en ligne de strol.

Je suis pas d'accord pourquoi s'embrouyer avec un buffer, un simple

Code:

---

scanf("%d%*c", &entier);

---

Et tout va bien plus de probleme de retour chariot.

Citation:

---

Envoyé par Yakurena

J'ai remarqué que la lecture au clavier, en langage C, pose souvent des problèmes

---

Si on utilise pas les bonnes fonctions, oui. Sinon, ça fonctionne très bien :

http://emmanuel-delahaye.developpez....tes.htm#saisie
http://emmanuel-delahaye.developpez....s.htm#fichiers

http://emmanuel-delahaye.developpez.com/clib.htm
Module IO

Citation:

---

Envoyé par Skyrunner

Je suis pas d'accord pourquoi s'embrouyer avec un buffer, un simple

Code:

---

scanf("%d%*c", &entier);

---

Et tout va bien plus de probleme de retour chariot.

---

Si tu es un expert de scanf(), je te suggère d'écrire un mode d'emploi détaillé et de l'ajouter aux documents du forum.

Citation:

---

Envoyé par Emmanuel Delahaye

Si tu es un expert de scanf(), je te suggère d'écrire un mode d'emploi détaillé et de l'ajouter aux documents du forum.

---

Moi ça m'intéresse. Pas de l'écrire, de le lire.
Les fonctions ...scanf m'inspirent toujours aussi peu confiance, je ne sais pas les utiliser. :oops:

Citation:

---

Envoyé par Gamdwin

Citation:

---

Envoyé par Emmanuel Delahaye

Si tu es un expert de scanf(), je te suggère d'écrire un mode d'emploi détaillé et de l'ajouter aux documents du forum.

---

Moi ça m'intéresse. Pas de l'écrire, de le lire.
Les fonctions ...scanf m'inspirent toujours aussi peu confiance, je ne sais pas les utiliser. :oops:

---

Pas de quoi rougir, moi c'est pareil :wink:

Salut

Je suis debutant en C et je voudrais savoir pourquoi il en faut pas utiliser scanf(); ?

Moi je l'utilise et je n'est aucun problème.

Merci

Citation:

---

Envoyé par phoenix440

Je suis debutant en C et je voudrais savoir pourquoi il ne faut pas utiliser scanf(); ?

---

On peut l'utiliser si on maîtrise. Problème, très peu de gens maitrisent scanf(), qui est une fonction extrêmement complexe et certainement pas pour débutants.

Citation:

---

Moi je l'utilise et je n'est aucun problème.

---

Ok. Montre comment tu résoudrais le problème posé par le posteur original.

Même si il y a des façons de bidouiller scanf() pour obtenir de bons résultats, pourquoi perdre ce temps et ne pas utiliser fgets() ?

Pour limiter la saisie à 10 caractères en prenant en compte les espaces, on peut faire :

Code:

---

1 2	char test[10]; scanf("%9[^\n]s",test);

---

printf() donne la possibilité de fixer le nombre de caractère à afficher grâce à une variable (donc plus pratique que de mettre en dur la valeur). Avec scanf(), ca ne semble pas marcher...

Nas'

Citation:

---

Envoyé par Skyrunner

Je suis pas d'accord pourquoi s'embrouyer avec un buffer, un simple

Code:

---

scanf("%d%*c", &entier);

---

Et tout va bien plus de probleme de retour chariot.

---

Mouais... Mais si je fais une saisie de ce genre la :

Citation:

---

123a

---

Le '\n' reste dans le buffer d'entrée, et on boucle toujours....

Bref, pour en revenir au code initial, je propose la solution suivante :

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

#include <stdio.h> #include <stdlib.h>   int main(void){   int entier = 1; do { puts("Entrez un entier : 0 pour arreter"); if ( ! scanf("%d%*[^\n]%*c",&entier) ){ char c; /* erreur d'entree */ fprintf(stderr, "erreur de format, veuillez resaisir un nombre entier\n"); /* de plus, le buffer n'a pas ete vide (echec de scanf...) */ while ( (c = getchar() ) && c != '\n'); } else { printf("entier = %d\n",entier); } } while(entier); exit(0);   printf("vous etes sortis\n");   return 0; }

---

On remarque plusieurs choses.
Tout d'abord, la ligne

Code:

---

scanf("%d%*[^\n]%*c",&entier)

---

permet de capturer l'entier saisi, ET de vider le buffer d'entree.
Malgré cela, si la scanf ECHOUE (sur le format d'entree), il ne poursuit pas les conversions suivantes : le buffer ne sera pas vidé !! il faut le faire manuellement dans le cas d'erreur du retour de scanf, avec

Code:

---

while ( (c = getchar() ) && c != '\n');

---

:arrow: Cela sous-entend qu'il faut a tout prix tester le retour de scanf (surtout quand on cherche à récuperer autre chose que des chaînes de caractères (cas plus permissif) ).

Par ailleurs, je n'ai pas traité dans mon code les problèmes suivant, qui peuvent (et doonc doivent être traités) suivants :
1) l'entier saisi est supérieur à INT_MAX
2) les saisies de la forme "123azeazeaze" sont considérées comme correctes (on récupère 123), ce qui n'est pas forcément le comportement attendu..

Bref, pour ce genre de saisie, scanf est loin d'être évident à utiliser (correctement en tout cas ;) )

Citation:

---

Envoyé par Nasky

Même si il y a des façons de bidouiller scanf() pour obtenir de bons résultats, pourquoi perdre ce temps et ne pas utiliser fgets() ?

Pour limiter la saisie à 10 caractères en prenant en compte les espaces, on peut faire :

Code:

---

1 2	char test[10]; scanf("%9[^\n]s",test);

---

printf() donne la possibilité de fixer le nombre de caractère à afficher grâce à une variable (donc plus pratique que de mettre en dur la valeur). Avec scanf(), ca ne semble pas marcher...

Nas'

---

On peut bien évidemment faire de même avec scanf, en utilisant à fond les avantages du préprocesseur.

Exemple :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19

#include <stdio.h>   /* twin macros : do not separate */ #define xstr(s) str(s) #define str(s) #s /* ***************************** */   #define TAILLE 80 #define S_TAILLE xstr(TAILLE)   int main(void) {   char tab[TAILLE] = {0}; scanf("%"S_TAILLE"s", tab); <...>   return 0; }

---

On utilise deux define afin d'expanser la macro passée ET de la transformer en chaine de caractere (sinon ca ne marche pas ;) )
Cette solution est plus "propre".

Mais on aurait pu bêtement faire :

Code:

---

1 2 3

#define TAILLE 80 #define S_TAILLE "80"

---

Ce qui est pénible en cas de maintenance...

<edit>
Je viens de lire les posts que j'avais "sauté", et je veux bien (sans pour autant me prétendre expert) écrire une documentation sur l'utilisation de scanf..
(Par contre, ça viendra quand mon stage m'en laissera le temps :P )
</edit>