Bonjour,
Merci pour le partage. J’ai cependant quelques remarques à faire, qui pourraient te permettre de t’améliorer.
Pour commencer, il serait bon d’utiliser un type de variable dit « booléen » pour
FirstPhraseMasquer. De plus, les deux noms de variables globales (définies en haut du code source) contiennent une faute, puisque le nom du joueur est
masqué et non
masquer.
Ensuite, dans la ligne suivante :
if (strcmp("/masquernom", cmdtext, true, 10) == 0)
La commande
/masquernom faisant 11 caractères, le quatrième argument de la fonction
strcmp est faux.
Enfin, quelques petits détails me sautent aux yeux, bien qu’ils n’aient pas une importance capitale. Pour commencer, l’indentation laisse à désirer. Je sais par expérience que c’est parfois de la faute de Pastebin, mais je préfère te le faire remarquer, au cas où ce serait volontaire. Puis pour finir, le mélange de l’anglais et du français dans le nom des variables n’est pas génial. Il faudrait choisir, pour une meilleure compréhension.
Je terminerai en t’avouant que pour des codes de cette taille (et de cette simplicité), y proposer des crédits est un peu inutile. Voilà, prends-en note et libre à toi de modifier ça en conséquence.
Bonne journée,
Gzsume.
