Discussion :

[swain.dakota]

Bonjour à tous,

j'essaye d'apprendre à utiliser les threads en c++ avec std::thread sous vs2012,
mais j'ai un petit problème lorsque je test ce bout de code :

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
 
#include <thread>
#include <iostream>
#include <string>
 
class Test
{
public:
 
	void Create()
	{
		std::thread t(&Test::MyThread, this);
	}
 
	void MyThread()
	{
		std::cout<<"hi"<<std::endl;
	}
 
};
 
int main()
{
	Test t = Test();
	t.Create();
}

y'a pas de problème pour l'utilisation basique de std::thread mais lorsque j'essaye de faire ça dans une classe ça ne passe plus (fenêtre abort avec boutons "ignorer", "recommencer" et "continuer")

Une idée ?

merci

[swain.dakota]

Bon je crois que j'ai compris mon erreur

je sais pas si c'est "bon" de faire ça mais quand je passe le std::thread en donnée membre ça marche impec

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
33
 
#include <thread>
#include <iostream>
#include <string>
 
class Test
{
public:
	std::thread m_Thread;
 
	void creer()
	{
		m_Thread = std::thread(&Test::affiche, this);
	}
 
	void affiche()
	{
		while(true)
		    std::cout << "hi" << std::endl;
	}
 
};
 
int main()
{
	Test t = Test();
	t.creer();
 
	while(true)
		std::cout << "ho" << std::endl;
 
	return 0;
}

[Arzar]

Bonjour,
Dans ce cas :

Code :

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

1
2
3
4
5
 
void Create()
{
   std::thread t(&Test::MyThread, this);
}

le std::thread est crée localement dans la fonction, donc son destructeur est appellée à la sortie de la fonction :

Code :

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

1
2
3
4
void Create()
{
	std::thread t(&Test::MyThread, this);
} // <- ici destructeur de t appelé

Hors si un std::thread atteint son destructeur et que join() n'a pas été appelé auparavant alors aussitôt le destructeur appelle std:terminate et le programme s'arrête sur le champ.
Donc le code suivant est légal :

Code :

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

1
2
3
4
5
void Create()
{
   std::thread t(&Test::MyThread, this);
   t.join();
}

mais bien sûr join() est bloquant et attend que le thread se termine, donc Create() va bloquer et donc le thread ne sert pas à grand chose au final.

La bonne solution consiste en effet à faire passer le std::thread comme donnée membre de la classe pour que sa durée de vie ne soit plus liée à celle de la fonction. Par contre il faudra quand même faire attention à ce que join() soit appelé avant que l'objet qui contient le thread soit détruit plus tard.

[swain.dakota]

Ok donc j'avais bien cerné le problème.

Merci d'avoir confirmé