Perte variable globale

**Rocou** · 24/05/2007, 17h59

Avec GCC, mes variable globales semblent prendre des valeurs de manière aléatoire. voici un bout de code:

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
char machaine[20]; /*c'est ma variable globale*/
 
void main(void)
{
   mafonction();
   affichechaine();
}
 
void mafonction(void)
{
   strcpy(machaine,"toto");
}
 
void affichechaine(void)
{
   printf("machaine: %s\n", machaine);
}

Il devrait y avoir "toto" dans 'machaine' or ce n'est pas toujours le cas!

**diogene** · 24/05/2007, 18h17

- Utilise les balises CODE
-

Code :

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

void main(void)

tu vas te faire incendier : main renvoie toujours un int
-

Il devrait y avoir "toto" dans 'machaine' or ce n'est pas toujours le cas!

Et qu'est-ce qu'il y a d'autre ?

Avec GCC, mes variable globales semblent prendre des valeurs de manière aléatoire.

Conseil : N'utilise pas de variables globales si tu peux t'en passer : Ce sont souvent des pestes ingérables dès que le code devient un peu étoffé

**Rocou** · 24/05/2007, 18h36

Envoyé par diogene

- Utilise les balises CODE
-

Code :

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

void main(void)

tu vas te faire incendier : main renvoie toujours un int

Envoyé par diogene

- Et qu'est-ce qu'il y a d'autre ?

La valeur d'une autre variable présente dans le code

Envoyé par diogene

Conseil : N'utilise pas de variables globales si tu peux t'en passer : Ce sont souvent des pestes ingérables dès que le code devient un peu étoffé

J'ai besoin de conserver une valeur générée dans une fonction qui soit lisible par toutes les autres. Comment faire autrement?
De toutes les façons, mon programme est tout petit et n'est pas destiné à grandir (le but est de lire un fichier texte et d'alimenter une base de données avec)

**diogene** · 24/05/2007, 18h45

La valeur d'une autre variable présente dans le code

Il n'y a pas d'autres variables dans le code présenté. Est-ce que le code présenté fait cette bizarrerie ?
Sinon, tu es en train peut être d'apprécier les joies des variables globales

.
Présente le code en défaut

**odsen.s** · 24/05/2007, 19h15

J'ai besoin de conserver une valeur générée dans une fonction qui soit lisible par toutes les autres. Comment faire autrement?

En C, on peut passer des arguments aux différentes fonctions. Exemple avec ton premier code :

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
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
 
void mafonction(char *s);
void affichechaine(const char *s);
 
int main(void)
{
   char s[32] = { 0 };
 
   mafonction(s);
   affichechaine(s);
 
   return 0;
}
 
void mafonction(char *s)
{
   strcpy(s,"toto");
}
 
void affichechaine(const char *s)
{
   printf("s: %s\n", s);
}

On peut tout à fait se passer de variables globales dans la grande majorité des cas, et c'est indispensable pour tout projet un minimum sérieux.

**Rocou** · 25/05/2007, 09h13

Envoyé par odsen.s

En C, on peut passer des arguments aux différentes fonctions. On peut tout à fait se passer de variables globales dans la grande majorité des cas, et c'est indispensable pour tout projet un minimum sérieux.

En l'occurrence, je ne sais pas comment je pourrais me passer d'une variable globale car quand j'appelle mes fonctions (dans le main(), via un switch) la valeur à conserver n'est pas connue. C'est une des fonctions appelée qui la détermine et ce sont les autres fonctions qui l'utilisent.
Il y a certes presque 20 ans que je n'avais pas programmé une ligne de C mais j'avais encore souvenir du passage de paramètres

Je pensais que ce phénomène était connu visiblement, ce n'est pas le cas.
Mon idée afin de contourner le problème est d'écrire dans un fichier temporaire (ou de me servir d'une table de ma base) afin de conserver et d'utiliser ces données globales selon les besoins.

C'est un peu lourdingue mais je ne vois pas d'autre solution

Voici ce que je dois faire:

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
 
int i;
Char MaChaine[20];
 
int main(int argc, char **argv)
{
/*....*/
	switch (i) 
		{
		case '1': fonction1();
			  break;
		case '2': fonction2();
			  break;
		case '3': fonction3();
			  break;
 
		default : break;
		}
}
 
