Роли в боте вк sql?

Question

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

@command(name="+хелп")
    async def adm(self, ctx, member=None):
        user = await self.bot.get_user(ctx.from_id)
        adm = await basa.register.main.chekc_adm(user)
        if member is None:
            await ctx.send("Перешлите или укажите айди пользователя")
        elif adm == "Разраб":
            mem = re.findall(f"[0-9]+", member)[0]
            await ctx.send(format(user.first_name) + ", выдал роль администратора пользователю " + format(member))
            await basa.register.main.adm(int(mem))
        elif member is adm == "Хелпер":
            await ctx.send("У пользователя имеется данная роль")
        else:
            await ctx.send("Не достаточно прав")

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

Я пробовал сделать так
self.adm {
'1': list(map(int, "Участник"))
}

и так дойти до 7-ой роли, 7 роль это сам создатель и что ты место этого(elif adm == "Разраб") писалось следующее (elif adm >= 5)
5 роль это разраб, 7 - зам

Comments

[shurshur]

list(map(int, "Участник"))

Эм... И что тут имелось в виду? Очевидно, что это выдаст ошибку, потому что попытается применить int к буквам.

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

shurshur, str тоже не работает

[shurshur]

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

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

Вторая ошибка - в качестве класса использовать его текстовое название.

Третья ошибка - текстовые названия потом не получается как-то нормально сравнивать.

Правильно было бы завести в базе поле типа integer с названием вроде user_class (а может и user_level, или как-то ещё), которое бы принимало числовое значение. В коде завести эти классы и массив отображения их числовых значений в названия. Примерно так (я не призываю прям повторять, просто демонстрация идеи):

UC_GUEST = 0
UC_USER = 1
UC_FLOODER = 2
UC_CHUDO_UDO = 3
UC_DONAT = 4
UC_MODERATOR = 10
UC_ADMIN = 11
UC_GOD = 20

user_class_names = {
    UC_GUEST: "Гость",
    UC_USER: "Пользователь",
    UC_FLOODER: "Очень Активный",
    UC_CHUDO_UDO: "Монстр Активности",
    UC_DONAT: "Хорошо помог :)",
    UC_MODERATOR: "Модератор",
    UC_ADMIN: "Администратор",
    UC_GOD: "Самый главный",
}

Префиксы UC_ будут признаками того, что это user class. При этом, хотя в python нет констант, существует соглашение, что набранные КАПСОМ переменные представляют из себя неизменяемые значения. Я оставил дырки в последовательности классов, что позволит в случае чего добавлять некоторые классы без перенумерации всей базы. Ну, скажем, потребуется ввести суперадмина - вон там для него есть пространство.

При выполнении тех или иных действий можно проверять класс пользователя. Например, пользователь может выдать предупреждение, если его класс не ниже модератора:

if user_class >= UC_MODERATOR:

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

if user_class >= UC_MODERATOR:
    if user_class > get_user_class(member_id):
        # выдаём предупреждение
    else:
        # этот модератор не может выдать этому пользователю предупреждение

Можно ещё больше усложнять логику. Например, пусть пользователям уровня UC_DONAT предупреждения могут выдавать только админы, обычные модераторы не могут (такая вот привилегия).

Использовать числовые величины для понятия с простым отсортированным набором значений удобно тем, что их можно сразу же и сравнивать. А вот когда нам потребуется не просто класс пользователя, а его название, мы можем сделать user_class_names[user_level] и получить нужное имя.

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

shurshur,
vk_botting.exceptions.CommandInvokeError: Command raised an exception: TypeError: '>=' not supported between instances of 'dict' and 'int'

[shurshur]

Артём Смирнов, очевидно, что в сравнении используются dict и int. Например, вместо user.id передаётся какой-нить декодированный json.

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

shurshur, И что с этим сделать?

[shurshur]

Артём Смирнов, искать ошибку и исправлять её.

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

shurshur,

@command(name="+хелп")
    async def adm(self, ctx, member=None):
        user = await self.bot.get_user(ctx.from_id)
        adm = await basa.register.main.chekc_adm(user)

        UC_USER = 1
        UC_FLOODER = 2
        UC_CHUDO_UDO = 3
        UC_DONAT = 4
        UC_MODERATOR = 5
        UC_PROGRAMMER = 6
        UC_GOD = 7

        user_class = {
            UC_USER: "Участник",
            UC_FLOODER: "Хелперлео",
            UC_CHUDO_UDO: "Монстр Активности",
            UC_DONAT: "Хорошо помог :)",
            UC_MODERATOR: "Модератор",
            UC_PROGRAMMER: "Разраб",
            UC_GOD: "Зам",
        }

        print(user_class)

        if member is None:
            await ctx.send("Перешлите или укажите айди пользователя")
        if user_class >= UC_PROGRAMMER:
            if user_class > user.id:
                mem = re.findall(f"[0-9]+", member)[0]
                await ctx.send(format(user.first_name) + ", выдал роль администратора пользователю " + format(member))
                await basa.register.main.adm(int(mem))
            else:
                await ctx.send("Не достаточно прав")

Не понимаю что тут не так

[shurshur]

Артём Смирнов, зачем user.id сравнивать с user_class? И предлагаю угадать, кто из них int, а кто dict...

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

shurshur, UC_PROGRAMMER int, user_class dick если не ошибаюсь

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

shurshur, get_user_class(user.id) за что отвечает?

[shurshur]

Артём Смирнов,

UC_PROGRAMMER int, user_class dick если не ошибаюсь

Именно так. Причём как сравить словарь с числом - непонятно. Причём непонятно и как вообще сравнивать, и зачем вообще сравнивать. Кстати, если уж копируешь мой пример, зачем явно определённое название user_class_names изменил на user_class?

get_user_class(user.id) за что отвечает

Это функция, которую ещё надо написать. Она может брать user_class из базы, или может из какого-то кэша (например, глобальной переменной-словаря со всеми пользователями), может генерировать класс по умолчанию для пользователя, которого нет в базе...

Я же говорил, что привёл пример для описания принципа, а не законченное решение задачи.