Тестирование пустого итератора в цикле Python for

Вам не нужно проверять, пуст ли итератор. Цикл for сделает это за вас и остановится, когда итератор станет пустым. Это так просто.

Кроме того, вам не нужен else после sys.exit () или возврата.

После этого ваш код выглядит так:

def my_func(choice_pattern, input):
    # Search in input for some things to choose from.
    choice_iterator = choice_pattern.finditer(input, re.M)
    if not choice_iterator:
        print "No choices. Exiting..."
        sys.exit()

    # Show choices to the user. For each one, ask user for a yes/no response. If
    # choice accepted, return a result. Otherwise show user next choice. If no
    # choices accepted by user, quit.
    for choice in choice_iterator:
        print choice
        # get_yes_no_answer() returns True or False depending on user response.
        if get_yes_no_answer():
            return choice
    # Loop exited without matches.
    print "No matches. Exiting..."
    sys.exit()

Вот и все!

Что происходит, так это то, что вы в цикле также получаете следующий элемент. В результате вы фактически показываете только каждый второй ответ.

Фактически, вы можете упростить его еще больше:

def my_func(choice_pattern, input):
    choice_iterator = choice_pattern.finditer(input, re.M)
    if choice_iterator:
        for choice in choice_iterator:
            print choice
            if get_yes_no_answer():
                return choice
    # If there is no choices or no matches, you end up here:
    print "No matches. Exiting..."
    sys.exit()

Итераторы используются почти так же, как и любой другой тип последовательности. Не нужно относиться к нему иначе, чем к списку.

зачем ты вообще это делаешь? почему бы и не просто:

def get_choice(pattern, inpt):
    choices = pattern.finditer(inpt, re.M)
    if not choices:
        sys.exit('No choices')
    for choice in choices:
        print(choice.group(0))
        if get_yes_no_answer():
            return choice
    sys.exit('No matches')

Я не знаю, какова длина вашего ввода, но я сомневаюсь, что это того стоит.

См. Попарный итератор из этого вопрос. Затем вы можете проверить последний элемент следующим образом:

MISSING = object()
for choice, next_choice in pairwise(chain(choice_iterator, [MISSING])):
    print(choice.group())
    if get_yes_no_answer():
        return choice.group()
    if next_choice is MISSING:
        print("No matches. Exiting...")
        sys.exit()

В показанном вами примере в этом нет необходимости. Вам не нужно проверять, вернул ли finditer итератор, потому что он всегда это делает. И вы можете просто пропустить цикл for, если не найдете то, что хотите:

def my_func(choice_pattern, input):
    """Search in input for some things to choose from."""
    for choice in choice_pattern.finditer(input, re.M):
        print(choice.group())
        if get_yes_no_answer():
            return choice.group()
    else:
        print("No choices. Exiting...")
        sys.exit()

Почему такой комплексный подход?

Я думаю, что это должно делать примерно то же самое:

def my_func(pattern, data):
  choices = pattern.findall(data, re.M)
  while(len(choices)>1):
    choice = choices.pop(0)
    if get_yes_no_answer():
      return choice
    else:
      choices.pop(0)
  else:
    return None