Liste de course

Je suis rincer, j'ai passé mon dimanche la dessus :)

Hello Claude !

Quel est ton soucis exactement avec ton code ? Comme ça je pourrais t'aider à y avoir plus clair :)

hello Kévin,
C'est surtout pour voir, si vous trouvez des grosse erreurs ?!?
merci :)

Je ne vois aucune erreur de mon côté, tout à l'air fonctionnel !

Je peux néanmoins te suggérer des axes d'améliorations :)

• La ligne "print(f"==>> liste_de_courses: {liste_de_courses}")" n'est pas nécéssaire étant donné que ta liste de courses sera toujours vide au début de l'exécution du programme

• Tu pourrais ajouter une base de données (par exemple en sqlite3 en local pour commencer) afin de stocker ta liste de courses, même après arrêt & redémarrage du programme, et ainsi ta première ligne deviendrai intéressante.

• Et ensuite, tu pourrais également passer tout ton programme en programmation orientée objet, avec des classes et des attributs.

Si tu as des questions, n'hésites pas!

Salut Claude,

Je rajoute quelques éléments à la réponse de Kévin :

• Attention avec l'utilisation de isalpha, cela vérifie uniquement si la chaîne de caractères contient des lettres de A à Z. En l'état ton script plante si par exemple je donne un symbole ($, %, ?, etc). Dans ce cas-ci il est préférable de vérifier si l'élément envoyé par l'utilisateur est un nombre (avec isdigit).

• Pour la suppression d'un élément dans ta liste, la méthode remove va retourner une erreur si tu essaies d'enlever un élément qui n'est pas dans ta liste. Tu pourrais vérifier si c'est le cas avant pour éviter les erreurs.

• Pour cette partie :

if len(liste_de_courses) == 0 :
    print("la liste de course est vide")
elif len(liste_de_courses) >0:
    for i, valeur in enumerate(liste_de_courses, 1):
        print(i, valeur)

Tu peux grandement simplifier ces conditions en évitant l'utilisation de la fonction len. Une liste vide est évaluée comme False, tu peux donc juste faire :

if liste_de_courses :
    for i, valeur in enumerate(liste_de_courses, 1):
        print(i, valeur)
else:
    print("la liste de course est vide")

• Pour la suppression de tous les éléments dans la liste, pour un souci de compréhension j'utiliserais clear plutôt que de re-créer un vide liste en mémoire. Les deux fonctionnent, mais avec clear tu indiques plus explicitement tes intentions.

Parfait pour le reste et bravo d'avoir fait tout ça 👏 Bonne continuation dans le parcours :)

Salut Kévin & Thilbaut ,

Merci bien pour vos réponses et j'ai pris bon note :)
Je réecrit ta solution Thilbaut en évitant de repasser les videos de la solution, il faut que je consolide les choses.
Et il faut que je m'habitue a lire les blocs d'instruction dans les boucles avec les indentations . :)