Problème de saisie qui boucle

Version imprimable

bonjour

le code que j'ai à un bug,
il boucle après la saisie du numéro de téléphone en répétant
voulez-vous enregister cette fiche ? (oui/non)
merci pour votre aide

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
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
 
#include <stdio.h>
#include <conio.h>
#include <string.h>
 
 
struct enreg
{
    char *nom[30];
    char *tel[10];
 
};
 
FILE *fd;
#define NOM_FICHE "fiche"
 
//sous programme de creation de fiche
int creation ()
{
    struct enreg en;
    char res[4];
    printf("Entrer le nom\n");
   scanf("%s",en.nom);
    printf("Entrer le numero de telephone");
    scanf("%s",en.tel);
    do
    {
        printf("voulez-vous enregister cette fiche ? (oui/non)");
        scanf("3%s",res);
    }
    while (strcmp(res,"non")&& strcmp(res,"oui"));
    if (res[0]=='o')
    {
        if (fseek(fd,0,SEEK_END)||
                fprintf(fd,"%s\n%s\n",en.nom,en.tel)==-1)return-1;
    }
    return 0;
}
 
//sous programme de recherche de fiche
int recherche ()
{
    printf("sous-programme de recherche d'une fiche\n");
    printf("appuyer sur une touche...");
    getch();
    //on attent jusqu'à ce que la touche soit frapper
}
 
 
//programme d'affichage du menu
int main()
{
    //menu pour le choix de la selection
    int selection, res;
    fd=fopen(NOM_FICHE,"a+");
    if(!fd)
    {
        printf("impossible d'ouvrir le fichier!");
        return-1;
    }
    //faire tant que Fin de programmme n'est pas selectionner
    do
    {
        // faire tant que le choix 1 ou 2 n'est pas selectionner
        do
        {
            printf("Choisissez une option:\n"
                   "1. création d'une nouvelle fiche\n"
                   "2. Recherche d'une fiche\n"
                   "3. Fin de programmme\n");
            // lire la selection
            scanf("%d",&selection);
            if (selection <1||selection>3)printf ("selection non valide\nVeuillez saisir les valeur 1, 2 ou 3\n");
        }
        while (selection <1||selection>3);
        if (selection==1) res=creation();
        if (selection==2) res=recherche();
    }
    while (selection!=3 && res==0);
    fclose(fd);
    return 0;
}

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
    char res[4];
 
    res[0] = '\0'; res[1] = '\0'; res[2] = '\0'; res[3] = '\0';
 
    do {
        printf("Voulez-vous enregistrer cette fiche ? (oui/non) ");
        scanf("%3s", res);
 
#ifdef _DEBUG
        printf("\n-> Valeur: %s\n\n", res);
#endif // _DEBUG
    }
//  while (strcmp(res, "non") && strcmp(res, "oui"));
//  while ((strcmp(res, "non") != 0) && (strcmp(res, "oui") != 0));
    while ((strncmp(res, "non", 3) != 0) && (strncmp(res, "oui", 3) != 0));

06/04/2015, 18h56
Sve@r
Citation:
Envoyé par bosk1000

Code:

1 2 3 4 5 6 struct enreg { char *nom[30]; char *tel[10]; };
Bonjour

En dehors de ton pb de saisie, tu as aussi un problème dans ces deux éléments de ta sturcture. Si, comme je le pense, ce sont des éléments dédiés au stockage d'un nom et d'un n° de téléphone, alors ce sont de simples tableaux de caractères qu'il te faut définir et non des tableaux de pointeurs.

Si maintenant tu définis des tableaux de pointeurs, ça signifie alors qu'une structure "enreg" est dédiée à pouvoir stocker 30 noms et 10 n° de téléphones max. Mais dans ce cas, chaque pointeur stockant un nom (ou un tel) devra alors allouer de la mémoire pour pouvoir stocker réellement les caractères composant ce nom (ou ce tel). Faisable bien entendu mais moins évident.

Accessoirement utiliser des variables globales est une très mauvaise idée. Ca évite des problèmes immédiats mais ça en provoque cent fois plus ensuite.
Citation:
Envoyé par foetus

Code:

1 2 3 4 5 6 char res[4]; res[0] = '\0'; res[1] = '\0'; res[2] = '\0'; res[3] = '\0'; do { printf("Voulez-vous enregistrer cette fiche ? (oui/non) "); scanf("%3s", res); while (...)
Mouais. On se demande pourquoi mettre des '\0' vu que scanf() mettra celui qu'il faut où il faut mais si c'est nécessaire, alors autant utiliser les bons outils...
Code:

1 2 char res[4]={'\0', '\0', '\0', '\0'}; memset(res, '\0', 4);
...
06/04/2015, 19h20
foetus
Citation:
Envoyé par Sve@r

Mouais. On se demande pourquoi mettre des '\0' vu que scanf() mettra celui qu'il faut où il faut mais si c'est nécessaire, alors autant utiliser les bons outils...

Code:

1 2 char res[4]={'\0', '\0', '\0', '\0'}; memset(res, '\0', 4);

...
Ce n'est pas ce que j'ai vu avec mon printf juste après la saisie ;) ;), mais bon

Et de plus je préfère utiliser ma notation sur des petits tableaux :mrgreen: :mrgreen: (pas plus de 10 éléments) parce que:
- C'est plus facile pour moi. Parce pour un tableau de 10 éléments (par exemple), il ne faut pas en louper un avec la notation {'\0', '\0' /*, ...*/ }. Versus 10 copiés-collés que l'on peut suivre avec les indices.
- Et je pense que la compilateur l'interprète comme la notation {'\0', '\0' /*, ...*/ } (ou vice et versa)
- Un appel de fonction inutile, à moins que memset soit inline (quoique pour du C)

salut,

Citation:

Envoyé par foetus

Ce n'est pas ce que j'ai vu avec mon printf juste après la saisie ;) ;), mais bon

Sve@r a pourtant raison, scanf() rajoute automatiquement le nulbyte à la fin

Citation:

Un appel de fonction inutile

ça va pousser une trentaine d'octets sur la pile qui par défaut dispose de +8Mo, on parle même pas de compter les cycles cpu ça n'est pas pertinent du tout

pour la petite info, char res[4] = { '\0', '\0', '\0', '\0' }; et char res[4]; res[0]=0; res[1]=0; res[2]=0; res[3]=0; génèrent tous les deux le même code machine avec 4 affectations byte par byte
Code:

1 2 3 4 movb $0x0,-0x10(%rbp) movb $0x0,-0xf(%rbp) movb $0x0,-0xe(%rbp) movb $0x0,-0xd(%rbp)
tandis que char res[4] = {0}; génèrera un code plus optimisé en une seule instruction du style movl $0x0,-0x10(%rbp), soit l'équivalent d'un char res[4]; *(long *)&res = 0;
memset() quant à lui fait sa sauce classique :
Code:

1 2 3 4 mov $0x4,%edx mov $0x0,%esi mov %rax,%rdi callq 0x400520 <memset@plt>
en revanche sur un tableau plus grand (d'une centaine d'éléments) ce n'est plus pareil :
la méthode de Sve@r
Code:

