Discussion :

[chris7522]

Bonjour a toutes et a tous,
J'ai trouvé peu d'explication pour ne pas dire rien , sur cette partie de code que je ne comprends pas .
Au prealable le code entier :

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
#include <stdio.h>
 
#define NAMES_MAX_LENGTH    19  /* Maximum length of names, not including '\0' */
 
#define STRINGIFY2( x) #x
#define STRINGIFY(x) STRINGIFY2(x)
 
static const char names_scan_fmt[]= "%" STRINGIFY(NAMES_MAX_LENGTH) "s";
 
typedef struct player
{ 
        char prenom[NAMES_MAX_LENGTH + 1];  /* +1 for the '\0' */
        char nom[NAMES_MAX_LENGTH + 1];
        int age;
} player;
 
void changement(player *p)
{ 
        printf("quel est ton prenom ?\n");
        scanf(names_scan_fmt,p->prenom);
        printf("quel est ton nom ?\n");
        scanf(names_scan_fmt,p->nom); 
        printf("quel est ton age ?\n");
        while (!scanf("%d",&p->age))
                scanf ("%*c");
}
 
int main(void)
{
        player p1,p2;
 
        printf("\n____1er JOUEUR____\n");
        changement(&p1);
        printf("\n____2eme JOUEUR____\n");
        changement(&p2);
        printf("Le prenom du 1er joueur est %s,le nom %s et l'age est %d\n",p1.prenom,p1.nom,p1.age);
        printf("Le prenom du 2 eme joueur est %s,le nom %s et l'age est %d\n",p2.prenom,p2.nom,p2.age);
 
        return 0;
}

La partie que je ne comprends pas est la suivante:

Code :

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

1
2
3
4
5
6
#define NAMES_MAX_LENGTH    19  /* Maximum length of names, not including '\0' */
 
#define STRINGIFY2( x) #x
#define STRINGIFY(x) STRINGIFY2(x)
 
static const char names_scan_fmt[]= "%" STRINGIFY(NAMES_MAX_LENGTH) "s";

Merci de votre aide

[Bousk]

Si tu ne mets pas 2 niveaux de macro, quand tu écris STRINGIFY(NAMES_MAX_LENGTH) le résultat sera "NAMES_MAX_LENGTH".

[chris7522]

Merci .Je suis debutant en C , seulement depuis mai de cette année , et je me demandais a quoi servait :
#define STRINGIFY2( x) #x
#define STRINGIFY(x) STRINGIFY2(x)

Il ne s'agit pas de quelque chose qui permet de securisé le scanf au bug overflow ?
Mais j'ai sans doute pas le niveau pour comprendre , je devrai peut etre revenir a des choses de mon niveau !

[Bousk]

https://c.developpez.com/faq/?page=L...on-d-une-macro

[chris7522]

Merci , c'est deja + clair !
Un dernier petit point a eclaircir . Je ne comprends pas l'expression :

Code :

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

1
2
3
while (!scanf("%d",&p->age))
                scanf ("%*c");
}

[dalfab]

Bonjour,

C'est, en effet, une séquence subtile. Heureusement on ne rencontre que très rarement ce genre de code.
Le premier scanf chercher à lire un nombre entier à stocker dans p->age.
Le second scanf est totalement équivalent à appeler getc(). Il va lire un caractère tout en ne stockant aucunement la valeur lue.
Il reste à expliquer le rôle du point d'exclamation et du while :
- la valeur retournée par scanf() est le nombre de champs qu'elle a pu extraire. Ici il y juste un entier à extraire. La fonction va donc retourner 1 si peut lire un entier, va retourner 0 si elle ne peut rien lire, et va retourner une valeur négative en cas d'erreur.
- le point d'exclamation est l'opérateur 'not'. S'il voit false ou une valeur nulle, il va retrourner true, s'il voit true ou une valeur non nulle il va retourner false.
Je te propose d'écrire le code sous la forme :

Code :

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

1
2
3
while ( scanf("%d", &p->age) == 0 )
                getc();
}

C'est exactement la même chose.

Donc cela essayer de lire un nombre entier, si c'est impossible (par exemple car la saisie commence par une lettre au lieu d'un chiffre) on va lire un caractère (donc la lettre) et recommencer jusqu'à ce finalement nombre entier soit lu.

[chris7522]

Merci dalfab d'avoir pris le temps de toutes ces explications

[Sve@r]

Bonjour

Donc cela essayer de lire un nombre entier, si c'est impossible (par exemple car la saisie commence par une lettre au lieu d'un chiffre) on va lire un caractère (donc la lettre) et recommencer jusqu'à ce finalement nombre entier soit lu.

C'est vrai que ce code produit ce comportement. Et donc si la saisie contient "abc123def", il en ressortira "123".

Toutefois perso ce n'est pas un code que je te conseillerais de reproduire parce que déjà je ne suis pas fan de codes qui "veulent réparer ce qui a été mal entré" car en général ils réparent pas super bien (si la saisie est "abc123def456" ce code récupèrera "123" alors que celui qui saisit (ok il saisit comme un couillon d'où le nom "test du couillon" de ce genre de benchmark) aurait peut-être voulu "123456".
Donc perso je préfère le style de saisie de type "tout ou rien". Soit l'utilisateur remplit correctement son truc et ça va, soit c'est un crétin et il n'a rien à faire dans mon programme.

Et en plus ce n'est pas super efficace. Déjà quand tu entres "abc123def" il récupère bien "123" mais il laisse "def" dans le buffer clavier qui risque d'aller pourrir la saisie suivante ; et en plus que se passe-t-il pour une saisie style "abcdef" sans chiffre à récupérer ? J'ai pas testé mais quelque chose me dit que comme scanf() renverra 0 mais que le fgetc() ne récupèrera rien ça va partir en vrille.

Perso je préfère coder ce type de saisie de cette façon

Code c :

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

1
2
3
4
5
6
char saisie[1024];
while (1) {
	fgets(saisie, 1024, stdin);
	if (sscanf(saisie, "%d", &p->age) > 0) break;
	fputs("Tu concentres ton demi-neurone et tu n'entres que des chiffres !!!\n", stderr);
}

Donc je fais saisir une string (ce qui laisse alors le clavier toujours vide) puis je regarde si je peux en extraire un nombre et sinon on recommence. Quitte à mettre ça dans une fonction dédiée. (style saisieInt()).

Certains esprits particulièrement affutés remarqueront la limitation à 1023 ce qui est, je l'admets, une source de souci potentiel (si on saisit 3000 caractères le truc n'en consomme que 1023 et laisse le reste) mais déjà cela fait partie du ratio "probabilité du risque/coût de programmation" (ratio ici directement fonction de la taille définie par le programmeur) et d'autre part on peut aussi remplacer fgets() par getline() qui, elle, est virtuellement illimité" (ne pas oublier alors le free() de la chaine saisie ce qui augmente alors le "coût de programmation" dont j'ai parlé).

Et en plus comme le crétin va recommencer jusqu'à ce qu'il s'instruise, je trouve déjà ce type de saisie un peu plus éducatif (ou alors il meurt de faim et de soif et ça joue un rôle de sélection naturelle... )