Discussion :

[diogene]

Sur ta fonction check_handle_HKLM()
- Elle ne peut pas retourner un unsigned int : tu retournes -1
- Elle doit retourner HKEY. Mais alors, comment signaler une erreur ? (je ne connais pas les valeurs possibles pour un HKEY valide ni leur type effectif)

La meilleure solution n'est-elle pas de retourner un indicateur de réussite et de passer par la liste des paramètres le HKEY ?

..... check_handle_HKLM(struct s_registre *ptr_registre, HKEY * hKey)

[diogene]

En ce qui concerne la fonction compare_ch,

- Je n'aime pas beaucoup l'utilisation de ces strlen() qui font à chaque fois parcourir la chaine pour trouver le zéro terminal, alors que nous allons être obligé de parcourir les chaînes pour les comparer (donc on pourra détecter à cette occasion le zéro terminal).

- Je n'aime pas non plus ces +- 32 (en plus, ces tests sont faux si les caractères ne sont pas alphabétiques).

- Le type des paramètres pourrait indiquer que les chaîne ne sont pas modifiées par la fonction (qualificatif const)

Compte tenu de mes allergies, je verrais plutôt quelque chose comme :

Code :

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

1
2
3
4
5
6
7
8
9
10
11
int compare_ch(char const *str1, char const *str2)
{
    int res = str1 == NULL || str2 == NULL;
    while(res == 0 && (*str1 != 0 || * str2 != 0))
    {
        res =  tolower(*str1) != tolower(*str2);
        str1++;
        str2++;
    }
    return res;
}

[Merillym]

Sur ta fonction check_handle_HKLM()
- Elle ne peut pas retourner un unsigned int : tu retournes -1
- Elle doit retourner HKEY. Mais alors, comment signaler une erreur ? (je ne connais pas les valeurs possibles pour un HKEY valide ni leur type effectif)

La meilleure solution n'est-elle pas de retourner un indicateur de réussite et de passer par la liste des paramètres le HKEY ?

..... check_handle_HKLM(struct s_registre *ptr_registre, HKEY * hKey)

Merci Diogene, je regarde tout cela, idem pour la fonction comparer_ch.