[C] problème avec scanf()

Recherche :

Mot : Pseudo : Filtrer
Bas de page
Auteur	Sujet : [C] problème avec scanf()

dr4g0n

amateur de princesse en sauc

Bonjour a tous.
Voila j'ai un petit souci
je demande un numéro a l'utilisateur compris entre 1 et 1000
je test la valeur entrée et je lui répond
Seul problème lorsque l'on rentre une lettre au lieu d'un chiffre mon programme boucle et plante ... il ne redemande plus la valeur et boucle
une idée pour eviter cela ?
Pour complément je suis sous un cygwin et je compile avec gcc.
ci dessous mon code

Code :

#include <stdio.h>
#include <signal.h>
int i;
int nbuser;
verifentree(nbuser,petit,grand){
if ( nbuser < 1 || nbuser > 1000){
printf(" votre nombre n'est pas dans la fourcette \n" );
//comptcoup();
i = 0;
return i, nbuser;
}
else {
printf("votre nb est bien dans la fourchette attendu \n il va etre envoyé au serveur \n" );
//comptcoup();
i = 1;
return i;
}
}
main() {
//int nbuser;
do{
printf(" PERE >> Entrez votre nombre entre 1 et 1000 \n" );
scanf("%d", &nbuser);
printf(" vous avez saisie la valeur %d \n", nbuser);
verifentree(nbuser);
//printf(" coup = %i",coup);
}
while (i != 1);
printf("PERE >> ok \n" );
}

Publicité

Joel F

Real men use unique_ptr

ne pas utiliser scanf mais fgets+strtol ou sscanf qui permettent de controler l'entrée

Elmoricq

Modérateur

Toutes les fonctions ...scanf() (scanf, fscanf, sscanf) ne doivent être utilisées que si l'on a conscience de deux choses : le pattern de lecture doit être conçu de façon à gérer tous les cas, et il faut vérifier le code retour pour gérer les problèmes de lecture.

Il vaut donc mieux utiliser fgets(), couplé à une fonction de conversion : strtol, strtod et consors.
On peut aussi utiliser fgets+sscanf comme le suggère Joel F, mais je trouve que c'est sortir l'artillerie lourde lorsqu'il n'y a qu'une seule valeur à convertir, sans formatage particulier, comme ici.

Trap D

Prend un livre de C et corrige ce code.
scanf n'est pas une fonction pour les débutants, encore moins pour des entrées utilisateurs.

Elmoricq

Modérateur

Bon sinon, en dehors de scanf, il y a des problèmes avec ton code :

Citation :

taiste.c:5: warning: return type defaults to `int'
taiste.c: In function `verifentree':
taiste.c:10: warning: left-hand operand of comma expression has no effect
taiste.c: At top level:
taiste.c:19: warning: return type defaults to `int'
taiste.c: In function `main':
taiste.c:30: warning: control reaches end of non-void function

Utilise l'option "-Wall" de gcc pour voir ces avertissements.

Code :

int i;
int nbuser;

À moins de savoir vraiment ce que tu fais et d'avoir une très bonne raison pour ça, ce qui n'est pas le cas ici, n'utilise pas de variable globale. C'est mal.

Code :

verifentree(nbuser,petit,grand){

Manquent les types de retour et des arguments.
Par défaut le compilateur assume qu'il s'agit d'entiers (int), mais c'est mal de laisser l'ambiguïté s'installer.

Code :

return i, nbuser;

Cette ligne ne fait pas ce que tu crois.

Elle signifie : "retourner i, puis évaluer nbuser" (ce qui n'arrive jamais).
Une fonction ne peut jamais retourner qu'une seule valeur.

Code :

main() {

Les deux seuls prototypes de main() valides sont :
int main(void)
et
int main(int argc, char **argv)

Code :

}

main() est censé retourner une valeur int, pour indiquer à l'appelant du programme dans quel état est sorti le programme (on assume qu'un code retour à 0 signifie que tout s'est bien passé).
Donc il faut ajouter un return.

Message cité 1 fois

dr4g0n

amateur de princesse en sauc

merci a tous pour ces présicions.
Désolé d'etre brouillons je débute en C
( cours du cnam powa )

Trap D

Elmoricq a écrit :

Code :

return i, nbuser;

Cette ligne ne fait pas ce que tu crois.

Elle signifie : "retourner i, puis évaluer nbuser" (ce qui n'arrive jamais).
Une fonction ne peut jamais retourner qu'une seule valeur.

Tu en es sûr ??
J'aurais plutôt dit qu'elle retournait la valeur de nbuser.
Si je ne m'abuse, i est évalué, puis nbuser et c'est la valeur du dernier qui est retourné (à moins que la précédence de return soit plus élvée que celle de la virgule ?)

Elmoricq

Modérateur

Ah tiens, bonne question (même si au final c'est pas le genre de cas que tu rencontres souvent [:ddr555] ).

C'est facile d'avoir la réponse, voyons ça :

Code :

#include <stdio.h>
int blabla(void)
{
int i = 0, j = 1;
return i, j;
}
int main(void)
{
printf("And teh result is : %d\n", blabla());
return 0;
}

Bon déjà, ça part mal pour moi [:joce]

Citation :

taiste.c: In function `blabla':
taiste.c:7: warning: left-hand operand of comma expression has no effect

Exécutons le bazar :

$ a.out
1

J'ai perdu [:zytrasnif]

FORUM HardWare.fr

Programmation

[C] problème avec scanf()

Sujets relatifs
[Resolu] Ajout dynamique onclick et problème d'auto-execution	Probleme mise en ligne d'un site web
petit problème d'allignement	probleme de regexp
[opera] problème avec attribut title	Problème de tri A à L, M à Z,...
Problème d'affichage sur blog	Creation site, problème après tentative upload
[ MySQL ] - Problème avec GROUP BY - Help !	Problème de miniaturisation avec frontpage
Plus de sujets relatifs à : [C] problème avec scanf()

Page générée en 0.079 secondes