Как сделать проще?

Question

[bybacapitan]

У меня есть функция которая проверяет подписан ли человек на канал, в неё нужно постоянно user_id, но у меня много хендлеров.

user_id = message.from_user.id
full_name = message.from_user.full_name
chat_id = message.chat.id
if await check_sub_channel(user_id, full_name, chat_id):

Думаю это можно упростить и сделать правильнее, но как?

Comments

[mayton2019]

Тут наверное мало информации для рефакторинга.

Answer 1

[Александр Нестеров]

По идее вам достаточно фильтров. Примеры:
aiogram
pytelebotapi

Answer 2

[fenrir]

Ну как минимум если все три значения лежат в message, передавайте просто message, а внутри уже проверяйте
if await check_sub_channel(message):
И зачем плодить переменные, если и так читаемость сохраняется, особенно если указывать что передаете

check_sub_channel(user_id=message.from_user.id, name=message.from_user.full_name, chat_id=message.chat.id):

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