Сообщения отправляются одному человеку хотя должный двум, как сделать так чтобы сообщения отправлялись 2 пользователям?

Question

[1parahon]

вот кусок кода в котором все происходит

bot.send_message(chat_id = poroh, text="Идет запуск сервера. Сервер запустится в течении 3 минут")
        bot.send_message(chat_id = user_id, text="Идет запуск сервера. Сервер запустится в течении 3 минут")
        os.chdir(r"C:\server_bot\servers\server1")
        os.startfile(r"C:\server_bot\servers\server1\start_server.bat")
        markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
        btn1 = types.KeyboardButton("Запуск сервера")
        btn2 = types.KeyboardButton("Остановка сервера")
        markup.add(btn1, btn2)
        markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
        bot.send_message(chat_id = user_id, text="Этим ботом управляет порох (poroh1). Что сделать боту??",reply_markup=markup)

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

Answer 1

[Vindicar]

Проверь, не одинаковые ли значения в user_id и poroh.
Если так, проверяй, откуда берётся значение user_id.

EDIT: Когда ты нажимаешь на "Да", это отдельное сообщение - отправленное от тебя!
Т.е. server() вызывается дважды, один раз для запустившего (после выбора карты), второй раз для тебя (ответ да или нет). Это, к слову, несёт в себе ещё одну проблему - если два человека попросят запустить сервер одноврнменно, боту будет тяжело понять, на чей запрос ты ответил.
Тебе надо использовать не ReplyKeyboardMarkup, а InlineKeyboardMarkup, так как там можно привязать к кнопке дополнительную инфу (строку), по которой бот сможет потом понять, к какому сообщению относилась кнопка.

Comments

[1parahon]

они разные проверял принтом, юзер берется выше кодом и все ок, я не понимаю хоть id разные пишется именно мне

[Vindicar]

1parahon, вот прямо так проверял?

bot.send_message(chat_id = poroh, text="Идет запуск сервера. Сервер запустится в течении 3 минут")
print(poroh == user_id, poroh, user_id)
bot.send_message(chat_id = user_id, text="Идет запуск сервера. Сервер запустится в течении 3 минут")

[1parahon]

Vindicar, не,проверял
print(poroh)
print(user_id)
а в чем разница?

[1parahon]

Vindicar, твоим способом проверил одинаковы id, хотя когда я проверял как выше указал были разные

[Vindicar]

1parahon, ну тут вопрос скорее "проверял именно в этом месте кода (+/- строка)"?

[Vindicar]

1parahon, вооот, я это и имел ввиду. У тебя где-то одна из переменных перезаписывается, значит. Где-то между твоей первой проверкой и использованием.

[1parahon]

import subprocess
import os
import telebot
from telebot import types
poroh = 1910413638
bot = telebot.TeleBot("")
@bot.message_handler(commands=["start"])
def start(message):
    markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
    btn1 = types.KeyboardButton("Запуск сервера")
    btn2 = types.KeyboardButton("Остановка сервера")
    markup.add(btn1, btn2)
    bot.send_message(message.from_user.id, "Этим ботом управляет порох (poroh_1parahon). Что сделать боту?", reply_markup=markup)
@bot.message_handler(content_types=["text"])
def server(message):
    user_id = message.from_user.id
    if message.text == "Запуск сервера":
        markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
        btn1 = types.KeyboardButton("Снайперы")
        markup.add(btn1)
        bot.send_message(message.from_user.id, "Выбирите карту", reply_markup=markup)
    elif message.text == "Снайперы":
        if user_id == poroh:
            markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
            os.chdir(r"C:\server_bot\servers\server1")
            os.startfile(r"C:\server_bot\servers\server1\start_server.vbs")
            bot.send_message(message.from_user.id, "Идет запуск сервера. Сервер запустится в течении 3 минут", reply_markup=markup)
            btn1 = types.KeyboardButton("Запуск сервера")
            btn2 = types.KeyboardButton("Остановка сервера")
            markup.add(btn1, btn2)
            bot.send_message(message.from_user.id, "Этим ботом управляет порох (poroh_1parahon). Что сделать боту?", reply_markup=markup)
        else:
            markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
            bot.send_message(message.from_user.id, "Ожидайте ответа пороха", reply_markup=markup)
            username = message.from_user.username
            btn1 = types.KeyboardButton("Да")
            btn2 = types.KeyboardButton("Нет")
            markup.add(btn1, btn2)
            bot.send_message(chat_id = poroh, text=f"@{username} хочет включить сервер, вы согласны с этим?", reply_markup=markup)
    elif message.text == "Да":
        bot.send_message(chat_id=poroh, text="Идет запуск сервера. Сервер запустится в течении 3 минут")
        print(poroh == user_id, poroh, user_id)
        bot.send_message(chat_id=user_id, text="Идет запуск сервера. Сервер запустится в течении 3 минут")
        os.chdir(r"C:\server_bot\servers\server1")
        os.startfile(r"C:\server_bot\servers\server1\start_server.bat")
        markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
        btn1 = types.KeyboardButton("Запуск сервера")
        btn2 = types.KeyboardButton("Остановка сервера")
        markup.add(btn1, btn2)
        markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
        bot.send_message(chat_id = user_id, text="Этим ботом управляет порох (poroh_1parahon). Что сделать боту??",reply_markup=markup)
    elif message.text == "Нет":
        markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
        bot.send_message(message.from_user.id,"Порох отказал в запуске сервера. Спрашивайте у него в лс в дс", reply_markup=markup)
        btn1 = types.KeyboardButton("Запуск сервера")
        btn2 = types.KeyboardButton("Остановка сервера")
        markup.add(btn1, btn2)
        bot.send_message(message.from_user.id,"Этим ботом управляет порох (poroh_1parahon). Что сделать боту?",reply_markup=markup)
    if message.text == "Остановка сервера":
        user = str(message.from_user.id)
        user = int(user)
        if user == 1910413638:
            markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
            subprocess.run(['cmd', '/c', 'taskkill /im java.exe'])
            bot.send_message(message.from_user.id, "Сервер выключен. Что сделать боту?", reply_markup=markup)
        else:
            markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
            x = bot.send_message(message.from_user.id, message.from_user.username)
            bot.send_message(1910413638, f"@{x} хочет выключить сервер, вы согласны с этим?", reply_markup=markup)
            btn1 = types.KeyboardButton("Да")
            btn2 = types.KeyboardButton("Нет")
            markup.add(btn1, btn2)
            if message.text == "Да":

                bot.send_message(message.from_user.id, "Сервер выключен. Что сделать боту?", reply_markup=markup)
                bot.send_message(1910413638, "Сервер выключен. Что сделать боту",reply_markup=markup)
            elif message.text == "Нет":
                bot.send_message(message.from_user.id, "Порох отказал в остановке сервера. Спрашивайте у него в лс в дс",reply_markup=markup)
                btn1 = types.KeyboardButton("Запуск сервера")
                btn2 = types.KeyboardButton("Остановка сервера")
                markup.add(btn1, btn2)
                bot.send_message(message.from_user.id, "Этим ботом управляет порох (poroh_1parahon). Что сделать боту?",reply_markup=markup)
