Discussion :

[totoscill]

Bonjour,

Je rencontre un problème pour lire une variable statique de ma classe singleton "model".
En effet dans cette classe, je declare:

.h:

Code :

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

1
2
3
4
5
 
public:
static int maVariable;
int getVariable()
void setVariable(int *p_varaible)

.c:

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
 
int maVariable;
 
int model::getVariable()
{
   return maVariable;
}
 
void model::setVariable(int *p_varaible)
{
   maVariable = *p_variable;
}
 
model* model::getInstance()
{
	if ( NULL == instance )
	{
		instance =  new model();
	}
	return instance;
};

Jusque la, aucun problème. Mon constructeur et destructeur sont bien privé et mon instance n'est créée qu une seule fois (principe du singleton).

Le problème est lors de l'accès à ma variable. Lorsque j'initialise ma variable dans une classe donnée et que je lis cette même variable dans la même classe, aucun problème. Le problème est lorsque je veux relire la variable initialisée dans la classe précédente dans une autre classe, model::getVariable() retourne "0". Je recupère bien l'instance avant d'accéder à ma variable dans l'autre classe avec getInstance().

J'ai essayé avec une variable "en dur" dans ma classe et pas de problème (ce qui est normale).

Je veux accéder à cette variable dans n'importe quelle classe de mon projet d'où cette question:

Le singleton est il bien fait pour ce type de problème?

Est il préférable d'utiliser un segment commun (dataseg) pour ce type de problème?

D'où pourrait venir le problème?

Je vous remercie.

[ymoreau]

Je ne vois pas où serait le problème désolé.

Par contre, petite critique : l'idée d'un objet singleton est qu'il soit instancié une seule fois et qu'il soit accessible de n'importe grâce à la fonction statique getInstance(). Du coup on a bien une instance de classe (même si elle est unique), alors que tu utilises une variable statique pour maVariable, puisque ta classe est instanciée tu devrais utiliser une donnée membre normale et pas statique.

Edit : je viens de voir que tu redéclares ta variable maVariable dans ton .cpp ! J'imagine que tu voulais l'initialiser, mais pour bien faire appel à ta donnée membre statique il faut préciser qu'elle vient de ta classe. Tel qu'il est ton code déclare une variable globale au .cpp qui porte le même nom que ta variable de classe, il est probable qu'il y ait confusion.
Tu devrais mettre dans .cpp :

Code :

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

int model::maVariable = 0;

Mais je pense quand même qu'il vaut mieux utiliser une donnée membre normale plutôt que statique, que tu initialiserais dans ton constructeur.

[totoscill]

Edit : je viens de voir que tu redéclares ta variable maVariable dans ton .cpp ! J'imagine que tu voulais l'initialiser, mais pour bien faire appel à ta donnée membre statique il faut préciser qu'elle vient de ta classe. Tel qu'il est ton code déclare une variable globale au .cpp qui porte le même nom que ta variable de classe, il est probable qu'il y ait confusion.
Tu devrais mettre dans .cpp :

Code :

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

int model::maVariable = 0;

Merci,

Petite erreur de frappe, je déclare bien ma variable de cette façon.
Je vai voir ce que ça donne en la passant en membre normal plutot que statique.

[3DArchi]

Salut,
Pourquoi ne pas tirer profit des références et utiliser le const :

Code :

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

1
2
3
int getVariable() const;
void setVariable(int const &r_varaible);
model& getInstance();

[totoscill]

Salut,
Pourquoi ne pas tirer profit des références et utiliser le const :

Code :

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

1
2
3
int getVariable() const;
void setVariable(int const &r_varaible);
model& getInstance();

J'ai essayé quelque chose comme ça mais sans effet. Je retest ça dès que possible. Merci.

[totoscill]

J'ai retravailler sur ce problème. Le problème viens du fait que je fais appel au singleton dans des process différent d'ou le problème d'accès à la variable puisque les process ont chacun leur propre zone mémoire.

Etes vous d'accord?

Si oui on peut dire que le problème au résolu. Je vais partager une zone mémoire pour resoudre le problème.