J'aime trop ton bool

Version imprimable

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
//---------------------------------------------------------------------------
 
#pragma hdrstop
#pragma argsused
#include <stdio.h>
#include <conio.h>
#include <string.h>
 
/* ---------------- Variables globales -------------------------------------*/
enum bool{faux,vrai};
struct identite {
                      int age;
                      char prenom[20];
                      char nom[20];
};
struct contact {
                      char num[20];
                      char prenom[20];
                      char nom[20];
        };
 
//-----------------------------------------------------------------------------
enum bool affichage(struct contact m,struct identite m2) {
        enum bool bouboul;
 
        getch();
        if (strcmp(m.prenom,m2.prenom)==0)
                bouboul=vrai;
 
        return bouboul;
 
}
//-----------------------------------------------------------------------------
int main()
{
//------------   declarations locales -----------------------------------------
        struct contact moi;
        struct identite moi2;
        enum bool mon;
 
//------ DEBUT --------------------------------------------------------------
 
        printf( "\n Entrer votre prenom :  " );
        scanf("%s",&moi2.prenom);
 
        strcpy(moi.prenom,moi.prenom);
 
        printf(" \n entrer votre nom  :  ");
        scanf("%s",&moi2.nom);
 
        strcpy(moi.prenom,moi2.prenom);
 
        mon=affichage(moi,moi2);
 
        printf("\n monbool : %d ",mon);
        getch();
 
 
 
        return 0;
}
//---------------------------------------------------------------------------

Bonjour ,

je comprend pas pourquoi mon printf du main concernant laffichage de la valeur de mon bool naffiche pas 1.
alors que jai tout fait por que ca le fasse ...
de plus lorsque dans la focntion , je mets un

Code:

1
2
else 
        bouboul=faux;

et que cette fois ci je fais tout pourquil soit faux bah lorsuqe je jessais daficher sa valeur num. ca maffiche un vieux truc moisi

besoin de votre aide
jai limpression que le compilateur me renvoi une valeur de mon bool aleatoire .. meme si je sais que cest pas possible ... :P

Salut,

Code:

1
2
3
4
5
6
7
8
9
10
enum bool affichage(struct contact m,struct identite m2) {
        enum bool bouboul;
 
        getch();
        if (strcmp(m.prenom,m2.prenom)==0)
                bouboul=vrai;
 
        return bouboul;
 
}

bouboul peut ne pas être valorisé, la valeur retournée peut être n'importe quoi

05/12/2007, 22h32
forthx

en effet... et tu peu y remedier en initialisant bouboul a faux par exemple.

a ma conaissance une variable n'est automatiquement initialisée que dans le cas ou elle est "static"
05/12/2007, 22h41
fremen167

Si tu corriges ça :

Code:

strcpy(moi.prenom,moi.prenom);

Tu arriveras peut-être à avoir un comportement compréhensible...

Citation:
Envoyé par Gruik
Salut,
Code:

1 2 3 4 5 6 7 8 9 10 enum bool affichage(struct contact m,struct identite m2) { enum bool bouboul; getch(); if (strcmp(m.prenom,m2.prenom)==0) bouboul=vrai; return bouboul; }
bouboul peut ne pas être valorisé, la valeur retournée peut être n'importe quoi
questce que tu veux dire ?
je suis juste etudiant et nous commencons le C.
lorsque je mets le debugueur ... je bloque juste apres la focntion et je vois bien que bouboul retourné est egal à 1 ... or avec le printf , il a une autre valeur ...
idem pour "faux" ...

Citation:

Envoyé par Pugebad

jai limpression que le compilateur me renvoi une valeur de mon bool aleatoire .. meme si je sais que cest pas possible ... :P

Le compilateur n'a rien à voir. Une fois ton code remis en forme et débarrassé de ses erreurs basiques

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
 
 
//---------------------------------------------------------------------------
 
#include <stdio.h>
#include <string.h>
 
