Как сократить/заменить списковое включение Python?

Question

[Андрей]

У меня в процессе "очистки" текста (txt = str(...)), получилось очень длинное списковое включение (недостаток опыта), можно ли его, как-то сократить/оптимизировать (может какие-то альтернативные функции или варианты записи есть)

' '.join([i for i in txt.split() if i.isupper() or i.isdigit() or [x.isdigit() for x in i].count(True) > 0 or [x.isupper() for x in i].count(True) >= 2 or translator.detect(i).lang == 'en' and stemmer.stem(i.lower()) not in trash_words])

Comments

[Александр]

Приведите минимальный воспроизводимый пример

[Андрей]

Например, мне необходимо вычленить из текста модель продукта.

l1 = 'Fernglas KILO3000 BDX 10x42 mit Entfernungsmesser'

Слово будет являться элементом модели, если оно попадает хотя бы под одно условие: все титульные, две и и более титульные, есть цифры, является английским словом, и не входит в список из стоп-слов [trash_words]). Было бы ваще отлично как-то по смыслу еще идентифицировать (возможны случае что слово не на английском, то тоже является элементом модели), но пока даже не представляю как это реализовать.

Answer 1

[Andrey]

Нечитабельно и сложно поддерживать - ты завтра уже забудешь что тут написано
К тому же у тебя уже логические ошибки - неправильная группировка условия or / and - ты забыл поставить скобки

Твой вариант

txt = 'sdf s7878 adsf '
trash_words = ['s7878']
s = ' '.join([i for i in txt.split() if i.isupper() or i.isdigit() or [x.isdigit() for x in i].count(True) > 0 or [x.isupper() for x in i].count(True) >= 2 and i.lower() not in trash_words])
print(s)

выдает
s7878
а должен ничего не выдавать

Правильно должно быть так

s = ' '.join([i for i in txt.split() if (i.isupper() or i.isdigit() or [x.isdigit() for x in i].count(True) > 0 or [x.isupper() for x in i].count(True) >= 2) and i.lower() not in trash_words])

Я бы сделал вот так

def check_str(i):
    if ((i.isupper() or
        i.isdigit() or
        [x.isdigit() for x in i].count(True) > 0 or
        [x.isupper() for x in i].count(True) >= 2) and
        i.lower() not in trash_words):
        return True
    return False

txt = 'sdf s7878 adsf '
trash_words = []
print(' '.join([i for i in txt.split() if check_str(i)]))

Comments

[Андрей]

дак правильно что выдает
s7878
так и должно быть, ибо за это отвечает условие
[x.isdigit() for x in i].count(True) > 0

а в Вашем случае, тогда наверное уж лучше воспользоваться конструкцией map c filter....

[Andrey]

дак правильно что выдает

Нет, не правильно. Смотри как у тебя работают условия or/and

У тебя не выполняется вот это условие
and stemmer.stem(i.lower()) not in trash_words