Игнорирует условие if в пайтон и выполняется даже если nmonth что угодно. Как исправить?

Question

[GENius_s]

while True: #бесконечный цикл
    allmon =  -1 # cумма
    nmoney = int(input()) # кол. средств
    nmonth = input('Это новый месяц? ') # новый месяц? в if варианты 
    if nmonth == '1' or nmonth == 'да' or nmonth == 'Да' or nmonth == 'ДА' or nmonth == 'lf' or nmonth == 'Lf' or nmonth == 'LF' 
            or nmonth == 'yes' or nmonth == 'Yes' or nmonth == 'YES' or nmonth == 'нуы' or nmonth == 'Нуы' or nmonth == 'НУЫ' 
            and allmon >= 0:
        allmon = allmon // 100 * 7.5 + allmon + nmoney
    elif nmonth == '0' or nmonth == 'нет' or nmonth == 'Нет' or nmonth == 'НЕТ' or nmonth == 'ytn' or nmonth == 'Ytn' or 
            nmonth == 'YTN' 
            or nmonth == 'no' or nmonth == 'No' or nmonth == 'NO' or nmonth == 'тщ' or nmonth == 'Тщ' or nmonth == 'ТЩ' \
            and 0 < allmon <= 0:
        allmon = allmon + nmoney
    else:
        print('ошибка')
# если я ввожу nmonth что угодно то цикл не выдает ошибку

Comments

[Модератор]

Фрагменты кода надо размещать в виде текста и оборачивать тэгом code для корректного отображения. Удобно делать кнопкой </>
Это обязательно, см.п.3.8 Регламента.
Сюда же относится traceback, ввод и вывод в консоли и другая структурированная текстовая инфа.

[Модератор]

Также обратите внимание на п.3.6

[Алан Гибизов]

Исправьте для начала вопрос. Приведите в порядок код, это же не код а груда мусора.

[dmshar]

А чего вы ожидаете получить, если сначала спрашиваете
nmonth = input('Это новый месяц? ')
получаете, например, ответ "да", а потом
выполняете вот это
allmon = allmon // 100 * 7.5 + allmon + nmoney
Как вы себе представляете прибавление значения строки "да" к числу?

Ну, а про это ЧУДО я вообще не говорю.
0 < allmon <= 0
Вы можете объяснить, что тут вы пытаетесь сказать?

[GENius_s]

dmshar, ошибка исправлена, спасибо

[Elvis]

На будущее, не пишите так

if nmonth == '1' or nmonth == 'да' or nmonth == 'Да' or nmonth == 'ДА' or nmonth == 'lf' or nmonth == 'Lf' or nmonth == 'LF' or nmonth == 'yes' or nmonth == 'Yes' or nmonth == 'YES' or nmonth == 'нуы' or nmonth == 'Нуы' or nmonth == 'НУЫ' and allmon >= 0:

пишите правильно:

if nmonth.lower() in ['1', 'да', 'lf', 'yes', 'нуы'] and allmon >= 0:

[VoVcH1C]

Elvis, раз уж на то пошло, то правильнее будет использовать кортеж, так как это константа, список будет использовать лишнюю память.

[fenrir]

VoVcH1C, не будет. Интерпретатор сам оптимизирует подобные моменты

In [2]: dis('"1" in ["1", "yes", "True"]')
  0           0 RESUME                   0

  1           2 LOAD_CONST               0 ('1')
              4 LOAD_CONST               1 (('1', 'yes', 'True'))
              6 CONTAINS_OP              0
              8 RETURN_VALUE

In [3]: dis('"1" in ("1", "yes", "True")')
  0           0 RESUME                   0

  1           2 LOAD_CONST               0 ('1')
              4 LOAD_CONST               1 (('1', 'yes', 'True'))
              6 CONTAINS_OP              0
              8 RETURN_VALUE

Answer 1

[theurus]

yes_list = ('да','ага','угу')

if nmonth.lower() in yes_list and allmon >= 0:

Comments

[Mirozr]

не очень ефективно исползовать кортедж но если это не м=серйозная задача то всероно но лутше исползовать список

[theurus]

Mirozr, поясни

[Mirozr]

theurus, Тобиш если это нейросеть в неё нужно будет добавлять данние а кортедж это неизменяемый тип данных. Но так как я сказал так как это не серйозная какаета програма можно использовать кортедж но желательно списко - [].

Answer 2

[Mirozr]

Не понимаю о чем вы говорите код прекрасно исполняется но только я могу вам посоветовать не делать такой громадский код в блоке if это не читабельно и не соответсвует нормам написания кода советую сделать два списка или хотяби не делать nmonth == 'no' or nmonth == 'No' or nmonth == 'NO' ведь можно сделать так как минимум :

if nmonth == '1' or 'да' or 'Да' or 'ДА' or 'lf' or 'Lf' or 'LF' or 'yes' or 'Yes' or 'YES' or 'нуы' or 'Нуы' or 'НУЫ' and allmon >= 0:

Советую проверить это также как и я:

allmon =  -1
nmoney = 0
nmonth = 0

while True:
    allmon =  -1
    nmoney = int(input())
    nmonth = input('Это новый месяц? ') 
    if nmonth == '1' or 'да' or 'Да' or 'ДА' or 'lf' or 'Lf' or 'LF' or 'yes' or 'Yes' or 'YES' or 'нуы' or 'Нуы' or 'НУЫ' and allmon >= 0:
        allmon = allmon // 100 * 7.5 + allmon + nmoney
        print("if")
    elif nmonth == '0' or 'нет' or 'Нет' or 'НЕТ' or 'ytn' or 'Ytn' or 'YTN' or 'no' or 'No' or 'NO' or 'тщ' or 'Тщ' or 'ТЩ' and 0 < allmon <= 0:
        allmon = allmon + nmoney
        print("else")

Comments

[GENius_s]

Почему у меня всегда выдает 1 if?