Bien coder en C : avis sur bout(s) de code

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)

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:

---

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; }

---

Citation:

---

Envoyé par 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)

---

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

:ccool: