Discussion :

[10_GOTO_10]

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)).

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.

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.

[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 :

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

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 :

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

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.

[10_GOTO_10]

Après correction des erreurs de syntaxe mineures:

Code :

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

static T& Instance();

remplacé par

Code :

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

static MonSingleton& Instance();

et

Code :

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

MonSingleton::Instance()

remplacé par

Code :

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

MonSingleton& MonSingleton::Instance()

J'obtiens:

[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.

[Matthieu Brucher]

Tu fais pas l'édition des liens avec le fichier .lib ???

[10_GOTO_10]

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

@@Unit2@Finalize
@@Unit2@Initialize
GetData
SetData
__CPPdebugHook

[Matthieu Brucher]

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

[10_GOTO_10]

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 :

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
#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 :

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
#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 :

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
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 :

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
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

[Matthieu Brucher]

Pas de pb, on est là pour chercher

[Patriarch24]

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
 
//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 !

Code :

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

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.

[Matthieu Brucher]

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