Bonjour,

Je suis débutant en informatique et j'ai une petite question par rapport à un devoir en informatique que je dois rendre pour demain.
La consigne est la suivante :
Vous devez créer une fonction is_valid_liste(liste) qui vérifie la liste donné en paramètre et renvoie
True si la combinaison est valide, c’est à dire, qu’elle fait référence aux couleurs
possibles et qu’elle a la bonne taille ; False est renvoyé si la combinaison
n’est pas valide.

Les couleurs possibles sont une liste COLOURS = ['R','J','V','B','O','C','T','F']

J'ai donc écrit ce code : COLOURS = ['R','J','V','B','O','C','T','F'] def is_valid_pattern(pattern): """ Vérifie la liste "pattern" donnée en paramètre et renvoie "True" si la combinaison est valide (c'est à dire que les conditions de longueur et d'appartenance à la liste "colours" sont vérifiées) ou "False" si la combinaison n'est pas valide. """ correct_colours = 0 for i in pattern: if i in COLOURS: correct_colours += 1 return (correct_colours == len(pattern) == 4)

Il m'a l'air d'être correct mais je me demandais s'il n'y avait pas des problèmes de bonne pratique, par exemple un soucis de stabilité. Il y a t'il moyen de faire mieux? Merci d'avance