Какие категории и как разделить файлы?

Question

[Артём Смирнов]

Какие папки создать и как разделить файлы?
Мне постоянно приходится в каждой команде делать одни и теже переменные
например вот:

@command(name="стата")
    async def profileuser(self, ctx):
        user = await self.bot.get_user(ctx.from_id)
        sms = await basa.register.main.chekc_sms(user)
        adm = await basa.register.main.chekc_adm(user)
        await ctx.send(f"сообщения всего: {sms}\n"
                       f"Роль: {adm}\n")

эти переменные почти везде будут указываться

basa.register.main.chekc_adm - Это путь и название базы данных, что можно сделать?

Comments

[shurshur]

От chekc и basa глазам больно...

Делить надо не на "категории и файлы", а логику и функциональность. Например, вот интерфейс к базе, вот реестр пользователей (может использовать базу через первый модуль), а вот логика самого бота (её тоже можно делить на части: тут считаем статистику, там обрабатываем команды показа рандомного котика...).

[Артём Смирнов]

shurshur, Спасибо за ответ, я так понял всё таким можно сделать в отдельном классе все переменные?
И подскажите ещё пожалуйста, я щас делаю и настраиваю администрацию в sql какой тип должен быть adm?
Там не цифры, а статусы, например 1 = Участник, 2 = хелпер, 3 = админ

[shurshur]

Артём Смирнов, ну если там одно из трёх чисел, то просто числовое поле.

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

groups:
- id
- level - уровень группы, чем выше, тем больше прав
- name
- role_id - соответствующая роль дискорда, которую назначать пользователям

При этом если бот будет для разных серверов предназначен, то всё ещё усложняется тем, что роли у каждого сервера свои.

Но нужно ли делать так сложно?

я так понял всё таким можно сделать в отдельном классе все переменные?

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

Например, если мы в каком-то классе будем хранить db_name, db_user, db_password и обращаться из разных мест своего кода для получении информации для коннекта к базе, то это плохое решение. В итоге у нас работа с базой размазана по коду, во множестве мест делается коннект к ней, а если, например, мы захотим при коннекте к базе вызывать какой-то запрос (типа set names utf8;), то нам придётся добавлять это в куче мест.

Вместо этого мы можем сдеалать некий класс DB, который принимает параметры базы в конструкторе, а потом в self.connection записывает объект соединения, а запросы передавать в методе execute. Пример такого класса (там Oracle, да и сделано не очень хорошо, например, курсор надо для каждого запроса создавать новый, а не как сейчас сделано, но для понимания идеи сойдёт) https://pastebin.com/y2VLN8Tm

Далее, вместо того, чтобы информацию по пользователям из базы дёргать запросами в разных частях кода, можно сделать отдельный класс User, у которого конструктор умеет загружать данные из базы и записывать в поля класса. Соответственно, можно реализовать, чтобы метод save() этого класса записывал данные обратно в базу. Работа с данными пользователя станет намного нагляднее и удобнее.

Наконец, вместо того, чтобы самому строить красиво организованную работу с базой, можно просто взять готовую реализацию ORM (object-relational model), типа sqlalchemy или peewee, где уже многое сделано заранее, можно просто описать структуру своих объектов и всё остальное доверить ORM. Будет примерно как я описал выше, но намного меньше кода.

Для того и делят логику, чтобы информация о пользователях была не куча переменных из разных кусков кода, а вот такой один объект, в котором есть все данные и реализована какая-то логика. Кстати, о логике: у классов, как правило, должны быть какие-то методы, это не просто мешочек с данными, это некая логическая единица с какой-то функциональностью.

Можно посмотреть для сравнения, как всё устроено в самом discord.py: вот пользователь (User), вот сервер (Guild), вот член гильдии (Member), он же одновременно и пользователь, но не совсем, так как пользователь на каждом сервере имеет свой объект Member. Возможности и назначение каждого класса свои. Например, если нам надо послать пользователю личное сообщение, то его надо слать User, а если упомянуть его в чате конкретного сервера - то Member. И у всех этих объектов есть не только поля (переменные), но и методы (функции).

