Как можно написать этот код короче (предложите свои варианты)?

Question

[Yevgeni]

def check_username_and_description(self, status):
        if any(x in status.user.screen_name.lower() for x in BOT_SETTINGS['banned_username_userinfo_words']):
            return False
        if any(x in status.user.description.lower() for x in BOT_SETTINGS['banned_username_userinfo_words']):
            return False
        return True

Answer 1

[Astrohas]

def check_username_and_description(self, status):
    return not any(
            x in " ".join([status.user.screen_name, status.user.description]).lower()
            for x in BOT_SETTINGS['banned_username_userinfo_words']
        )

или

def check_username_and_description(self, status):
    return not bool(
        set((status.user.screen_name + " " +  status.user.description).lower()).intersection(
            set(BOT_SETTINGS['banned_username_userinfo_words'])
        )
    )

Comments

[Yevgeni]

Спасибо. Оба варианта хороши. Пошел практиковаться с сетами. А то что-то я не уделяю им должного внимания.

Answer 2

[Анатолий]

def check_username_and_description(self, status):
    return False if any(x in status.user.screen_name.lower() for x in BOT_SETTINGS['banned_username_userinfo_words']) or any(x in status.user.description.lower() for x in BOT_SETTINGS['banned_username_userinfo_words']) else True

Comments

[Yevgeni]

но мне ведь нужно проверить две строки а не одну.

status.user.screen_name.lower()
status.user.description.lower()

[Анатолий]

Yevgeni, так он и проверяет 2 строки. У вас изначально код не очень корректный, т.к. если сработает if и вернет False, то True уже не вернется.
А так, этот код полностью соответствует тому, что вы написали.

[Astrohas]

старайтесь не писать такие длинные выражения

[Анатолий]

Astrohas, в оригинальном коде длина строки 101 символ без отступов -> человек не соблюдает PEP -> зачем напрягать его лишними переносами.

[Astrohas]

Анатолий, да, но воспитать топикстартера то нужно

Answer 3

[Вадим Шаталов]

def check_username_and_description(self, status):
    return not any([word.lower() in BOT_SETTINGS['banned_username_userinfo_words'] 
        for word in (status.user.description, status.user.screen_name)])