Discussion :

[apt]

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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.

[wiztricks]

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

[marco056]

Il va falloir bosser les tutos !
Une proposition :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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)

[MPython Alaplancha]

Bonjour,
Autre proposition:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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)

[wiztricks]

Autre proposition:

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[MPython Alaplancha]

ç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

[apt]

Bonjour ,

J'ai modifié la condition de if en :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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é :

--| 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é :

Ressayez !

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

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 !')

[papajoker]

bonjour

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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

[Sve@r]

Bonjour

J'ai modifié la condition de if en :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

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 python :

Sélectionner tout - Visualiser dans une fenêtre à part

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 python :

Sélectionner tout - Visualiser dans une fenêtre à part

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 !')

... and not choix == " ":

... and choix != " ":

[apt]

Bonjour papajoker, sve@r,

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

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

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

... and choix != " ":

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 :

Sélectionner tout - Visualiser dans une fenêtre à part

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