Discussion :

[dwarfman78]

Bonjour � tous, j'ai un probl�me d'impl�mentation du singleton.

ConfigParser.hpp :

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
 
#ifndef ConfigParser_h
#define ConfigParser_h
#include <map>
#include "tinyxml/tinyxml.h"
class ConfigParser
{//singleton
 
    private:
    static ConfigParser* myInstance;
    ConfigParser();
 
    public :
    static ConfigParser* getInstance();
    ~ConfigParser();
    float getParam(char*); //return the param with the specified name, param are always of float type
 
    private:
    void addParam(const char*, float);//key, value
    std::map<const char*,float> myConfigVars;
};/#endif

ConfigParser.cpp

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
11
#include "ConfigParser.hpp"
ConfigParser* ConfigParser::getInstance()
{
    if(!ConfigParser::myInstance)
    {
        ConfigParser::myInstance=new ConfigParser();
    }
    return ConfigParser::myInstance;
}
 
etc...

L'erreur est "undefined reference to ConfigParser::myInstance.

De plus je n'initialise pas le pointeur et je ne sais pas o� le faire.

Merci d'avance !

[Mat007]

Salut,

Ca ne r�pond techniquement pas � ta question (cf. la FAQ pour �a) mais pourquoi ne pas faire plus simple ?

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
class ConfigParser
{//singleton
  public :
    static ConfigParser& getInstance();
    float getParam(char*); //return the param with the specified name, param are always of float type
 
  private:
    void addParam(const char*, float);//key, value
    std::map<const char*,float> myConfigVars;
};
 
ConfigParser& ConfigParser::getInstance()
{
    static ConfigParser parser;
    return parser;
}

MAT.

[aziz jim]

Le membre static doit �tre initialis� dans le fichier .cpp .

Code :

S�lectionner tout - Visualiser dans une fen�tre � part

1
2
3
4
5
6
7
8
9
10
11
12
#include "ConfigParser.hpp"
 
ConfigParser* ConfigParser::myInstance = NULL;
 
ConfigParser* ConfigParser::getInstance()
{
    if(!ConfigParser::myInstance)
    {
        ConfigParser::myInstance=new ConfigParser();
    }
    return ConfigParser::myInstance;
}

[dwarfman78]

J'ai r�solu le probl�me en mettant tout le code (header+impl) dans un seul fichier ...

Bizarre cependant je suis pas fan de cette technique ...

Merci quand m�me.

[vdumont]

J'ai r�solu le probl�me en mettant tout le code (header+impl) dans un seul fichier ...

Bizarre cependant je suis pas fan de cette technique ...

Merci quand m�me.

En effet c'est tr�s laid de proc�der ainsi.

[aziz jim]

http://come-david.developpez.com/tut...Singleton#LIII