Постоянно else хотя должно бить True как исправить?

Question

[ARKRAFTER]

@client.command()
async def addblack(ctx,userid):
	owner = 634060873731670018
	author = ctx.message.author
	if author.id == owner:
		sql.execute(f'SELECT userid FROM badusers')
		if sql.fetchone() is None:
			sql.execute(f'INSERT INTO badusers VALUES (?) ', ({userid}))
			db.commit()
			await ctx.send('готово!')
		else:
			await ctx.send('он уже есть в бд!')
	else:
		await ctx.send('вы не имеете достаточного права чтоб добавлять его как крашера!')

Постоянно виводится он уже есть в бд! хотя его там нет

Comments

[Модератор]

ARKRAFTER дорогой пользователь, настоятельно рекомендуем еще раз обратить самое пристальное внимание на п. 3.1 регламента работы сервиса (и, в особенности, на его последний абзац). В противном случае, ваши вопросы будут удаляться по причине тег-спама, а систематические нарушения приведут к блокировке учетной записи.

[devdb]

Не бейте True, ведь он говорит правду, а за правду не бьют! :)

Answer 1

[o5a]

Потому что неправильно используете SQL. Запрос выбирает все записи из badusers, а не только конкретного user_id. Насколько вижу, предполагается проверить, есть ли user и занести.

sql.execute(f'SELECT userid FROM badusers WHERE user_id = ?', (user_id, ))

А лучше добавить уникальный индекс/первичный ключ по этому user_id и не придется делать проверки перед добавлением.

Во-вторых, неправильный синтаксис в execute для INSERT, не нужно там фигурных скобок, просто

sql.execute(f'INSERT INTO badusers VALUES (?) ', (userid, ))

Answer 2

[Алан Гибизов]

Смотри что sql.fetchone() возвращает. Там явно что-то есть, но не None.

Comments

[ARKRAFTER]

ARKRAFTER, ARKRAFTER,
userid INT

[Алан Гибизов]

ARKRAFTER, ну int это явно не None, вот и не исполняется твой if.

[Алан Гибизов]

ARKRAFTER, что касается ошибки с параметром - это вам надо разбираться, какие именно параметры каких типов в каком порядке принимает coro()