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

Question

[Дмитрий]

Приветствую всех! Прошу помочь с кодом, можете сказать как подправить код, чтоб бот на команду "/weather Москва" отсылал погоду в Москве, а не писал, что "Город не найден"

import telebot 
import pyowm
bot = telebot.TeleBot("token")
owm = pyowm.OWM("token2", language='ru')

@bot.message_handler(commands=["weather"])
def send_weather(message):
    try:
        observation = owm.weather_at_place(message.text)  
        weather = observation.get_weather()
        temp = weather.get_temperature('celsius')['temp']

        answer = 'В городе ' + message.text + ' сейчас ' + weather.get_detailed_status() + '\n'
        answer += 'Температура в районе ' + str(round(temp)) + ' градусов' + '\n\n'

        if temp < 5:
            answer += 'Очень холодно, оденься потеплее ☺️'
        elif temp < 15:
            answer += 'Прохладно, лучше оденься '
        else:
            answer += 'Не холодно, можешь хоть в трусах идти '

        bot.send_message(message.chat.id, observation)
    except pyowm.exceptions.api_response_error.NotFoundError:
        bot.send_message(message.chat.id, 'Город не найден ')

bot.polling(none_stop=True)

Спасибо за помощь!! Прошу одаренных проходить мимо,если вы не можете дать нормальную формулировку ответа

Comments

[GavriKos]

КОД В ТЕГ CODE!!!
Тем более питоний! Как понять где у вас отступы??

Answer 1

[o5a]

1. message.text содержит полный текст сообщения, соответственно для запроса погоды вы его полностью и передаете, а owm не может найти погоду в городе с названием "/weather Москва". Можно просто порезать через split(maxsplit=1), чтобы не разрезать город с пробелами в названии.

2. commands=["/weather"] не надо указывать слэш при указании команды, иначе его придется дублировать

Comments

[Дмитрий]

Извиняюсь,что неточно написал команду,потом в коде подправил. Удалил слэш в квадратных скобках,но код все равно на команду /weather,пишет город не найден. Я понимаю,что надо писать вместе с городом команду,но он все равно не может найти погоду в городе. Вы можете куском кода отправить верный ответ?

[o5a]

Дмитрий, я же написал, проблема в том, что в качестве города передается вся строка message.text, а не только город. Изменить можно например так:

city = message.text.split(maxsplit=1)[1]
observation = owm.weather_at_place(city)

# и соответственно изменить в информационной строке
answer = 'В городе ' + city + ...

И кстати лучше использовать f-строки, а не конкатенацию строк, т.е. так:

answer = f'В городе {city} сейчас {weather.get_detailed_status()} \n'

[Дмитрий]

o5a, сейчас отредактирую код,посмотрю работать будет или нет

[Дмитрий]

o5a, да спасибо огромное,все работает. А можете ещё подсказать,как подправить,чтоб бот не падал,если пользователь пишет просто команду /weather?

[o5a]

Дмитрий, можно так (вместо строки city = )

_, *params = message.text.split(maxsplit=1)
# если параметр команды указан, то забираем, иначе устанавливаем город пустым
city = params[0] if params else ''

# аналогично можно просто прекратить выполнение команды, если параметр у команды не указан, что наверное логичнее
if params:
    city = params[0]
else:
    bot.send_message(message.chat.id, 'Укажите город')
    return

[Дмитрий]

o5a,

def send_weather(message):
    try:
        params = message.text.split(maxsplit= 1)[1]
        if params:
            city = params[0]
            observation = owm.weather_at_place(city)
            w = observation.get_weather()
            temp =w.get_temperature('celsius')['temp']

            answer = 'В городе ' + city + ' сейчас ' + w.get_detailed_status() + '\n'
            answer += 'Температура в районе ' + str(round(temp)) + ' градусов' + '\n\n'
            if temp < 5:
                answer += 'Очень холодно, оденься тепло ☺️'
            elif temp < 15:
                answer += 'Прохладно, лучше оденься потеплее '
            else:
                answer += 'Не холодно, можешь хоть в трусах идти '
            
            bot.send_message(message.chat.id, answer)
        else:
            bot.send_message(message.chat.id, help.ERR_MSG)
    except pyowm.exceptions.api_response_error.NotFoundError:
        bot.send_message(message.chat.id, 'Город не найден ')

Терминал выдает ошибку: IndexError: list index out of range. Написал все как ты.

[o5a]

Дмитрий, вместо строки params = должно быть новое
_, *params = message.text.split(maxsplit=1)

[Дмитрий]

o5a, в смысле новое?Не до конца понимаю

[Дмитрий]

o5a, IndexError: list index out of rang, терминал выдает
все равно выдает

[Дмитрий]

o5a, все понял, починил. Огромное спасибо!!!

Answer 2

[0ralo]

Код нечитабилен и не приложена ошибка, но судя по всему, вы передаёте функции поиска погоды не "Москва", а "/weather Москва".Поэтому она и не может найти такой город

Comments

[Дмитрий]

Код читабелен,если вы его скопируете и вставите в среду разработки. Суть проблемы,когда я отсылаю /weather код пишет,город не найден,хотя если вы по коду просмотрите,то увидите когда должно оно писаться.

[0ralo]

Код читабелен,если вы его скопируете и вставите в среду разработки.

это неправда.
В среде разработки будет ошибка синтаксиса так как отсутствуют отступы.Поищите разницу между этим

def Hello():
   print("Hello, World!")

и этим
def Hello():
print("Hello, World!")

[Дмитрий]

Так скажите спасибо хабру или я не знаю как оформить код, который вы указали выше. Весь код я скопировал с с vs Code

[0ralo]

[Дмитрий]

0ralo, специально для Вас отредактировал вопрос так,чтоб код был Вам понятен. Поэтому можете с лёгкостью размышлять над ответом

[0ralo]

Дмитрий, Спасибо огромное, как я говорил выше, и o5a Расписал для вас ниже, вы передаёте функции не название города, а полную строку сообщения.Если писать сыро то у вас
owm.weather_at_place("/weather Москва")
вместо
owm.weather_at_place("Москва")

Answer 3

[Storm95]

o5a Скиньте исходник пожалуйста полностью