Forum Programmation.c Terminé

Posté par  .
Étiquettes : aucune
0
31
mar.
2005
Bonsoir à tous,

J'ai enfin fini mon projet tutoré pour l'IUT, il est assez stable maintenant (j'entend par là que je n'arrive plus à le faire planter mais qu'il reste surement des bugs fourbes tapis dans les recoins sombres).

Je l'ai mis sous GPL dans mon coin, mais je me disais que ça pourrait etre sympa de le rendre disponible sur internet si ça intéresse quelques personnes. Donc en fait, je voudrais savoir si mon bousin vaut le coup d'etre mis en ligne, ou si en fait il est nul :)

En gros, c'est un serveur multi services qui tourne sous Linux et Win32 (aussi sous Mac OS X mais j'ai moins testé), basé sur un système de plugins. Pour l'instant il n'y a qu'un plugin qui fait du http sommaire (en gros, ça gère GET et HEAD).

Voici un lien vers les sources :
http://membres.lycos.fr/jaguar_agency/(...)
(désolé, c'est toujours la page lycos pourrie...)

Voilà voilà, je remercie d'avance ceux qui auront la patience et la gentillesse de jeter un coup d'oeil :)
  • # question de style

    Posté par  . Évalué à 8.

    Je n'ai pas le temps de regarder en profondeur mais au premier coup d'oeil, j'ai regardé le fichier mm_core_parser.c, et j'ai remarqué quelques points :
    • pourquoi rajouter les prototypes au début du .c alors qu'ils sont dans le .h ?
    • dans ParseAlloc(), tu fais :
      parse_t *pRet = NULL;
      pRet = malloc(sizeof(parse_t));
      
      plutot que de mettre sizeof(parse_t), tu devrais mettre sizeof(*pRet) pour qu'un changement de type n'ai pas a etre modifié partout. de plus, pourquoi ne pas faire directement : parse_t *pRet = malloc(sizeof(*pRet));
    • à plusieurs endroits tu utilises l'expression a ? b : c de cette facon : ( cond ) ? a = b : a = c; c'est mieux de faire : a = ( cond ) ? b : c; Ok je cherche la petit bête ;). A part ça le code est bien commenté et se lit bien et je ne pourrais que t'encourager à le mettre en ligne, cela peut toujours interesser des gens, ne serait ce qu'a but éducatif. Bonne continuation
    • [^] # Re: question de style

      Posté par  . Évalué à 1.

      Merci pour les conseils, je ne savais pas qu'on pouvait utiliser sizeof() de cette manière, et c'est très pratique !

      Les prototypes en tete du fichier, je les avait laissés car ils me servaient de table des matières pour les gros modules ;)

Suivre le flux des commentaires

Note : les commentaires appartiennent à celles et ceux qui les ont postés. Nous n’en sommes pas responsables.