Как связать if и определенный массив в телеграмм боте?

Question

[madreyk]

Python 3.9.2
PyTelegramBotAPI
Я должен создать приватного телеграмм бота, для определенной группы людей. В массив "users" я планирую занести пользователей, у которых есть доступ

users=['<usernames>']
@bot.message_handler(commands='Начать')
def handle_start(message):
	if message.from_user.username==(users):
		bot.send_message(message.from_user.id, 'Вот', reply_markup=user_markup, parse_mode='HTML')
	if message.from_user.username!=(users):
		bot.send_message(message.from_user.id, 'В доступе отказано', reply_markup=user_markup, parse_mode='HTML')

или я думаю

users=['<usernames>']
@bot.message_handler(commands='Начать')
def handle_start(message):
	if message.from_user.username==(users):
		bot.send_message(message.from_user.id, 'Вот', reply_markup=user_markup, parse_mode='HTML')
	else:	
        bot.send_message(message.from_user.id, 'В доступе отказано', reply_markup=user_markup, parse_mode='HTML')

Но вот дебаг:

Traceback (most recent call last):
  File "start.py", line 25, in <module>
    bot.polling(none_stop=True, interval=0)
  File "/home/mem/.local/lib/python3.8/site-packages/telebot/__init__.py", line 485, in polling
    self.__threaded_polling(none_stop, interval, timeout, long_polling_timeout)
  File "/home/mem/.local/lib/python3.8/site-packages/telebot/__init__.py", line 544, in __threaded_polling
    raise e
  File "/home/mem/.local/lib/python3.8/site-packages/telebot/__init__.py", line 506, in __threaded_polling
    polling_thread.raise_exceptions()
  File "/home/mem/.local/lib/python3.8/site-packages/telebot/util.py", line 87, in raise_exceptions
    raise self.exception_info
  File "/home/mem/.local/lib/python3.8/site-packages/telebot/util.py", line 69, in run
    task(*args, **kwargs)
  File "/home/mem/.local/lib/python3.8/site-packages/telebot/__init__.py", line 323, in __retrieve_updates
    self.process_new_updates(updates)
  File "/home/mem/.local/lib/python3.8/site-packages/telebot/__init__.py", line 384, in process_new_updates
    self.process_new_messages(new_messages)
  File "/home/mem/.local/lib/python3.8/site-packages/telebot/__init__.py", line 410, in process_new_messages
    self._notify_command_handlers(self.message_handlers, new_messages)
  File "/home/mem/.local/lib/python3.8/site-packages/telebot/__init__.py", line 2183, in _notify_command_handlers
    if self._test_message_handler(message_handler, message):
  File "/home/mem/.local/lib/python3.8/site-packages/telebot/__init__.py", line 2149, in _test_message_handler
    if not self._test_filter(message_filter, filter_value, message):
  File "/home/mem/.local/lib/python3.8/site-packages/telebot/__init__.py", line 2170, in _test_filter
    return test_cases.get(message_filter, lambda msg: False)(message)
  File "/home/mem/.local/lib/python3.8/site-packages/telebot/__init__.py", line 2166, in <lambda>
    'commands': lambda msg: msg.content_type == 'text' and util.extract_command(msg.text) in filter_value,
TypeError: 'in <string>' requires string as left operand, not NoneType

Comments

[Сергей Горностаев]

Во-первых, у вас ошибка не связана с вопросом. Во-вторых, строку со списком строк сравнивать очевидно бесполезно.

[Ищю в поисковиках]

Попробуйте заменить
message.from_user.username==(users)
На
message.from_user.username in users

[Алан Гибизов]

Действительно, ошибка

TypeError: 'in ' requires string as left operand, not NoneType

говорит, что ошибка в операндах операции in - а у вас в коде такой операции не видно.

В общем случае оно должно выглядеть примерно так:

if “a” in “abc”:
    print(“a present”)
else:
    print(“a absent”)

Или

if “a” in [“a”, “b”, “c”]:
    print(“a present”)
else:
    print(“a absent”)

