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

Question

[Puffer_fish]

При попадании дротика в цель должно отправляться сообщение с победой(оно отправляется), но так же вместе с ним отправляется сообщение о неудаче. Как это исправить?

@bot.message_handler(commands=['start'])
def send_message(message):
	keyboard1 = telebot.types.ReplyKeyboardMarkup(True)
	keyboard1.row('Давай поиграем!')
	mess = f' Привет, <b>{message.from_user.username}<u></u></b>! Хочешь поиграть?'
	bot.send_message(message.chat.id ,mess, parse_mode='HTML', reply_markup=keyboard1)
	@bot.message_handler(content_types=['text'])
	def func(message):
		if(message.text =='Давай поиграем!'):
		  	bot.send_message(message.chat.id, ' Ну тогда поехали!', reply_markup=types.ReplyKeyboardRemove())
		  	sent = bot.send_dice (message.chat.id, '')
		  	if sent.dice.value == 6:
		  		time.sleep(5)
		  		mess = f'Ого, тебе повезло!'
		  		bot.send_message(message.chat.id, mess, parse_mode='html')
		  	if sent.dice.value == 1 or 2 or 3 or 4 or 5:
		  		time.sleep(5)
		  		mess3 = f'Не повезло! Хочешь попробовать снова?'
		  		bot.send_message(message.chat.id, mess3, parse_mode='html', reply_markup = keyboard1)

Comments

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

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

Answer 1

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

if sent.dice.value == 1 or 2:
означает если (значение равно 1) или истина:
то-есть всегда истинно.
Внимательнее изучите, как правильно такие вещи делать.
Можно
if 1 <= value < 6:
Можно
if value in range(1, 6):
Можно
if value in [1, 2, 3, 4, 5]:

Comments

[Константин Нагибович]

if value in range(1, 6)
На практике столкнулся с тем, что это выражение при больших диапазонах здорово тупит. Было примерно range(1, 10000000)

[shurshur]

Константин Нагибович, конечно, ведь range - генератор. Он сгенерирует все миллионы вместо двух простых сравнений :)

[Константин Нагибович]

shurshur, я в курсе. Просто не рекомендую такую конструкцию для такого простого сравнения использовать.