Singleton sur plusieurs DLL

Version imprimable

Voir 40 message(s) de cette discussion en une page

27/04/2006, 11h29
10_GOTO_10

Citation:

Envoyé par Miles

OK, c'est ce que je pensais. La raison provient sans doute de l'inline du singleton dans unit2.hpp.

Le singleton en lui-même, il n'y a pas de raisons qu'il ne marche pas. J'ai fait un copié-collé exact d'ici: http://fr.wikipedia.org/wiki/Singlet...de_conception)).

Citation:

Envoyé par Miles

Vire la classe template en mettant ce qu'il faut dans ta classe, mais ne mets le corps de la fonction que dans unit2.cpp. Tu vois ce que je veux dire ?

Tout à fait, c'est ce qu'il y avait dans ma première version (voir plus haut), et qui avait exactement le même comportement.

Citation:

Envoyé par Miles

Ca pourra peut-être marcher.

Ca marche ou ça marche pas ? Parce que sans vouloir te vexer, et en te remerciant encore du temps que tu m'accordes, je n'ai pas l'impression que tu maitrises mieux le sujet que moi. Essayer des trucs au hasard en pensant que "ça pourra peut-être marcher", ça fait trois jours que je le fais, sans résultat.
27/04/2006, 11h41
Matthieu Brucher
OK, on va y aller avec du code directement. N'oublie pas que ce qu'il y a sur Wikipedia, c'est pour un singleton dans le cas général, il n'est pas précisé qi ça marchait dans le cas que tu as.
Dans Unit2.cpp :
Code:

1 2 3 4 5 6 7 8 9 10 11 class MonSingleton { private: MonSingleton(); //...définir ici le reste de l'interface public: static T& Instance(); // C'est ce que je fais int Data; };
Dans Unit2.cpp
Code:

1 2 3 4 5 6 MonSingleton::Instance() { static MonSingleton aMoi; return aMoi; }
C'est pas la même chose que ce que tu avais. Essaie une fois.

Je ne peux jamais dire si ça va marcher, ce que je propose, je n'ai pas le temps d'essayer les solutions pour toi, mais je donne ce que je ferai en premier lieu pour arranger ce problème si je le rencontrai. C'est parfois judicieux, pafois pas.
27/04/2006, 12h10
10_GOTO_10

Après correction des erreurs de syntaxe mineures:

Code:

static T& Instance();

remplacé par

Code:

static MonSingleton& Instance();

et

Code:

MonSingleton::Instance()

remplacé par

Code:

MonSingleton& MonSingleton::Instance()

J'obtiens:

Citation:

[Lieur Erreur] Unresolved external 'MonSingleton::Instance()' referenced from D:\TEST_32\TESTSINGLETON\UNIT3.OBJ

En compilant Projet. Evidemment, puisque cette méthode est définie dans Unit2.cpp qui n'est pas dans Projet.
27/04/2006, 13h51
Matthieu Brucher

Tu fais pas l'édition des liens avec le fichier .lib ???
27/04/2006, 14h37
10_GOTO_10

Si, mais le .lib ne contient pas la méthode Instance. Il ne contient que ceci:

Citation:

@@Unit2@Finalize
@@Unit2@Initialize
GetData
SetData
__CPPdebugHook
27/04/2006, 16h02
Matthieu Brucher

Ah oui, c'est ma faute, j'ai oublié le _export devant la méthode Instance() :aie::aie::aie:

:yaisse: C'est fantastique, ça marche.

Ah oui, pour le prochain qui va se prendre la tête avec ça, voilà le code définitif:

Unit2.cpp:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
#pragma hdrstop
#include "Unit2.h"
//---------------------------------------------------------------------------
#pragma package(smart_init)
 
extern "C" int _export GetData() {
  return MonSingleton::Instance().Data;
}
 
extern "C" void _export SetData(int Data) {
  MonSingleton::Instance().Data = Data;
}
 
MonSingleton::MonSingleton()
{
}
 
MonSingleton& _export MonSingleton::Instance()
{
    static MonSingleton aMoi;
    return aMoi;
}

Unit2.h:

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
#ifndef Unit2H
#define Unit2H
 
class MonSingleton
{
private:
    MonSingleton();
    //...définir ici le reste de l'interface
public:
    static MonSingleton& _export Instance();
    // C'est ce que je fais
    int Data;
};
 
extern "C" int _export GetData();
extern "C" void _export SetData(int Data);
 
//---------------------------------------------------------------------------
#endif

Unit3.cpp:

Code:

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
#include <vcl.h>
#pragma hdrstop
#include "Unit3.h"
#include "Unit2.h"
//---------------------------------------------------------------------------
 
#pragma package(smart_init)
#pragma resource "*.dfm"
TForm1 *Form1;
//---------------------------------------------------------------------------
 
__fastcall TForm1::TForm1(TComponent* Owner)
  : TForm(Owner)
{
}
//---------------------------------------------------------------------------
 
void __fastcall TForm1::GetDataDllClick(TObject *Sender)
{
  Edit1->Text = GetData();
}
//---------------------------------------------------------------------------
 
void __fastcall TForm1::SetDataDllClick(TObject *Sender)
{
  SetData(Edit1->Text.ToInt());
}
//---------------------------------------------------------------------------
 
void __fastcall TForm1::GetDataSingletonClick(TObject *Sender)
{
  Edit1->Text = MonSingleton::Instance().Data;
}
//---------------------------------------------------------------------------
 
void __fastcall TForm1::SetDataSingletonClick(TObject *Sender)
{
  MonSingleton::Instance().Data = Edit1->Text.ToInt();
}

Unit3.h:

Code:

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
#ifndef Unit3H
#define Unit3H
//---------------------------------------------------------------------------
#include <Classes.hpp>
#include <Controls.hpp>
#include <StdCtrls.hpp>
#include <Forms.hpp>
//---------------------------------------------------------------------------
class TForm1 : public TForm
{
__published: // Composants gérés par l'EDI
  TEdit *Edit1;
  TButton *GetDataDll;
  TButton *SetDataDll;
  TButton *GetDataSingleton;
  TButton *SetDataSingleton;
  void __fastcall GetDataDllClick(TObject *Sender);
  void __fastcall SetDataDllClick(TObject *Sender);
  void __fastcall GetDataSingletonClick(TObject *Sender);
  void __fastcall SetDataSingletonClick(TObject *Sender);
private: // Déclarations de l'utilisateur
public:  // Déclarations de l'utilisateur
  __fastcall TForm1(TComponent* Owner);
};
//---------------------------------------------------------------------------
extern PACKAGE TForm1 *Form1;
//---------------------------------------------------------------------------
#endif

Merci, pardon si j'ai à un moment douté, et :resolu:

27/04/2006, 18h53
Matthieu Brucher

Pas de pb, on est là pour chercher :)

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
 
//xx.h
class Singleton {
 
public:
    Singleton& getInstance();
    // public interface
private: //ou protected
    Singleton();
    static Singleton* _instance;
    //autre données
};
 
 
xx.cpp
Singleton* Singleton::_instance = 0;
 
Singleton& Singleton::getInstance(){
   if(_instance==0)  // lazy initialization
      _instance = new Singleton;
   return _instance;
}

Note : jamais le constructeur ne doit être public !

Citation:

Code:

1
2
3
4
5
MonSingleton& _export MonSingleton::Instance()
{
    static MonSingleton aMoi;
    return aMoi;
}

Ceci n'est pas du C++, mais du C. static doit être réservé aux données statiques.

28/04/2006, 11h14
Matthieu Brucher

Dit ça à Meyers, Alexandrescu et les autres.
static, c'est du C++ aussi. On a le droit de le mettre ici.

Voir 40 message(s) de cette discussion en une page