Discussion :

[Plomeg]

bonjour,

Comme l'explique le titre je voudrais initialiser un static map:

Code :

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

 static std::map<T1,T2> conteneur_map;

Je voudrais qu'il soit vide au départ.

J'ai trouvé des doc et autres dessus mais je n'ai rien compris...

Merci d'avance

[corrector]

Comme l'explique le titre je voudrais initialiser un static map:

Code :

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

 static std::map<T1,T2> conteneur_map;

Je voudrais qu'il soit vide au départ.

Il y a static et static. static comment?

• variable statique déclarée à portée globale?
• variable statique déclarée dans une fonction?
• membre statique d'une classe?

[ram-0000]

Je voudrais qu'il soit vide au départ

Le constructeur d'un map fait qu'il est vide au départ

[vimes]

Si la map est un membre statique d'une classe - elle sera vide par défaut - mais il me semble que la solution la plus 'sûre' serait de créer un couple de fonctions qui initialise et de-initialise la partie statique de la classe.

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
 
class A
{
  static bool ms_bInitialised = false;
  static std::map<T1,T2> conteneur_map;
  static void sInit();
  static void sClose();
}
static void A::sInit()
{
    if(!ms_bInitialised)// à remplacer par un assert si nécessaire
    {
       conteneur_map = std::map<T1,T2>();
       ms_bInitiliased = true;
    }
 
}
static void A::sClose()
{
 
    if(ms_bInitialised)// à remplacer par un assert si nécessaire
    {
       conteneur_map.clear();
       ms_bInitiliased = false;
    } ;
}

Il faudrait donc appeler la fonction sInit avant d'instancer la classe et appeler sClose en fin de programme ou lorsque la classe n'est plus utile.

[corrector]

Si la map est un membre statique d'une classe - elle sera vide par défaut - mais il me semble que la solution la plus 'sûre' serait de créer un couple de fonctions qui initialise et de-initialise la partie statique de la classe.

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
 
class A
{
  static bool ms_bInitialised = false;
  static std::map<T1,T2> conteneur_map;
  static void sInit();
  static void sClose();
}
static void A::sInit()
{
    if(!ms_bInitialised)// à remplacer par un assert si nécessaire
    {
       conteneur_map = std::map<T1,T2>();
       ms_bInitiliased = true;
    }
 
}
static void A::sClose()
{
 
    if(!ms_bInitialised)// à remplacer par un assert si nécessaire
    {
       conteneur_map.clear();
       ms_bInitiliased = false;
    } ;
}

Pourquoi plus sûr?

Tu penses que le constructeur par défaut et le destructeur de std::map sont bugués?

[ram-0000]

Si la map est un membre statique d'une classe - elle sera vide par défaut - mais il me semble que la solution la plus 'sûre' serait de créer un couple de fonctions qui initialise et de-initialise la partie statique de la classe.

Plus sûr : non
plus propre : peut être.

Cela a au moins le mérite de faire (et de demander au développeur ?) un appel explicite mais cela n'apporte rien de plus.

[corrector]

Plus sûr : non
plus propre : peut être.

En quoi c'est plus propre?

Cela a au moins le mérite de faire (et de demander au développeur ?) un appel explicite mais cela n'apporte rien de plus.

En gros on créé une contrainte sur l'utilisation de la map; pourquoi?

[ram-0000]

En quoi c'est plus propre?



En gros on créé une contrainte sur l'utilisation de la map; pourquoi?

Parce que je n'aime faire le nettoyage de manière implicite par le destructeur. J'aime bien le faire de manière explicite. Peut être une habitude que j'ai prise du C.

Bon, c'est sur que dans ce cas, il y a la méthode map.clear() qui fait la même chose (et qui est certainement appelée par le destructeur)

[corrector]

Parce que je n'aime faire le nettoyage de manière implicite par le destructeur. J'aime bien le faire de manière explicite.

Oui, mais pourquoi?

Peut être une habitude que j'ai prise du C.

Est-ce encore pertinent en C++?

Bon, c'est sur que dans ce cas, il y a la méthode map.clear() qui fait la même chose (et qui est certainement appelée par le destructeur)

Peut-être, mais pas forcément : clear() a une post-condition (exprimée en terme de *this), le destructeur non, par définition.

[vimes]

Plus sûr : non
plus propre : peut être.
Cela a au moins le mérite de faire (et de demander au développeur ?) un appel explicite mais cela n'apporte rien de plus.

Tu as raison : en fait, je voulais dire que c'était plus sûr en terme de developpement.

[Plomeg]

désolé pour la précision mais c'est bien static de classe, donc à initialiser à l'extérieur sinon erreur de linker (c'est mon cas). Il faudrait donc faire quelque chose comme:

Code :

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

std::map<T1,T2> ClasseProprietaire::conteneur_map =  "quelque chose";//en dehors de la classe

Enfin je crois. Je pense qu'il faudrait créer un map du même type et faire égal (mais j'ai tester et ça ne veut pas non plus).

[corrector]

désolé pour la précision mais c'est bien static de classe, donc à initialiser à l'extérieur sinon erreur de linker (c'est mon cas). Il faudrait donc faire quelque chose comme:

Code :

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

std::map<T1,T2> ClasseProprietaire::conteneur_map =  "quelque chose";//en dehors de la classe

Enfin je crois. Je pense qu'il faudrait créer un map du même type et faire égal (mais j'ai tester et ça ne veut pas non plus).

Non, il suffit de la définir à l'extérieur de la classe (une seule fois dans le programme) :

Code :

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

1
2
3
// dans ClasseProprietaire.cpp :
std::map<T1,T2> ClasseProprietaire::conteneur_map; 
// initialisation par défaut (vide)

ou bien

Code :

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

1
2
std::map<T1,T2> ClasseProprietaire::conteneur_map = une_autre_map; 
// initialisation par recopie

ou

Code :

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

1
2
std::map<T1,T2> ClasseProprietaire::conteneur_map (debut, fin); 
// initialisation avec la range [debut, fin)