Il est préférable d'initialiser un pointeur à NULL plutôt que 0.
Techniquement, c'est du C, et en C, NULL, c'est 0. Après, oui, c'est plus lisible NULL.
Il vérifie que l'argument débute par un tiret mais ne vérifie pas la taille pour être sûr que l'argument a bien plus d'un caractère ce qui serait je pense pertinent.
En même temps, je vérifie le 2nd caractère dans le switch. Du coup, déjà je check si '0円' ou '-', puis je vérifie si le 2nd est connu. Si le 2nd est pas connu, ce qui implique le '0円', je pars en erreur.
Puis il vérifie deux fois que arg n'est pas nul, c'est inutilement redondant.
Pas faux.
Bref, c'était ma revue de code. Elle vaut ce qu'elle vaut.
Et je vous remercie tous les deux, c'est toujours sympa une revue.
C'est clair que le code est améliorable, j'ai fini de bricoler ça a tôt hier matin faut dire ^ (bien que ça ne soit qu'une mauvaise excuse... bon, on va faire comme si jétais prof: c'étais pour l'exercice... non?)
[^] # Re: Petite review
Posté par freem . En réponse au journal bout de code pour relancer une commande dans certaines conditions. Évalué à 2.
Techniquement, c'est du C, et en C, NULL, c'est 0. Après, oui, c'est plus lisible NULL.
En même temps, je vérifie le 2nd caractère dans le switch. Du coup, déjà je check si '0円' ou '-', puis je vérifie si le 2nd est connu. Si le 2nd est pas connu, ce qui implique le '0円', je pars en erreur.
Pas faux.
Et je vous remercie tous les deux, c'est toujours sympa une revue.
C'est clair que le code est améliorable, j'ai fini de bricoler ça a tôt hier matin faut dire ^ (bien que ça ne soit qu'une mauvaise excuse... bon, on va faire comme si jétais prof: c'étais pour l'exercice... non?)