Discussion :

[bruce-willis]

Bonjour,

Voici mon 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
#include <string>
 
std::wstring GetNewGUID()
{
	USES_CONVERSION;
	CString strGUID = ""; 
	GUID sessionGUID = GUID_NULL;
	HRESULT hr = CoCreateGuid(&sessionGUID);
	if (sessionGUID != GUID_NULL && HRESULT_CODE(hr) == RPC_S_OK)
        strGUID.Format("{%08lX-%04X-%04x-%02X%02X-%02X%02X%02X%02X%02X%02X}", 
		sessionGUID.Data1, sessionGUID.Data2, sessionGUID.Data3,
		sessionGUID.Data4[0], sessionGUID.Data4[1], sessionGUID.Data4[2], sessionGUID.Data4[3],
		sessionGUID.Data4[4], sessionGUID.Data4[5], sessionGUID.Data4[6], sessionGUID.Data4[7],
		sessionGUID.Data1, sessionGUID.Data2, sessionGUID.Data3,
		sessionGUID.Data4[0], sessionGUID.Data4[1], sessionGUID.Data4[2], sessionGUID.Data4[3],
		sessionGUID.Data4[4], sessionGUID.Data4[5], sessionGUID.Data4[6], sessionGUID.Data4[7]);
	return A2BSTR(strGUID);
}
 
int main()
{
	return 0;
}

En fait, GetNewGUID() est une fonction que j'ai piquée sur des forums mais ça marche pas

[Médinoc]

Ce truc utilise MFC d'une manière parfaitement inutile.
Utiliser la fonction StringFromCLSID().

[bruce-willis]

Quelle en-tête alors pour ta fonction?

Et si on continue avec ma fonction précédemment, qu'est-ce qui cloche plus précisemment?

[Médinoc]

Alors, la fonction précédente:

• Elle utilise MFC.
• Elle suppose que tu compiles en "ANSI"/Multi-byte et pas en Unicode (elle devrait utiliser des TCHAR à la place).
• Elle utilise les vieilles macros de conversion de ATL/MFC (les nouvelles sont des classes C++ plus "safe").
• Le test sur le HRESULT est inutilement élaboré, un simple SUCCEEDED() suffirait.

PS: Pour éviter une allocation de mémoire, tu peux aussi utiliser StringFromGUID2() en lui passant un buffer de 40 WCHARs en paramètre...

[bruce-willis]

StringFromGUID2 semble me convenir mais j'ai pas besoin de tous ces paramètres à entrer
C'est pour cela que j'ai cherché cette fonction qui génère tout simplement un GUID à utiliser

[Médinoc]

"tous" ces paramètres?
StringFromGuid2 demande juste un GUID et un buffer avec sa taille!

Si tu veux une wstring, c'est simple:

Code C++ :

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

1
2
3
4
5
6
7
8
9
10
11
12
std::wstring GetNewGUID()
{
	std::wstring ret = L"";
	GUID sessionGUID = GUID_NULL;
	HRESULT hr = CoCreateGuid(&sessionGUID);
	if(SUCCEEDED(hr)) {
		WCHAR szGUID[40];
		if(StringFromGUID2(sessionGUID, szGUID, 40) > 0)
			ret = szGUID;
	}
	return ret;
}

[bruce-willis]

Oui c'est ça !
Au niveau des #include?

[Gabrielly]

Salut voici un peu de code très pratique où les constructeurs font le travail.

Code :

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

1
2
3
4
5
6
7
 
GUID sessionGUID = GUID_NULL;
HRESULT hr = CoCreateGuid(&sessionGUID);  // New GUID 
CComBSTR bstrGUID(sessionGUID);  // GUID en BSTR
CStringW wstrGUID(bstrGIUD);   // Si on est famillier avec les CString donc en UNICODE
CStringA astrGUID(bstrGUID);  // sinon en ANSI
CString strGUID(bstrGUID);  // Switch entre ANSI ou UNICODE

ou bien

Code :

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

1
2
3
4
5
6
 
GUID sessionGUID = GUID_NULL;
HRESULT hr = CoCreateGuid(&sessionGUID);
 
// j'ai ma chaine en une ligne de code
CString strGUID(CComBSTR(sessionGUID)); // Facile non!!!!

Vu que tu fais soit de l'ATL ou MFC tu as ce sample

[Médinoc]

Attention, le premier code de Gabrielly nécessite une version pas trop obsolète de Visual (comprendre: Pas Visual 6)