Skip to content

Instantly share code, notes, and snippets.

@lapwat
Created April 18, 2016 17:02
Show Gist options
  • Save lapwat/453adde9c8cd79ec2cd8163b978b8097 to your computer and use it in GitHub Desktop.
Save lapwat/453adde9c8cd79ec2cd8163b978b8097 to your computer and use it in GitHub Desktop.
Revue du TP Fractales
==== 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment