Ошибка TypeError: marks_message() missing 1 required positional argument: 'marks' почему?

Question

[chtopisatsuda]

Есть код в котором проверяется если сообщение == оценки
то идет вывод оценок в сообщениях, если я напишу оценки в чате один раз, то бот выводит оценки, но если напишу еще раз, выводит такую ошибку:

TypeError: marks_message() missing 1 required positional argument: 'marks'

marks_message():

@bot.message_handler(content_types=['text'])
def marks_message(message, marks):
    if message.text.lower() == 'оценки':
        for mr in marks:
            bot.send_message(message.chat.id, f"{' ' + mr[1] + ':'}\n{' ' + mr[2].translate({ord ('['): None})}")

reg():

def reg(message, loginDb, passwordDb):
    sqlite_select_query = """SELECT * from dn"""
    cursor.execute(sqlite_select_query)
    cursor.close
    records = cursor.fetchall()
    print("Всего строк:  ", len(records))
    print("Вывод каждой строки")
    for row in records:
        loginDb = row[0]
        passwordDb = row[1]
        print("Логин:", row[0])
        print("Пароль:", row[1])
    d = Dnevnik(login=loginDb, password=passwordDb)
    marks = d.marks()
    bot.send_message(message.chat.id, ' Вы успешно вошли в аккаунт ' + loginDb)
    markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
    btn1 = types.KeyboardButton(" Домашняя работа")
    btn2 = types.KeyboardButton("оценки")
    markup.add(btn1, btn2)
    bot.send_message(message.chat.id, 'Выберите в меню действие... ', reply_markup=markup)
    marks_message(message, marks)

Answer 1

[Vindicar]

@bot.message_handler(content_types=['text'])
def marks_message(message, marks):

marks_message(message, marks)
Плохая идея. Ты уже решай - или функция является обработчиком события от бота, и тогда её параметры определяются библиотекой, или функция вызывается тобой.

Comments

[chtopisatsuda]

а если я не вызову ее то как я передам в нее параметры из другой функции

[Vindicar]

chtopisatsuda, значит, нечего пытаться передавать в неё параметры из другой функции.
Если тебе нужно один и тот же код выполнять по разным поводам - вынеси его в отдельную функцию с параметрами, и уже её вызывай из нужных мест, давая ей нужные параметры.
Вот как, по твоему, бот догадается, что такое marks и откуда его взять?

[chtopisatsuda]

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

[chtopisatsuda]

у меня marks это список с оценками как мне выносить в другую функцию если у меня marks это d, а d можно получить только после регистрации пользователя

from ast import Str, While
from cgitb import text
from datetime import date
from multiprocessing import AuthenticationError
from turtle import home

from dnevnikru import Dnevnik
from dnevnikru import exceptions as DnevnikError
from telebot import types


from pprint import pprint

import json
import sqlite3
import telebot

import threading

#sqlite
try:
    sqlite_connection = sqlite3.connect('sqlite_python.db', check_same_thread=False)
    cursor = sqlite_connection.cursor()
    print("База данных создана и успешно подключена к SQLite")

    sqlite_select_query = "select sqlite_version();"
    cursor.execute(sqlite_select_query)
    sqlite_connection.commit()
    record = cursor.fetchall()
    print("Версия базы данных SQLite: ", record)

except sqlite3.Error as error:
    print("Ошибка при подключении к sqlite", error)
finally:
    if (sqlite_connection):
        sqlite_connection.commit()
        print("Соединение с SQLite закрыто")


bot = telebot.TeleBot("5799037534:AAErhJf782SaBMmQgFdllYpPxEXVZC_XL7s")
bot.delete_webhook()


@bot.message_handler(commands=['start', 'go'])
def start_handler(message):
    bot.send_message(message.chat.id, "Привет , чтобы войти в дневник ру введи данные в виде логина и пароля (пример: логин123 пароль123) ❗️")
    bot.register_next_step_handler(message, auth)



def auth(message):
    loginDb, passwordDb = message.text.split()
    cursor.execute('INSERT INTO dn (login, password) VALUES (?, ?)', (loginDb, passwordDb))
    sqlite_connection.commit()

    print(loginDb, passwordDb)
    reg(message, loginDb, passwordDb)
    bot.register_next_step_handler(message, reg, loginDb, passwordDb)

def reg(message, loginDb, passwordDb):
    sqlite_select_query = """SELECT * from dn"""
    cursor.execute(sqlite_select_query)
    cursor.close
    records = cursor.fetchall()
    print("Всего строк:  ", len(records))
    print("Вывод каждой строки")
    for row in records:
        loginDb = row[0]
        passwordDb = row[1]
        print("Логин:", row[0])
        print("Пароль:", row[1])
    d = Dnevnik(login=loginDb, password=passwordDb)
    marks = d.marks()
    bot.send_message(message.chat.id, ' Вы успешно вошли в аккаунт ' + loginDb)
    markup = types.ReplyKeyboardMarkup(resize_keyboard=True)
    btn1 = types.KeyboardButton(" Домашняя работа")
    btn2 = types.KeyboardButton("оценки")
    markup.add(btn1, btn2)
    bot.send_message(message.chat.id, 'Выберите в меню действие... ', reply_markup=markup)
    marks_message(message, marks)

@bot.message_handler(content_types=['text'])
def marks_message(message, marks):
    if message.text.lower() == 'оценки':
        for mr in marks:
            bot.send_message(message.chat.id, f"{' ' + mr[1] + ':'}\n{' ' + mr[2].translate({ord ('['): None})}")



bot.polling(none_stop=True)

[Vindicar]

chtopisatsuda, а неужели не ясно, что то, что происходит в reg() - остаётся в reg() и пропадает по его завершении, если явно не сохранено?
Очевидно, нужно как-то хранить сведения о зарегистрированном пользователе.
Для начала - для каждого пользователя хранить логин и пароль, чтобы по дальнейшим запросам сконструировать экземпляр Dnevnik и вызвать метод получения актуальных оценок.
Следующий шаг - кэшировать экземпляр Dnevnik, создавая его только если для этого пользователя объекта нет в кэше. Тогда если пользователь делает несколько запросов подряд, мы не будем каждый раз конструировать Dnevnik (а я подозреваю ты там логинишься на сайт).
Тут возникают свои сложности - нужно выкидывать неимпользуемые экземпляры Dnevnik, нужно смотреть, не отвалилась ли сессия на сайте, и перелогиниваться (ну это сам Dnevnik пускай делает).

[chtopisatsuda]

Vindicar, данные в бд хранятся

[chtopisatsuda]

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

Answer 2

[OCCASS OCCASSOVICH]

Потому что хэндлер принимает только сообщения
Откуда берутся marks? И что это вообще такое?

Comments

[chtopisatsuda]

переменная marks для вывода оценок почитай код

[OCCASS OCCASSOVICH]

Тебе нужно убрать декоратор, потому что ты вызываешь функцию сам, а не хочешь чтобы ее обрабатывал диспатчер

[OCCASS OCCASSOVICH]

chtopisatsuda, сработало?