Discussion :

[jcloupgarou]

Bonjour,

je fais appel à une fonction "Common()" définie dans une DLL pour effectuer un traitement qui peut etre trés long dans une partie de mon programme. J'aimerais que ce traitement s'arrete si son temps d'execution dépasse les 30s, meme s'il n'est pas terminé. Je mise sur les threads pour gérer cette problématique, mais ca ne fonctionne pas correctement...

J'ai écrit une classe héritant de CWinThread : CCommonThread
Dans la propriété "Run()" de cette classe, j'ai placé la fonction "Common()" de ma DLL.
Dans mon programme, je lance ainsi le nouveau thread :

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
 
CCommonThread* pAdapter = (CCommonThread*)AfxBeginThread( RUNTIME_CLASS(CCommonThread), THREAD_PRIORITY_NORMAL, 0, CREATE_SUSPENDED);
pAdapter->m_bAutoDelete = FALSE;
pAdapter->ResumeThread();
 
DWORD exitCode;
if ( ::WaitForSingleObject( pAdapter->m_hThread, 30000) == WAIT_TIMEOUT )
{
    exitCode = 1;		
}
else
{	
    exitCode = 0;
}
pAdapter->PostThreadMessage( WM_QUIT, 0, 0);

Cela fonctionne au premier passage, mais ensuite, dés que je veux utiliser des fonctionnalités de la DLL, mon appli plante. J'ai l'impression que la DLL n'est pas proprement libérée à la fermeture du thread...

Auriez-vous une idée pour résoudre ce genre de problème. Y a-t-il une issue sans utiliser les threads ?
Tous les exemples de mise en place d'une solution threadée (trouvés sur le web) montre une boucle de traitement dans l'execution du thread. La terminaison du thread s'effectue de lui-meme en mettant une fin à cette boucle. Je ne suis pas dans cette situation, car dés que le thread appelle la fonction "Common()" de la DLL, je n'ai plus la main...

Merci pour votre coup de main !!!

[Médinoc]

WM_QUIT signifie seulement que la boucle de messages du thread destinataire doit se terminer après le prochain GetMessage(). Si ton thread n'a pas de boucle de messages, ça ne lui fera rien.

[jcloupgarou]

Je pense que mon thread gère bien une boucle de message : c'est un thread UI créée avec DECLARE_DYNCREATE, IMPLEMENT_DYNCREATE et DECLARE_MESSAGE_MAP() dans la déclaration et l'implémentation de la classe !

Voici le code de l'entete et de la définition des propriétés :

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
34
35
36
37
38
39
40
41
42
43
44
class CCommonThread : public CWinThread
{
	DECLARE_DYNCREATE(CCommonThread)
public:
	CCommonThread();  
	virtual ~CCommonThread();
        virtual int Run() ;
        virtual int ExitInstance() ;
        virtual BOOL InitInstance();
 
	TopoDS_Shape forme1_;
	TopoDS_Shape forme2_;
 
protected:
	DECLARE_MESSAGE_MAP()
};
 
IMPLEMENT_DYNCREATE( CCommonThread, CWinThread)
CCommonThread::CCommonThread()
{
}
CCommonThread::~CCommonThread()
{
}
BEGIN_MESSAGE_MAP(CCommonThread, CWinThread)
END_MESSAGE_MAP()
int CCommonThread::Run()
{	
	Common( forme1_, forme2_ );	       
 
	return (0) ;
}   
BOOL CCommonThread::InitInstance()   
{       
   return ::InitInstance();   
}   
int CCommonThread::ExitInstance()   
{   
	//AfxEndThread( 0 ); 
	//ExitThread( 0 );
	PostQuitMessage(0);
 
	return CWinThread::ExitInstance();   
}

[Médinoc]

Es-tu sûr que c'est un thread UI? Si j'ai bien compris, ton calcul long est dans la fonction Common(). Si oui, c'est cette fonction qui doit contenir une boucle de messages pour que ton thread soit un thread UI.

[jcloupgarou]

Je ne peux malheureusement rien faire avec la fonction "Common()" qui est définie dans une DLL qui n'est pas de mon ressort et dont je n'ai pas le code source...

[Médinoc]

Hé bien c'est mort alors. On ne peut interrompre proprement un thread que s'il nous aide à l'interrompre.
Sinon, on ne peut l'interrompre que salement, ce qui comporte des risques pour l'intégrité du programme (imagine un TerminateThread() qui tomberait au beau milieu d'un malloc(), pendant que les listes chaînées du tas sont dans un état intermédiaire, et que le thread possède le verrou dessus!)

[TheGzD]

Commence par changer ceci :

Code :

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

pAdapter->m_bAutoDelete = TRUE;

Ensuite dans le cas où ton thread a dépassé les 30s d'exécution essaye de passer les instructions suivantes :

Code :

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

1
2
pAdapter->Delete();
::WaitForSingleObject(pAdapter->m_hThread, INFINITE);

En espérant que ça règle le problème.

[Médinoc]

Pardonne mon franc-parler, mais c'est pourri:

1. Le destructeur de CWinThread (et à fortiori CWinThread:elete()) ne fait rien pour interrompre le thread, il se contente de fermer son handle.
2. Tu fais un Delete() (qui fait un delete this) juste avant d'accéder à un membre de la classe: Boum! comportement indéfini.
3. Tu utilises le handle qui comme dit en 1, vient d'être fermé.

[jcloupgarou]

J'avais effectivement essayé ce genre de chose en controlant le "delete", mais évidemment sans succés : l'utilisation de la Dll contenant la fonction "Common()" est par la suite impossible, le moindre accés fait planter le soft...
Pourtant j'imagine ne pas etre la premiere personne à vouloir faire ce genre de chose, non ?... Déception ;o(

[Médinoc]

Hélas, sur la plupart des platformes, le seul moyen d'interrompre proprement une chose, c'est si la chose en question se laisse interrompre.

En natif sous Windows, la seule chose qu'on puisse interrompre salement sans trop de répercussions, c'est un processus entier. Interrompre salement un thread porte un risque significatif de corruption du processus autour.