Discussion :

[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 :
Si je fais ce code là, pas de problème :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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!

[turbo_chess]

Bonjour,

pense à la balise

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

turbo_chess

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

[stephl]

Une solution consiste à écrire:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[Laurran]

d'accord, merci beaucoup

[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

[stephl]

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?

[souviron34]

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

[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 ?

[souviron34]

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

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

[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

[stephl]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[Médinoc]

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

Code C :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[stephl]

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.

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

[stephl]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[Médinoc]

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

Code C :

Sélectionner tout - Visualiser dans une fenêtre à part

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 sortie :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[Médinoc]

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[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

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