0

Всех приветствую! Подскажите, как ограничить доступ к выполнению команд и пользовательского ввода с Reply-клавиатуры для user_id, которые отсутствуют в базе данных?

С командами пока не придумал.

Но с обработчиками Reply клавиатуры пробовал так:

def check_id(message):
    id = message.from_user.id
    cursor.execute('SELECT * FROM registration WHERE user_id=?', (id, )).fetchone()

@bot.message_handler(content_types=['text']) 
def statistics_step(message): 
    if message.text.lower() == 'анкета 1' and check_id is None:
        one_anketa_step(message)
    if message.text.lower() == 'анкета 2' and check_id is None:
        two_anketa_step(message)

Мой способ не сработал, бот совсем перестал реагировать на клавиатуру.

P.S. TelegramBotAPI+SQLite+Python

4
  • 1
    Почему функцию check_id вызываете без аргумента? И почему в SELECT выводите все поля а не одно?
    – dtroyan
    16 ноя 2021 в 22:39
  • С аргументом косяк, согласен. Пытался и с аргументом, все равно не работало. А select проверяет все user_id из базы с user_id пользователя.
    – ProNoob
    16 ноя 2021 в 22:41
  • 1
    Плохая практика писать в селекте звездочку, там достаточно одно поле выводить. Также функция ничего возвращает.
    – dtroyan
    16 ноя 2021 в 22:43
  • @dtroyan Если есть возможность, покажите примером. Не уверен, что в другом случае таблица сравнит все user_id. Но скорее всего, я просто что-то не понимаю)
    – ProNoob
    16 ноя 2021 в 22:47

1 ответ 1

1

Попробуйте так, что-то вроде этого. Кстати в условиях вы тоже вроде ошиблись, если я правильно понял вашу логику:

def check_id(message):
    id = message.from_user.id
    record = cursor.execute('SELECT * FROM registration WHERE user_id=?', (id, )).fetchone()
    if record:
        return True
    else:
        return False

@bot.message_handler(content_types=['text'])
def statistics_step(message):
    if (message.text.lower() == 'анкета 1') and check_id(message):
        one_anketa_step(message)
    if (message.text.lower() == 'анкета 2') and check_id(message):
        two_anketa_step(message)
7
  • Спасибо большое, все работает) А на счет выполнения команд /start /menu не подскажите? Видел подобный вопрос через lamda в примере делали, но там со списком. [ru.stackoverflow.com/questions/1230730/…
    – ProNoob
    16 ноя 2021 в 23:17
  • 1
    Можно сделать также как вы здесь реализовали, Там проверяется на вхождение в список, а вы сделайте на проверку True
    – dtroyan
    16 ноя 2021 в 23:26
  • А именно централизовано на все команды? А не в каждой команде по отдельной проверке.
    – ProNoob
    16 ноя 2021 в 23:30
  • 1
    Примерно так - func=lambda message: check_id(message) == True
    – dtroyan
    16 ноя 2021 в 23:57
  • @дроян, спасибо большое, позже попробую)
    – ProNoob
    17 ноя 2021 в 0:15

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge you have read our privacy policy.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.