Discussion :

[elix63]

Bonjour,
Je devellope actuellement un dll qui plante au link aprés la compilation.

voici ma 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
class Carte{ //Singleton
 
 
private:
 
	Carte(){
		//Initialisation via fichier texte de la liste  list<Debitmetre*> ld
	}
 
 
	private:
	static Carte *c;
 
	public:
	static Carte * getCarte(){ 
		if(c == NULL)
			c = new Carte();
		return c;
	}
 
	list<Debitmetre*> ld;//Liste des debitmetres associés à la carte
 
 
};

Quand je l'appelle dans fonctions.cpp ( qui contient les fonctions d'interactions avec ma dll ), plantage.
J'ai vu pas mal d'article qui parle de ce type d'erreur mais je comprend pas comment je dois le résoudre...

Appel de la méthode statique qui renvoi mon instance de CARTE :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
#include "carte.h"
#include "extern.h" //fichier contenant les fonctions de la dll fournies que je dois interfacer avec mon programme
 
 
static Carte* car;//variable globale destinée à stocker l'instance pour garder les informations entre chaque accés à la dll.
 
 
bool connect(char id)
{
car=Carte::getCarte();
 
return true;
}

Toute aide est la bienvenue merci.

[koala01]

Salut,

Je te conseillerais de passer faire un tour dans la FAQ qui traite des données et fonctionstatiques, et plus particulièrement l'entrée qui traite de l'initialisation des données statiques

Tu y verra que tu dois initialiser ton membre statique dans un et un seul fichier d'implémentation sous la forme de

Code :

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

Type TaClass::TonMembreStatic = saValeurDeDepart

Autrement dit, tu dois écrire

Code :

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

Carte* Carte::c = NULL;

Une fois que tu as ton singleton, il n'est absolument pas nécessaire de garder une variable globale:
Déclare un pointeur de type carte, demande l'instance, et roulez jeunesse

L'utilisation sera donc toujours de l'ordre de

Code :

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

1
2
3
4
5
6
7
8
 
void fonction()
{
   Carte* inst=Carte::getCarte();
   /* ...*/
   /*utilisation de inst pour ce qui t'intéresse */
 
}

[elix63]

Merci ^^

j'avais résolu le problème en faisant un autre style de singleton, le singleton C++ de Wikipedia.

Mais effectivement, j'avais jamais fait gaffe à ce détail xD.

Bonne journée