int fonction1(void)
	{
	strcpy(MaChaine,"toto");
	}
 
int fonction2(void)
	{
	/*utilisation de 'MaChaine' */
	}
 
int fonction3(void)
	{
	/*utilisation de 'MaChaine' */
	}

**Médinoc** · 25/05/2007, 09h23

Peux-tu poster un code minimal et compilable sans erreur ni warning qui montre le problème ?
Notamment, une version compilable directement du code du post original est-elle suffisante pour reproduire le problème ?

De plus, quelle est ta version de gcc ?

**Emmanuel Delahaye** · 25/05/2007, 15h42

Envoyé par Rocou

Avec GCC, mes variable globales semblent prendre des valeurs de manière aléatoire. voici un bout de code:

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
char machaine[20]; /*c'est ma variable globale*/
 
void main(void)
{
   mafonction();
   affichechaine();
}
 
void mafonction(void)
{
   strcpy(machaine,"toto");
}
 
void affichechaine(void)
{
   printf("machaine: %s\n", machaine);
}

Il devrait y avoir "toto" dans 'machaine' or ce n'est pas toujours le cas!

Pour de nombreuses raisons, ce code invoque un comportement indéterminé. Tout peut arriver. Mais si son compilateur est bien configuré :

http://emmanuel-delahaye.developpez....tm#cfg_compilo

on est prévenu :

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
 
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : C:\dev\forums\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c:4: warning: return type of 'main' is not `int'
main.c: In function `main':
main.c:5: warning: implicit declaration of function `mafonction'
main.c:6: warning: implicit declaration of function `affichechaine'
main.c: At top level:
main.c:10: error: conflicting types for 'mafonction'
main.c:5: error: previous implicit declaration of 'mafonction' was here
main.c: In function `mafonction':
main.c:11: warning: implicit declaration of function `strcpy'
main.c: At top level:
main.c:15: error: conflicting types for 'affichechaine'
main.c:6: error: previous implicit declaration of 'affichechaine' was here
main.c: In function `affichechaine':
main.c:16: warning: implicit declaration of function `printf'
Process terminated with status 1 (0 minutes, 1 seconds)
4 errors, 5 warnings

Ton code bat des records sur l'échelle de Goret :

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

on t'oblige à coder comme ça, on a menacé de violer ton poisson rouge, ou c'est autre chose ?

**Rocou** · 25/05/2007, 16h09

Envoyé par Emmanuel Delahaye

on t'oblige à coder comme ça, on a menacé de violer ton poisson rouge, ou c'est autre chose ?

Ce "code" n'était pas destiné à être compilé. Son but était d'illustrer ce que je voulais faire.

Cela dit, je ne m'en sort toujours pas et mes chaînes, quelles soient globales ou pas, ne contiennent pas ce que j'y ai mis.

Voici ma fonction de lecture de mon fichier texte:

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
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
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
120
121
122
123
124
125
126
127
128
129
130
131
132
133
134
135
136
137
138
139
140
141
142
143
144
145
146
147
148
149
150
151
152
153
154
155
156
157
158
159
160
161
162
163
164
165
166
167
168
169
170
171
172
173
174
175
176
177
178
179
180
181
182
183
184
185
186
187
188
189
190
191
192
193
194
195
196
197
198
199
200
201
202
203
204
int lecturefichiercsv(void)
{
//les variables qui vont contenir les champs de la ligne du fichier
 char IdTLigne;
 char Operateur[5]={0}; // CAB
 char LaDateDebut[10]={0};
 char HDeb[8]={0};
 char KMdepart[7]={0};
 char LeVehicule[5]={0}; // CAB
 char TypeDechet[2];
 char NumBac[5]; //CAB
 char Poids[7];
 char SignClient[5]; //CAB
 char LaDateScan[10];
 char LHeureScan[8];
 char KMFin[7];
 char LaDateFin[10];
 char LHeureFin[8];
 char VolumeCarbu[6];
 char LaDateCarbu[10];
 char LHeureCarbu[8];
 
 
  int i=0;
  int c=0;
  int FinFichier=0;
  char *LeDir;
 
 
  DIR *dp;
  struct dirent *entry;
 
 LeDir = malloc(255);
 
if((dp = opendir("/Volumes/Disque\ dur/Users/ftp/VU/DATA/")) == NULL) 
    {
	fprintf(stderr,"impossible d'ouvrir le répertoire: %s\n", "/Volumes/Disque\ dur/Users/ftp/VU/DATA/");
    return EXIT_FAILURE;
    }
 
while((entry = readdir(dp)) != NULL) {	
if(!strcmp(entry->d_name,".") || !strcmp(entry->d_name,"..") || !strcmp(entry->d_name,".DS_Store"))
			continue;
 
 strcpy(LeDir,"/Volumes/Disque\ dur/Users/ftp/VU/DATA/");
 strcat(LeDir,entry->d_name);
 
   if((fIn= fopen(LeDir, "r")) == NULL)
   {
      return EXIT_FAILURE;
   }
 
	FinFichier=0;
	while(FinFichier==0)
	{
	 IdTLigne=fgetc(fIn); // lecture de IdTLigne
	 if(IdTLigne==EOF)
		{
		 FinFichier=1;
		 continue;
		}
	 if(IdTLigne=='\n')
		{
		  continue;
		}
 
	 for (i=0;i<5;i++)
		{
		 Operateur[i]=fgetc(fIn);
		}
 	 Operateur[i]='\0';
 
	 for (i=0;i<10;i++)
		{
	 	 LaDateDebut[i]=fgetc(fIn);
		}
	 LaDateDebut[i]='\0';
 
	 for (i=0;i<8;i++)
		{
		 HDeb[i]=fgetc(fIn);
		}
	 HDeb[i]='\0';
 
	 for (i=0;i<7;i++)
		{
		 KMdepart[i]=fgetc(fIn);
		}
	 KMdepart[i]='\0';
 
	 for (i=0;i<5;i++)
		{
		 LeVehicule[i]=fgetc(fIn);
		}
	 LeVehicule[i]='\0';
 
	 for (i=0;i<2;i++)
		{
		 TypeDechet[i]=fgetc(fIn);
		}
	 TypeDechet[i]='\0';
 
	 for (i=0;i<5;i++)
		{
		 NumBac[i]=fgetc(fIn);
		}
	 NumBac[i]='\0';
 
	 for (i=0;i<7;i++)
		{
		 Poids[i]=fgetc(fIn);
		}
	 Poids[i]='\0';
 
	 for (i=0;i<5;i++)
		{
		 SignClient[i]=fgetc(fIn);
		}
	 SignClient[i]='\0';
 
	 for (i=0;i<10;i++)
		{
		 LaDateScan[i]=fgetc(fIn);
		}
	 LaDateScan[i]='\0';
 
	 for (i=0;i<8;i++)
		{
		 LHeureScan[i]=fgetc(fIn);
		}
	 LHeureScan[i]='\0';
 
	 for (i=0;i<7;i++)
		{
		 KMFin[i]=fgetc(fIn);
		}
	 KMFin[i]='\0';
 
	 for (i=0;i<10;i++)
		{
		 LaDateFin[i]=fgetc(fIn);
		}
	 LaDateFin[i]='\0';
 
	 for (i=0;i<8;i++)
		{
		 LHeureFin[i]=fgetc(fIn);
		}
	 LHeureFin[i]='\0';
 
	 for (i=0;i<6;i++)
		{
		 VolumeCarbu[i]=fgetc(fIn);
		}
	 VolumeCarbu[i]='\0';
 
	 for (i=0;i<10;i++)
		{
		 LaDateCarbu[i]=fgetc(fIn);
		}
	 LaDateCarbu[i]='\0';
 
	 for (i=0;i<8;i++)
		{
	 	 LHeureCarbu[i]=fgetc(fIn);
		}
	 LHeureCarbu[i]='\0';
 
	 //lecture du caractère de fin retour à la ligne ou de fin de fichier
	 c=fgetc(fIn);
 
	 if(c==EOF)
		{
		 FinFichier=1;
		 continue;
		}
		switch (IdTLigne) 
				{
				case '1':	
							DebTournee(&ReferenceTournee);
							break;
				case '2':	
							LaCollecte(&ReferenceTournee);
							break;
				case '3':	
							FinTournee(&ReferenceTournee);
							break;
				case '4':	
							PleinCarburant(&ReferenceTournee);
							break;
				case '5':	PeseeClient(&ReferenceTournee); 
							break;
				default: printf("id ligne default: %c\n",IdTLigne);
				}
 
	 }
    fclose(fIn);
	free(LeDir);
	}
 
closedir(dp);
 
return EXIT_SUCCESS;
}

