Discussion :

[dede92]

Bonjour,

J'essaie de retrouver le chemin complet d'une source de données ODBC
mais je ne comprends rien au resultat de mes tests :
Je passe sytématiquement dans la routine d'erreur qui m'affiche "L'Opération a réussi" !!!
Voici 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
16
17
18
19
20
21
22
23
24
25
26
27
 
static HKEY  hKey;
LPBYTE data;
char Buf[MAX_PATH];
DWORD  dwBufLen = sizeof(Buf);
LPDWORD lpType;
LPVOID lpMsgBuf;
char SubKey[] = "Software\\ODBC\\ODBC.INI\\compta";
//    data=Buf[0];
//
  if (RegOpenKeyEx (HKEY_CURRENT_USER, SubKey, 0, 0, &hKey) == ERROR_SUCCESS) {
    if (RegQueryValueEx( hKey, "DBQ", NULL, NULL, (LPTSTR)&data, &dwBufLen) == ERROR_SUCCESS)
       MessageBox(NULL, (LPCTSTR)data, "Erreur RegQueryValueEx", MB_OK|MB_ICONINFORMATION);
    else {
        FormatMessage( FORMAT_MESSAGE_ALLOCATE_BUFFER | 
                       FORMAT_MESSAGE_FROM_SYSTEM | 
                       FORMAT_MESSAGE_IGNORE_INSERTS, NULL, GetLastError(), 0, (LPTSTR) &lpMsgBuf, 0, NULL);
        MessageBox(NULL, (LPCTSTR)lpMsgBuf, "Resultat RegQueryValueEx", MB_OK|MB_ICONINFORMATION);
     }
   }
  else {
        FormatMessage( FORMAT_MESSAGE_ALLOCATE_BUFFER | 
                       FORMAT_MESSAGE_FROM_SYSTEM | 
                       FORMAT_MESSAGE_IGNORE_INSERTS, NULL, GetLastError(), 0, (LPTSTR) &lpMsgBuf, 0, NULL);
      MessageBox(NULL, (LPCTSTR)lpMsgBuf, "Erreur RegOpenKeyEx", MB_OK|MB_ICONINFORMATION);
  }
  RegCloseKey(hKey);

Certes, je n'ai plus programmé en C depuis un certain temps,
mais si quelqu'un peut me dépanner ...

Merci d'avance

[Médinoc]

Bonjour,

1. Les fonctions de registre n'utilisent pas GetLastError() : Tu dois mémoriser leur valeur de retour et la passer à FormatMessage().
2. Pourquoi lpMsgBug est-il de type LPVOID plutôt que LPCTSTR LPTSTR?
3. Tu devrais mettre l'affichage de message dans une fonction séparée.

Code C :

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

1
2
3
4
5
6
7
8
9
10
11
12
void ErrorMessage(HWND hWndParent, DWORD err, LPCTSTR title)
{
	LPTSTR lpMsgBuf = NULL;
	FormatMessage( FORMAT_MESSAGE_ALLOCATE_BUFFER | 
		FORMAT_MESSAGE_FROM_SYSTEM | 
		FORMAT_MESSAGE_IGNORE_INSERTS, 
		NULL, err, 0, (LPTSTR) &lpMsgBuf, 0, NULL);
 
	MessageBox(hWndParent, lpMsgBuf, title, MB_OK|MB_ICONINFORMATION);
 
	LocalFree(lpMsgBuf);
}

[dede92]

Bonjour,

J'ai fait comme tu m'as dis, c'est plus propre ...
Mais j'ai toujours des warnings à la compil ...

Code :

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

1
2
3
4
 
1>e:\dev_2011\chap macompta\macompta.c(69) : warning C4090: 'fonction'*: qualificateurs 'const' différents
1>e:\dev_2011\chap macompta\macompta.c(120) : warning C4047: 'fonction'*: 'LPBYTE' diffère de 'char (*)[260]' dans les niveaux d'indirection
1>e:\dev_2011\chap macompta\macompta.c(120) : warning C4024: 'RegQueryValueExA'*: types différents pour le paramètre formel et réel 5

Je pense sue ça vient du fait que Buf n'est pas LPBYTE, non ?
Nouveau 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
 
//
    retour = RegOpenKeyEx (HKEY_CURRENT_USER, SubKey, 0, KEY_QUERY_VALUE, &hKey);
    if (retour == ERROR_SUCCESS) {
      retour=RegQueryValueEx( hKey, "DBQ", NULL, NULL, &Buf, &dwBufLen);
      if (retour == ERROR_SUCCESS)
        MessageBox(NULL, (LPCTSTR)Buf, "Resultat RegQueryValueEx", MB_OK|MB_ICONINFORMATION);
      else 
        ErrorMessage (hWnd, retour, "Erreur RegOpenKeyEx");
    }
    else 
        ErrorMessage (hWnd, retour, "Erreur RegQueryValueEx");
    RegCloseKey(hKey);
//

Merci.

[Médinoc]

En effet, buf devrait être un tableau de BYTE.
Pour bien faire, tu devrais le convertir en autre chose (DWORD, tableau de TCHAR, etc.) après que RegQueryValueEx ait retourné, et après avoir vérifié le type de la valeur (quatrième paramètre).

PS: J'ai fait une erreur dans ma fonction, lpMsgBuf doit être de type LPTSTR et non pas LPCTSTR.

[dede92]

C'est corrigé et ça fonctionne correctement !
Merci !
Et tous mes voeux pour 2011 ...