Wie kann ich dieses einfache Python-Programm komprimieren, um es effizienter zu machen?

Question

Dies ist nur ein einfaches Spiel, das ich zusammengestellt habe, das dir ein paar Fragen stellt und dann automatisch einen neuen Passcode aus den Antworten formatiert.

Da ich ein Anfänger bin, würde ich gerne einige Techniken kennen lernen, die ich verwenden könnte, die die Effizienz dieses Codes erhöhen würde, und vielleicht die verrückte Menge von Funktionen entfernen, die ich erklärt habe. Danke!

minimum = 3 
maximum = 10 

name = input("What is your name? ") 

def nameInput(): 

    if len(name) < minimum: 
     print("Name is too small") 
     return nameInput() 
    elif len(name) > maximum: 
      print("Name is too large") 
      return nameInput() 

nameInput() 

food = input("What is your favorite food? ") 

def foodInput(): 

    if len(food) < minimum: 
     print("Enter a food name longer than " + (food)) 
     return foodInput() 
    elif len(food) > maximum: 
      print("Enter a food name shorter than " + (food)) 
      return foodInput() 

foodInput() 

birthday = input("What is your date of birth? (mmddyyyy) ") 

nameIndex = name[0:2] 
foodIndex = food[2: ] 
birthIndex = birthday[0:3] 

passcode = nameIndex + foodIndex + birthIndex 

print("Your password is " + passcode) 

Answer 1

Wenn Sie nach Kürze sind, versuchen Sie eine einzelne Funktion nicht mehrfach zu verwenden, als mehrere Funktionen ein einziges Mal

def get_input(variable_name, min_size, max_size, begin, end):     
    data = input("What is your {}? ".format(variable_name))     
    if len(data) < min_size:             
     raise ValueError('{} is too small'.format(variable_name.capitalize())) 
    elif len(data) > max_size:             
     raise ValueError('{} is too big'.format(variable_name.capitalize())) 
    return data[begin:end]              

name_index = get_input("name", 3, 10, 0, 2)         
food_index = get_input("food", 3, 10, 2, 100)         
birth_index = get_input("date of birth (mmddyyyy)", 3, 10, 0, 3)    
passcode = name_index + food_index + birth_index        

print("Your passcode is", passcode)