Почему row[0] обращается к None объекту?

Question

[Reaper_sds]

@commands.Cog.listener()
async def on_dropdown(self, inter: disnake.MessageInteraction):
    c = self.conn.cursor()
    c.execute(f"SELECT role_id, cost FROM shop WHERE id = ?", (inter.data['values'][0],))
    row = c.fetchone()
    role = inter.guild.get_role(row[0])
    if role in inter.author.roles:
        embed = disnake.Embed(title="Магазин ролей", description=f"У вас уже есть роль {role}", color=0xFF0000)
        await inter.response.send_message(embed=embed, ephemeral=True)
        return
    if c.execute(f"SELECT cash FROM users WHERE id = {inter.author.id}") < row[1]:
        embed = disnake.Embed(title="Магазин ролей", description=f"У вас недостаточно денег для покупки роли {role_name}", color=0xFF0000)
        await inter.response.send_message(embed=embed, ephemeral=True)
        return
    else:
        c.execute(f"UPDATE users SET cash = {c.execute(f'SELECT cash FROM users WHERE id = {inter.author.id}') - row[1]} WHERE id = {inter.author.id}")
        self.conn.commit()
        await inter.author.add_roles(role)
        embed = disnake.Embed(title="Магазин ролей", description=f"Вы успешно приобрели роль {role}", color=0x2f3136)
        await inter.response.send_message(embed=embed, ephemeral=True)

Comments

[Vindicar]

c.execute(f"UPDATE users SET cash = {c.execute(f'SELECT cash FROM users WHERE id = {inter.author.id}') - row[1]} WHERE id = {inter.author.id}")

Пожалуйста, не надо так делать. Вам что, по пальцу отрезают за каждую промежуточную переменную?
И не надо формировать запросы через f-строки, для этого есть подстановка параметров в запрос. Будет аккуратнее и безопаснее.

[Reaper_sds]

Vindicar, а разве из за этого код не будет работать? Вроде я про другую ошибку спрашивал

[Vindicar]

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

[Reaper_sds]

Vindicar, это всё круто и я решил проблему с row[0] теперь у меня проблема с cost(row[2])

[Everything_is_bad]

Reaper_sds, у тебя нету в коде вызова cost(row[2]), короче учи основы python, ботов тебе писать рано

Answer 1

[Everything_is_bad]

Почему row[0] обращается к None объекту?

Потому что в row None. А почему в row None? Потому что None вернул sql запрос. Элементарная же логика

Comments

[Reaper_sds]

эма ну ладно

[Reaper_sds]

и почему же вернул?:)

[Everything_is_bad]

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

[Reaper_sds]

типа такой роли нету в базе данных?

[Reaper_sds]

типа я не сохранил эту роль в базу данных?