Discussion :

[Changedman]

Question surement très simple pour la plupart d'entre vous mais dont je ne connais pas la reponse, comment faire pour prendre une chaine contenant un espace avec un scanf

Ex :

Code :

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

1
2
3
4
5
 
...
...
scanf("%s", nom);
...

Où "nom" est une chaine contenant un espace.

Merci d'avance pour vos reponses.

[Thierry Chappuis]

Voici une manière sécurisée de saisir une ligne entrée par l'utilisateur avec scanf():

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
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
43
44
45
46
#include <stdio.h>
#include <stdlib.h>
 
void vider_stdin(void)
{
    scanf("%*[^\n]");
    getchar();
}
 
int main(void)
{
    int c;
    int saisie_OK = 0;
    char s_buffer[51] = {0};
 
    while (saisie_OK != 1)
    {
        printf("Entrez une phrase: ");
        fflush(stdout);
 
        if (scanf("%50[^\n]", s_buffer) == 1)
        {
            /* La saisie avec scanf() a fonctionné */
            if ( (c = getchar()) == '\n')
            {
                saisie_OK = 1;
            }
            else
            {
                /* saisie trop longue */
                fprintf(stderr, "La saisie est trop longue!\n");
                vider_stdin();
            }
        }
        else
        {
            fprintf(stderr, "Erreur durant la saisie\n");
            vider_stdin();
        }
    }
 
    puts(s_buffer);
 
 
    return EXIT_SUCCESS;
}

Ce code est inspiré de l'excellent tutoriel de Xavier Renault http://xrenault.developpez.com/tutoriels/c/scanf/

Lorsqu'il s'agit de saisir des chaînes de caractères, je préfère toutefois utiliser la fonction fgets():

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
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
void vider_stdin(void)
{
    scanf("%*[^\n]");
    getchar();
}
 
int main(void)
{
    char *pc;
    char s_buffer[51] = {0};
    int saisie_OK = 0;
 
    while (saisie_OK == 0)
    {
        printf("Entrez une phrase: ");
        fflush(stdout);
 
        fgets(s_buffer, sizeof s_buffer, stdin);
        if ((pc = strchr(s_buffer, '\n')) != NULL)
        {
            *pc = '\0';
            saisie_OK = 1;
        }
        else
        {
            /* saisie trop longue */
            fprintf(stderr, "La saisie est trop longue!\n");
            vider_stdin();
        }
    }
 
    puts(s_buffer);
 
    return EXIT_SUCCESS;
}

Thierry

[Emmanuel Delahaye]

Question surement très simple pour la plupart d'entre vous mais dont je ne connais pas la reponse, comment faire pour prendre une chaine contenant un espace avec un scanf

Code :

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

1
2
3
4
5
 
...
...
scanf("%s", nom);
...

Où "nom" est une chaine contenant un espace.

Si tu n'est pas un expert de scanf(), je te conseille d'utiliser plutôt fgets() ou de fabriquer ta propre fonction d'entrée :

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

Fabriqué avec 100% de caractères recyclés.

[Changedman]

Etant au debut de mon apprentissage du C, je dois avouer que ceci est assez complexe pour moi.

Je vous poste mon code, c'est une application très simple ayant pour but d'ajouter des signets à la PSP.
Attention, il se peut que ça pique les yeux je ne pense pas que le code soit très propre.

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
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
43
44
45
46
47
48
/* Par Changedman le 21/02/07, créé pour planete-psp, le site de toute l'actualité de la PSP
Mise à jour V0.70
Merci à developpez.com, un site parfait pour toutes vos questions sur tous les langages*/
 
#include <string.h>
#include <stdio.h>
#include <stdlib.h>
 
