Что не так в моем коде?

Question

[Kapuzev]

import telebot
import constants
from telebot import apihelper
proxy = constants.ip+":"+constants.port
apihelper.proxy = {'https': proxy}
bot = telebot.TeleBot(constants.token1)



@bot.message_handler(commands=['admin'])
def hendle_admin(message):
    user_markup = telebot.types.ReplyKeyboardMarkup(True, True)
    user_markup.row('Отмена')
    msg = bot.send_message(message.chat.id, "Введите пароль", reply_markup = user_markup)
    bot.register_next_step_handler(msg, handle_admin_password)


def handle_admin_password(message):

    if message.text == constants.password:

        bot.send_message(message.chat.id, "Пароль верен")

        user_markup = telebot.types.ReplyKeyboardMarkup(True, True)
        user_markup.row('Алгебра', 'Геометрия', 'Русский', 'Литература')
        user_markup.row('Химия', 'Физика', 'Немецкий', 'Биология')
        user_markup.row('История', 'Обществознание', 'География', 'Информатика')
        user_markup.row('АнглийскийСА', 'АнглийскийТА', 'Физкультура', 'Обж')
        user_markup.row('/start')
        msg = bot.send_message(message.chat.id, "Выберите предмет", reply_markup=user_markup)

        bot.register_next_step_handler(msg, handle_admin_dz)


    elif message.text == 'Отмена':
        user_markup = telebot.types.ReplyKeyboardMarkup(True, True)
        user_markup.row('/admin')
        user_markup.row('/start')
        bot.send_message(message.chat.id, "Вы уверенны?", reply_markup = user_markup)


def handle_admin_dz(message):
    if message.text == "Алгебра":
        Algebra = bot.send_message(message.chat.id, "Запишите дз")

    elif message.text == "Геометрия":
        Geom = bot.send_message(message.chat.id, "Запишите дз")
    elif message.text == "Русский":
        Rus = bot.send_message(message.chat.id, "Запишите дз")
    elif message.text == "Литература":
        Lit = bot.send_message(message.chat.id, "Запишите дз")
    elif message.text == "Химия":
        bot.send_message(message.chat.id, "Запишите дз")
    elif message.text == "Физика":
        bot.send_message(message.chat.id, "Запишите дз")
    elif message.text == "Немецкий":
        bot.send_message(message.chat.id, "Запишите дз")
    elif message.text == "История":
        bot.send_message(message.chat.id, "Запишите дз")
    elif message.text == "Обществознание":
        bot.send_message(message.chat.id, "Запишите дз")
    elif message.text == "География":
        bot.send_message(message.chat.id, "Запишите дз")
    elif message.text == "Информатика":
        bot.send_message(message.chat.id, "Запишите дз")
    elif message.text == "АнглийскийСА":
        bot.send_message(message.chat.id, "Запишите дз")
    elif message.text == "АнглийскийТА":
        bot.send_message(message.chat.id, "Запишите дз")
    elif message.text == "Физкультура":
        bot.send_message(message.chat.id, "Запишите дз")
    elif message.text == "Обж":
        bot.send_message(message.chat.id, "Запишите дз")



@bot.message_handler(commands=['help', 'start'])
def hendle_start(message):

    user_markup = telebot.types.ReplyKeyboardMarkup(True, True)
    user_markup.row('Понедельник', 'Вторник', 'Среда')
    user_markup.row('Четверг', 'Пятница', 'Суббота')
    user_markup.row('/admin')
    bot.send_message(message.chat.id, "Выберите день недели", reply_markup = user_markup) #включен

    @bot.message_handler(content_types='text')
    def hendle_text(message):
        if message.text == "Понедельник":
            den = 'ПОНЕДЕЛЬНИК'
            '#Алгебра:' + handle_admin_dz.Algebra
