Discussion :

[Heliopraetor]

Bonjour !


C'est un étrange problème que je soumets aujourd'hui à votre perspicacité. Dans une application, j'utilise RegisterClassEx. Cette méthode échoue (0 est renvoyé).

Pourtant, si je m'en fiche (pas de if (RegisterClassEx(..)==0) return 0; ), l'application fonctionne apparemment parfaitement, la fenêtre est crée comme je le souhaite, et il n'y a pas de problème.

Je voudrais savoir pourquoi, car malgré tout je préfererais une valeur de retour non nulle.

Merci d'avance

[Twindruff]

Utilise GetLastError et FormatMessage pour avoir plus de détails sur l'échec.
Peut-être que la classe est déjà enregistrée et c'est pour ça que ça marche ? (avec GetClassInfo tu peux savoir si une classe est déjà enregistrée)

[delire8]

Effectivement, nous avions pensé au fait que la classe puisse être déjà enregistrée. Mais nous ne la voyons pas sur spy++ . De plus, le 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
if( GetClassInfoEx(NULL,szClassName,&wc) )
		g_CLogM->AddEvent("Application déjà enregistrée");
	else
	{ 
(construction de wc)

f(!RegisterClassEx(&wc))
		{
			g_CLogM->AddEvent("Méthode RegisterClassEx échouée");
			return 0;
		}

		g_CLogM->AddEvent("Application enregistrée");
}

nous renvoie une fois sur deux un échec, et l'autre "Application Enregistrée", sans jamais passer par "Application déjà enregistreé"... Ce qui tend à nous faire penser que la classe n'est pas déjà enregistrée (et ce ne serait donc pas lié à une mauvaise fermeture du programme).

Pourriez-vous nous indiquer quelques pistes supplémentaires ?

Merci

Cordialement

[Twindruff]

Eh ben déjà utiliser GetLastError et FormatMessage pour avoir plus de détails sur l'erreur

[delire8]

Ca m'a complétement échappé !

Je fais ça tout de suite.

Merci

[delire8]

C'est fait. j'ai modifié mon code pour obtenir le suivant :

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
if(!RegisterClassEx(&wc))
		{
	#ifndef _DEBUG
			CSplashScreen::Exit();
	#endif
			LPVOID lpMsgBuf;
			
			FormatMessage(FORMAT_MESSAGE_ALLOCATE_BUFFER | 
						  FORMAT_MESSAGE_FROM_SYSTEM |
						  FORMAT_MESSAGE_IGNORE_INSERTS,
						  NULL,
						  GetLastError(),
						  MAKELANGID(LANG_NEUTRAL, SUBLANG_DEFAULT),
						  (LPTSTR) &lpMsgBuf,
						  0,
						  NULL);
			MessageBox(g_hDlg,(LPTSTR)lpMsgBuf,"",0);


			g_CLogM->AddEvent("Méthode RegisterClassEx échouée");
			
			return 0;
		}

La plupart du temps, ça passe. Et quand RegisterClassEx rate, il m'envoie un "La classe n'existe pas." . Y'a pas à dire, je suis très avancé !

Alors, pour quelles raisons RegisterClasseEx peut-il échouer ?

Merci

Cordialement

[Heliopraetor]

J'ai utilisé cette fonction. Mais même si la classe n'existe pas, registerclassex échoue de toute manière, l'application pouvant cependant marcher par la suite. Une suggestion ?

[Médinoc]

Bonjour,

1. Pour commencer, je ne suis pas sûr de ce que fait ton CSplashScreen(), mais tu devrais appeler GetLastError() AVANT pour être sûr.
2. Ensuite, tu oublies de libérer la mémoire de ton buffer.
3. Et pour finir, on peut voir la définition précise de ta classe de fenêtre ?

PS: Pense à regarder aussi sur les forums Microsoft : Peut-être quelqu'un a-t-il déjà eu ton problème...

[Heliopraetor]

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
//------Création de la classe de fenêtre
wc.hInstance = hInstance;
wc.lpszClassName = szClassName;     //pointeur vers une chaîne constante
wc.lpfnWndProc = WindowProcedure;
wc.style = 0;
wc.cbSize = sizeof(WNDCLASSEX);
wc.hIcon = LoadIcon(hInstance,MAKEINTRESOURCE(IDI_MYICON));
wc.hIconSm = LoadIcon(hInstance,MAKEINTRESOURCE(IDI_MYICON));
wc.hCursor = LoadCursor(NULL, IDC_ARROW);
wc.lpszMenuName = NULL;
wc.cbClsExtra = 0;
wc.cbWndExtra = 0;
wc.hbrBackground = (HBRUSH) GetStockObject(BLACK_BRUSH);

if(!RegisterClassEx(&wc))
{
	LPVOID lpMsgBuf;
	
	FormatMessage(FORMAT_MESSAGE_ALLOCATE_BUFFER | 
				  FORMAT_MESSAGE_FROM_SYSTEM |
				  FORMAT_MESSAGE_IGNORE_INSERTS,
				  NULL,
				  GetLastError(),
				  MAKELANGID(LANG_NEUTRAL, SUBLANG_DEFAULT),
				  (LPTSTR) &lpMsgBuf,
				  0,
				  NULL);
	
	//on écrit des événements dans un fichier log
	g_CLogM->AddEvent((LPTSTR)lpMsgBuf);
	g_CLogM->AddEvent("Méthode RegisterClassEx échouée");
	
#ifndef _DEBUG
	CSplashScreen::Exit();   //pas de problème avec cette fonction !
#endif
	return 0;
}

Pour info : la fenêtre est destinée à ne recevoir "que du DirectX", elle est volontairement rudimentaire.

Pour la mémoire à libérer, faut-il comprendre que formatmessage alloue de la mémoire ?

[Médinoc]

Eh bien, je ne vois pas grand-chose de bizarre dans la WNDCLASSEX, à part peut-être le GetStockObject() (j'ignore si c'est OK de le mettre dans une classe ou si c'est nuisible comme pour GetSysColorBrush()).

Pour la mémoire à libérer, faut-il comprendre que formatmessage alloue de la mémoire ?

À quoi crois-tu que sert le flag FORMAT_MESSAGE_ALLOCATE_BUFFER ?

[Twindruff]

Parfois le RegisterClass échoue si la MessageProc n'est pas bien formée aussi.
Edit: ah non c'est la CreateWindow qui peut échouer si la MessageProc n'est pas bien formée