int main()
{
    char adresse[100];
    char nom[100];
    char Nom_Fichier[250];
    char lettre = 'C';
    long validation = 1;
    long compteur = 0;
    FILE* fichier = NULL;
 
        printf("Bookmarks Adder V0.70 By Changedman\n\n\n\n");
        printf("La lettre du lecteur de la PSP : ");
        scanf("%c", &lettre); // detection de l'emplacement de la PSP
        do
        {
        printf("Adresse a enregistrer : ");
        scanf("%s", adresse); //enregistrement de l'adresse
        printf("Nom a afficher dans le navigateur internet : ");
        scanf("%s", nom); //enregistrement du nom
        sprintf ( Nom_Fichier, "%c:\\PSP\\SYSTEM\\BROWSER\\bookmarks.html", lettre);
        fichier = fopen(Nom_Fichier, "a+"); //ouverture du fichier en mode ajout
 
            if (fichier != NULL)
        {
            fprintf(fichier, "<DT><A HREF=\"%s\" ADD_DATE=\"000000000%ld\" LAST_VISIT=\"000000000%ld\" LAST_MODIFIED=\"000000000%ld\" LAST_CHARSET=\"UTF-8\">%s</A></DL><p>  \n", adresse, compteur, compteur, compteur, nom); //balises HTMl ecrites dans le fichier
            fclose(fichier);
        }
        else
        {
            printf("Impossible d'ouvrir bookmarks.html\n"); // Message d'erreur
        }
        compteur++;
        printf("Appuyer sur 1 pour ajouter un nouveau signet ou sur 0 pour quitter : "); // propose à l'utilisateur d'en ajouter un autre
        scanf("%ld", &validation);
        }while (validation != 0);
 
        printf("\nA bientot et n hesitez pas a repasser"); // message d'au revoir (la politesse n'a jamais tué personne :p )
 
        return 0;
}

Ne comprenant pas entierement le code que vous m'avez fourni, je ne sais pas comment l'integrer à ce code. Pourriez-vous me le detailler, ce serait vraiment extraordinaire ou alors me dire comment l'integrer au code, j'essaierai d'en comprendre le fonctionnement ensuite.

EDIT : Posté avant lecture de la reponse, je vais m'interesser de plus près à fgets et ses avantages

EDIT n°2 : Je peux resoudre ce problème avec la methode des %c qui ne supprime pas les espaces automatiquement mais ce ne sera qu'une astuce et cela pourrait me manquer sur d'autres codes, je ne veux pas commencer à garder des lacunes.

[Melem]

Quand on utilise "%s" dans scanf() la lecture s'arrête à la rencontre d'un espace, d'une tabulation ou d'une fin de ligne. Si tu veux saisir une chaîne comportant des espaces tu dois utiliser gets() ou mieux fgets(). Exemple:

Code :

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

1
2
char s[20+1];
fgets(s, sizeof(s), stdin);

[Emmanuel Delahaye]

tu dois utiliser gets()

Une connerie par post. Tu vas t'arrêter un jour, ou il faut que j'en parle à Leon ?

[Melem]

tu dois utiliser gets()

Non Melem a écrit

tu dois utiliser gets() ou mieux fgets()

.

[Médinoc]

C'est suffisant pour être faut.
On ne doit JAMAIS utiliser gets(), y'a pas de "ou" quoi que ce soit. C'est JAMAIS.

PS: Tiens, je viens d'avoir une nouvelle idée pour miner complètement un programme:
#define fgets(a,b,c) gets(a)

[Thierry Chappuis]

Salut,

La fonction scanf() est une fonction très complexe, malgré ce que laissent parfois penser les cours d'introduction au langage C. La plupart des spécialistes la laisse même de côté au profit de fgets() dont le comportement est simple appréhender. C'est d'ailleurs l'alternative que je préconise. Mais regardons de plus prêt comment utiliser la fonction scanf() avec les chaînes de caractères.

La saisie de chaînes de caractère avec scanf() engendre problème de sécurité majeur:

Si l'utilisateur introduit une chaîne trop longue par rapport à l'espace prévu pour stocker cette chaîne, il y aura un débordement de tampon. Cette faille peut avoir des conséquences fâcheuses et peut être exploitée par un utilisateur mal intentionné dans le but de faire planter ton programme. Dans le contexte d'un exercice de programmation, ça ne porte pas à conséquence, mais dans le monde professionnel, cela représente un risque inacceptable (d'autant qu'avec un minimum de précautions, il est facile de contrôler ce problème). Dans ce contexte, il est donc nécessaire de contrôler le nombre de caractères saisis afin d'éviter les débordements de tampon. Ainsi les utilisations telles que scanf("%s", s_tampon) ou gets(s_tampon) (comme j'ai pu le lire dans ce fil) sont des BUGS et sont à banir du vocabulaire de tout programmeur C dès ses débuts.

Voici l'exemple minimal pour la saisie d'une chaîne de caractère avec scanf():

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
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
#include <stdio.h>
 
void vider_stdin(void);
 
int main(void)
{
    /* On définit un tableau de caractères qui servira de tampon pour la
        saisie de chaînes de caractères */
    char s_tampon[21] = {0};
 
    /* On demande à l'utilisateur d'entrer une phrase */
    printf("Entrez une phrase: ");
    fflush(stdout);
 
    /* On saisit au maximum 20 caractère pour éviter les débordements
        de tampon et place un caractère nul à la fin */
    scanf("%20s", s_tampon);
    /* Au minimum, il reste le caractère de fin de ligne sur le flux d'entrée.
        Je vide le flux entrant avec une fonction maison. 
        Voir:http://xrenault.developpez.com/tutoriels/c/scanf/ */
    vider_stdin();
 
    /* On constate que seul le 1er mot a été stocké dans le tampon, que
        les espaces de tête ont été ignoré, et que si le 1er mot est trop long,
        il a été tronqué automatiquement */
    puts(s_tampon);
 
    return 0;
}
 
void vider_stdin(void)
{
    scanf("%*[^\n]");
    getchar();
}

• L'entrée

  Code :

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

  Entrez une phrase: bienvenue sur developpez

  affiche "bienvenue". (Avec le spécificateur de format %s, tous les caractères sont saisis jusqu'à la rencontre d'un caractère d'espacement)
• L'entrée

  Code :

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

  Entrez une phrase:         bienvenue sur developpez

  affiche "bienvenue". (Les espaces de tête ont été ignorés)
• L'entrée

  Code :

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

  Entrez une phrase:  abcdefghijklmnopqrstuvwxyz

  affiche "abcdefghijklmnopqrst". (La chaîne a été tronquée après saisie de 20 caractères)

Dans le code ci-dessus:

Code :

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

1
2
3
scanf("%20s", s_tampon);
/* est équivalent à */
scanf(" %20[^ \n\t]", s_tampon);

ou la chaine de format " %20[^ \n\t]" se compose d'un espace afin d'ignorer les espaces de tête, du nombre de caractères à saisir afin d'éviter les débordements de tampon, d'une expression rationnelle qui dit à scanf() de saisir tout caractères qui n'est pas un espace (espace, tabulation ou fin de ligne). La saisie s'arrête à la rencontre du 1er caractère d'espace.

Maintenant, pour accepter la lecture de plusieurs mots séparés par des espaces ou des tabulations, il suffit de le préciser au niveau de l'expression rationnelle. Ainsi, le code suivant permet de lire plusieurs mots en évitant les débordement de tampon:

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
16
17
18
19
20
21
22
23
#include <stdio.h>
 
void vider_stdin(void);
 
int main(void)
{
    char s_tampon[21] = {0};
 
    printf("Entrez une phrase: ");
    fflush(stdout);
 
    scanf(" %20[^\n]", s_tampon);
    vider_stdin();
 
    puts(s_tampon);
    return 0;
}
 
void vider_stdin(void)
{
    scanf("%*[^\n]");
    getchar();
}

Tout cela est comme tu peux le voir beaucoup plus complexe que ce que disent les cours d'introduction au C sur scanf(). La fonction fgets() à mon avis plus simple maîtriser, et facilite la maintenance (en effet, rien ne dit que la personne chargée de la maintenance de ton programme sera un gourou de scanf() ). Voici l'exemple ci-dessus ré-écrit avec fgets():

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
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
43
44
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
void vider_stdin(void);
 
int main(void)
{
    char s_tampon[21] = {0};
    char *pc;
 
    printf("Entrez une phrase: ");
    fflush(stdout);
 
    /* On saisit la ligne entrée par l'utilisateur avec fgets() */
    fgets(s_tampon, sizeof s_tampon, stdin);
    /* Si la chaîne entrée par l'utilisateur est trop longue, on ne trouve pas
       de caractère de fin de ligne '\n' dans s_tampon. On utilise la fonction
       strchr() de la bibliothèque standard pour effectuer cette recherche */
    if ((pc = strchr(s_tampon, '\n')) == NULL)
    {
        /* On affiche un avertissement à l'utilisateur pour lui signifier
           que son entrée a été tronquée */
        fprintf(stderr, "L'entrée est trop longue et a été tronquée!!!\n");
        /* On vide le tampon du flux d'entrée standard en prévision des
           saisies futures */
        vider_stdin();
    }
    else /* La chaîne a été saisie complètement */
    {
        /* On élimine le caractère de fin de ligne qui ne sert à rien */
        *pc = '\0';
    }
 
    puts(s_tampon);
 
    return 0;
}
 
void vider_stdin(void)
{
    scanf("%*[^\n]");
    getchar();
}

Désolé d'avoir été un peu long. J'espère que ce poste est plus clair que le dernier que j'ai posté, un peu avare en commentaires.

Salutations

Thierry

[Changedman]

C'est moi qui ait été long, desolé. Merci beaucoup pour ce post instructif, je vais l'integrer à mon code et le securisé au mieux.

[dj.motte]

Salut,

J'ai essayé l'exemple de mujigka pour la saisie avec scanf. Malheureusement ça na change rien, si un espace est introduit la conversion cesse...

[Changedman]

Salut,

J'ai essayé l'exemple de mujigka pour la saisie avec scanf. Malheureusement ça na change rien, si un espace est introduit la conversion cesse...

Tu parles bien du 2eme ?

[Emmanuel Delahaye]

J'ai essayé l'exemple de mujigka pour la saisie avec scanf. Malheureusement ça na change rien, si un espace est introduit la conversion cesse...

Pourquoi je ne suis pas surpris ? Laissez tomber cette fonction du diable.

[Changedman]

Pourquoi je ne suis pas surpris ? Laissez tomber cette fonction du diable.

Ben je vais suivre le conseil mais elle fonctionne : http://img252.imageshack.us/img252/9...nstitrelw1.png

[dj.motte]

Salut,

Oui c'est la deuxième version ( scanf ) . Ok ça marche, il y avait une erreur dans mon copier-coller.

[Thierry Chappuis]

Salut,

J'ai essayé l'exemple de mujigka pour la saisie avec scanf. Malheureusement ça na change rien, si un espace est introduit la conversion cesse...

J'ai testé et ça fonctionne. Mais Emmanuel a raison, il y a des solutions plus appropriées que scanf() pour la gestion des entrées utilisateur, en particulier fgets(). Pour apprendre à se construire des entrées/sorties solides en C:

• http://emmanuel-delahaye.developpez.com/inputs.htm
• La fonction strtol() http://man.developpez.com/man3/strtol.3.php
• La fonction strtod() http://man.developpez.com/man3/strtod.3.php

Thierry

[dj.motte]

Salut,

Le problème avec la version scanf est que l'on ne peut pas quitter la saisie sans entrer au moins un caractère. Un simple retour de chariot ne peut pas valider la saisie.

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
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
 
#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#include <assert.h>
 
void vider_stdin(void)
{ scanf("%*[^\n]");
  getchar();
}
 
void do1()
{ char buf[20] ;
  while( 1 )
  { memset(buf,'\0', sizeof(buf) );
    fflush(stdout);
    printf("Saisir fin pour quitter\n" ) ;
 
    scanf(" %20[^\n]", buf ) ;
 
    vider_stdin();
    //fgets(buf, sizeof(buf), stdin ) ;
    //buf[ strlen(buf)-1 ] = '\0' ;
    if( 0 == strcmp(buf,"fin") ) break;
    printf( "len= %d\n%s\n", strlen(buf),  buf ) ;
 
  }
}
 
int main( int argc , char * argv [] )
{ do1() ;
  return 0 ;
}

Par contre la version fgets permet le passage au suivant dans l'algo, si on entre simplement un retour de chariot.

Comme quoi ce scanf c'est pas terrible .

[Emmanuel Delahaye]

Comme quoi ce scanf c'est pas terrible .

Je pense qu'on ne peut pas bien utiliser scanf() tant qu'on a pas sacrifié deux ou trois vierges en l'honneur de l'Ange Déchu...

[Thierry Chappuis]

Je pense qu'on ne peut pas bien utiliser scanf() tant qu'on a pas sacrifié deux ou trois vierges en l'honneur de l'Ange Déchu...

Alors je laisse tomber scanf()

Thierry

[miron]

Par contre la version fgets permet le passage au suivant dans l'algo, si on entre simplement un retour de chariot.
Comme quoi ce scanf c'est pas terrible .

Un inconvéniant de fgets, je trouve est qu'il enregistre le '\n', ceu qui lors d'utilisation des chaines peu etre embetant...