/* ---------------- Variables globales -------------------------------------*/
enum bool
{ faux, vrai };
struct identite
{
   int age;
   char prenom[20];
   char nom[20];
};
struct contact
{
   char num[20];
   char prenom[20];
   char nom[20];
};
 
//-----------------------------------------------------------------------------
enum bool affichage (struct contact m, struct identite m2)
{
   enum bool bouboul;
 
   if (strcmp (m.prenom, m2.prenom) == 0)
      bouboul = vrai;
 
   return bouboul;
 
}
 
//-----------------------------------------------------------------------------
int main (void)
{
//------------   declarations locales -----------------------------------------
   struct contact moi;
   struct identite moi2;
   enum bool mon;
 
//------ DEBUT --------------------------------------------------------------
 
   printf ("\n Entrer votre prenom :  ");
   scanf ("%s", moi2.prenom);
 
   strcpy (moi.prenom, moi.prenom);
 
   printf (" \n entrer votre nom  :  ");
   scanf ("%s", moi2.nom);
 
   strcpy (moi.prenom, moi2.prenom);
 
   mon = affichage (moi, moi2);
 
   printf ("\n monbool : %d ", mon);
 
   return 0;
}
 
//---------------------------------------------------------------------------

il reste :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : C:\dev\forums\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c: In function `affichage':
main.c:26: warning: 'bouboul' might be used uninitialized in this function
main.c: In function `main':
main.c:48: warning: statement with no effect
Linking console executable: console.exe
Process terminated with status 0 (0 minutes, 0 seconds)
0 errors, 2 warnings

Ce qui est largement suffisant pour justifier un comportement indéfini...

06/12/2007, 10h04
Emmanuel Delahaye

Citation:

Envoyé par Pugebad

questce que tu veux dire ?
je suis juste etudiant et nous commencons le C.
lorsque je mets le debugueur ... je bloque juste apres la focntion et je vois bien que bouboul retourné est egal à 1 ... or avec le printf , il a une autre valeur ...
idem pour "faux" ...

On se tue à te dire qu'une variable locale n'a pas de valeur définie par défaut. Ca peut donc être n'importe quoi. Si tu ne passes pas par la branche qui assigne bouboul, la valeur retournée est indéfinie (moisie, comme tu le dis si bien...). C'est si difficile que ça à comprendre ?
06/12/2007, 12h19
Gruik

Citation:

Envoyé par Emmanuel Delahaye

Code:

main.c:26: warning: 'bouboul' might be used uninitialized in this function

Notez que ce warning n'apparait pas en -O0 avec gcc 3, ce qui est quand meme un probleme vu qu'en général on veut pouvoir debugger en phase de test et qu'on enleve l'optimisation
06/12/2007, 12h28
Emmanuel Delahaye
Citation:

Envoyé par Gruik

Notez que ce warning n'apparait pas en -O0 avec gcc 3, ce qui est quand meme un probleme vu qu'en général on veut pouvoir debugger en phase de test et qu'on enleve l'optimisation

Bizarre. Mon reglage actuel :
Code:

1 2 3 -Wall -Wextra -O -Wwrite-strings -Wstrict-prototypes -Wuninitialized -Wunreachable-code
P.S. Ah, OK, je vois ce que tu veux dire. L'optimisation n'empêche pas le déboguage. Simplement, parfois, l'exécution pas à pas est un peu surprenante, c'est tout....
06/12/2007, 17h32
Panaméen

Citation:

Envoyé par Emmanuel Delahaye

On se tue à te dire qu'une variable locale n'a pas de valeur définie par défaut. Ca peut donc être n'importe quoi. Si tu ne passes pas par la branche qui assigne bouboul, la valeur retournée est indéfinie (moisie, comme tu le dis si bien...). C'est si difficile que ça à comprendre ?

passer par la branche qui assigne bouboul ??? je vois pas trop dsl
06/12/2007, 17h37
Emmanuel Delahaye

Citation:

Envoyé par Pugebad

passer par la branche qui assigne bouboul ??? je vois pas trop dsl

