problème avec la fonction gets

Version imprimable

Voir 40 message(s) de cette discussion en une page

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!

22/03/2007, 09h31
turbo_chess

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
22/03/2007, 09h33
stephl

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.
22/03/2007, 09h36
stephl

Une solution consiste à écrire:

Code:

scanf("%d\n",&i);
22/03/2007, 11h22
Laurran

d'accord, merci beaucoup
22/03/2007, 11h25
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
22/03/2007, 11h33
stephl

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?
22/03/2007, 11h42
souviron34

et aussi tu parles de fgets mais tu utilises gets.....
22/03/2007, 13h59
Médinoc

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 ?
22/03/2007, 14h02
souviron34

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..)....
22/03/2007, 14h42
Laurran

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
22/03/2007, 14h46
stephl
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;
}

22/03/2007, 15h00
stephl

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.
22/03/2007, 15h05
Médinoc

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().
22/03/2007, 15h15
stephl

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.
22/03/2007, 15h31
Médinoc
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".

22/03/2007, 15h38
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
22/03/2007, 15h40
Médinoc

Ç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().

Voir 40 message(s) de cette discussion en une page