Discussion :

[jfouche]

Bonsoir

Supposons :

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
class Module
{
protected:
    typedef bool (Module::*TMessageFct)(const Message& rcMessage);
 
private:
    typedef std::map<int, TMessageFct> TMapMessageFct;
    TMapMessageFct m_mapMessages;
 
public:
    Module() { }
    virtual ~Module() {}
 
    void Connect(int nMsg, TMessageFct pFonction)
    {
        m_mapMessages[nMsg] = pFonction;
    }
};

et :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
class MonModule : public Module
{
public:
    MonModule()
    {
        Connect(1, &MonModule::OnMessage1);
    }
 
    void OnMessage1(const Message& rcMessage)
    {
    }
};

Mon compilateur rale sur le connect, car il trouve que la signature n'est pas bonne. Je peux comprendre, Module::Fonction est différent de MonModule::UnAutreFonction.

Y a t il une methode pour s'en sortir ? Un vieux caste de la mort est il propre ?
En fait, je pensais que comme il y avait héritage, ca pouvait passer...
--
Jérémie

[méphistopheles]

Bonjour

il faut appeler la fonction dans la classe mère:

Code :

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

Module::Connect(1, &MonModule::OnMessage1);

bonne chance

[Nogane]

Je doute que le compilo puisse convertir un "Module::Fonction" en "MonModule::UnAutreFonction", même avec l'héritage. (enfin je n'ai jamais essayé)

Pour ce genre de chose j'utilise boost::signals

[Médinoc]

Pour l'héritage de pointeur de fonction membre, si un simple static_cast<> marche, c'est "safe".
S'il faut autre chose qu'un static_cast<> pour que ça rentre, c'est que tu cherches à faire un truc incorrect (typiquement dans des cas d'héritage multiple).

[Nogane]

Un static_cast ne semble pas marcher.
Ca me fait:

Code :

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

error: invalid static_cast from type 'void (MonModule::*)(const std::Message&)' to type 'bool (Module::*)(const std::Message&)'

Je propose une solution:

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
class Module
{
protected:
  typedef bool (*TMessageFct)(const Message& rcMessage);
  //plus de Module:: ici

private:
  typedef std::map<int, TMessageFct> TMapMessageFct;
  TMapMessageFct m_mapMessages;

public:
  Module() { }
  virtual ~Module() {}

  void Connect(int nMsg, TMessageFct pFonction)
  {
    m_mapMessages[nMsg] = pFonction;
  }
};

class MonModule : public Module
{
public:
  MonModule()
  {
    Connect(1,(TMessageFct)&bind(&MonModule::OnMessage1, this, _1));
  }

  void OnMessage1(const Message& rcMessage)
  {
  }
};

bind se trouve dans boost::bind

[Médinoc]

Le type de retour de la fonction est mauvais: Elle doit retourner un bool

[Nogane]

Woups!!
Autant pour moi!
j'avais pas vu

En effet, en corrigeant ça, avec un static_cast, ça compile

EDIT: et ça fonctionne

[jfouche]

Merci pour vos point de vue.
Apres une bonne nuit de sommeil, j'ai pensé à qqchose du genre :

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
template <class T>
class Module
{
protected:
    typedef void (T::*TMessageFct)(const Message& rcMessage);
 
private:
    typedef std::map<int, TMessageFct> TMapMessageFct;
    TMapMessageFct m_mapMessages;
 
public:
    Module() { }
    virtual ~Module() {}
 
    void Connect(int nMsg, TMessageFct pFonction)
    {
        m_mapMessages[nMsg] = pFonction;
    }
};

et donc une implementation possible par :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
class MonModule : public Module<MonModule>
{
public:
    MonModule()
    {
        Connect(1, &MonModule::OnMessage1);
    }
 
    void OnMessage1(const Message& rcMessage)
    {
    }
};

Pas testé car pas de compilateur sous la main, mais je regarde ça ce soir.
--
Jérémie

[3DArchi]

Ca compile, ca marche, c'est formidable.