[MFC]Problème de boucle if

Version imprimable

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:

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

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:

Citation:

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:

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 !!! 8O 8O

Citation:

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 !!

01/02/2006, 21h40
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 ?
:D
01/02/2006, 21h47
tus01
Citation:

Envoyé par farscape

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

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

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!
8O
01/02/2006, 21h51
farscape

ok tu as fais l'implementation de la fonction ?
:D
01/02/2006, 21h54
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!

la partie dans le .cpp:

Code:

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

01/02/2006, 22h04
tus01
Mais mon code fait partie de la fonction
Code:

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:

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?

Ma classe avec ma fonction ressemble à ceci:

Code:

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:

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

8O 8O

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:

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:

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?

02/02/2006, 15h33
tus01

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