Почему не работает код в примере?

Question

[Андрей Андреев]

Команда award должна выдавать деньги, но почему то не работает, в консоли ошибок нету, но работает то что нельзя отправить деньги если не ввел имя, и нельзя когда сумма меньше 1, а команда take тоже самое только забирать деньги должна, а почему не работает?.
ПРОБЛЕМА: Не добавляется баланс

@client.command(aliases= ['award'])
async def __award(ctx, member: discord.Member = None, amount: int = None):
	if member is None:
		await ctx.send(f"**{ctx.author}**, укажите пользователя, которому желаете выдать определенную сумму")
	else:
		if amount is None:
			await ctx.send(f"**{ctx.author}**, укажите сумму, которую желаете начислить на счет пользователя")
		elif amount <1:
			await ctx.send(f"**{ctx.author}**, укажите сумму большо 1 :leaves:")
		else:
			if fetch := cursor.execute("UPDATE users SET cash = cash + {} WHERE id = {}".format(amount, member.id)).fetchone():
				connection.commit()

			await ctx.message.add_reaction('✅')


@client.command(aliases = ['take'])
async def __take( ctx, member: discord.Member = None, amount = None):
	if member is None:
		await ctx.send(f"**{ctx.author}**, укажите пользователя, которому желаете снять определенную сумму со счета")
	else:
		if amount is None:
			await ctx.send(f"**{ctx.author}**, укажите сумму, которую желаете забрать со счета пользователя")
		elif amount == 'all':
			if fetch := cursor.execute("UPDATE users SET cash = {} WHERE id = {}".format(0, member.id)).fetchone():
				connection.commit()
		elif int(amount) <1:
			await ctx.send(f"**{ctx.author}**, укажите сумму большо 1 :leaves:")
		else:
			if fetch := cursor.execute("UPDATE users SET cash = cash - {} WHERE id = {}".format(int(amount), member.id)).fetchone():
				connection.commit()

			await ctx.message.add_reaction('✅')

Comments

[Shandy]

Какая ошибка возникает при этом?

[Андрей Андреев]

Владимир Аршинов, не добавляется баланс

[Shandy]

Андрей Андреев, а в консоли никаких ошибок нет?

[Андрей Андреев]

Владимир Аршинов, нету

[Shandy]

Андрей Андреев, упс, извиняюсь. Перечитал и увидел

[Андрей Андреев]

Владимир Аршинов, не это я добавил только что)

[MinTnt]

Андрей Андреев, добавь принты для дебага. Может проблема с sql

[Shandy]

Андрей Андреев, я не особо шарю в sql и не могу точно сказать, но разве можно делать так?
SET cash = cash - {}

[Shandy]

Я в плане SQL же не знает сколько там этот cash

[Андрей Андреев]

Владимир Аршинов, ну так, у меня уже это есть у команды take

[Shandy]

Андрей Андреев, а take работает?

[Андрей Андреев]

Владимир Аршинов, не знаю я же не могу начислить себе деньги

[Андрей Андреев]

MinTnt, не шарю(

[Shandy]

Андрей Андреев, попробуй делать 2 запроса, а не подобной штукой страдай. Сначала узнать значение баланса человека, потом сложить баланс и отправить это обратно

balance = SELECT баланса пользователя по id
new_balance = balance+1
UPDATE users SET {new_balance} WHERE id = {}

Сам напишешь как надо :)

[soremix]

Потому что смесь невалидного кода.

amount: int

Почему int? Не шарю в дискорде, но если это параметр из команды, то разве она интом будет?

cash = cash + {}

Без комментариев

int(amount)

А здесь это уже число

Кто вообще использует format в sql запросах?

а

fetch :=

зачем?

.fetchone():

зачем после апдейта? что возвращает?

[Андрей Андреев]

SoreMix, Я убрал fetch, насчет int не знаю, amount будет с типом данных int, просто исправить что бы заработало?

Answer 1

[Руслан .]

А у вас ранее есть добавление записи в таблицу users?
Этот запрос:
UPDATE users SET cash = cash + {} WHERE id = {}
сработает только при наличии записи с нужным id в таблице users.

Comments

[Андрей Андреев]

@client.event
async def on_ready():
	cursor.execute("""CREATE TABLE IF NOT EXISTS users (
		name TEXT,
		id INT,
		cash BIGINT,
		rep INT,
		lvl INT
	)""")
	connection.commit()

[Руслан .]

Андрей Андреев, это код по созданию таблицы. А нужно ещё потом в эту таблицу запись положить, например так:
INSERT users (id, cash) VALUES (id, 0)

Тут id это идентификатор пользователя которому потом можно увеличить или уменьшить баланс.

[Андрей Андреев]

Руслан ., Вот так?

async def on_ready():
	cursor.execute("""CREATE TABLE IF NOT EXISTS users (
		name TEXT,
		id INT,
		cash BIGINT,
		rep INT,
		lvl INT,
		INSERT users (id, cash) VALUES (id, 0)
	)""")
	connection.commit()

[Руслан .]

Андрей Андреев, не совсем.
Добавление записи нужно делать отдельным вызовом.
cursor.execute("INSERT users (id, cash) VALUES (1, 0)")

И в запросе я привел пример запроса по добавлению записи в таблицу users, вам видимо нужно не только поля id и cash заполнять при добавлении нового user-а но и все остальные.