Discussion :

[socrate]

Bonjour!
Je ne comprends pas pourquoi cet exemple ne marche pas. Comment devrais-je faire cela selon vous?

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
 
class A
{
public:
  void f(){}
};
 
int main(int argc,char* argv[],char* envp[])
{
  typedef void (A::*FCT)();
  FCT fct = A::f;
  fct();
}

Erreur de compilation à la ligne fct():
error C2064: le terme ne correspond pas à une fonction qui prend les arguments 0

Merci d'avance

[JolyLoic]

Code :

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

FCT fct = &A::f;

[Rely]

pour compléter la reponse de JolyLoic j'ajouterai qu'il faut déclarer (ou créer) une instance car la fonction f ne peut être appelée si ce n'est par une instance.

Code :

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

1
2
3
4
5
6
7
8
 
int main(int argc,char* argv[],char* envp[]) 
{ 
  typedef void (A::*FCT)(); 
  A a;
  FCT fct = &A::f; 
  (&a->*fct)(); 
}

[socrate]

Oui pardon, l'appel deveit être fait depuis une fonction membre. Mais mettre un '&' ne résoud pas mon problème, je crois que ca ne change rien (il est implicite): le code suivant génère la même erreur.

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 A
{
public:
  void f(){}
  void g()
  {
    typedef void (A::*FCT)();
    FCT fct = &A::f;
    fct();
  }
};
 
int main()
{
}

Petite précision: j'ai besoin si possble que g soit non statique

[Rely]

pour le & tu as raison.

en fait le problème se trouve dans la façon dont tu appelles la fonction membre. Je pense que tu n'as pas bien lu mon post. il faut appeler f dans le contexte d'une instatnce de la classe car c'est une fonction memebre non static. ça donne

Code :

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

1
2
3
4
5
6
7
8
 
int main(int argc,char* argv[],char* envp[]) 
{ 
  typedef void (A::*FCT)(); 
  A a; 
  FCT fct = A::f; 
  (&a->*fct)(); 
}

[Rely]

Excuse moi c'est moi qui n'est pas lu ton post. je crois que la solution à ton problème c'est

Code :

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

1
2
3
4
5
6
7
 
void g() 
  { 
    typedef void (A::*FCT)(); 
    FCT fct = &A::f; 
    (this->*fct)(); 
  }

[socrate]

Oui ca marche! J'avais essayé, mais sans mettre le this. Tu comprends pourquoi (*fct)() ne marche pas? Merci en tous cas, cas sans ca j'étais obligé de le faire en assmbleur et c'était pas très joli

[Rely]

Oui ca marche! J'avais essayé, mais sans mettre le this. Tu comprends pourquoi (*fct)() ne marche pas? Merci en tous cas, cas sans ca j'étais obligé de le faire en assmbleur et c'était pas très joli

Je vais tenter d'expliquer pourquoi (*fct)() ne marche pas. fct telle que déclarée (typedef void (A::*FCT)(); FCT fct peut adresser n'importe quelle fonction de prototype 'void fonction()' de n'importe quelle instance de A; Il est donc nécessaire de préciser l'intance dont la fonction 'f' sera addressé par 'fct' d'où la nécessité de mettre '(this->*fct)()' (cela permet par exemple de faire la différencier si on avait créer une intance a de la classe A).
J'espère t'avoir aidé à comprendre dans cette tentative d'explication.

[socrate]

Oui je suis d'accord, mais normalement dans les méthodes d'une classe, le this-> est implicite.
Ainsi quand on fait m() ca veut dire this->m(), et a->m() fait l'appel pour a, et il n'y a jamais d'ambigüité

[Rely]

this n'est implicite que pour les données ou méthodes membres!
Etant donné que fct qui est une variable locale, n'est pas membre de la classe A c'est tout à fait normal de préciser explicitement l'instance avec this en l'occurence.

[socrate]

D'accord, je crois que j'ai compris. Merci

[JolyLoic]

Effectivement, pour l'appel, j'avais loupé qu'il manquait un objet auquel appliquer la fonction. Par contre, pour le &, j'insiste. Il n'est implicite que pour les fonctions libres, pas pour les fonctions membres.

Je crois que c'est dans le D&E que Bjarne Stoustrup indique que s'il avait eu le choix, il l'aurait aussi rendu explicite pour les fonctions libres.