Discussion :

[mitherkiller]

Bonjour, je suis en plein dans les structures et j'ai un petit 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
15
16
17
18
19
20
21
#include <stdio.h>
#include <stdlib.h>
struct personne
{
   char nom[100];
   char prenom[100];
};
 
 
 
int main()
{
	struct personne point;
	printf("Quel est votre nom?");
	scanf("%s", personne.nom);
	printf("\nQuel est votre prenom?\n", personne.prenom);
	scanf("%ld", personne.nom);
	printf(" Vous vous apelez %s %s\n", personne.nom, personne.prenom);
 
	return 0;
}

Merci, je comprends vraiemnt pas sa marche pas??

[freecircus]

Code :

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

struct personne point;

ici tu déclares la variable "point" de type "personne" c'est ok ?
Donc là :

Code :

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

scanf("%s", personne.nom);

ce n'est pas "personne.nom" mais "point.nom" qu'il faut utiliser.

Code :

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

printf("\nQuel est votre prenom?\n", personne.prenom);

Tu veux faire quoi là avec "personne.prenom" ? l'afficher? => il manque un formateur.

Code :

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

scanf("%ld", personne.nom);

" %ld " c'est quoi ça?
Pour une chaine c'est %s, tu l'as bien fait dans le reste du code!
un peu de rigueur..

[Franck.H]

Tu veux faire quoi là avec "personne.prenom" ? l'afficher? => il manque un formateur.

Code :

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

scanf("%ld", personne.nom);

Ouais sauf que là il veut pas l'afficher mais plutôt saisir puis stocker dans le tableau de char "nom" mais l'utilisation de scanf est à bannir, préfère plutôt fgets et avec celle-ci, tu lis le flux standard d'entrée stdin.

N'oublie pas non plus qu'il faut également vérifier la présence du caractère de saut de ligne \n que fgets ajoute si elle en trouve un. Tu peux nettoyer ta chaine en ajoutant à la place le \0 de fin de chaîne comme ceci:

Code :

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

1
2
3
4
5
6
char * p = strchr (buffer, '\n');
 
if (p != NULL)
{
   *p = 0;
}

Voilà, je pense avoir tout dit !

[Franck.H]

Code :

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

scanf("%s", personne.nom);

Ce n'est pas la structure "personne" qu'il faut utiliser ici, mais la variable du type de ta structure que tu as déclarée dans ton main soit "point", idem pour les autres !

[Emmanuel Delahaye]

struct personne
{
char nom[100];
char prenom[100];
};

int main()
{
struct personne point;
scanf("%s", personne.nom);

Quel est le nom de l'objet ?

scanf("%ld", personne.nom);

Ensuite, %ld pour une chaine, ça va pas le faire...

[Emmanuel Delahaye]

Voilà, je pense avoir tout dit !

... et purger le flux en cas d'absence du '\n', sinon, on se retrouve avec le syndrome de scanf().. 'ça s'arrête pas'...

[Thierry Chappuis]

Ouais sauf que là il veut pas l'afficher mais plutôt saisir puis stocker dans le tableau de char "nom" mais l'utilisation de scanf est à bannir, préfère plutôt fgets et avec celle-ci, tu lis le flux standard d'entrée stdin.

N'oublie pas non plus qu'il faut également vérifier la présence du caractère de saut de ligne \n que fgets ajoute si elle en trouve un. Tu peux nettoyer ta chaine en ajoutant à la place le \0 de fin de chaîne comme ceci:

Code :

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

1
2
3
4
5
6
char * p = strchr (buffer, '\n');
 
if (p != NULL)
{
   *p = 0;
}

Voilà, je pense avoir tout dit !

Si vraiment tu veux nettoyer, il te faut encore vider le tampon du flux entrant en cas d'absence du caractère de fin de ligne '\n':

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
void fclean(char *buffer, FILE *stream)
{
    /* On recherche la présence de '\n' dans le tampon buffer */
    char * p = strchr (buffer, '\n');
 
    if (p != NULL) /* '\n' a été trouvé */
    {
       *p = 0;
    }
    else /* L'entrée a été tronquée: on vide le tampon du flux stream */
    {
        int c;
        while ((c = fgetc(stream)) != '\n' && c != EOF)
        {
            continue;
        }
    }
}

Le mieux est de se construire ses propres fonctions qui intègrent la saisie et le nettoyage.

Thierry

[Thierry Chappuis]

Voici une réécriture de ton programme en utilisant 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
45
46
47
48
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
struct Personne
{
    char nom[30];
    char prenom[30];
};
 
static void fclean(char *buffer, FILE *stream);
 
int main(void)
{
    struct Personne pers;
    printf("Quel est votre nom?\n");
    /* On réalise la saisie et on nettoie*/
    fgets(pers.nom, sizeof pers.nom, stdin);
    fclean(pers.nom, stdin);
 
    printf("Quel est votre prenom?\n");
    /* On réalise la saisie et on nettoie*/
    fgets(pers.prenom, sizeof pers.prenom, stdin);
    fclean(pers.prenom, stdin);
 
    printf("Vous vous appelez %s %s\n", pers.nom, pers.prenom);
 
    return EXIT_SUCCESS;
}
 
static void fclean(char *buffer, FILE *stream)
{
    /* On recherche la présence de '\n' dans le tampon buffer */
    char * p = strchr (buffer, '\n');
 
    if (p != NULL) /* '\n' a été trouvé */
    {
        *p = 0;
    }
    else /* L'entrée a été tronquée: on vide le tampon du flux stream */
    {
        int c;
        while ((c = fgetc(stream)) != '\n' && c != EOF)
        {
            continue;
        }
    }
}

Thierry

[Franck.H]

... et purger le flux en cas d'absence du '\n', sinon, on se retrouve avec le syndrome de scanf().. 'ça s'arrête pas'...

Ouais exacte, j'avais oublié de le mentioner