1 2 3 4 5 6 char res[100] = { '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0', '\0'};
ainsi que la déclaration en char res[100] = {0}; vont toutes les deux générer un code machine optimisé par le compilateur (en tenant compte de l'architecture 32/64, en dépassant un peu si besoin pour aligner etc.), sorte de fast-copy en écrivant 13 qwords (13x8 en 64bits, 25x4 en 32bits) dans le buffer
Code:

1 2 3 4 mov $0x0,%eax mov $0xc,%ecx mov %rdx,%rdi rep stos %rax,%es:(%rdi)
la méthode de foetus en revanche... fait rigoureusement ce qu'elle semble faire
Code:

1 2 3 4 5 char res[100]; res[0] = 0; res[1] = 0; res[2] = 0; res[3] = 0; res[4] = 0; res[5] = 0; res[6] = 0; res[7] = 0; res[8] = 0; res[9] = 0; res[10] = 0; res[11] = 0; res[12] = 0; res[13] = 0; res[14] = 0; res[15] = 0; res[16] = 0; res[17] = 0; res[18] = 0; res[19] = 0; (...) res[90] = 0; res[91] = 0; res[92] = 0; res[93] = 0; res[94] = 0; res[95] = 0; res[96] = 0; res[97] = 0; res[98] = 0; res[99] = 0;
donne son équivalent en code machine avec une succession interminable de movb (affectation char après char donc)

07/04/2015, 09h26
foetus

@BufferBob: merci l'ami :ccool:

Et effectivement, avec un multiple de 4 (peut-être aussi de 8 ou de 2), le compilateur va insérer un int ou un long .

Donc cela me rassure dans un sens :mrgreen:: avec un petit tableau (pas plus de 10 éléments) on l'initialise complètement et le code ne génère pas trop d'instructions.
07/04/2015, 11h02
Sve@r

Citation:

Envoyé par BufferBob

...tandis que char res[4] = {0}; génèrera un code plus optimisé en une seule instruction du style movl $0x0,-0x10(%rbp), soit l'équivalent d'un char res[4]; *(long *)&res = 0;

J'ai failli aussi écrire dans cette syntaxe mais j'avais un petit doute à ce sujet: écrire char res[4] = {0}; va-t-il remplir tout le tableau de 0 ou seulement son premier élément ??? Me semble me souvenir que c'est tout le tableau (où pour être plus exact le premier élément est intialisé avec la valeur demandée et les autres sont mis à 0) mais je ne suis plus certain...
07/04/2015, 12h40
Bktero

Cette écriture initialise tout le tableau à 0.

J'ai écrit un article à ce sujet : https://gradot.wordpress.com/2012/09...ructures-en-c/
Voir vers "Allons maintenant à la partie 6.7.8 Initialization. On lit avec intérêt le point 10......"

Merci pour toutes vos informations, je me remet sur le code et je fais suivre ma correction
Citation:
Envoyé par foetus
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 char res[4]; res[0] = '\0'; res[1] = '\0'; res[2] = '\0'; res[3] = '\0'; do { printf("Voulez-vous enregistrer cette fiche ? (oui/non) "); scanf("%3s", res); #ifdef _DEBUG printf("\n-> Valeur: %s\n\n", res); #endif // _DEBUG } // while (strcmp(res, "non") && strcmp(res, "oui")); // while ((strcmp(res, "non") != 0) && (strcmp(res, "oui") != 0)); while ((strncmp(res, "non", 3) != 0) && (strncmp(res, "oui", 3) != 0));
Ta solution fonctionne effectivement, merci

mais je vais me pencher aussi sur le reste de ce qui est décrit plus haut

Citation:

Envoyé par Sve@r

Bonjour

En dehors de ton pb de saisie, tu as aussi un problème dans ces deux éléments de ta structure. Si, comme je le pense, ce sont des éléments dédiés au stockage d'un nom et d'un n° de téléphone, alors ce sont de simples tableaux de caractères qu'il te faut définir et non des tableaux de pointeurs.

quel est le problème de saisie ?

pour les étoiles c'est une erreur que javais oublié d'enlever...

Citation:

Envoyé par Sve@r

Accessoirement utiliser des variables globales est une très mauvaise idée. Ca évite des problèmes immédiats mais ça en provoque cent fois plus ensuite.
[/code]...

De quel variable globale parles-tu ?

08/04/2015, 08h37
Sve@r

Citation:

Envoyé par bosk1000

Citation:

Envoyé par bosk1000

le code que j'ai à un bug,
il boucle après la saisie du numéro de téléphone en répétant
voulez-vous enregister cette fiche ? (oui/non)

quel est le problème de saisie ?

Ben je sais pas trop. Peut-être celui dont tu te plaignais en ouvrant ce topic...

Citation:

Envoyé par bosk1000

De quel variable globale parles-tu ?

FILE *fd ligne 14
08/04/2015, 20h25
bosk1000
le programme fonctionne, j'ai rectifié ma variable "fd" en "repertoire"

maintenant je vais posé ma GROSSE question de débutant

pourquoi dois-je faire
Code:

1 2 3 char res[4]={'\0', '\0', '\0', '\0'}; memset(res, '\0', 4);
je pense que ça à un rapport au pointeur si compliqué à comprendre....

Merci pour vos informations
08/04/2015, 20h47
foetus

Tu n'as pas besoin d'initialiser ton tableau de 4 caractères (*) puisque la méthode scanf mettra un caractère final '\0' à la fin des caractères qu'elle a trouvé.

C'est le code sécurisée "qui ne fait pas confiance à la méthode scanf". Et pas besoin de réinitialiser à chaque tour de boucle puisque la tableau a une case en plus ;)

Ton erreur: c'est le chiffre 3 qui fallait mettre dans le "format" entre le % et le s :langue2:

* -> et soit dit en passant c'est soit la première ligne soit la deuxième puisque les 2 lignes font la même chose
08/04/2015, 21h05
Sve@r
Citation:

Envoyé par bosk1000

le programme fonctionne, j'ai rectifié ma variable "fd" en "repertoire"

Je n'ai pas dit que le soucis venait du nom de la variable, mais du fait qu'elle soit globale. Tu peux l'appeler comme tu veux, ça ne changera rien à ce fait...
Citation:
Envoyé par bosk1000

maintenant je vais poser ma GROSSE question de débutant

pourquoi dois-je faire

Code:

1 2 3 char res[4]={'\0', '\0', '\0', '\0'}; memset(res, '\0', 4);
Déjà quand j'ai écrit ça, j'ai dit qu'on n'était pas obligé de le faire. C'était suite au post de foetus qui remplissait chaque élément du tableau de façon individuelle. J'ai alors proposé deux alternatives qui font la même chose mais de façon que je trouve plus concise
- l'initialisation lors de la définition
- le passage par une fonction dédiée au remplissage d'un tableau
Mais j'ai dit aussi que ce remplissage n'était pas nécessaire vu qu'il y a un scanf() juste après qui va lui-aussi écrire dans le tableau. On se retrouve alors dans le cas d'une variable écrite deux fois sans être lue entre temps. Fatalement dans ce cas là il y a une écriture de trop.

Citation:

Envoyé par bosk1000

je pense que ça à un rapport au pointeur si compliqué à comprendre....

Effectivement dans memset() le premier argument est le nom du tableau ce qui correspond alors à l'adresse de son premier élément, adresse que la fonction stockera dans un pointeur. Mais le pointeur ici est une conséquence de la façon dont le C travaille et non une cause de ces instructions. Ceci dit, penser que le pointeur est "compliqué" à comprendre n'est qu'une idée préconçue (voire une légende urbaine) car quand on s'y met, étape par étape, avec une bonne doc, on s'en sort très bien.
08/04/2015, 21h38
bosk1000
Citation:

Envoyé par foetus

Ton erreur: c'est le chiffre 3 qui fallait mettre dans le "format" entre le % et le s :langue2:

tu parle de cette ligne ?
Code:

1 2 scanf("%3s", res);
Citation:

Envoyé par Sve@r

Je n'ai pas dit que le soucis venait du nom de la variable, mais du fait qu'elle soit globale. Tu peux l'appeler comme tu veux, ça ne changera rien à ce fait...

ok, j'avoue avoir toujours mal compris, j'ai cru que l'on critiquait quand j'utilisais des nom comme "i" au lieu de "compteur" (ho le boulet que je suis....)
du coup j'ai plus qu'a chercher ce que veux dire "globale" dans le sens des variables et comprendre ce vers quoi je doit allez

merci pour tout ça
09/04/2015, 08h07
Sve@r

Citation:

Envoyé par bosk1000

ok, j'avoue avoir toujours mal compris, j'ai cru que l'on critiquait quand j'utilisais des nom comme "i" au lieu de "compteur" (ho le boulet que je suis....)
du coup j'ai plus qu'a chercher ce que veux dire "globale" dans le sens des variables et comprendre ce vers quoi je doit allez

Uutiliser "i" comme indice de boucle est assez courant. Et c'est bien plus pratique d'écrire "i++" que "compteur++".
A une certaine époque (pas tellement lointaine), une grande société, connue pour ses OS moisis et dont la devise est "puisqu'on n'arrive pas à le faire bien alors on le fait beau", a tenté de poser un copyright sur le for (i=...; test sur i; i++). Fort heureusement pour tous les programmeurs du monde, la justice américaine a, cette fois, fait preuve de réalisme.

Dans le même tuto tu as un chapitre sur la visibilité des variables. Ca t'expliquera le concept de "global"...
09/04/2015, 18h47
bosk1000

merci
tes informations me sont très utiles