Revue du TP Fractales

fractales_revue.txt

==== Fiche d'évaluation ====

=== Cette évaluation porte sur le travail de ===
identifiant du rendu : ???

=== Identité du relecteur ===
Nom : Lapointe
Prénom : Quentin

=== Compétence du relecteur en programmation C ===
4 : très élevée

=== Appréciation générale ===
2 : bien que certains concepts soient à revoir, la majorité des concepts mis en oeuvre dans ce TP sont acquis

=== Evaluation détaillée ===

      == Compilation ==

Warnings de typage
Soustraction entre 2 unsigned
Valeur absolue d'un unsigned

      == Exécution ==

Une demande de 3 itérations avec all_images à true ne génère que les itérations 0, 1, 2
=> La boucle dans main devrait avoir "i <=" plutôt que "i <" comme condition d'arrêt
./koch 1000 9 0x000000 0xFF0000 kochlow.ppm all => plante à partir de 5 itérations ???
Noms de fichier bizarres pour un nom de fichier petit

      == Algorithmique ==

Pour all_images = true, vous repartez de 0 itération à chaque fois que vous voulez une itération supplémentaire. generer_koch est appelée i(i+1)/2 fois avec i le nombre d'itérations demandé.
=> Vous devriez ne pas réinitialiser la liste chaînée mais demander generer_koch(koch, 1) à chaque tour de boucle pour avoir l'itération suivante. generer_koch sera appelée i fois seulement.
Vous n'avez pas besoin d'arrondir un float avant de le mettre dans un int. Un cast suffit.

      == Mémoire ==

Vous ne libérez pas l'image ni les paramètres.

      == Modularité ==

xxx

      == Lisibilité ==

Absence de fichier README.txt, une personne ne connaissant pas l'énoncé serait perdue.

      == Autres remarques ==

Vous avez modifié certains prototypes.
Vous utilisez un tableau en 2 dimensions pour stocker pic, l'énoncé demandait un simple tableau[size_x * size_y]
Vous oubliez un 0 dans le nom de ficher devant le numéro d'itération => koch1.ppm au lieu de koch01.ppm