DansTonChatConsole

Signaler
Messages postés
11
Date d'inscription
lundi 24 octobre 2011
Statut
Membre
Dernière intervention
6 novembre 2017
-
Messages postés
11
Date d'inscription
lundi 24 octobre 2011
Statut
Membre
Dernière intervention
6 novembre 2017
-
Cette discussion concerne un article du site. Pour la consulter dans son contexte d'origine, cliquez sur le lien ci-dessous.

https://codes-sources.commentcamarche.net/source/101560-danstonchatconsole

Messages postés
3833
Date d'inscription
dimanche 12 décembre 2004
Statut
Modérateur
Dernière intervention
11 juin 2021
122
Bonjour.

Le code est indenté, les fonctions sont bien découpées, et pas trop monolithique. C'est assez propre. Je n'ai pas trouvé de bugs, et ça fonctionne bien.

Quelques critiques (assez minimes):
  • Curl c'est bien, mais c'est peut-être un peu lourd comme dépendance. Un simple socket aurait pu suffire, non ?
  • La gestion des options est assez bancale. Si tu veux faire un truc propre, utilise "getopt", c'est simple, standard, et plus efficace que ce que tu proposes.
  • La ligne de compilation que tu donnes est incorrecte: Il manque un "-std=c99".
  • Petites typos: lenght => length, connection (en anglais) => connexion (en français).
  • chunk.memory = malloc(1); => Je saisis mal l'intérêt de cette ligne. Un realloc(NULL, size), c'est valide, et ça équivaut à un malloc(size). J'aurais clairement mis un: chunk.memory = NULL.
  • Ça manque de commentaires (en doxygen si possible).
Messages postés
11
Date d'inscription
lundi 24 octobre 2011
Statut
Membre
Dernière intervention
6 novembre 2017
>
Messages postés
3833
Date d'inscription
dimanche 12 décembre 2004
Statut
Modérateur
Dernière intervention
11 juin 2021

Voilà, c'est corrigé. Merci
Messages postés
11
Date d'inscription
lundi 24 octobre 2011
Statut
Membre
Dernière intervention
6 novembre 2017
>
Messages postés
3833
Date d'inscription
dimanche 12 décembre 2004
Statut
Modérateur
Dernière intervention
11 juin 2021

Bonjour,
Tout d'abord, merci pour ta réponse.

J'ai fait ce projet pour me familiariser avec curl, pour une utilisation future dans d'autres projets (plus gros?)

Je ne connaissais pas "getopt", je vais allez y jeter un oeil!

Pour le -std=c99, c'est maintenant corrigé.

Les fautes d'orthographes vont être corrigées, merci de me les avoirs soulignés.

Dans le tutoriel que j'avais lu, il disait de mettre un malloc(1) pour ensuite re allouer plus de mémoire. Si ta méthode marche, je l'utiliserai car elle me semble plus propre.

Ah les commentaires, toujours le même problème ;)