problème avec la fonction gets

Bonjour, j'ai un problème avec la fonction gets.
Apaprement, elle ne marche pas si avant je fais un scanf, ce qui me parait très bizarre :
Si je fais ce code là, pas de problème :

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14

#include <stdio.h> main() { int util1; char chaine[50]; FILE *sortie; int a=1; if (a==1){ sortie=fopen("bonjour","w"); fprintf(sortie,"BONJOUR\n"); gets(chaine); fprintf(sortie," %s\n",chaine); } }

---

Mais si je fais un scanf juste avant, ça ne marche pas : la chaine n'est pas demandée à l'utilisateur, alors que bonjour est écrit dans le fichier (ce qui prouve qu'on est bien rentré dans la boucle).

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15

#include <stdio.h> main() { int util1; char chaine[50]; FILE *sortie; int a; scanf("%d",&a); if (a==1){ sortie=fopen("bonjour","w"); fprintf(sortie,"BONJOUR\n"); gets(chaine); fprintf(sortie," %s\n",chaine); } }

---

Est ce que vous pouvez m'aider? Merci!

Bonjour,

pense à la balise :tagcode:

sinon j'ai rien compris c'est quoi le problème :roll: , normalement tu as intialisé la variable a , où se situe le problème 8O

turbo_chess

Il reste le '\n' qui n'a pas été ôté par scanf() dans le buffer d'entrée. gets() le trouve et considère que la chaîne est déjà entrée et est vide.

Une solution consiste à écrire:

Code:

---

scanf("%d\n",&i);

---

d'accord, merci beaucoup

Mais je ne comprends pas la solution que vous avez donné, car alors, on ne rentre plus dans la boucle.
Je ne comprends vraiment pas comment je peux faire

Citation:

---

Envoyé par Laurran

Mais je ne comprends pas la solution que vous avez donné, car alors, on ne rentre plus dans la boucle.
Je ne comprends vraiment pas comment je peux faire

---

De quelle boucle parlez-vous?

et aussi tu parles de fgets mais tu utilises gets.....

Je n'ai rien vu ici qui parle de fgets().

Mais gets() est un bug et ne doit pas être utilisée du tout. Utiliser fgets() à la place (même si ça n'aurait pas résolu le problème du P.O.).

PS: Quand sera-t-elle enfin officiellement deprecated ?

Citation:

---

Envoyé par Médinoc

Je n'ai rien vu ici qui parle de fgets().

---

Opps autant pour moi :oops: c'était tellement évident pour moi (fopen..)....

Mais je ne veux pas utiliser de fichier dans ce cas là, car je veux que ce soit l'utilisateur qui choisisse.
Alors comment faire pour lire un entier à l'aide de scanf puis une chaine (avec espaces autorisés)???

Merci

Citation:

---

Envoyé par Laurran

Alors comment faire pour lire un entier à l'aide de scanf puis une chaine (avec espaces autorisés)???

---

Comme je l'ai déjà écrit, voici une solution possible:

Code:

---

1 2 3 4 5 6 7 8 9 10 11 12 13

#include <stdio.h>     int main(void) { char s[50]; int i;   scanf("%d\n",&i); gets(s); printf("%d\t%s\n",i,s); return 0; }

---

Et Comme je l'ai déjà écrit, on n'utilise pas gets() :

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 41 42

#include <stdio.h> #include <string.h>   void purge(void) { int c; while( (c=getchar()) != '\n' && c != EOF) { } }   void purgeString(char *s) { /* Soit il y a un \n dans la chaîne, soit il est toujours dans stdin */ char * pLF = strchr(s, '\n'); if(pLF != NULL) { /* Il est dans la chaîne, on le supprime. */ *pLF = '\0'; } else { /* Il est toujours dans stdin, on vide la ligne */ purge(); } }     int main(void) { char s[50]; int i;   scanf("%d", &i); purge();   fgets(s, 50, stdin); purgeString(s);   printf("%d\t%s\n",i,s); return 0; }

---

Citation:

---

Envoyé par Médinoc

Et Comme je l'ai déjà écrit, on n'utilise pas gets() :

---

C'est bon; ici il s'agit d'un programme tout simple où l'utilisateur et le programmeur sont la même personne. Alors toutes ces précautions sont un peu superflues. Si l'utilisateur/programmeur est assez bête pour rentrer trop de caractères et que ça plante, tant pis pour lui. En outre, les dégâts causés par un éventuel plantage ne devraient pas être trop dramatiques.

C'est vrai qu'on ne code pas sur une DeathStation 9000, mais c'est un coup à prendre...

Et ce n'est pas "toutes ces précautions", la seule précaution ici est d'utiliser fgets() à la place de gets().
Le reste est nécessaire pour résoudre le problème originel : Que gets() ou fgets() marche même si on ne tape rien de valide pour scanf().

Citation:

---

Envoyé par Médinoc

Et ce n'est pas "toutes ces précautions", la seule précaution ici est d'utiliser fgets() à la place de gets().
Le reste est nécessaire pour résoudre le problème originel : Que gets() ou fgets() marche même si on ne tape rien de valide pour scanf().

---

J'ai écrit "toutes" car les fonctions de purge ne sont pas nécessaires; ceci fonctionne:

Code:

---

scanf("%d\n",&i);

---

edit: Je suis d'accord que pour des programmes plus sérieux, il convient d'utiliser des routines d'entrée plus sécurisées. Mais pour de simples programmes console comme celui-ci, il ne me semble pas important de gérer ce genre de cas.

Perdu, si on entre une bêtise au lieu de "rien du tout", la seconde question n'est pas posée:

Code:

---

1 2 3 4 5 6

a = -10; puts("Entrez a :"); scanf("%d\n", &a); puts("Entrez buf :"); gets(buf); /* Oui, gets(). Pour être sûr. */ printf("%d\t%s\n",a, buf);

---

Code:

---

1 2 3 4

Entrez a : x Entrez buf : -10 x

---

Ici, le printf() a eu lieu sans que je puisse taper quoi que ce fût dans "entrez buf" --> Problème originel non-résolu.

Et c'est encore plus marrant si on fait une boucle de scanf :

Code:

---

1 2 3 4 5 6 7 8

a = 0; do { puts("Entrez a entre 1 et 10:"); scanf("%d\n", &a); printf("Vous avez saisi : %d\n.", a); } while( a < 1 || a > 10);

---

Et tu tapes "x".

Merci beaucoup pour toutes vos réponses.
Je ne suis pas sure d'avoir tout bien compris lorsqu'on n'utilise pas gets, mais avec gets ça marche.
Merci beaucoup

Ça marche autant avec gets() qu'avec fgets(), sauf quand on tape plus que le buffer.

Et aussi, fgets() ne retire pas tout seul le \n, c'est à ça que sert la fonction purgeString().

Citation:

---

Envoyé par Médinoc

Ici, le printf() a eu lieu sans que je puisse taper quoi que ce fût dans "entrez buf" --> Problème originel non-résolu.

---

Selon moi, le problème originel n'est pas de permettre à un crétin d'utiliser le programme, mais juste d'ôter le '\n' du buffer d'entrée. Si l'utilisateur s'amuse à entrer des caractères quand un entier est attendu, c'est son problème. En tout cas, je ne pense pas que c'était la raison de ce thread.

Tu as partiellement raison, j'avais mal lu.
Déformation professionnelle due à l'habitude du "test du singe"...

Citation:

---

Envoyé par Médinoc

Déformation professionnelle due à l'habitude du "test du singe"...

---

Je ne saisis pas là...

Tu colles un singe (ou /dev/random) devant le clavier et tu regardes si ça crashe ou non.

OK, merci pour l'explication. Je ne pense pas que ce programme-ci aura à passer ce genre de tests ;) .

Citation:

---

Envoyé par Laurran

Merci beaucoup pour toutes vos réponses.
Je ne suis pas sure d'avoir tout bien compris lorsqu'on n'utilise pas gets, mais avec gets ça marche.
Merci beaucoup

---

Comme l'a expliqué Médinoc, fgets() permet une utilisation plus sécurisée en cela que cette fonction prend comme paramètre une limite pour le buffer. Cela évite les dépassements.

Je suis désolée mais il doit me manquer encore une astuce. Le code suivant ne marche pas :(

Code:

---

1 2 3 4 5 6 7 8 9 10

int main() { char s[50]; int i; printf("Entrez un nombre\n"); scanf("%d\n",&i); printf("Entrez la chaine\n"); gets(s); printf("%d\t%s\n",i,s); }

---

Citation:

---

Envoyé par Laurran

Je suis désolée mais il doit me manquer encore une astuce. Le code suivant ne marche pas :(

Code:

---

1 2 3 4 5 6 7 8 9 10

int main() { char s[50]; int i; printf("Entrez un nombre\n"); scanf("%d\n",&i); printf("Entrez la chaine\n"); gets(s); printf("%d\t%s\n",i,s); }

---

---

Il faut qu'on m'explique... Qu'est-ce qui ne va pas avec la méthode proposée par Médinoc? Elle a le mérite d'être simple et de fonctionner (on peut encore contrôler la valeur de retour de scanf()). Perso, j'éliminerai également le 1er scanf() au profit de fgets/strtol ou fgets/sscanf. C'est peut-être un usine à gaz pour un code aussi basique, mais c'est dans ce genre de codes simplistes qu'on prend de bons réflexes. En tous les cas, je ne vois rien pour justifier l'utilisation de gets(), même dans un exemple rapide.

Thierry

La méthode marche très bien si je la copie telle qu'elle est là.
Mais si je rajoute un printf entre les deux ça ne marche pas, et ça ne marche pas non plus si on enlève le \n.
Merci

Citation:

---

Envoyé par Laurran

Bonjour, j'ai un problème avec la fonction gets.
Apaprement, elle ne marche pas si avant je fais un scanf, ce qui me parait très bizarre :

---

gets() est un bug qu'il ne faut pas utiliser. scanf() est difficile à utiliser correctement (la preuve).

Je conseille d'utiliser uniquement et correctement fgets().

http://emmanuel-delahaye.developpez.com/inputs.htm

Citation:

---

Envoyé par Laurran

La méthode marche très bien si je la copie telle qu'elle est là.
Mais si je rajoute un printf entre les deux ça ne marche pas, et ça ne marche pas non plus si on enlève le \n.
Merci

---

Là, je dois bien m'avouer vaincu :oops: ! Le code de Médinoc fonctionne à la perfection, mais pas le mien. Les printf() utilisés font apparaître que mon raisonnement est faux. Je pensais que:

Code:

---

scanf("%d\n",&i);

---

lisait un entier, ôtait le '\n' du buffer d'entrée et retournait. Apparemment, il n'en est rien. Ca bloque tant que l'on ne tape pas quelque chose d'autre. Ce comportement avait été caché dans mon code par l'absence de printf(). Personnellement, je ne comprends pas trop pourquoi ça bloque...
La meilleure solution est donc d'utiliser le code de Médinoc. Le programme donné par Laurran peut fonctionner en utilisant la méthode "purge" de Médinoc:

Code:

---

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

#include <stdio.h>     int main(void) { char s[50]; int i,c;   printf("Entrez un nombre\n"); scanf("%d",&i); while ((c=getchar())!='\n' && c!=EOF); printf("Entrez la chaine\n"); gets(s); printf("%d\t%s\n",i,s); return 0; }

---

Néanmoins, le code de Médinoc fait prendre de bonnes habitudes et est plus robuste.
Médinoc, je pense que vous pouvez modifier votre post#22 en "Tu as tort,..." ;) .

Merci!!