Discussion :

[tus01]

Bonjour j'ai un problème avec le else de ma boucle de test: je teste l'existence d'une clef dans le registre Windows, si celle ci n'existe pas je la crée en l'ouvrant (puisque lorsque on essaye d'ouvrir une clef qui n'existe pas elle est crée par défaut):

Uilisation des bobliothèques suivantes:

Code :

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

1
2
#include "Registry.h"
#include "RegEntry.h"

Code :

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

1
2
3
4
5
6
7
8
9
10
for(i=0;i<m_ListMonday.size();i++)
	{	if ( CRegistry :: KeyExists( "Software\\Workshift editor", HKEY_LOCAL_MACHINE )) 
	//la clef existe on ne fait rien
 
		else 
			//la clef n'existe pas on la crée
			RegMyReg.Open( "Software\\Workshift editor", HKEY_LOCAL_MACHINE );
 
 
	}

L'erreur est la suivante:

C:\activeX\Workshift\WorkshiftPpg.cpp(300) : error C2181: illegal else without matching if

Je change donc mon code et encadre les statements par des {} :

Code :

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

1
2
3
4
5
6
7
8
9
10
for(i=0;i<m_ListMonday.size();i++)
	{	if ( CRegistry :: KeyExists( "Software\\Workshift editor", HKEY_LOCAL_MACHINE )) {
	    //la clef existe on ne fait rien
 
	} else {
			//la clef n'existe pas on la crée
		RegMyReg.Open( "Software\\Workshift editor", HKEY_LOCAL_MACHINE ); }
 
 
	}

et j'obtiens cette fois-ci 5 erreurs !!!

WorkshiftPpg.obj : error LNK2001: unresolved external symbol "public: bool __thiscall CRegistry::Open(char const *,struct HKEY__ *,unsigned long,bool)" (?Open@CRegistry@@QAE_NPBDPAUHKEY__@@K_N@Z)

WorkshiftPpg.obj : error LNK2001: unresolved external symbol "public: static bool __cdecl CRegistry::KeyExists(char const *,struct HKEY__ *)" (?KeyExists@CRegistry@@SA_NPBDPAUHKEY__@@@Z)

WorkshiftPpg.obj : error LNK2001: unresolved external symbol "public: __thiscall CRegistry::CRegistry(unsigned long)" (??0CRegistry@@QAE@K@Z)

WorkshiftPpg.obj : error LNK2001: unresolved external symbol "public: void __thiscall CRegistry::Close(void)" (?Close@CRegistry@@QAEXXZ)

Debug/Workshift.ocx : fatal error LNK1120: 4 unresolved externals
Error executing link.exe.

Merci pour votre aide !!

[farscape]

verifie que tu as bien implementé les fonctions en question
ou que le signature de l'implemention correspond bien a celle definit dans la classe.
autre chose CRegistry :: KeyExists est une fonction static ?
si c'est le cas static est bien present dans la classe ?

[tus01]

CRegistry :: KeyExists est une fonction static ?
si c'est le cas static est bien present dans la classe ?

Voila ce que j'ai dans ma classe pour cette fonction:

Code :

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

1
2
3
4
5
6
7
8
class CRegistry {
public:
//......
static bool	KeyExists(LPCTSTR lpszRegPath, HKEY hRootKey = HKEY_LOCAL_MACHINE);
	bool		SubKeyExists(LPCTSTR lpszSub);
//....
protected:
//....

Donc oui elle est static et oui elle est déclarée!

[farscape]

ok tu as fais l'implementation de la fonction ?

[tus01]

L'implémentation?? Qu'est ce que ça veut dire?
Tu veux dire en passant par MFC ClassWizard?
Dans cce cas non. J'ai juste importé mon Registry.h et c'est tout!

[farscape]

la partie dans le .cpp:

Code :

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

1
2
3
4
5
 
