Как пройтись циклом for в файле txt?

Question

[Igor def]

Простой фильтр запрещенных слов:

@dp.message_handler()
async def mat(message: types.Message):
    #mat = ["слово1", "слово2"]
    file_mat = open("all_mat.txt", "r")
    mat = file_mat
    for i in mat:
        if len(message.text) > len(message.text.replace(i, "")):
            await message.delete()
            await message.answer(f"{message.from_user.first_name}, ваше сообщение удалено!")
        break
    file_mat.close()

если использовать список внутри функции mat = ["слово1", "слово2"] то естественно работает.
Хотел сделать все это внутри файла all_mat.txt. Изучил документацию, пытаюсь пройтись циклом в файле, но я понимаю, что я как то не так это делаю, подскажите правильный метод пожалуйста.

Comments

[Everything_is_bad]

напиши эту реализацию отдельной функций в отрыве от aiogram, а то ты сейчас творишь какую-то дичь

[Igor def]

Everything_is_bad, а можно пояснить немного? Потому что я не такой мега спец.. Я писал отдельной функцией, просто ничего не происходит, когда пользователь пишет слово из файла "all_mat.txt"

Answer 1

[Igor def]

решил другим способом:
создал Py файл list_mat в котором есть список list

from list_mat import list

@dp.message_handler()
async def mat(message: types.Message):
    for i in list:
        if len(message.text) > len(message.text.replace(i, "")):
            await message.delete()
            await message.answer(f"{message.from_user.first_name}, ваше сообщение удалено!")
        break

Comments

[Everything_is_bad]

проблема что твое решение не уменьшило количество дичи

[Vindicar]

Это может работать, но это трижды неправильно.
1. from list_mat import list за переопределение встроенных имён (а list - именно такое) в приличном обществе бьют по рукам.
2.

len(message.text) > len(message.text.replace(i, ""))

- а ещё более через задницу слабо решить? Чем просто i in message.text не подошло? Впрочем, и тот, и тот вариант не справятся с другим регистром текста (большие буквы или маленькие).
3. Сначала удаляем сообщение, потом на него отвечаем? Нелогично. Это может работать, но нет гарантий, что это будет работать всегда. Лучше в обратном порядке.

Твоя первоначальная попытка упиралась в две простые вещи. Первое, если ты делаешь цикл for по файлу, ты перебираешь его строки, т.е. каждое слово должно идти на отдельной строке, без кавычек и прочего. Второе, при этом символ перевода строки тоже читается, и его надо убрать (например, методом strip()).
Так что сделать список из строк файла можно очень просто:

with open('file.txt', 'rt', encoding='utf-8') as f:  # гарантирует, что файл будет закрыт
    # перебирает строки в файле, помещает каждую в line, делает strip(), результат помещает в список word_list
    word_list = [ line.strip() for line in f ]

Это можно будет сделать один раз, при запуске бота, а не читать файл при каждом сообщении.
Впрочем, в этом случае для обновления списка слов потребуется перезапуск бота, а если делать это при обработке сообщения, то обновлённый список подхватится сразу же.

[Igor def]

Vindicar, спасибо большое за пояснение! Все сработало

[Igor def]

Vindicar, сначала сделал, проверил, потом понял что он не удалят сообщение, если это запрещенное слово внутри какого либо предложения. Сделал вот так:

@dp.message_handler(lambda msg: any(word in msg.text.lower() for word in word_list))
async def mat(message: types.Message):
    await message.delete()
    await message.answer(f"{message.from_user.first_name}, ваше сообщение удалено!")

правильно ли я сделал?

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

[Vindicar]

Igor def, можно и так, хотя такой подход по-прежнему среагирует на старое-доброе "гребля заштрихуй оскорблять" (т.е. на ситуации, когда слово не из чёрного списка содержит в себе слово из чёрного). Так куда аккуратнее, во всяком случае.

А насчёт удаления - да, возможно, я перепараноил немного. С моей точки зрения, после того как сообщение удалено, с ним уже ничего нельзя делать - его больше нет. Но телега, походу, просто скрывает удалённые сообщения, как минимум на первых порах, так что они всё ещё существуют...

[Igor def]

Vindicar, понял, спасибо. Ну я не собирался вычесывать такие предложения как "гребля заштрихуй оскорблять" - уж больно заморочено.. Может в будущем, когда опыта наберусь. В таких случаях модератор в ручном порядке может мутить пользователя.. А эту уже совсем другая история :) Еще раз спасибо за пояснения и помощь!

[Vindicar]

Igor def, не, ты не понял =) Бот на них среагирует, но ругательств-то там нет. Так что модератору придётся размутить несправедливо замученных.