Как упростить список исключений?

Question

[Лаврентий Федотов]

Суть такая: у меня длинный список исключений с if то-то or то-то or то-то и так где-то 100 исключений. И этот кусок кода у меня в коде повторяется гдето 70 раз
Можно ли как-то упростить это? Сделать как-то, чтобы в одном месте были эти исключения, а в коде на них ссылаться?

Comments

[Алексей Черемисин]

Интересно, откуда их столько? может быть сделать одно исключение, а в нем статус? И функцию, которая это исключение выбрасывает со статусом?

[nirvimel]

кусок кода у меня в коде повторяется где-то 70 раз

В аду за такое дают клавиатуру без клавиши Ctrl и заставляют каждый день переписывать заново все 70 одинаковых фрагментов вручную без возможности копипаста, а за ошибку в один символ счетчик сбрасывается и все приходится писать с начала.

[Влад Григорьев]

70 раз копипастить и не использовать функцию это жесть.
Покажите хоть пример, может там все изи упростить

[Лаврентий Федотов]

Ну, примерно такое

if ((m.text in '/Ethash' and len(m.text)>7) 
	or (m.text in '/DCR' and len(m.text)>3) 
	or (m.text in '/SIA' and len(m.text)>3) 
	or (m.text in '/LBC' and len(m.text)>3)
	or (m.text in '/PASC' and len(m.text)>3) 
	or (m.text in '/PASL' and len(m.text)>3) 
	or (m.text in '/Equihash' and len(m.text)>4) 
	or (m.text in '/CryptoNight' and len(m.text)>4) 
	or (m.text in '/LBRY' and len(m.text)>4) 
	or (m.text in '/X11Gost' and len(m.text)>4) 
	or (m.text in '/Groestl' and len(m.text)>4) 
	or (m.text in '/Lyra2REv2' and len(m.text)>4) 
	or (m.text in '/NeoScrypt' and len(m.text)>4) 
	or (m.text in '/Blake14r' and len(m.text)>4) 
	or (m.text in '/Blake2b' and len(m.text)>4) 
	or (m.text in '/Pascal' and len(m.text)>4) 
	or (m.text in '/DCRDUAL' and len(m.text)>4) 
	or (m.text in '/SIADUAL' and len(m.text)>4) 
	or (m.text in '/LBCDUAL' and len(m.text)>4) 
	or (m.text in '/PASCDUAL' and len(m.text)>4) 
	or (m.text in '/Sha256' and len(m.text)>4) 
	or (m.text in '/Scrypt' and len(m.text)>4) 
	or (m.text in '/X11' and len(m.text)>3) 
	or (m.text in '/Qubit' and len(m.text)>4) 
	or (m.text in '/Quark' and len(m.text)>4) 
	or (m.text in '/290' and len(m.text)>3) 
	or (m.text in '/390' and len(m.text)>3) 
	or (m.text in '/1080' and len(m.text)>3) 
	or (m.text in '/Fury' and len(m.text)>3) 
	or (m.text in '/280x' and len(m.text)>3) 
	or (m.text in '/380' and len(m.text)>3) 
	or (m.text in '/470' and len(m.text)>3) 
	or (m.text in '/480' and len(m.text)>3) 
	or (m.text in '/750Ti' and len(m.text)>3) 
	or (m.text in '/1060' and len(m.text)>4) 
	or (m.text in '/1070' and len(m.text)>4)
	or (m.text in 'Отмена' and len(m.text)>5) 
	or (m.text in 'Cancel' and len(m.text)>5) 
	or (m.text in '\xE2\xAC\x85') 
	or (m.text in '\xF0\x9F\x91\xBE Calculate' and len(m.text)>3) 
	or (m.text in '\xF0\x9F\x91\xBE Рассчитать' and len(m.text)>5) 
	or (m.text in '/сalculate') 
	or (m.text in '/start') 
	or (m.text in '\xF0\x9F\x93\x96') 
	or (m.text in '/ads')  
	or (m.text in '\xE2\x9D\x93') 
	or (m.text in '/info') 
	or (m.text in '\xE2\x98\x8E') 
	or (m.text in '/report') 
	or (m.text in '/account' and len(m.text)==8) 
	or (m.text in '\xF0\x9F\x99\x87') 
	or (m.text in '\xF0\x9F\x99\x87 Account' and len(m.text)>5) 
	or (m.text in '\xF0\x9F\x99\x87 Профиль' and len(m.text)>5) 
	or (m.text in '\xF0\x9F\x93\x88 Exchange' and len(m.text)>3) 
	or (m.text in '\xF0\x9F\x93\x88 Биржи' and len(m.text)>4) 
	or (m.text in '/exchange')
	or (m.text in '\xE2\x9A\x93 Pools' and len(m.text)>3) 
	or (m.text in '\xE2\x9A\x93 Пулы' and len(m.text)>5) 
	or (m.text in '/pools')
	or (m.text in 'SHA-256' and len(m.text)==7) 
	or (m.text in 'Scrypt' and len(m.text)==6) 
	or (m.text in 'X11' and len(m.text)==3) 
	or (m.text in 'Qubit' and len(m.text)==5) 
	or (m.text in 'Quark' and len(m.text)==5) 
	or (m.text in 'Ethash' and len(m.text)==6) 
	or (m.text in 'CryptoNight' and len(m.text)==11) 
	or (m.text in 'Equihash' and len(m.text)==8) 
	or (m.text in 'LBRY' and len(m.text)==4) 
	or (m.text in 'X11Gost' and len(m.text)>4) 
	or (m.text in 'Groestl' and len(m.text)>4) 
	or (m.text in 'Lyra2REv2' and len(m.text)==6) 
	or (m.text in 'NeoScrypt' and len(m.text)==11) 
	or (m.text in 'Blake14r' and len(m.text)==8) 
	or (m.text in 'Blake2b' and len(m.text)>4) 
	or (m.text in 'Pascal' and len(m.text)>4) 
	or (m.text in 'ETH+PASL' and len(m.text)==8) 
	or (m.text in 'ETH+DCR' and len(m.text)==7)):
			return

