того же UserDao, или лучше явно передавать в конструктор 4-5 нужных параметров?
что такое UserDao?
Ну я как пример взял, это может быть например датакласс, маппящий message в объект пользователя из бд, для последующей работы с ним в дата слое
ух, прям тырпрайзом пахнуло
Да не я не умею
Кстати, лучше старый шаблон Кости убери и возьми от Тишки (это я про твоего бота-агрегатора)
это какойт-такой от тишки? Можно ссыль? Я просто как-то его стащил, он мне не очень нравится, но особо не парился по этому поводу
Тут много странной фигни. open(...) можно заменить на InputFile (иначе ты сейчас не закрываешь файл), зачем к двум сообщениям подряд одна и та же реплай-клава? Почему вообще два сообщения?
ну реплай я кидаю автоматом на каждый мессаж, с целью обновить ее у юзера если я ее на бакенде обновил. Как это по-другому реализовать — не придумал
Лишаешь себя возможности редактировать сообщения
Про InputFile спасибо, согласен.
Про редактирование — вообще хз как это использовать, юзкейсов не вижу в исправлении сообщений помимо некоторых групповых ботов
Ну крч я предлагаю сначала переписать с тишкиным шаблоном (см. первый закреп), а потом уже чинить всё остальное)
Оке, я посмотрю, спасибо)
Если начнёшь ещё и на aiogram 3.x переходить, то можешь сделать один роутер для сообщений в ЛС, другой роутер для сообщений в группах, чтобы не тащить лишний фильтр постоянно
роутеры вообще таскать не надо
В смысле?
шаблон тишки предполагает прокидывание роутера в регистрацию хендлеров, а не импортирование его постоянно
Вот это вообще плохо. Как минимум, хотя бы логировать. Как максимум, удалять пользователя из БД (вроде хранишь)
А я где-то говорил, что надо импортировать?
"тащить лишний фильтр" -> chat_type="private" / chat_type = "group"
Да, храню, логировать смысла вообще не вижу (срать ненужной информацией) + он не коммерческий, для себя делал, там 15 человек.
Обсуждают сегодня