bot.polling(none_stop=True, interval=0)

а где тут только в начале я указал poroh и чуть позже user_id

[Vindicar]

1parahon, так, минутку. У меня глупый вопрос: ты проверяешь бота с того же аккаунта (1910413638)? Тогда оба значения будут совпадать "естественно", так как сообщение пришло от тебя.

[1parahon]

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

[1parahon]

Vindicar, с двух аккаунтов

[1parahon]

Vindicar, то что где выкл серв, я там еще не делал, пока разбираюсь с запуском

[Vindicar]

1parahon, странное дело... а проверка сразу после

def server(message):
    user_id = message.from_user.id

что показывает? Одинаковые или разные?

[1parahon]

Vindicar, щас проверил, по началу id другого аккаунта, а когда идет выбор да или нет с моего аккаунта почему то присваивается мой id

[1parahon]

Vindicar, я кажется понял почему так происходит у меня стоит во втором def запуск при любом тексте и из-за этого меняется id, но я не понимаю как это исправить

[Vindicar]

1parahon, а, блин. Я тупой. =)
Когда ты нажимаешь на "Да", это отдельное сообщение - отправленное от тебя!
Т.е. server() вызывается дважды, один раз для запустившего (после выбора карты), второй раз для тебя (ответ да или нет). Это, к слову, несёт в себе ещё одну проблему - если два человека попросят запустить сервер одноврнменно, боту будет тяжело понять, на чей запрос ты ответил.
Тебе надо использовать не ReplyKeyboardMarkup, а InlineKeyboardMarkup, так как там можно привязать к кнопке дополнительную инфу (строку), по которой бот сможет потом понять, к какому сообщению относилась кнопка.

[1parahon]

Vindicar, ну ботом будут не часто пользоваться, так что это не проблема, а как сделать то что ты написал?

[1parahon]

1parahon, до этого я пытался так сделать, но у меня ошибки были

[Vindicar]

1parahon, если первое - не проблема, ты можешь сохранять в user_id в глобальную переменную и потом читать её. Но имей ввиду, что если поступит несколько запросов прежде чем ты ответишь - все, кроме последнего, будут потеряны.
Второе - читай пример, разбирайся. Если основные моменты, то:

markup = InlineKeyboardMarkup()  # создаём клавиатуру
btn1 = InlineKeyboardButton(  # создаём кнопку
    "Да",   # что будет написано на кнопке
    # скрытая строка, которую пользователь не увидит
    callback_data=f"yes:{message.chat.id}"  # в ней кодируем ответ и id пользователя.
)
btn2 = InlineKeyboardButton(  # создаём кнопку
    "Нет",   # что будет написано на кнопке
    # скрытая строка, которую пользователь не увидит
    callback_data=f"no:{message.chat.id}"  # в ней кодируем ответ и id пользователя.
)
markup.add(btn1, btn2)  # довешиваем кнопку на клавиатуру
bot.send_message(  # клавиатуру всегда отправляем вместе с сообщением
    message.chat.id,  # кому?
    "Yes/no?",   # текст сообщения
    reply_markup=markup  # прикрепляемая клавиатура
)

А когда нужно будет обработать кнопку:

# говорим, что этот обработчик будет обрабатывать ТОЛЬКО кнопки "Да",
# так как у них всех скрытая строка (call.data) начинается с "yes:"
@bot.callback_query_handler(func=lambda call: call.data.startswith("yes:"))
def callback_yes(call):
    user_id = int(call.data[len("yes:"):])  # убираем yes: из начала, оставляя только user_id
    ...  # тут делаешь реакцию на "да"

# а этот обработчик будет обрабатывать ТОЛЬКО кнопки "Нет",
# так как у них всех скрытая строка (call.data) начинается с "no:"
@bot.callback_query_handler(func=lambda call: call.data.startswith("no:"))
def callback_no(call):
    user_id = int(call.data[len("no:"):])  # убираем no: из начала, оставляя только user_id
    ...  # тут делаешь реакцию на "нет"

Это в общих чертах, код нужно будет доработать под себя.

[1parahon]

Vindicar, СПАСИБО БОЛЬШОЕ, У МЕНЯ ПОЛУЧИЛОСЬ!!!! УРА