Discussion :

[poca10]

Bonjour!

J'ai un petit souci , je voudrais déclarer une section critique dans mon pgm comme une variable globale, on m'a dit que c'était possible.
Mais lors de la compilation j'ai qq messages d'erreur

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
 
UINT monThread(LPVOID pParam);
CCriticalSection g_csThread;
 
 
class CMaClasseDlg : public CDialog
{
// Construction
...
}

Le résultat

MaClasseDlg.obj : error LNK2005: "class CCriticalSection g_csThread" (?g_csThread@@3VCCriticalSection@@A) already defined in MaClasse.obj
MaClasseDlg.obj : error LNK2005: "class CCriticalSection g_csThread" (?g_csThread@@3VCCriticalSection@@A) already defined in MaClasse.obj
Debug/MaClasse.exe : fatal error LNK1169: one or more multiply defined symbols found
Error executing link.exe.


J'aime pas trop bcp le fatal error... Il est vrai aussi q je suis pas tres douée nan plus avec les histoire de threads et de section critique
Bref si qqu a une idée, elle serait la bienvenue!


Merci!

[poca10]

Chch plus! Je viens de trouver mon erreur... Si j'avais chch plutot sur MSDN!
Bref il fallait écrire

Code :

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

1
2
 
static CCriticalSection g_csThread;

Mais pkoi il faut la déclarer en static????

[farscape]

le pb c'est que ta variable est declarer dans le header de la classe et non dans la classe
donc à chaque fois que ce header est mis dans un source la variable est declarer d'ou le pb rencontre si on l'a declare static elle existe qu'un seule fois.
si on ne veut pas la declarer static on peut toujours faire ça :

Code :

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

1
2
3
4
5
6
 
#ifndef _DECLAREVAR
CCriticalSection g_csThread; 
#else
extern CCriticalSection g_csThread; 
#endif

et dans un des sources comprenant le header juste avant le #include mettre
#define _DECLAREVAR

[poca10]

Merci pour ton aide

@+

[farscape]

une autre maniere de proceder un peu plus objet

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
 
// dans le header
class MY_GLOBALS
{
public:
   static CCriticalSection g_csThread;
// etc ...
};
 
// declaration dans le .cpp
CCriticalSection MY_GLOBALS::g_csThread ;
 
// utilisation
MY_GLOBALS::g_csThread.Lock(100) ;

cette forme de declaration à ma preference pour des raisons pratiques visibilite claire dans le classview ,regroupement dans une classe container
plus facile pour les recherches et utilisation de IntelliSense

[joker180]

J'ai voulu utilisé ta méthode Objet pour déclarer des variables globales. tout compile comme il faut mais j'ai des erreurs au link du genre:

Code :

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

1
2
 
hreadCalcCore.obj : error LNK2001: unresolved external symbol "public: static void * MY_GLOBALS::hReprise_Mesure" (?Reprise_Mesure@MY_GLOBALS@@2PAXA)

et je ne vois pas d'ou ca peut venir.
une idée peut etre?

[farscape]

salut,
ça veut dire que hReprise_Mesure n'est pas definie :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
 
// dans le header
class MY_GLOBALS
{
public:
   static HANDLE hReprise_Mesure;
// etc ...
};
 
// declaration dans le .cpp
HANDLE MY_GLOBALS::hReprise_Mesure=NULL ;

[joker180]

Et bien pourtant si. voici la declaration complète de ma classe globale:

Code :

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

1
2
3
4
5
6
7
8
9
class MY_GLOBALS 
{ 
public: 
	static CCriticalSection CriticalSection; 
	static HANDLE hReprise_Mesure;
	static HANDLE hStandBy_Calc;
	static HANDLE hStandBy_Core;
	static HANDLE hEvents[2]; 
};

[joker180]

et j'ai une erreur de link pour chaque handle ainsi que pour ma section critique.

[farscape]

et la partie initialisation en dehors de la classe ?

Code :

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

1
2
3
 
// declaration dans le .cpp
HANDLE MY_GLOBALS::hReprise_Mesure=NULL ;

????

[joker180]

Si je le déclare comme tu dis dans mon .cpp j'obtient l'erreur suivante à la compilation:

Code :

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

H:\My Documents\Jean-Christophe\BEST\best_jc\source\essai multithread\multithread\ThreadMeasures.cpp(27) : error C2655: 'hReprise_Mesure' : definition or redeclaration illegal in current scope