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