#Физика:
#Обществознание:
#История:
#Информатика:'
            bot.send_message(message.chat.id, den)
        elif message.text == "Вторник":
            den = 'ВТОРНИК' \

            bot.send_message(message.chat.id, den)
        elif message.text == "Среда":
            den = 'СРЕДА'
            bot.send_message(message.chat.id, den)
        elif message.text == "Четверг":
            den = 'ЧЕТВЕРГ'
            bot.send_message(message.chat.id, den)
        elif message.text == "Пятница":
            den = 'ПЯТНИЦА'
            bot.send_message(message.chat.id, den)
        elif message.text == "Суббота":
            den = 'СУББОТА'
            bot.send_message(message.chat.id, den)




bot.polling(none_stop=True, interval=0)

Comments

[Kapuzev]

Код не дописан, но суть видна

[alex1478]

Ошибка то какая?

[Kapuzev]

alex1478, File "C:/Users/progr/Desktop/ВСтоле/музло, проекты/Py/main.py", line 90, in hendle_text
'#Алгебра:' + handle_admin_dz.Algebra
AttributeError: 'function' object has no attribute 'Algebra'

[alex1478]

Kapuzev, На 90 строке вы не правильно работате с функцией handle_admin_dz, пытаясь получить атрибут Algebra, которого у функции нет. Мб вам надо вызывать handle_admin_dz(Algebra), я не знаю как там у вас всё. Киньте текст функции hendle_text, только форматирование используйте.

[Daniil Markelov]

handle_admin_dz.Algebra

вы передаете переменную Algebra, как аттрибут метода handle_admin_dz. Так делать нельзя.

[Kapuzev]

Зёма Подушкин, если писать просто Algebra все равно не работает

[Влад Григорьев]

Kapuzev, что бы кодить, надо сначала понимать основы.
Вы в локальной области функции handle_admin_dz в определенном случае создаете переменную Algebra ни куда ее не передаете, и никуда не сохраняете
Она создаться только при вызове handle_admin_dz(), и только внутри этой функции.

ПС постарался максимально просто объяснить

[Олег]

никогда не писал на питоне, но тут
den = 'ПОНЕДЕЛЬНИК'
'#Алгебра:' + handle_admin_dz.Algebra
явно пропущена операция соединения строковых литералов.
без нее на второй строчке получается попытка присвоить константному строковому литералу нового значения.

[Kapuzev]

Олег, вот и у меня еще вопрос как записать Algebra строкой и, ну типа всунуть в строку))

простите меня ламера, первый бот)

[Олег]

den = 'ПОНЕДЕЛЬНИК' + '#Алгебра:' + handle_admin_dz.Algebra

[Kapuzev]

Олег, тут такая же ошибка будет

[Олег]

Вы питон учили ?
Я нет. Но простая логика
def hendle_admin(message):
типа это функция. Во всех языках аргументы передаются в скобках,
а через точку пишут обращения к пропертям(свойствам).

[o5a]

Kapuzev,
Насколько я понял, handle_admin_dz используется для записи ДЗ админом. Т.е. результатом работы этой функции должно стать занесение данных в какую-то внутреннюю структуру. Обычно делается через базу данных, самое простое - sqlite, уже встроенная в python. Другие варианты - хранение в текстовых файлах или использование модуля shelve.

Для получения информации о ДЗ пользователем же должна использоваться другая функция, которая по выбранным пользователем данным (день/предмет) сделает запрос к нашей базе и выдаст в чат результат в читаемом виде.

[Kapuzev]

o5a, ооо, спасибо огромное. но я читаю на сайтах как пользоваться и не понимаю как записать сообщение от бота в таблицу. по правде говоря я понял только как вручную записать значения в таблицу.

не могли бы вы показать в ответе как добавить значения в таблицу из текста сообщения пользователя?)

заранее спасибо!

[o5a]

Kapuzev, лучше прочитать хотя бы основы SQL и работы с sqlite, потому что с нуля это просто не опишешь. В sqlite создается соединение с базой данных (файлом .db), а для исполнения команд SQL создается курсор, в который передается команда с необходимыми данными и в зависимости от вида команды можем получить на выходе результат (если это SELECT), с которым потом работаем как с обычным массивом.

