0

У меня есть три состояния ответа пользователя: "yes", "no" и любые другие ответы. Соответственно три ветки. Есть одна из трех веток, которая если user ввел что-либо кроме "yes" или "no" возвращает его в начало

if autorisation  != 'no' and 'yes':

    while autorisation  != 'yes' and autorisation != 'no':

        print('Write only "yes" or "no". Try again')
        autorisation = str(input("Are you registered? :")).lower().replace(" ","")

Мне не хватает знаний и умения чтобы сделать код красивым и читаемым. Если использовать else то мы не вернемся в начало ветвления

2
  • 1
    input() возвращает строку. Зачем Вы строку кастуете к строке? 23 авг 2019 в 8:40
  • if autorisation != 'no' and 'yes': - в этой строке ошибка. ru.stackoverflow.com/questions/846296/… И str в строке с input не требуется. 26 авг 2019 в 13:18

5 ответов 5

3
while True:
    autorisation = input("Are you registered? :").lower().replace(" ","")

    if autorisation=='no' or autorisation=='yes':
        break
2
  • ну str то лишнее :) 26 авг 2019 в 13:20
  • да, лишнее. Исправлено, спасибо.
    – S. Nick
    26 авг 2019 в 13:32
1

Здесь бы помог do..while, но такого цикла в Python нет, так что:

autorisation = str(input("Are you registered? :")).lower().replace(" ","")

while (autorisation  != 'yes') and (autorisation != 'no'):

        print('Write only "yes" or "no". Try again')
        autorisation = str(input("Are you registered? :")).lower().replace(" ","")

Этот код сначала спрашивает. Если неверно – повторяет, если верно – условие цикла не выполняется и выполнение программы идёт дальше.

Если вместо строки autorisation = str(input("Are you registered? :")).lower().replace(" ","") был бы большой многострочный блок кода, его следовало бы вынести в отдельную функцию по принципу DRY.

1
def asksYesNo(question):
    response = None
    while response not in ('yes', 'no'):
        response = input(question).lower()
    return response
5
  • после .lower() результат response not in ('Yes', 'No') будет всегда true. 26 авг 2019 в 13:19
  • Все исправил, обычная невнимательность) 26 авг 2019 в 17:49
  • @МишаМладший и что тогда делать с этой строкой ? print('Write only "yes" or "no". Try again') 27 авг 2019 в 18:48
  • @MusterYoooda можно так askYesNo('You a registered? Write only yes or no: ') 28 авг 2019 в 8:46
  • @МишаМладший Так не пойдет(( Я поэтому и не использовал там функцию 28 авг 2019 в 12:22
1

Вы можете определить варианты, при которых нужно прекратить запрашивать ответы в отдельную переменную. Чтобы не тратить время выполнения на создание объекта, варианты стоит определить за за пределами цикла.

Далее в вечном цикле запрашиваете ответ на вопрос. Если введенное значение совпало с одним из вариантов (конструкция qanswer in variants), то прекращаете цикл словом break`.

Так как введенное значением может содержать проблемы и разный регист, введеное значение надо преобразовать в оптимальный формат. В данном случае привели строку в нижний регистр (.lower()) и убрали пробелы перед и после введенных симоволов (.strip())

Код примера ниже

variants = ('yes', 'no',)
while True:
    answer = input("Are you registered? :").lower().strip()
    if answer in variants:
        break
print('Ok')
0
0

if autorisation != 'no' and 'yes': не имеет смысла так как не пустая строка возвращает истину то есть смысл этого выражения не изменится убрав and 'yes'

по вопросу:


while au:=str(input('yes\no? :')).lower() not in ['yes','no','not','n','y']:
   print('u write'+str(au)+'it is not yes or not')

if au in 'yes':
   print('letter y in yes then only one expression needed')

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.