Более вещий способ гнездовых функций

Question

Вот код, у меня есть:

def reallyquit(): 
     from easygui import boolbox 
     if __debug__: print ("realy quit?") 
     answer = boolbox("Are you sure you want to Quit?") 
     if answer == 1: 
      sys.exit() 
     return 

columns = ['adr1','adr2','spam','spam', 'spam'] 

Street = choicebox("Street Address?", title, columns) 
    while Street == None: 
     reallyquit() 
     Street = choicebox("Street Address?", title, columns) 

Есть ли более вещий способ рекурсивно задать вопрос, если пользователь закрывает окно неосторожно? В основном ищут советы на последних 4 строках, но не стесняйтесь критиковать остальных.

Answer 1

import sys 
from easygui import boolbox, choicebox 

def reallyquit(): 
    "Ask if the user wants to quit. If so, exit." 
    if __debug__: 
     print("really quit?") 
    answer = boolbox("Are you sure you want to Quit?") 
    if answer == 1: 
     sys.exit() 

columns = ['adr1', 'adr2', 'spam', 'spam'] 
street = None 

# ask for address until given or user decides to quit 
while street is None: 
    street = choicebox("Street Address?", title, columns) 
    if street is None: 
     reallyquit() 

В основной логики: Обычно вы бы не проверить, если значение равно None, так как это синглтон (только один в системе, и все вхождения None являются одной и той же логической None). Таким образом, is None - лучший тест. Я также использовал немного более форматирование кода Pythonic (пробелы после запятых в списке, имена переменных нижнего регистра) и порядок тестирования, который не дублирует вызов взаимодействия с пользователем.

Альтернативная форма предложенная Eric будет что-то вроде:

while True: 
    street = choicebox("Street Address?", title, columns) 
    if street is not None: 
     break 
    reallyquit() 

Это имеет силу не повторять тест street is None, но немного длиннее. Ваше предпочтение, для которого вариант кажется более четким и логичным, может отличаться.

Для подготовки: Импорт осуществляется из функций, а choicebox импортируется там, где он ранее отсутствовал. Функция задается определяющей строкой комментария. Выполняется одно утверждение для каждого стандартного стандарта. И ненужный return удален.

Это еще довольно грубая программа, но теперь она «более Pythonic».

Не должно быть подавляющим, но стандартом для внешнего вида Pythonic является PEP 8. Существуют такие инструменты, как pep8, которые вы можете установить для проверки соответствия. Более глубокая или «семантическая» часть Pythonic-кода выложена в PEP 20, также известном как «Zen of Python». К сожалению, нет никакого средства судить о соблюдении этих более глубоких принципов. Требуется опыт.

Answer 2

Я думаю, что это более идиоматическое, чем при использовании переменного дозорный:

while True: 
    street = choicebox("Street Address?", title, columns) 
    if street is None: 
     reallyquit() 
    else: 
     break 

Кроме того, рассмотреть вопрос об использовании logging модуля для отладки печати, поэтому вместо if __debug__: print(whatever) вы просто logging.debug(whatever) и установить уровень журналирования.