A la base, il y a un truc qui ne me plait pas dans ton code : void nombreDeCaractere
Tu ne nommes pas bien ta fonction.
Il faut choisir entre int nombreDeCaracteres et void printNombreDeCaracteres mais choisis un nom qui reflète ce que fait ta fonction.
J'aurais fait cela ainsi :
Code :
- int countLetters(char *string) {
- int count = 0;
- if (string != NULL) {
- do {
- if (isalpha(*string)) ++count;
- } while(*++string) ;
- }
- return count;
- }
|
et pour du code employé en production, j'aurais vérifié en plus que count n'atteignais pas INT_MAX, et utilisé une valeur négative pour signaler une erreur.
A+,
Message édité par gilou le 22-03-2020 à 14:53:03
---------------
There's more than what can be linked! -- Iyashikei Anime Forever! -- AngularJS c'est un framework d'engulé! --