Discussion :

[vonemya]

Chalut,

J'ai deux fichiers fichier1 et fichier2.

Dans fichier2, je fais les déclarations suivantes:

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
static char * var1 = (char*)malloc(....);
static char * var2 = (char*)malloc(....);
  ....
  ....
void fct()
{
    ....
    ....
    a = var1;
    b = var2;
}
    ....

Dans fichier1

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
int main()
{
   .....
   .....
   var1 = "test1";
   var2 = "test2";
   .....
   //appel à fct qui est déclarée dans fichier2
   fct(param);
   .....
   .....
}

Mais a et b ne prennent pas les valeurs attendues.
Je lance mon prog en mode pas à pas:
Dans le fichier1 juste après les affectations de var1 et var2, ces dernières ont les bonnes valeurs.
Mais après l'appel de fct qui est définie dans le fichier2 là où les vaiables ont été declarées, var1 et var2 n'ont plus ces bonnes valeurs

quelqu'un a une explication pour ça??

Merci d'avance!

[craryb]

quelles sont les valeurs de var a la fin?

[Emmanuel Delahaye]

Dans fichier2, je fais les déclarations suivantes:
[code]
static char * var1 = (char*)malloc(....);
static char * var2 = (char*)malloc(....);
....

Pas du C. On a pas le droit d'appeler une fonction pour initialiser une variable persistante.

[Beldom]

Déjà, y'a un truc qui va pas du tout, si on malloc une chaine, pour y mettre des caractères, c'est

Code :

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

1
2
3
str[0] = 'a';
str[1] = 'b';
str[n] = 0;

ou encore les fonctions qui vont bien : strcpy par exemple.
Parceque sinon, tu as alloué de la mémoire pour rien. Et ca doit être la que ca plante.
Quoi que ton a = var1 sera pas forcément plus valide.
(Me souviens plus de tout ce qui est globale, je m'en sert rarement perso, et quasi jamais pour des chaines)

Je conseillerais quand à moi de ne pas utiliser de globales dans ce cas, et l'usage de la fonction strdup qui alloue elle même... En déclarant les chaines dans le main et en les passant en parametre.
Mais sans le tout le code, je sais pas si tu peux le faire ou pas...

[vonemya]

Pas du C. On a pas le droit d'appeler une fonction pour initialiser une variable persistante.

Effectivement, c'est du C++. D'ailleurs je viens de me rendre compte que j'ai poster dans le forum du C

Est ce que c'est pareil pour le C++: on a pas le droit d'appeller une fonction pour initialiser une variable persistante?

Dans mon cas l'appel à fct n'a pas pour but d'initialiser ces deux variables globales, à un moment donné du programme on modifie leurs valeurs.

[poukill]

Effectivement, c'est du C++. D'ailleurs je viens de me rendre compte que j'ai poster dans le forum du C

Est ce que c'est pareil pour le C++: on a pas le droit d'appeller une fonction pour initialiser une variable persistante?

Dans mon cas l'appel à fct n'a pas pour but d'initialiser ces deux variables globales, à un moment donné du programme on modifie leurs valeurs.

En même temps, les malloc, c'est pas vraiment du C++ ....

[valefor]

Hors mis les problèmes d'allocation, pour le problème de la visibilité des variables, je te recommande :
Fichier 1 :

Code :

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

1
2
3
4
5
6
7
8
 
char * var1 = 0;
char * var2 = 0;
void fct()
{
    a = var1;
    b = var2;
}

Fichier 2 :

Code :

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

1
2
3
4
5
6
7
8
9
 
extern char * var1;
extern char * var2;
int main()
{
   var1 = "test1";
   var2 = "test2";
   fct(param);
}

Enfin point de vue conception c'est pas net ce que tu fait. Est-ce que tes variables devraient être externe pour le fichier 1, pour le fichier 2 (donc qui initialise les variables) ? Pourquoi ne pas passer tes var1 et var2 en paramètre de fct ?