[Артём Смирнов]

shurshur, У меня стоит VARCHAT, стандартная роль Участник, вот думаю, правильно или нет

[Артём Смирнов]

shurshur, И у меня бот для вк

[shurshur]

Артём Смирнов, не varchat, а varchar. Не думаю, что там будут миллионы пользователей и тысячи запросов в секунду, так что конкретный тип не столь важен.

Ну, ctx.send обычно в дискорд-ботах, нетрудно ошибиться.

[Артём Смирнов]

shurshur, Подскажи пожалуйста, как сделать что бы место цифр были нормальные роли как админ хелпер и т.д, я поставил тип БИГИНТ, и не могу установить роли, только int

[shurshur]

Артём Смирнов, если хочется роли словами писать, тогда надо varchar(#), где # - максимальная длина. Ну и вообще почитать о том, какие типы бывают в используемой базе данных (у некоторых бывают разные типы: char, varchar, varchar2, nvarchar итд, они могут оказаться где-то одинаковыми, а где-то нет, например, могут быть строго однобайтными или поддерживать юникоды с эмодзями).

int - это целоче число (скорее всего, 4 байта, то есть от -2^31 до 2^31-1), bigint - "большое" целое (скорее всего, 8 байт), но каждая конкретная база может иметь свою систему типов, могут быть типы с разным именем и одинаковой имплементацией или типы с одним именем в разных базах иметь разные особенности...

Например:
https://dev.mysql.com/doc/refman/8.0/en/integer-ty...
https://www.postgresql.org/docs/current/datatype-n...
https://www.sqlite.org/datatype3.html

[Артём Смирнов]

shurshur,
Я не до конца понял, если сделать так, то я не смогу выдать роль т.к "Участник" ошибку выдаст

[shurshur]

Какую ошибку? Где именно?

[Артём Смирнов]

shurshur,В коде в самом, name 'Участник' is not defined, если писать только цифры, но я не имею представления как сделать что бы место цифр были роли

Answer 1

[Alexndrev]

создай классы и обращайся к каждому

Comments

[Артём Смирнов]

Внутри классах писать это?
adm = await basa.register.main.chekc_adm(user)
или как то сократить можно?

[Alexndrev]

Артём Смирнов, тут ты уже ничего не сократишь, это статик выражение

[Артём Смирнов]

Alexndrev, Хорошо, спасибо, а какой тип должен быть у роли админ?
Что бы были роли как участник и т.д

[Alexndrev]

Артём Смирнов, str или text

[Артём Смирнов]

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

@command(name="+адм")
    async def prov(self, ctx, member=None):
        user = await self.bot.get_user(ctx.from_id)
        adm = await basa.register.main.chekc_adm(user)
        uca = ("Участник")
        admin = ("Разраб")
        if member is None:
            await ctx.send("Перешлите или укажите айди пользователя")
        elif adm != admin:
            await ctx.send(format(user.first_name) + ", выдал роль администратора пользователю" + member)
        else:
            await ctx.send("Не достаточно прав")

Это последнее о чём прошу, я только учу sql, иза этого прошу прощения, я не до конца понимаю что в документации написано, иза этого провожу тесты и прошу помощи

[Alexndrev]

Артём Смирнов, ну банально придай каждой значение
юзер 0, админ 1, создатель 2
и уже проверку
если роль == админ or роль >= "1"

[Артём Смирнов]

Alexndrev, elif adm == admin >= "5":
await ctx.send(format(user.first_name) + ", выдал роль администратора пользователю" + member)

Так как то?

[Артём Смирнов]

Alexndrev, Я посторался сделать отдельный класс под перемнные

class LeoUser:
    def __init__(self, bot, *args):
        self.bot = bot
        self._last_user = None
        self.user = args[0]
        self.sms = args[1]
        self.adm = args[2]
        self.name = args[3]
        self.clan = args[4]

выдаёт ошибку что
self.user = args[0]
IndexError: tuple index out of range