Le_BuG63
Messages postés11Date d'inscriptionlundi 24 octobre 2011StatutMembreDernière intervention 6 novembre 2017 25 juil. 2016 à 15:19
Voilà, c'est corrigé. Merci
Le_BuG63
Messages postés11Date d'inscriptionlundi 24 octobre 2011StatutMembreDernière intervention 6 novembre 2017 25 juil. 2016 à 12:39
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 ;)
cptpingu
Messages postés3837Date d'inscriptiondimanche 12 décembre 2004StatutModérateurDernière intervention28 mars 2023123 25 juil. 2016 à 12:24
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).
25 juil. 2016 à 15:19
25 juil. 2016 à 12:39
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 ;)
25 juil. 2016 à 12:24
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):