Discussion :

[robinsondesbois]

Bonjour,

J'ai une class A de ce type :

Code :

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

1
2
3
4
5
6
7
8
 
class A
{
public:
    static void foo();
private:
    int a;
}

J'initialise un pointeur sur ma fonction foo : (void)(*ptr)(void). Je passe ce pointeur à un thread pour qu'il rappelle foo quand cela sera nécessaire. Ceci fonctionne très bien mais (parce que sans mais je ne poserais pas de question) je souhaite modifier la variable 'a' via foo.

Le problème que j'ai est donc :
- pour modifier a je ne doit pas mettre le mot clé static
- pour utiliser un pointeur sur fonction d'une méthode, il faut qu'elle soit static.

Existe-il un tricks pour contourner mon problème ?

Cordialement,
Robin

[Médinoc]

Si le seul prototype auquel tu aies droit est void (*ptr)(void), alors tu n'as pas d'autre solution que d'utiliser une variable globale/statique/TLS/etc. pour transporter le this.

[robinsondesbois]

Bien vu.

Pour le moment je faisais ceci :

Code :

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

1
2
void (*ptr)(void);
ptr = &A::foo;

Tu me propose de faire :

Code :

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

1
2
void (*ptr)(A*);
ptr = &A::foo;

Et a mon thread je lui envoie le nouveau pointeur sur fonction ET le pointeur sur mon instance de A ?

Après pour des soucis de classe auto-imbriqué je préférerais ne pas avoir a inclure A dans mon thread.
Peut être qu'avec un système d'héritage ça pourrait être propre.

EDIT : Dans mon projet, un objet de ma classe A instancie un objet de ma classe B qui fabrique un thread effectuant une MAJ sur mon objet A.

[Médinoc]

La manière la plus générique, mais pas la plus C++ (c'est la manière qu'on voit le plus souvent pour les threads, d'ailleurs), c'est ceci:

Code C++ :

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
class A
{
public:
	void foo();
	static void Sfoo(void *p) { static_cast< A* >(p)->foo(); };
private:
	int a;
};
 
...
 
void (*ptr)(void*);
void *ptrData;
 
A* pA = new A;
ptr = &A::Sfoo;
ptrData = pA;
 
LaunchNewThread(ptr, ptrData);

[robinsondesbois]

Merci

Je vois enfin une utilité concrète au (void*).
Les appelles de fonction se font très bien.

Bon après j'ai toujours un petit problème au niveau de mes mutex mais ça sera peut-être pour un autre thread

Merci beaucoup pour ton aide !

[Iradrille]

Peux tu modifier le type du pointeur de fonction ?

Si oui transforme ça en std::function<void()> et l'utilisation est simple

Code :

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

1
2
3
4
5
6
7
8
9
10
11
class A {
   std::function<void()> getFunction() const {
      return [this]() { foo(); };
   }
 
   void foo() { a = 5; }
   int a;
};
 
A a;
std::function<void()> func = a.getFunction();

[robinsondesbois]

Je ne connaissais pas le conteneur function.
Je vais tester les deux méthode pour voir.

Merci a vous