Проблема с операторами if,elif в цикле?

Question

[Макс Богачов]

На днях написал игрушку типу русской рулетки:

import time
import random

def tm(x):
    time.sleep(x)

rf = random.randint(1,8)
rn = random.randint(1,7)
rc = random.randint(1,6)

while True:
    ans_1 = str(input('Добро пожаловать в русскую рулетку!\nФлобер(8 патронов),Наган(7) ибо Кольт(6)?\nВыберите револьвер:'))
    if ans_1 == 'Флобер' or 'флобер':
        ans_2_1 = int(input('Введите количеством патронов в барабане(от 1 до 8):'))
        if ans_2_1 == 1:
            print('Барабан крутиться...')
            tm(rf)
            if rf in range(ans_2_1):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_1 == 2:
            print('Барабан крутиться...')
            tm(rf)
            if rf in range(ans_2_1):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_1 == 3:
            print('Барабан крутиться...')
            tm(rf)
            if rf in range(ans_2_1):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_1 == 4:
            print('Барабан крутиться...')
            tm(rf)
            if rf in range(ans_2_1):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_1 == 5:
            print('Барабан крутиться...')
            tm(rf)
            if rf in range(ans_2_1):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_1 == 6:
            print('Барабан крутиться...')
            tm(rf)
            if rf in range(ans_2_1):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_1 == 7:
            print('Барабан крутиться...')
            tm(rf)
            if rf in range(ans_2_1):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_1 == 8:
            print('Ну ты же знаешь что с полным барабаном у тебя нет шансов...')
        else:
            print('Некорректное количество патронов')
        break
    elif ans_1 == 'Наган' or 'наган':
        ans_2_2 = int(input('Введите количеством патронов в барабане(от 1 до 7):'))
        if ans_2_2 == 1:
            print('Барабан крутиться...')
            tm(rn)
            if rn in range(ans_2_2):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_2 == 2:
            print('Барабан крутиться...')
            tm(rn)
            if rn in range(ans_2_2):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_2 == 3:
            print('Барабан крутиться...')
            tm(rn)
            if rn in range(ans_2_2):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_2 == 4:
            print('Барабан крутиться...')
            tm(rn)
            if rn in range(ans_2_2):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_2 == 5:
            print('Барабан крутиться...')
            tm(rn)
            if rn in range(ans_2_2):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_2 == 6:
            print('Барабан крутиться...')
            tm(rn)
            if rn in range(ans_2_2):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_2 == 7:
            print('Ну ты же знаешь что с полным барабаном у тебя нет шансов...')
        else:
                print('Некорректное количество патронов')
        break
    elif ans_1 == 'Кольт' or 'кольт':
        ans_2_3 = (input('Введите количеством патронов в барабане(от 1 до 6):'))
        if ans_2_3 == 1:
            print('Барабан крутиться...')
            tm(rc)
            if rc in range(ans_2_3):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_3 == 2:
            print('Барабан крутиться...')
            tm(rc)
            if rc in range(ans_2_3):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_3 == 3:
            print('Барабан крутиться...')
            tm(rc)
            if rc in range(ans_2_3):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_3 == 4:
            print('Барабан крутиться...')
            tm(rc)
            if rc in range(ans_2_3):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_3 == 5:
            print('Барабан крутиться...')
            tm(rc)
            if rc in range(ans_2_3):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_3 == 6:
            print('Ну ты же знаешь что с полным барабаном у тебя нет шансов...')
        else:
            print('Некорректное количество патронов')
        break
    else:
        print('Неверный револьвер')
        continue

Но когда я выбираю второе либо третье оружие у меня операторы если не работают выбивают всё к первому:

Извините если где-то тупонул но де-баггер ничего не нашёл

Comments

[Roman K]

Господи, дай мне это развидеть

[Даниил Демидко]

Редкостный образец китайского кода.
"На днях написал игрушку", рано тебе игрушки писать, программировать научись.
Хотя... Я до сих пор помню свою игрушку-говорилку на паскале...
Попробуй вынести повторяющийся код в функцию.
А копипаста - зло

[Sly_tom_cat .]

Где учат писать такой индийский код? Сообщите адрес - программисты скинутся на бульдозер что-бы разровнять это заведение, дабы не насиловать свой мозг чтением такого говнокода.

Answer 1

[Sly_tom_cat .]

Никогда не пишите такой говногкод!

Зачем кучу раз повторять эти IF-ы?
Если действия никак не отличаются для любого количества патронов - то какой смысл их писать отдельным кодом? С пистолетами разница только в размере барабана - тоже решается все через простую переменную.

import time
import random

guns = {'флобер': 8, 'наган': 7, 'кольт': 6}
while True:
  while True:
    gun = input('Добро пожаловать в русскую рулетку!\n'
                'Флобер(8 патронов),Наган(7) либо Кольт(6)?\n'
                'Выберите револьвер:')
    if gun.lower() not in guns.keys():
      print('Неверный револьвер')
    else:
      break
  bullets = guns[gun]
  while True:
    choice = int(input('Введите количеством патронов в барабане'
                       ' (от 1 до %d):'%bullets))
    if choice == bullets:
      print('Ну ты же знаешь что с полным барабаном у тебя нет шансов...')
    elif choice <= 0 or choice >= bullets:
      print('Некорректное количество патронов')
    else:
      break
  print('Барабан вращается')
  turn = random.randint(1, bullets)
  time.sleep(turn)
  if turn >= choice:
    print('Ты выиграл')
  else:
    print('Ты проиграл!')

