Discussion :

[zoya23]

Bonjour,

J'ai déclaré une variable globale "factsPhone" comme étant une liste de faits (classe Fact) dans un fichier header "initialization.h" et j'ai crée une fonction "initializeFactsPhone()" pour remplir "factsPhone" comme suit :

"initialization.h" :

Code :

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

1
2
3
4
5
6
7
8
9
10
    list<Fact> * factsPhone = new list<Fact> ();   //facts of phone
    void initializeFactsPhone(){
 
        Fact f1 (lp1, "justified", "collected");
        Fact f2 (lp2, "justified", "collected");
        Fact f3 (lp3, "justified", "collected");
        factsPhone->push_back(f1);
        factsPhone->push_back(f2);
        factsPhone->push_back(f3);
    }

Dans un 2ème fichier Entity.cc, j'ai créé deux fonctions : la première sert à appeler la fonction "initializeFactsPhone()" pour initialiser la variable globale "factsPhone" et la 2ème fonction sert à gérer la liste "factsPhone comme suit :

Entity.cc :

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
#include "initialization.h"
...
void Entity::initialize()
{
   if (strcmp("phone", getName()) == 0)
    {
        ....
        initializeFactsPhone();
        ....
    }
 
void Entity::handleMessage(cMessage *msg)
{
    ev << "factsPhone : " << factsPhone->empty() << endl;  //afficher 1 : liste remplie; 0 : liste vide
    int i=0;
    while (!selectedRules->empty())
    {....
     }
}

Mon programme commence par exécuter la fonction Entity::initialize(), j'ai pu afficher le contenu de la liste "factsPhone" dans la même fonction Entity::initialize().
Cependant, lorsque mon programme exécute la fonction Entity::handleMessage, la liste factsPhone est vide!!! le programme affiche "1" pour "factsPhone->empty()".

Je soupçonne que c'est un problème de gestion de variable globale modifiée au sein de la fonction "Entity::initialize()" et réinitialisée dans l'autre fonction "Entity::handleMessage".

J'ai modifié la déclaration de "factsPhone" en rajoutant une fois "static" et une fois "extern" mais ça ne marche toujours pas.

Comment gérer une variable globale modifiée par toutes les fonctions et non réinitialisée pour chaque fonction ?

Pourriez vous SVP m'aider?

Je vous remercie d'avance.

[oodini]

Merci de mettre ton code dans la balise idoine.

[zoya23]

Je m'excuse mais je n'ai pas pu modifier mon message pour mettre le code dans la balise.
Dorénavant je vais le faire, pourriez vous m'indiquer comment le faire?

[bacelar]

Je ne vois rien dans votre code qui permet d'affirmer que la méthode "initializeFactsPhone" soit obligatoirement appelée avant la méthode "handleMessage".

Evitez les variables globales. Utilisez le Design Pattern Singleton au pire.

[ternel]

Je te suggère de transformer ta variable globale en une classe, que tu utilise une seule fois si tu veux.

Par exemple, plutôt qu'avoir:

Code :

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

1
2
3
4
5
int main () {
    initialiser_truc();
    for(int i = 0; i<10; ++i) utiliser_truc(i);
    vider_truc();
}

il vaut mieux avoir ceci:

Code :

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

1
2
3
4
5
 
int main () {
    Truc truc();
    for(int i = 0; i<10; ++i) truc.utiliser(i);
}

Ainsi, il n'y a plus de variable globale (je peux avoir deux Truc), et je peux me reposer sur le destructeur de Truc pour garantir le vidage correct (c'est le RAII).

[Astraya]

Ou alors tu met factsPhone en static.
En déboguant je te met ma main à coupé que ton factPhone dans initializeFactsPhone n'a pas la même adresse mémoire que celui qui est dans ton Entity.cc.