MBOUDISSA / Cinema

Tp Python
0 stars 0 forks source link

Correction #2

Open liZe opened 3 years ago

liZe commented 3 years ago

(Correction supprimé.)

liZe commented 3 years ago

Hum… Je n’avais pas la dernière version de votre travail, mes commentaires ci-dessus sont sans doute faux… Je reviens vers vous avec la correction finale, désolé :/.

liZe commented 3 years ago

Votre application est globalement bien faite et fonctionnelle, c’est très bien.

Tout d’abord, au niveau des bonnes nouvelles : votre code est lisible, simple à comprendre, écrit avec une certaine rigueur. Vous avez globalement bien utilisé le langage, bien suivi le guide de style (même si certaines lignes sont bien trop longues !), et bien suivi les consignes.

On a tout ce qu’il faut dans le code pour créer et accéder à la base de données, avec du code concis. C’est très bien.

L’application est plutôt jolie et engageante quand on arrive sur la page d’accueil, mais … je ne note pas le style ;). On comprend bien comment l’utiliser, c’est fluide (même si certains liens peuvent parfois manquer, comme lors de l’édition des films).

Tout ce que j’ai pu tester fonctionne correctement (ou presque), c’est très bien ! La simplicité de votre code est un grand atout pour avoir un projet lisible et avec peu d’erreurs, c’est tout ce que j’attendais. J’ai eu quelques problèmes avec certains types qui ne sont pas vérifiés (par exemple les dates), mais sinon c’est très solide.

La recherche et l’édition de films fonctionnent bien, sans mauvaise surprise. Bravo à vous.

En conclusion : c’est un très bon travail ! Je suis vraiment désolé de la première correction, j’aurais dû faire plus attention…

MBOUDISSA commented 3 years ago

Pas de soucis concernant la petite erreur de version ^^ Et merci pour votre correction et vos remarques :)