Answer 2

[Макс Богачов]

Ну ребят насчёт скобок это сложно на каждой строке так проделывать я решил создать 2 списка для каждого револьвера,ой да что я вам рассказываю вот код:

import time
import random

def tm(x):
    time.sleep(x)

lrf = ['Флобер','флобер','1','8']
lrn = ['Наган','наган','2','7']
lrc = ['Кольт','кольт','3','6']

lrff = [1,2,3,4,5,6,7,8]
lrfn = [1,2,3,4,5,6,7]
lrfc = [1,2,3,4,5,6]

rf = random.choice(lrff)
rn = random.choice(lrfn)
rc = random.choice(lrfc)

while True:
    ans_1 = str(input('Добро пожаловать в русскую рулетку!\nФлобер(8 патронов),Наган(7) ибо Кольт(6)?\nВыберите револьвер:'))
    if ans_1 in lrf:
        ans_2_1 = int(input('Введите количеством патронов в барабане(от 1 до 8):'))
        if ans_2_1 == 1:
            print('Барабан крутиться...')
            tm(3)
            if rf in range(ans_2_1):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_1 == 2:
            print('Барабан крутиться...')
            tm(3)
            if rf in range(ans_2_1):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_1 == 3:
            print('Барабан крутиться...')
            tm(3)
            if rf in range(ans_2_1):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_1 == 4:
            print('Барабан крутиться...')
            tm(3)
            if rf in range(ans_2_1):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_1 == 5:
            print('Барабан крутиться...')
            tm(3)
            if rf in range(ans_2_1):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_1 == 6:
            print('Барабан крутиться...')
            tm(3)
            if rf in range(ans_2_1):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_1 == 7:
            print('Барабан крутиться...')
            tm(3)
            if rf in range(ans_2_1):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_1 == 8:
            print('Ну ты же знаешь что с полным барабаном у тебя нет шансов...')
        else:
            print('Некорректное количество патронов')
        break
    elif ans_1 in lrn:
        ans_2_2 = int(input('Введите количеством патронов в барабане(от 1 до 7):'))
        if ans_2_2 == 1:
            print('Барабан крутиться...')
            tm(3)
            if rn in range(ans_2_2):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_2 == 2:
            print('Барабан крутиться...')
            tm(3)
            if rn in range(ans_2_2):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_2 == 3:
            print('Барабан крутиться...')
            tm(3)
            if rn in range(ans_2_2):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_2 == 4:
            print('Барабан крутиться...')
            tm(3)
            if rn in range(ans_2_2):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_2 == 5:
            print('Барабан крутиться...')
            tm(3)
            if rn in range(ans_2_2):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_2 == 6:
            print('Барабан крутиться...')
            tm(3)
            if rn in range(ans_2_2):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_2 == 7:
            print('Ну ты же знаешь что с полным барабаном у тебя нет шансов...')
        else:
                print('Некорректное количество патронов')
        break
    elif ans_1 in lrc:
        ans_2_3 = int(input('Введите количеством патронов в барабане(от 1 до 6):'))
        if ans_2_3 == 1:
            print('Барабан крутиться...')
            tm(3)
            if rc in range(ans_2_3):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_3 == 2:
            print('Барабан крутиться...')
            tm(3)
            if rc in range(ans_2_3):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_3 == 3:
            print('Барабан крутиться...')
            tm(3)
            if rc in range(ans_2_3):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_3 == 4:
            print('Барабан крутиться...')
            tm(3)
            if rc in range(ans_2_3):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_3 == 5:
            print('Барабан крутиться...')
            tm(3)
            if rc in range(ans_2_3):
                print('Ты проиграл!')
            else:
                print('Ты выиграл')
        elif ans_2_3 == 6:
            print('Ну ты же знаешь что с полным барабаном у тебя нет шансов...')
        else:
            print('Некорректное количество патронов')
        break
    else:
        print('Неверный револьвер')
        continue

Comments

[Sly_tom_cat .]

Зачем вы за автором повторяете весь его индийский код? Все решается в 10 раз более коротким кодом.

[Алексей С.]

Sly_tom_cat .: :lol: посмотри автора ответа и автора вопроса =)

Answer 3

[pcdesign]

Имеет смысл отказаться от этих бесконечных if, else, а использовать словарь dict со всеми вариантами.
Тогда и ошибка сама исправится, да и сам код станет в 10 раз короче.

Answer 4

[Алексей С.]

if ans_1 == 'Наган' or 'наган': А вы в такой конструкции понимаете что с чем сравниваете .. может скобочек добавить ?

Comments

[Roman K]

Скобочки не помогут)

[Александр]

Здесь нужно использовать if ans_1 in ['Наган', 'наган'] Ну, или, что гораздо лучше, перед сравнением приводить всю строку к одному регистру.

[Даниил Демидко]

Скобочки это не по Python way) Здесь нужно понять дао

[Sly_tom_cat .]

вообще-то проще просто привести к прописным, а потом сравнивать.
if ans.lower() == 'наган'

Но тут мы имеем полный набор паттернов индийского.китайского кода, так что одним кривым if-ом дело не ограничивается.