Не понимаю суть ошибки, можете объяснить?

Question

[Timka4959]

имеется код:

в main.py:

from aiogram import Bot, types
from aiogram.dispatcher import Dispatcher
from aiogram.utils import executor

from config import TOKEN
from sqlite import db_start, create_profile, edit_profile

async def on_startup(_):
  await db_start()


bot = Bot(token=TOKEN)
dp = Dispatcher(bot)
balance = 0
limit = 15000000000000
use = 0


@dp.message_handler(commands=['start'])
async def process_start_command(message: types.Message):
    await message.reply("Привет!\nЯ бот-кликер валюты в боте BFG!\n\nЧтобы получить помощь напишите /help")
    await create_profile(user_id=message.from_user.id) 
    await edit_profile(user_id=message.from_user.id)

@dp.message_handler(commands=['help'])
async def help_command(message: types.Message):
    await message.reply("/click - кликать игровую валюту (1 клик - 500 ИГ)\n/balance - ваш баланс\n/sell - вывод\n/stat - статистика за этот день\n\nПРИМЕЧАНИЕ - баланс отчищается каждый день")

@dp.message_handler(commands=['stat'])
async def limit_command(message: types.Message):
    await message.reply(f"Владелец может вывести ещё - {limit}\nвыведено за сегодня: - {use}")

@dp.message_handler(commands=['balance'])
async def player_balance(message: types.Message):
    await message.reply(f"У вас на балансе {balance} тысяч")

@dp.message_handler(commands=['sell'])
async def sell_command(message: types.Message):
    await message.reply("Для вывода напишите ? в лс и отправь ему сообщение с балансом")

@dp.message_handler(commands=['click'])
async def plus_balance(message: types.Message):
    global balance
    await message.reply("+500 к балансу")
    balance += 0.5

@dp.message_handler(commands=['Minus_limit'])
async def game(message: types.Message):
    global limit
    global use
    args = message.get_args()
    limit -= int(args)
    use -= int(args)

                        
if __name__ == '__main__':
    executor.start_polling(dp, on_startup=on_startup)

sqlite.py:

import sqlite3 as sq
from main import balance

async def db_start():
  global db, cur
  db = sq.connect('new.db')
  cur = db.cursor()

  cur.execute("CREATE TABLE IF NOT EXISTS profile(user_id TEXT PRIMARY KEY, balance TEXT)")
  db.commit()

async def create_profile(user_id):
  user = cur.execute("SELECT 1 FROM profile WHERE user_id == '{key}'".format(key=user_id)).fetchone()
  if not user:
    cur.execute("INSERT INTO profile VALUES(?, ?",(user_id, balance))
    db.commit()

async def edit_profile(state, user_id):
  async with state.proxy() as data:
    cur.execute("UPDATE profile WHERE user_id == '{}' SET balance == '{}'".format(user_id, data['balance']))
    db.commit()

*есть ещё в config.py, но там токен

выскакивает ошибка:

Traceback (most recent call last):
  File "main.py", line 6, in <module>
    from sqlite import db_start, create_profile, edit_profile
  File "/home/runner/CrispWrongDigits/sqlite.py", line 2, in <module>
    from main import balance
  File "/home/runner/CrispWrongDigits/main.py", line 6, in <module>
    from sqlite import db_start, create_profile, edit_profile
ImportError: cannot import name 'db_start' from partially initialized module 'sqlite' (most likely due to a circular import) (/home/runner/CrispWrongDigits/sqlite.py)

помогите пожалуйста, как исправить?

Comments

[Модератор]

Не надо ставить как можно больше тэгов. Лучше оставить один, но конкретный, с которым проблема.
См.п.3.1 Регламента.

Answer 1

[Vindicar]

from sqlite import db_start, create_profile, edit_profile
from main import balance

main импортирует sqlite, который импортирует main, который импортирует sqlite, который импортирует main, который импортирует sqlite, который импортирует main, который импортирует sqlite, который импортирует main, который импортирует sqlite, который импортирует main, который импортирует sqlite, который импортирует main, который импортирует sqlite, который импортирует main, который импортирует sqlite, который импортирует main, который импортирует sqlite, который импортирует main, который импортирует sqlite, который импортирует main, который ...
Ну ты понял.

Быстрого волшебного решения тут нет. Разбирайся с областями видимости, избавляйся от глобальных переменных.
Например, почему у тебя при создании нового профиля используется balance? Т.е. новый пользователь получает столько денег, сколько было у последнего активного?
Почему вообще у тебя вещи, специфичные для одного пользователя, лежат то в глобальных переменных, то в данных машины состояний (state)? Бери их из БД , когда они потребуются.

Comments

[Timka4959]

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

[Timka4959]

спасибо! Попробовал ваше решение, заработало

[Timka4959]

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

[Vindicar]

Timka4959, а тут вопрос не в специфике БД.
Тут вопрос в понимании, что такое область видимости, умении разделить данные, относящиеся к отдельным сущностям, и в целом понимании того, как работает скрипт на питоне.

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

А раз так, значит, надо иметь какое-то хранилище, где по ключу (ID пользователя) можно будет вытащить его данные. Для учебного бота это мог бы быть простой словарь, но БД тоже прекрасно подходит, и имеет преимущество в персистентности (данные не теряются при перезапуске скрипта).

[Vindicar]

Timka4959, ВООБЩЕ ИЗБАВЬСЯ от глобальной переменной balance. Это хороший способ выстрелить себе во все конечности сразу.

[Timka4959]

Vindicar, уже. Я полностью переписал код и теперь он отлично работает)

Answer 2

[Александр Нестеров]

Циклические импорты в python запрещены.