Tu apprends l'informatique comment ? En lisant des mangas ?
Tu ne sais pas ce qu'est une branche ? Une assignation ?
06/12/2007, 18h19
fremen167

Bon, ce qu'on te dit c'est que dans affichage, tu as un "if".

Un if crée deux branches dans ton programme, tu suivras l'une ou l'autre selon son résultat.

Ici, tu fais quelque chose dans le cas "oui" : bouboul = vrai.

Mais dans le cas (la branche) "non", que vaut bouboul ?!

C'est ce que te dit gentiment ;) le compilateur par le message :

Code:

main.c:26: warning: 'bouboul' might be used uninitialized in this function

Il convient donc d'initialiser bouboul (à faux) au début de ta fonction.
06/12/2007, 20h05
Panaméen

Merci frement pour ta réponse clair et précise

jaffectes bouboul de fonction à faux en debut mais tjs le meme probleme

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
//---------------------------------------------------------------------------
 
#pragma hdrstop
#pragma argsused
#include <stdio.h>
#include <conio.h>
#include <string.h>
 
/* ---------------- Variables globales -------------------------------------*/
enum bool{faux,vrai};
struct identite {
                      int age;
                      char prenom[20];
                      char nom[20];
};
struct contact {
                      char num[20];
                      char prenom[20];
                      char nom[20];
        };
 
//-----------------------------------------------------------------------------
enum bool affichage(struct contact m,struct identite m2) {
        enum bool bouboul=faux;
 
        if (strcmp(m.prenom,m2.prenom)==0) {
                bouboul=vrai;
        }
        else  {
                bouboul=faux;
        }
        printf ("\n %d",bouboul);
 
        return bouboul;
 
}
//-----------------------------------------------------------------------------
int main()
{
//------------   declarations locales -----------------------------------------
        struct contact moi;
        struct identite moi2;
        enum bool mon=faux;
 
//------ DEBUT --------------------------------------------------------------
 
        printf( "\n Entrer votre prenom :  " );
        scanf("%s",&moi2.prenom);
 
        strcpy(moi.prenom,moi2.prenom);
 
        mon=affichage(moi,moi2);
        printf ("\n%d",mon);
        getch();
 
 
 
        return 0;
}
//---------------------------------------------------------------------------

10/12/2007, 22h57
fremen167

Salut Pugebad,

Je n'ai pas répondu + tôt car je n'avais pas de compilo sur ma becane. Je viens de tester ton code avec Code::Blocks, il affiche bien deux fois 1, comme le code le laissait prévoir :). Peut-être que si tu donnes des précisions sur ton environnement de travail, des spécialistes C du forum seront capable de t'expliquer ce qui ne marche pas sur ta machine ??

Juste une petite remarque : pourquoi as-tu codé un else (bouboul=faux) alors que bouboul est déjà initialisé à faux lors de sa déclaration. Cela ne sert à rien ! Idem pour l'initialisation de mon, puisqu'il est affecté systématiquement par l'instruction :

Code:

mon=affichage(moi,moi2);

Tu es passé d'une situation dans laquelle tu laissais une variable dans un état incertain (ce qui est fort risqué) à une autre où tu répètes partout la même chose. Ce n'est guère mieux, si tu te places dans la perspective du développement, non pas d'un tout petit programme d'apprentissage, mais d'un "vrai" programme au sein d'un application. Parce que le jour où tu devras le modifier, tu risques d'oublier un endroit, et d'être obligé de recommencer. Mieux le code est construit, plus facile sera une correction ou une évolution ;)

De la même manière, tu devrais faire davantage attention aux noms que tu emploies :
- "mon", ce n'est pas parlant ;
- "affichage", pour une fonction qui recopie une chaîne et retourne un booléen 8O
Je sais que le programme est assez artificiel, et qu'il visait pour toi à tester des fonctionnalités du langage, mais il est bon de prendre les bonnes habitudes dès le début. Le nom d'une donnée ou d'une fonction doit évoquer immédiatement, intuitivement son rôle dans le programme. Sinon, ton code sera illisible, même pour toi dans deux ou trois mois !!