[Python 3.X] Structure conditionnelle if à corriger

Structure conditionnelle if à corriger

Bonjour à tous,

J'aimerais corriger la structure conditionnelle if dans ce bout de code, ou l'utilisateur doit choisir dans un menu entre 1 et 5 :

Code:

1
2
3
4
5
6
7
8
choix = ""
while choix !=5:
    choix = input ("VOtre choix : ")
    if choix[0] < "1" and choix[0] > "5" \
        and not choix[0] == " " and len(choix) == 1:
        print("doit être entre 1-5. Essayez à nouveau"
    else: '''En cas ou choix = 5'''
        print("A bientôt !")

Merci d'avance.

Salut,

Si vous ne convertissez pas choix, chaîne de caractères, en un entier, tester qu'elle est de longueur 1 et que c'est une sous chaîne de "12345" (ou "1" <= choix <= "5") suffirait (modulo la condition de sortie...).
Si vous la convertissez en entier, le test sera 1 <= choix <= 5.

A vous de corriger...

- W

Il va falloir bosser les tutos !
Une proposition :

Code:

1
2
3
4
5
choix = 0
while choix not in [1,2,3,4,5]:
    question = input("Entre un choix ")
    choix = int(question)
print("choix : ", choix)

Bonjour,
Autre proposition:

Code:

1
2
3
4
while (choix := input("Entre un choix entre 1 et 5:")) not in "12345":
	print('erreur de saisie! doit être entre 1-5. Essayez à nouveau')
 
print("Votre choix:",choix)

Citation:
Envoyé par Hominidé
Autre proposition:
Code:

1 2 3 4 while (choix := input("Entre un choix entre 1 et 5:")) not in "12345": print('erreur de saisie! doit être entre 1-5. Essayez à nouveau') print("Votre choix:",choix)
ça ne marche pas car l'omission de test sur la longueur autorise "123" à être une réponse correcte.

- W

Citation:

Envoyé par wiztricks

ça ne marche pas car l'omission de test sur la longueur autorise "123" à être une réponse correcte.

- W

Je n'avais effectivement pas pensé à ce cas de figure :roll:

Citation:

Envoyé par Hominidé

Je n'avais effectivement pas pensé à ce cas de figure :roll:

Je l'avais pourtant précisé dans ma réponse...

- W

Citation:

Envoyé par wiztricks

Je l'avais pourtant précisé dans ma réponse...

- W

En effet. Mes yeux devaient s'égarer ailleurs ... Désolé.
Pour le schmilblick :
Code:

1 2 3 4 while (choix := int(input("Entre un choix entre 1 et 5:"))) not in range(1,6): print('erreur de saisie! doit être entre 1-5. Essayez à nouveau') print("Votre choix:",choix)
Code:

1 2 3 4 while not 0 < (choix := int(input("Entre un choix entre 1 et 5:"))) < 6: print('erreur de saisie! doit être entre 1-5. Essayez à nouveau') print("Votre choix:",choix)

Bonjour ,

J'ai modifié la condition de if en :

Code:

if len(choix) != 1 and choix not in "12345" and not choix == " ":

Mais cette condition ne teste que si l'entrée est à deux chiffres !

Texte affiché :

Citation:

--| Votre choix doit être, entre 1-5
--| Ressayez.

Une entrée d'espace ou de caractère n'est pas traitée.

Voici le texte affiché :

Citation:

Ressayez !

Pour avoir un bon aperçu, voici le code utilisé :

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
liste=[]
choix = ""
while choix not in "12345":
    print("------------------------------------")
    print("1: Ajouter un élément à la liste")
    print('2: Retirer un élément de la liste')
    print("3: Aficher la liste")
    print("4: Vider la liste")
    print("5: Quitter")
 
    choix = input ("Votre choix : ")
 
    if len(choix) != 1 and choix not in "12345" and not choix == " ":
        print("--| Votre choix doit être, entre 1-5")
        print("--| Ressayez.") 
 
    elif choix == "1":
    ...
    elif choix == "2":
    ...
    elif choix == "3":
    ...
    elif choix == "4":
    ...
    elif choix == "5":
        print('A bientôt !')
        break
    else:
        print('Ressayez !')

bonjour

Citation:

Envoyé par apt

Mais cette condition ne teste que si l'entrée est à deux chiffres !

oui, tout a fait normal car tu utilises des ET dans ta condition/test

je suppose que tu désirais : si pas de longeur 1 OU pas dans "1..5" OU ... alors j'ai une erreur.
Note , puisque tu utilises des chaines, il est bon de supprimer les espaces avant ta condition
Note: le jour ou tu as 10 entrées, ta logique devient mauvaise, donc si le menu peut grandir, ce n'est pas pas la meilleure solution (travail sur chaine)

pour info ... avec python 3.10 existe maintenant "match"
Code:

1 2 3 4 5 6 7 match entry: case "1": print("call ", "Function_1") case "5": exit(5) case _: print("pas dans menu...")

Bonjour

Citation:

Envoyé par apt

J'ai modifié la condition de if en :

Code:

if len(choix) != 1 and choix not in "12345" and not choix == " ":

Mais cette condition ne teste que si l'entrée est à deux chiffres !

Pourquoi faire compliqué avec 3 tests pour une évaluation (à savoir un choix faisant partie de valeurs autorisées) ???

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
liste=[]
while True:
	print("------------------------------------")
	print("1: Ajouter un élément à la liste")
	print('2: Retirer un élément de la liste')
	print("3: Aficher la liste")
	print("4: Vider la liste")
	print("5: Quitter")
 
	choix = input ("Votre choix : ")
 
	if choix not in ("1", "2", "3", "4", "5"):
		print("--| Votre choix doit être, entre 1-5")
		print("--| Ressayez.") 
 
	elif choix == "1":
		print(1)
	elif choix == "2":
		print(2)
	elif choix == "5":
		print('A bientôt !')
		break
	else:
		print('Ressayez !')

Après, pour faire plus indépendant, tu peux gérer la saisie dans une fonction chargée de vérifier avant de renvoyer. Et tu passes à la fonction juste ta liste de choix et elle, elle s'adapte à la liste...

Code:

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
def menu(prop):
	while True:
		print("------------------------------------")
		for (k, v) in prop.items(): print("%s: %s" % (k, v))
		choix = input ("Votre choix : ")
		if choix in prop.keys(): return choix
		print("--| Votre choix doit être, entre %s-%s" % (tuple(prop.keys())[0], tuple(prop.keys())[-1]))
		print("--| Ressayez.") 
	# while
# menu()
 
liste=[]
while True:
	choix=menu(
		{
			"1" : "Ajouter un élément à la liste",
			"2" : "Retirer un élément de la liste",
			"3" : "Aficher la liste",
			"4" : "Aficher la liste dans l'autre sens",
			"5" : "Vider la liste",
			"6" : "Quitter",
		}
	)
	if choix == "1":
		print(1)
	elif choix == "2":
		print(2)
	elif choix == "6":
		print('A bientôt !')
		break
	else:
		print('Ressayez !')

Citation:

Envoyé par apt

... and not choix == " ":

... and choix != " ": :whistle:

Bonjour papajoker, sve@r,

Citation:

Envoyé par papajoker

Note: le jour ou tu as 10 entrées, ta logique devient mauvaise, donc si le menu peut grandir, ce n'est pas pas la meilleure solution (travail sur chaine)

pour info ... avec python 3.10 existe maintenant "match"

Merci pour l'astuce :)

Citation:

Envoyé par sve@r

Pourquoi faire compliqué avec 3 tests pour une évaluation (à savoir un choix faisant partie de valeurs autorisées) ???

Cela explique pourquoi la structure conditionnelle ne fonctionne pas !

Merci pour le code :D

Citation:

Envoyé par sve@r

Après, pour faire plus indépendant, tu peux gérer la saisie dans une fonction chargée de vérifier avant de renvoyer. Et tu passes à la fonction juste ta liste de choix et elle, elle s'adapte à la liste...

Je vais prendre un peu plus de temps pour bien comprendre comment cela fonctionne :mouarf:

Citation:

Envoyé par sve@r

... and choix != " ": :whistle:

Cela fait partie d'un bout de code extrait à partir de Stack Overflow.

J'ai essayé le code sans recourir au traitement des entrées via des chaînes.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 liste=[] while True: print("------------------------------------") print("1: Ajouter un élément à la liste") print('2: Retirer un élément de la liste') print("3: Aficher la liste") print("4: Vider la liste") print("5: Quitter") choix = int(input("Votre choix : ")) if choix not in (1, 2, 3, 4, 5): print("--| Votre choix doit être entre 1-5") print("--| Ressayez.") elif choix == 1: print(1) elif choix == 2: print(2) elif choix == 5: print("A bientôt !") break else: print("Ressayez !")
Je pense que c'est nettement mieux ainsi ;)

Citation:

Envoyé par apt

Je vais prendre un peu plus de temps pour bien comprendre comment cela fonctionne :mouarf:

Ouais, un peu subtil et pas forcément super correct. J'utilise un dictionnaire. Chaque clef ce sont les valeurs des choix et les valeurs du dico ce sont les libellés. Là où ça merdoie c'est dans le fait que je pars du principe que les valeurs du dico sont données dans l'ordre pour afficher "entre X et Y" en prenant X à la première clef et Y à la dernière. C'est pas super super mais j'ai fait ça vite fait juste pour te donner des idées.

Citation:

Envoyé par apt

J'ai essayé le code sans recourir au traitement des entrées via des chaînes...Je pense que c'est nettement mieux ainsi ;)

Mouais. T'as tenté de répondre un truc style "toto" quand on te propose un choix ? ;)

Citation:

Envoyé par apt

Cela fait partie d'un bout de code extrait à partir de [I]Stack Overflow.

Le problème c'est qu'ensuite nous ne pouvons évaluer ton problème donc plus difficile de t'aider - En fait il ne faut copier depuis ce site que si l'on comprend très bien ce que l'on copie

Dans la suite de l'idée de sve@r, pour un menu plus générique (pas forcément utile ici), il est aussi possible d'attacher un code avec un item du menu
Exemple:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 liste = ["papa", "maman"] def ajouter(liste): liste.append("frère") print("run menu 1", liste) return liste def retirer(liste): liste = liste.pop() print("run menu 2", liste) return liste def quitter_app(liste): # save_liste(liste) exit(5) menus = [ {'msg': "ajout", 'call': ajouter}, {'msg': "supprimer", 'call': retirer}, {'msg': "dernier", 'call': quitter_app} ]
Ici, la sucture if..elseif n'a plus lieux d'exister, après contrôles, il est possible d'appeler directement la fonction
Code:

1 2 menu = menus[choix-1] liste = menu['call'](liste)