 bool  CRegistry:: KeyExists(LPCTSTR lpszRegPath, HKEY hRootKey/* = HKEY_LOCAL_MACHINE*/)
{
// ton code
}

[tus01]

Mais mon code fait partie de la fonction

Code :

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

1
2
3
void CWorkshiftPropPage::OnSave()
{ //ma fonction
}

qui est associée à un bouton.
Est-ce que je mets toute ma fonction dans le code de

Code :

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

1
2
3
4
bool  CRegistry:: KeyExists(LPCTSTR lpszRegPath, HKEY hRootKey/* = HKEY_LOCAL_MACHINE*/) 
{ 
// ton code 
}

ou est ce que je rajoute juste cette portion de code en haut avec les déclarations?

[tus01]

Ma classe avec ma fonction ressemble à ceci:

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
45
46
47
48
49
Begin CRegistry Class Definition
 * ==================================== */
 
using namespace std;
 
class CRegistry {
 
public:
 
	CRegistry	(DWORD flags = CREG_CREATE);	
	virtual		~CRegistry() { Close(); for (int i=0; i < _reEntries.size(); ++i) delete _reEntries[i]; delete [] _lpszSubKey; }
 
	CRegEntry&	operator[](LPCTSTR lpszVName);
	CRegEntry*	GetAt(size_t n) { assert(n < Count());  return _reEntries.at(n); }
 
	bool		Open(LPCTSTR lpszRegPath, HKEY hRootKey = HKEY_LOCAL_MACHINE,
				DWORD dwAccess = KEY_QUERY_VALUE | KEY_SET_VALUE, bool bAuto = false);
 
	bool		AutoOpen(DWORD dwAccess);
	void		AutoClose();
	void		Close();
	bool		Refresh();	
 
	static bool	KeyExists(LPCTSTR lpszRegPath, HKEY hRootKey = HKEY_LOCAL_MACHINE);
	bool		SubKeyExists(LPCTSTR lpszSub);	
 
	void		DeleteKey();	
 
	__inline	GetFlags()	{	return __dwFlags; }
	__inline	Count()		{	return _reEntries.size(); }
 
	HKEY		hKey;		/* Registry key handle */
 
protected:
 
	DWORD		__dwFlags;
	friend		void CRegEntry::MultiSetAt(size_t nIndex, LPCTSTR lpszVal);
	friend		void CRegEntry::MultiRemoveAt(size_t nIndex);
 
private:
 
	void		InitData();	
	void		DeleteKey(HKEY hPrimaryKey, LPCTSTR lpszSubKey);
 
	HKEY		_hRootKey;
	LPTSTR		_lpszSubKey;
 
	std::vector<CRegEntry *> _reEntries;
};

Je ne comprends pas très bien pourquoi tu me dis de mettre mon code dans:

Code :

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

1
2
3
4
 bool  CRegistry:: KeyExists(LPCTSTR lpszRegPath, HKEY hRootKey/* = HKEY_LOCAL_MACHINE*/) 
{ 
// ton code 
}

[tus01]

Ok je crois que je t'ai répondu n'importe quoi!
J'ai importé 3 fichiers qui me permette d'utiliser les fonctions de la class CRegistry:
RegEntry.h
Registry.h
Registry.cpp

Dans mon .cpp j'ai ceci:

Code :

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

1
2
3
4
5
bool CRegistry::KeyExists(LPCTSTR lpszRegPath, HKEY hRootKey) {
 
	CRegistry cregTemp( NULL );
	return cregTemp.Open(lpszRegPath, hRootKey, KEY_QUERY_VALUE, true);	
}

et non

Code :

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

1
2
3
4
5
6
7
8
class CRegistry { 
public: 
//...... 
static bool   KeyExists(LPCTSTR lpszRegPath, HKEY hRootKey = HKEY_LOCAL_MACHINE); 
   bool      SubKeyExists(LPCTSTR lpszSub); 
//.... 
protected: 
//....

Cette dernière partie de code est ce que j'avais dans mon .h !!
Est ce que ça te permet d'y voir plus clair?

[tus01]

D'accord, je comence à comprendre...
Merci pour ton aide en tout cas!