[Лаврентий Федотов]

Я знаю, что это жесть

[Лаврентий Федотов]

Я пытался делать через функции, но видимо делал что-то не так, ну и бросил пока 70 раз, чтобы работало

[aRegius]

lavr_fedotov: А с какой целью используется проверка длины переменной? На простом примере если? То есть, мы проверяем есть ли X в 'blabla' и далее проверяем что длина X чему-то равна (больше, меньше), для чего? И еще... Как понимать, например, эту строку кода

or (m.text in 'NeoScrypt' and len(m.text)==11) # ???

Как такое может быть, если в строке 'NeoScrypt' всего 9 символов ?

Answer 1

[Влад Григорьев]

А что в m.text?
Пример:
Ну и допустим что m.text != ETH+DCR, иначе можно просто все условия положить в список и проверять m.text in list_if
Сделайте функцию

def check_if(value):
    list_if = [
        ('ETH+PASL', 7),
        ('ETH+DCR', 8),
        ('/pools', None),

        # и так далее
    ]
    for check_v, check_len in list_if:
        if value in check_v and ((check_len == len(value)) if check_len else True):
            return True


check_if(m.text)

так и расширять в последствии будет проще

Answer 2

[sim3x]

def is_conditions_for_some_shit_pass(m):
  return ((m.text in '/Ethash' and len(m.text)>7) 
  or (m.text in '/DCR' and len(m.text)>3) 
  or (m.text in '/SIA' and len(m.text)>3) 
  # ....
  or (m.text in 'ETH+PASL' and len(m.text)==8) 
  or (m.text in 'ETH+DCR' and len(m.text)==7))

#####
if is_conditions_for_some_shit_pass(m): 
  pass

Answer 3

[Alexej Simakov]

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

по теме: попробуйте засунуть группы условий в лямбды

#каждое условие отдельно
isGreaterThanFive = lambda x: x >5
isSmallerThanTen = lambda x: x<10

# или в виде комбинированого условия
matches_criteria_1 = lambda v: v >5 and v<10

n = 4
if (isGreaterThanFive(n) and isSmallerThanTen(n)):
  print "matches"
else:
  print "dont match"

v = 9
if (matches_criteria_1(v)):
  print "matches"
else:
  print "dont match"

P.S.: покажите мне if с 100 условиями, я такого никогда в жизни не видел. Пожалуйста.

Comments

[Anatoly Scherbakov]

И мне! Я тоже хочу!

[Лаврентий Федотов]

Anatoly Scherbakov: Выше к комментариях к вопросу

[Alexej Simakov]

lavr_fedotov:
Из самого простого:
Вы можете использовать "in list" конструкцию питона;

if m.text in ['/сalculate', '/start', '\xF0\x9F\x93\x96', '/ads']:

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

or (m.text in 'Qubit' and len(m.text)==5) 
  or (m.text in 'Quark' and len(m.text)==5) 
#эквивалентно:
if len(m.text) == 5 and m.text in ["Qubit", "Quark"]:

Ну а в приницпе перейти на вариант предложеный Влад Григорьев.

Answer 4

[TonicZz]

Применение функций.