Jak przetestować tę zmienną nie jest równa wielu rzeczy? Python

Question

To jest kawałek kodu mam:

choice = "" 

while choice != "1" and choice != "2" and choice != "3": 
    choice = raw_input("pick 1, 2 or 3") 

    if choice == "1": 
     print "1 it is!" 

    elif choice == "2": 
     print "2 it is!" 

    elif choice == "3": 
     print "3 it is!" 

    else: 
     print "You should choose 1, 2 or 3" 

Chociaż to działa, czuję, że to naprawdę niezgrabny, szczególnie gdy klauzula. Co się stanie, jeśli będę miał więcej akceptowalnych wyborów? Czy istnieje lepszy sposób na wprowadzenie klauzuli?

Answer 1

while bit może być refactored trochę zrobić to trochę czystsze przez sprawdzenie, czy element jest w liście wyboru Podobnie jak

while choice not in [1, 2, 3]: 

to sprawdzanie jest wartością wyboru nie jest elementem tej listy

Answer 2

Można naciskać logikę do pętli i zastąpić

while choice != "1" and choice != "2" and choice != "3": 

z

while True: 

a następnie początkowa linia choice = "" jest niepotrzebna. Następnie, w każdym oddziale, gdy skończysz, co chcesz zrobić, możesz break.

Answer 3

while str(choice) not in "123" .....

Answer 4

myślę, że można użyć zestawu, który zawiera wszystkich możliwych wyborów i ich wykorzystywania „w” wypowiedzi judgefor część chwilę.

Jeśli chodzi o część if-else, print (wybór, "to jest!") Będzie w porządku.

Answer 5

Możesz użyć słownika do odwzorowania 1 na kod, który chcesz wykonać, gdy 1 jest wartością, i tak dalej ... W ten sposób pozbędziesz się ifs, a twój kod będzie obsługiwał inne wartości w przyszłości przez po prostu aktualizuję słownik. Jeśli chodzi o warunek w czasie, wystarczy sprawdzić, czy klucz znajduje się w słowniku.

Answer 6

Myślę, że coś takiego byłoby lepiej

possilities = {"1":"1 it is!", "2":"2 it is!", "3":"3 it is!"} 
choice = "" 

while True: 
    choice = raw_input("pick 1, 2 or 3") 
    if choice in possilities: 
     print possilities[choice] 
     break 
    else: 
     print "You should use 1, 2 or 3" 

Answer 7

Sugeruję mający a function który właśnie pętle aż prawidłowy opcja zostanie wybrana, a następnie zwraca wybraną wartość.

Oznacza to reszta kodu nie jest przeznaczony wewnątrz while, utrzymując wszystko ładne i płaskie ("Flat is better than nested")

def get_choice(options): 
    """Given a list of options, makes the user select one. 

    The options must be strings, or they will never match (because raw_input returns a string) 

    >>> yn_choices = ['y', 'n'] 
    >>> a = get_choice(options = yn_choices) 
    """ 
    prompt_string = "Pick " + ", ".join(options) 
    while True: 
     choice = raw_input(prompt_string) 
     if choice in options: 
      return choice 
     else: 
      print "Invalid choice" 

# Prompt user for selection 
choice = get_choice(["1", "2", "3"]) 

# Do stuff with choice... 
if choice == "1": 
    print "1 it is!" 

elif choice == "2": 
    print "2 it is!" 

elif choice == "3": 
    print "3 it is!" 

else: 
    print "You should choose 1, 2 or 3"