Как сделать перевод строки в телеграм боте на python?

Question

[Layerqa]

Имею код

Код

@bot.message_handler(commands=['stats'])
def start_message(message):
	response = requests.get("ссылка на json файл") #json
	todos = json.loads(response.text)
	for player in todos['data']['data']:
		name = player['name']
		kills = player['kills']
		deaths = player['deaths']
		bot.send_message(message.chat.id, 
			"Name: " + name + "\n" +"Kills: " + kills + "\n" + "Deaths: "+ deaths)

При выполнении команды в боте, он присылает каждое значение в новом сообщении

Скриншот

Comments

[alternativshik]

и? Что не так?

[Layerqa]

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

[alternativshik]

Ну так и сделай одно сообщение и отправь его один раз.

[Layerqa]

alternativshik, как?

[Layerqa]

alternativshik, ору

[Артур]

Layerqa, соедините в цикле всё в одну строку и после цикла один раз вызовите send_message

[Layerqa]

Артур, спасибо, решил таким

способом

user_list = ''
string_users = f"{str(name)} Убил: {str(kills)} Умер: {str(deaths)}\n"
user_list += string_users
bot.send_message(message.chat.id, ("" +  user_list))

но почему то выводится 30 сообщений и в каждом сообщении новый игрок(user)

[shurshur]

Layerqa, ну если в каждой итерации цикла for player in... вызывать send_message, то сколько раз будет вызван send_message? Надо в цикле набирать сообщение, а потом вывести его одним вызовом send_message.

Например, так:

messages = []
for player in todos['data']['data']:
  name = player['name']
  kills = player['kills']
  deaths = player['deaths']
  messages.append("Name: " + name + "\n" +"Kills: " + kills + "\n" + "Deaths: "+ deaths)
bot.send_message(message.chat.id, "\n\n".join(messages))

[Layerqa]

shurshur, ваш код выводит ошибку TabError: inconsistent use of tabs and spaces in indentation

[Layerqa]

Хотя стойте он не выдаёт ошибку но все равно выводит нового игрока в старом сообщении

[shurshur]

Layerqa, я сдвинул весь код влево, потому что у первой строки хабр упорно убирал отступ.

Что касается переносов, у меня с ними никогда проблем не было. Правда, я telebot не использовал, только python-telegram-bot.

p = os.popen('%s %s' % (cmd, " ".join(req)), 'r')
out = p.read()
bot.send_message(chat_id=update.message.chat_id, text=out)

[Layerqa]

shurshur, нечего не понел, может как нибудь можно сохранять name, kills и deaths в отдельный txt файл и потом оттуда брать и отправлять?

[Layerqa]

shurshur, или есть способ легче?

[shurshur]

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

Попробую-ка я сейчас своего тестового бота на telebot переделать...

[Layerqa]

shurshur, ок, я предложил

[shurshur]

Layerqa, вот такой код с использованием telebot прекрасно выводит сообщение с переносами строк:

def listener(messages):
  for m in messages:
    if m.from_user.id not in allowed:
      bot.send_message(m.chat.id, 'Ты кто такой??? Давай, до свидания!')
      return
    if m.content_type == 'text':
      req = m.text
      req = req.split(' ')
      cmd = req[0][1:] # cut /
    try:
      cmd = commands[cmd]
    except KeyError:
      bot.send_message(m.chat.id, 'Ничё не понял...')
      return
    req = req[1:]
    p = os.popen('%s %s' % (cmd, " ".join(req)), 'r')
    out = p.read()
    bot.send_message(m.chat.id, out)

[Layerqa]

shurshur, нечего непонятно как это поможет мне вывести сообщение в одном блоке?

[shurshur]

Layerqa, мы не можем за тебя написать твою программу. Если ты в функции 30 раз вызовешь send_message - то неудивительно, что твой бот отправит 30 сообщений. Ты должен все сообщения объединить в одну строку и отправить одним вызовом send_message.

[shurshur]

Layerqa, кстати, судя по последнему скриншоту, ты видимо send_message включил в цикл, который добавляет в новый элемент messages и сразу же выводит накопленные messages, поэтому в 1 сообщении данные по одному игроку, далее по 2, по 3 итд.

[Layerqa]

shurshur, ой бл* туплю сорян теперь понел

[Layerqa]

shurshur, осознал свою ошибку прошу прощения, нужно было просто вытащить send message из цикла for player............

[shurshur]

Layerqa, ну, я тоже на скриншот сначала внимательно не посмотрел, а как глянул - так там же всё очевидно...

[Layerqa]

shurshur, спасибо большое, еще такой вопрос можно ли вытащить первые 10 name(Имён игроков)?

[shurshur]

Layerqa, можно, например, если массив игроков todos['data']['data'], то todos['data']['data'][0:10] - это массив первых 10 игроков.

[Layerqa]

shurshur, ок спасибо за помощь

Answer 1

[fx0_]

Если я правильно понял, то вам нужно отправить все одним сообщением. В таком случае можно воспользоваться методом .append
В цикле for мы записываем строки в players, после чего отправляем его единым сообщением.

players = []
def start_message(message):
  response = requests.get("ссылка на json файл") #json
  todos = json.loads(response.text)
  for player in todos['data']['data']:
    name = player['name']
    kills = player['kills']
    deaths = player['deaths']
   players.append("Name: " + name + "\n" +"Kills: " + kills + "\n" + "Deaths: "+ deaths)
bot.send_message(message.chat.id, players)

Comments

[Layerqa]

Спасибо, отменяю решением