Как исправить вывод картинок?

Question

[Дмитрий Касаткин]

В def vote создаю рандомный запрос в бд и отправляю картинку через return
Затем оценку которую поставил юзер должна уйти к этой картинке в бд.
Но проблема, когда ставишь оценку, она засчитывается СЛЕДУЮЩЕЙ картинке которая еще не появилась. И вместо 1 картинке бот присылает 3 или 4 картинки.
Подскажите что неправильно сделал?

Код вывода картинки и счет оценок

@bot.message_handler(content_types=['text'])
def distribution(message):
    if message.text == 'Оценивать':
        vote(message)
        print(vote(message)[1])
    elif message.text == 'Назад':
        main_menu = types.ReplyKeyboardMarkup(resize_keyboard=True, row_width=2)
        main_1 = types.KeyboardButton('Мой профиль')
        main_2 = types.KeyboardButton('Оценивать')
        main_menu.add(main_1, main_2)
        bot.send_message(message.chat.id, 'Главное меню', reply_markup=main_menu)

    elif message.text == '1' or '2' or '3' or '4' or '5':
        print(vote(message)[1])
        print(message.text)
        conn = sqlite3.connect('vote.db')
        cursor = conn.cursor()
        cursor.execute("SELECT * FROM users WHERE chat_id == ?", (f'{vote(message)[1]}',))
        res = cursor.fetchone()
        rating = res[5] + int(message.text)
        total_votes = res[6] + 1
        cursor.execute(f"UPDATE users SET rating == ?, total_votes == ? WHERE chat_id == ?", (f'{rating}', f'{total_votes}', f'{vote(message)[1]}'))
        conn.commit()
        conn.close()
    else:
        bot.send_message(message.chat.id, 'Я не знаю такой команды')
			
def vote(message):
    conn = sqlite3.connect('vote.db')
    cursor = conn.cursor()
    cursor.execute(f"SELECT id FROM users")
    result1 = cursor.fetchall()
    queue = (random.choice(result1)[0])
    cursor.execute("SELECT * FROM users WHERE id == ?", (f'{queue}',))
    result = cursor.fetchone()
    chat_id = result[1]
    name = result[2]
    avatar = result[4]
    markup = types.ReplyKeyboardMarkup(resize_keyboard=True, row_width=5)
    btn1 = types.KeyboardButton('1')
    btn2 = types.KeyboardButton('2')
    btn3 = types.KeyboardButton('3')
    btn4 = types.KeyboardButton('4')
    btn5 = types.KeyboardButton('5')
    btn6 = types.KeyboardButton('Назад')
    btn7 = types.KeyboardButton('Жалоба')
    btn8 = types.KeyboardButton('Сообщение')
    markup.add(btn1, btn2, btn3, btn4, btn5, btn6, btn7, btn8)
    return [bot.send_photo(message.chat.id, avatar, caption=f'{chat_id}', reply_markup=markup), chat_id]

Comments

[Vindicar]

message.text == '1' or '2' or '3' or '4' or '5':
Блин, да с чего вы все берёте, что так можно сравнивать? Это НЕ работает так.

[Дмитрий Касаткин]

Vindicar, Понял, что это не правильно. Подскажите пожалуйста как правильно.

[Vindicar]

Дмитрий Касаткин, это описано в любом учебнике.
Для 2-3х вариантов можно написать (message.text == '1') or (message.text == '2')
Если вариантов много, будет проще написать message.text in ('1', '2', '3', '4', '5')

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

Vindicar, наверно, проще сделать, чтоб так работало, чем научить всех делать как правильно…

[Дмитрий Касаткин]

Алан Гибизов, Так я и спрашиваю у более опытных как правильно.

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

Дмитрий Касаткин, а как вы учите python?

[Дмитрий Касаткин]

Алан Гибизов, больше на практике, и онлайн курсы.

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

Дмитрий Касаткин, тогда вот вам совет: найдите (купите, скачайте) книгу «Изучаем Python» М.Лутц, и прорабатывайте её. Это будет многократно эффективней.

[Дмитрий Касаткин]

Алан Гибизов, Спасибо. Скачал, там 2 тома. Буду изучать.

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

Дмитрий Касаткин, кстати, издание российское в бумаге ужасно. Мелкий шрифт на полупрозрачной бумаге, толстый тяжеленный том. Купил за дикие тыщи, лежит, а я с iPad читаю pdf.

Answer 1

[Максим Кравчук]

Каждый vote(message) вызывает отправку новой картинки!
То есть

if message.text == 'Оценивать':
        vote(message)
        print(vote(message)[1])

тут вы уже две картинки отправили!

cursor.execute(f"UPDATE users SET rating == ?, total_votes == ? WHERE chat_id == ?", (f'{rating}', f'{total_votes}', f'{vote(message)[1]}'))

Здесь vote(message)[1] это уже следующее сообщение.
А Вам надо текущему проставить. Для этого chat_id получите из message:
message.from_user.id

Comments

[Дмитрий Касаткин]

Втом то и загвоздка. Задача получить чат айди соответствующий картинке.

[Максим Кравчук]

То есть это норм, что отправляется несколько картинок?
Тогда при выставлении оценки надо:
- либо отправлять оценку реплаем на картинку, тогда можно будет знать на что отправлена оценка
- либо к каждой картинке крепить инлайн кнопки и оценку принимать от них

[Дмитрий Касаткин]

Максим Кравчук, это все от незнания как работает return. Думал понимаю, а оказалось нет) А есть возможность получить 1 из 2 значений return, несколько раз в разных функциях?