Могу дать некоторые простые примеры курсоров, как это может выглядеть в данном случае. Это не готовый скрипт, просто наброски для примера.

# создается соединение
conn = sqlite3.connect('homework.db')
# объект-курсор
c = conn.cursor()
# одноразово создаем таблицу для хранения домашних заданий в разрезе дня недели и предмета
c.execute('''CREATE TABLE IF NOT EXISTS homework (weekday number, subject text, body text)''')
# затем нужно будет заполнить таблицу необходимыми данными

# по "ключевым" значениям "день недели" + "предмет" меняем текст домашней работы в нашей таблице
c.execute('''UPDATE homework SET body=? WHERE weekday=? AND subject=?''', (body, weekday, subject))
# изменения данных в базе должны "сохраняться" через commit
conn.commit()

# примерно так может выглядеть функция, запрашивающая данные по "день"+"предмет"
def get_homework(conn, weekday, subject):
    c = conn.cursor()
    # получаем данные (текст задания) по нашему ключу из двух полей (день+предмет)
    c.execute('''SELECT body FROM homework WHERE weekday=? AND subject=?''', (weekday, subject))
    return c.fetchone()[0]

Answer 1

[antonksa]

den = 'ПОНЕДЕЛЬНИК'
    '#Алгебра:' + handle_admin_dz.Algebra

ёпта, это в мемориз.
Особенно учитывая

Algebra = bot.send_message(message.chat.id, "Запишите дз")

Даже если бы можно было выковырять переменную из функции, нарушив принципы инкапсуляции, то что вы пытаетесь сделать? Сложить строку с чем? С функцией? Результатом исполнения функции?

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

Напишите функцию получения текста сообщения. Вызовите ее. Сохраните результат в переменную. Используйте ее для отправки сообщения.

def handle_admin_dz(message):
    if message.text in {
        "Алгебра", "Геометрия", "Русский", ...
    }:
        response_text = "Запишите дз"
    else:
        response_text = "Неизвестный раздел!"
    return response_text

@bot.message_handler(content_types='text')
def handle_text(message):
    if message.text == "Понедельник":
        response_text = handle_admin_dz(message)
        den = 'ПОНЕДЕЛЬНИК\n#Алгебра: ' + response_text

Мне кстати непонятно, как у вас может быть одновременно message.text == "Понедельник" и == "Алгебра"

Comments

[Kapuzev]

блин, спасибо огромное. постараюсь ответить развёрнуто.

задача у этого бота такая:

*если ты админ
то вводишь пароль, записываешь дз.
(то есть отвечаешь на "запишите дз", а бот записывает эту переменную)

*если ты просто пользователь
то этот бот кидает тебе дз на выбранный день списком
(то есть он берет заданную строку и после каждого тега урока рвет эту строку и вкисывает туда переменную)

вообще в перспективе он должен еще и отправлять составленое дз в выбранный канал

сейчас мы делаем это вручную. отсылаем список предметов и в течении дня записываем дз редактируя уже отправленное сообщение

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

и вообще хотелось бы сказать зачем мы это делаем. в нашей школе отменили бумажные журналы (идиотское решения) и ввели "ультрапродвинутый" сайт-журнал. деръмо еще то, нам не удобно, учителям тоже. каждый учитель плюётся что не удобно совсем. но делать нечего.
некоторые учителя и вовсе не говорят дз на уроке!!! а не у каждого есть возможность мониторить этот убогий сайт. и мы решили сделать канал в телеге (если интересно @dz_9a)

а теперь я придумал слелать бота..
и вот сижу за этим уже больше месяца.

еще в будующем было бы хорошо сделать возможность пользаваться этим и другим классам. просто давая боту ссылку на нужный канал.

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

*писал с телефона. извиняюсь за возможные опечатки, если не сложно можем списаться в телеге (@leerov), просто помощь очень нужна сами не справляемся :с. и спасибо за ответ)))