Телеграм бот на питоне: ошибка переменная is not defined?

Question

[spider2007]

Недавно начал разбираться с ботами на пайтоне. Нашел пример как парсить погоду. Добавил код к себе, но замучила ошибка.
Мой код:

import telebot
import requests
from bs4 import BeautifulSoup as BS

r = requests.get('https://sinoptik.ua/погода-харьков')
html = BS(r.content, 'html.parser')

bot = telebot.TeleBot('1113481836:AAGB_KMA0glwKctU2sKiH6bLS1zTyYYavlo')

keyboard1 = telebot.types.ReplyKeyboardMarkup()
keyboard1.row(' Rozklad', '⚙️ Кнопка4','Погода')
keyboard1.row(' Наш сайт', ' Зворотній зв`язок')

for el in html.select('#content'):
    t_min = el.select('.temperature .min')[0].text
    t_max = el.select('.temperature .max')[0].text
    text = el.select('.wDescription .description')[0].text


@bot.message_handler(commands=['start'])
def start_message(message):
    bot.send_message(message.chat.id,  'Привіт, '+ message.from_user.username)



@bot.message_handler(content_types=['text'])

def send_text(message):
   # global t_min
    if message.text == ' Rozklad':
            bot.send_message(message.chat.id,  'Обери курс:', reply_markup=kurs)

   # if message.text == 'Погода':
   #         global tmin
   #         bot.send_message(message.chat.id, tmin, reply_markup=kurs)
    if message.text == 'Погода':
            bot.send_message(message.chat.id, "Привет, погода на сегодня \n" +
                     t_min + ', ' + t_max + '\n' + text)

 
if __name__ == '__main__':
    bot.polling(none_stop=True)

После запуска и нажатия на погоду выбивает ошибки:
2020-06-21 08:07:50,838 (util.py:68 WorkerThread2) ERROR - TeleBot: "NameError occurred, args=("name 't_min' is not defined",)
Traceback (most recent call last):
File "C:\Python37\lib\site-packages\telebot\util.py", line 62, in run
task(*args, **kwargs)
File "C:/Users/spider2007/PycharmProjects/untitled/sample/ldgik.py", line 48, in send_text
t_min + ', ' + t_max + '\n' + text)
NameError: name 't_min' is not defined
"
Traceback (most recent call last):
File "C:/Users/spider2007/PycharmProjects/untitled/sample/ldgik.py", line 69, in
bot.polling(none_stop=True)
File "C:\Python37\lib\site-packages\telebot\__init__.py", line 415, in polling
self.__threaded_polling(none_stop, interval, timeout)
File "C:\Python37\lib\site-packages\telebot\__init__.py", line 439, in __threaded_polling
self.worker_pool.raise_exceptions()
File "C:\Python37\lib\site-packages\telebot\util.py", line 111, in raise_exceptions
six.reraise(self.exc_info[0], self.exc_info[1], self.exc_info[2])
File "C:\Python37\lib\site-packages\six.py", line 703, in reraise
raise value
File "C:\Python37\lib\site-packages\telebot\util.py", line 62, in run
task(*args, **kwargs)
File "C:/Users/spider2007/PycharmProjects/untitled/sample/ldgik.py", line 48, in send_text
t_min + ', ' + t_max + '\n' + text)
NameError: name 't_min' is not defined

Process finished with exit code 1

Answer 1

[Сергей Горностаев]

Разберитесь сначала с основами языка. Учебник какой-нибудь прочитайте.

Answer 2

[Diatomiccoder]

Ошибка говорит, что нет переменной t_min. Она сдается только внутри цикла. Т.е. скорее всего html.select(...) возвращает пустой список (или свой пустой итерируемый объект).
Можно попробовать объявить переменные t_min, t_max, text (например t_min = "") до цикла, и ошибка должна будет исчезнуть

Comments

[spider2007]

Сделал так
t_min=''
t_max=''
text=''

for el in html.select('#content'):
t_min = el.select('.temperature .min')[0].text
t_max = el.select('.temperature .max')[0].text
text = el.select('.wDescription .description')[0].text

Получил
2020-06-21 08:36:34,864 (util.py:68 WorkerThread2) ERROR - TeleBot: "UnboundLocalError occurred, args=("local variable 'text' referenced before assignment",)
Traceback (most recent call last):
File "C:\Python37\lib\site-packages\telebot\util.py", line 62, in run
task(*args, **kwargs)
File "C:/Users/spider2007/PycharmProjects/untitled/sample/ldgik.py", line 52, in send_text
t_min + ', ' + t_max + '\n' + text)
UnboundLocalError: local variable 'text' referenced before assignment
"

[spider2007]

Убрал переменную текст и ошибки пропали, но погоду не выдает. Переменные пустые.

[spider2007]

Diatomiccoder переменную текст назвал по другому - работает без ошибок, но переменные пустые. Я прочитал, что это может быть связано с видимостью переменных и нужно использовать глобал (перем). Пробовал вставлять, но не помогло. Может подскажите?

[spider2007]

Dr. Bacon, да, это первый бот, разбираюсь. Иногда лучше промолчать...

[milssky]

spider2007, даже если и первый, то основы языка, на котором он написан, вам придется выучить.

[alexusbon]

spider2007, ты уверен что парсишь то что надо? и даже если парсишь что надо, то сначало необходимо сделать ссылку переменной которая not defind на объект из парсинга, а уж потом производить математические операции

Answer 3

[Никита Верхогляд]

Изучи для начала страницу, которую парсишь.

Элементы которые ты ищешь, обладают лишь одним классом, а ты ищешь их по двум классам.
А элемента с классом "wDescription description" вообще не существует.

Тебе нужно сначала получить элемент класса "temperature", а уже из него вытаскивать элементы с температурой.