Что касается приведённых отрезков кода - тут просматривается сразу несколько проблем.
Во-первых, упомянутая Сергеем попытка сравнивать значение переменной message.from_user.username со списком.
Надо вместо == использовать in как в примерах выше.
Во-вторых, сразу видно, что совершенно нет понимания, что в python скобки - не просто скобочки. Я, когда начинал, тоже долго не понимал смысл скобочек. Лично мне помогла книга «Изучаем Python» М.Лутца, уже первые две сотни страниц дадут громадный скачок в понимании.
Ну и, на закуску, есть ощущение, что и со списками не всё понятно, и с PEP 8 не знаком…

Либо потратьте время на изучение Python, либо деньги на найм программиста. А то всё выглядит так, будто вы взялись переделать схему радиопередатчика на другой диапазон, не имея понятия о законе Ома и способах соединения радиодеталей. Не надо так.

[Hemul GM]

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

[madreyk]

Сергей Горностаев, да, я понимаю, что нельзя сравнивать разные типы данных, но когда я ставил скобку и указывал str, ничего не менялось

[madreyk]

Алан Гибизов, in не работает:

@bot.message_handler(commands='Начать')
def handle_start(message):
	if message.from_user.username in users:
		bot.send_message(message.from_user.id, 'Вот', reply_markup=user_markup, parse_mode='HTML')
	else:
		bot.send_message(message.from_user.id, 'В доступе отказано', reply_markup=user_markup, parse_mode='HTML')

ошибка:

Traceback (most recent call last):
  File "start.py", line 25, in <module>
    bot.polling(none_stop=True, interval=0)
  File "/home/mem/.local/lib/python3.8/site-packages/telebot/__init__.py", line 485, in polling
    self.__threaded_polling(none_stop, interval, timeout, long_polling_timeout)
  File "/home/mem/.local/lib/python3.8/site-packages/telebot/__init__.py", line 544, in __threaded_polling
    raise e
  File "/home/mem/.local/lib/python3.8/site-packages/telebot/__init__.py", line 506, in __threaded_polling
    polling_thread.raise_exceptions()
  File "/home/mem/.local/lib/python3.8/site-packages/telebot/util.py", line 87, in raise_exceptions
    raise self.exception_info
  File "/home/mem/.local/lib/python3.8/site-packages/telebot/util.py", line 69, in run
    task(*args, **kwargs)
  File "/home/mem/.local/lib/python3.8/site-packages/telebot/__init__.py", line 323, in __retrieve_updates
    self.process_new_updates(updates)
  File "/home/mem/.local/lib/python3.8/site-packages/telebot/__init__.py", line 384, in process_new_updates
    self.process_new_messages(new_messages)
  File "/home/mem/.local/lib/python3.8/site-packages/telebot/__init__.py", line 410, in process_new_messages
    self._notify_command_handlers(self.message_handlers, new_messages)
  File "/home/mem/.local/lib/python3.8/site-packages/telebot/__init__.py", line 2183, in _notify_command_handlers
    if self._test_message_handler(message_handler, message):
  File "/home/mem/.local/lib/python3.8/site-packages/telebot/__init__.py", line 2149, in _test_message_handler
    if not self._test_filter(message_filter, filter_value, message):
  File "/home/mem/.local/lib/python3.8/site-packages/telebot/__init__.py", line 2170, in _test_filter
    return test_cases.get(message_filter, lambda msg: False)(message)
  File "/home/mem/.local/lib/python3.8/site-packages/telebot/__init__.py", line 2166, in <lambda>
    'commands': lambda msg: msg.content_type == 'text' and util.extract_command(msg.text) in filter_value,
TypeError: 'in <string>' requires string as left operand, not NoneType

Постараюсь в свободное время ознакомится с данной книгой

[Сергей Горностаев]

madreyk, вам уже несколько человек сказали, что ошибка не связана с кодом сравнения. Она возникает в библиотеке.

[Алан Гибизов]

madreyk, ошибка говорит: левым операндом операции in должна быть строка, а не None. А у вас вот это:
message.from_user.username
возвращает None.
Почему? Так сходу не разберёшься. Возможно, вы его неправильно используете; возможно, оно кривое само по себе; но копать надо туда. Не in виноват, конечно.

Answer 1

[Cheabatto]

Попробуйте вот так:
users=['users']
@bot.message_handler(commands='start')
def handle_start(message):
print(message.from_user.username)
if message.from_user.username in users:
bot.send_message(message.from_user.id, 'Вот', parse_mode='HTML')
if message.from_user.username not in users:
bot.send_message(message.from_user.id, 'В доступе отказано', parse_mode='HTML')