Pourquoi (par exemple) la chaîne 'Operateur' qui est un tableau de 5 caractères, se trouve initialisée avec la quasi totalité de la ligne de mon fichier texte soit 121 caractères?

**Emmanuel Delahaye** · 25/05/2007, 16h14

Envoyé par Rocou

Ce "code" n'était pas destiné à être compilé.<...> Voici ma fonction de lecture de mon fichier texte:
<...>

On pourrait avoir le code qui est "destiné à être complié", parce que là, c'est pas ça...

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
 
Project   : Forums
Compiler  : GNU GCC Compiler (called directly)
Directory : C:\dev\forums\
--------------------------------------------------------------------------------
Switching to target: default
Compiling: main.c
main.c: In function `lecturefichiercsv':
main.c:30: error: `DIR' undeclared (first use in this function)
main.c:30: error: (Each undeclared identifier is reported only once
main.c:30: error: for each function it appears in.)
main.c:30: error: `dp' undeclared (first use in this function)
main.c:33: warning: implicit declaration of function `malloc'
main.c:35: warning: implicit declaration of function `opendir'
main.c:35:18: warning: unknown escape sequence: '\040'
main.c:35: error: `NULL' undeclared (first use in this function)
main.c:37: warning: implicit declaration of function `fprintf'
main.c:37: error: `stderr' undeclared (first use in this function)
main.c:37:60: warning: unknown escape sequence: '\040'
main.c:38: error: `EXIT_FAILURE' undeclared (first use in this function)
main.c:41: warning: implicit declaration of function `readdir'
main.c:41: warning: assignment makes pointer from integer without a cast
main.c:42: warning: implicit declaration of function `strcmp'
main.c:42: error: dereferencing pointer to incomplete type
main.c:42: error: dereferencing pointer to incomplete type
main.c:42: error: dereferencing pointer to incomplete type
main.c:45: warning: implicit declaration of function `strcpy'
main.c:45:15: warning: unknown escape sequence: '\040'
main.c:46: warning: implicit declaration of function `strcat'
main.c:46: error: dereferencing pointer to incomplete type
main.c:48: error: `fIn' undeclared (first use in this function)
main.c:48: warning: implicit declaration of function `fopen'
main.c:56: warning: implicit declaration of function `fgetc'
main.c:57: error: `EOF' undeclared (first use in this function)
main.c:180: warning: implicit declaration of function `DebTournee'
main.c:180: error: `ReferenceTournee' undeclared (first use in this function)
main.c:183: warning: implicit declaration of function `LaCollecte'
main.c:186: warning: implicit declaration of function `FinTournee'
main.c:189: warning: implicit declaration of function `PleinCarburant'
main.c:191: warning: implicit declaration of function `PeseeClient'
main.c:193: warning: implicit declaration of function `printf'
main.c:197: warning: implicit declaration of function `fclose'
main.c:198: warning: implicit declaration of function `free'
main.c:201: warning: implicit declaration of function `closedir'
main.c:203: error: `EXIT_SUCCESS' undeclared (first use in this function)
Process terminated with status 1 (0 minutes, 1 seconds)
15 errors, 22 warnings

**Jean-Marc.Bourguet** · 25/05/2007, 16h19

Envoyé par Rocou

Code :

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

1
2
3
4
5
6
7
8
9
int lecturefichiercsv(void)
{
 
 char Operateur[5]={0}; // CAB
	 for (i=0;i<5;i++)
		{
		 Operateur[i]=fgetc(fIn);
		}
 	 Operateur[i]='\0';

Pourquoi (par exemple) la chaîne 'Operateur' qui est un tableau de 5 caractères, se trouve initialisée avec la quasi totalité de la ligne de mon fichier texte soit 121 caractères?

Et le \0 final, tu le mets ou? char t[N] signifie tableau de N char, pas tableau de char dont le dernier indice est N.