Как прибавить +1 к значению в sqlite?

Question

[hey_umbrella]

Мне нужно,чтобы после нажатия на кнопку к star прибавляло +1,в чем ошибка?

if call.data == 'stars':
            #ursor.execute(f'UPDATE stars SET star = star + 1 WHERE photo = "{photo}"')
            #con.commit()
            photo = call.message.text
            con = sqlite3.connect("database.db")
            cursor = con.cursor()
            cursor.execute(f'SELECT star FROM stars WHERE photo = "{photo}"')
            star = cursor.fetchone()[0]
            con.commit()
            cursor.execute(f'UPDATE stars SET star = star +1; WHERE photo = "{photo}"')
            con.commit()
            markup = telebot.types.InlineKeyboardMarkup()
            markup.add(telebot.types.InlineKeyboardButton(text=f'Рейтинг - ⭐{star}', callback_data="stars"))

Сама ошибка - star = cursor.fetchone()[0]
TypeError: 'NoneType' object is not subscriptable

Comments

[hey_umbrella]

Захар Винокуров, тоже самое

Answer 1

[Vindicar]

> Сама ошибка - star = cursor.fetchone()[0]
> TypeError: 'NoneType' object is not subscriptable

not subscriptable означает что ты пытаешься взять индекс у объекта, который это не поддерживает. В твоем случае объект типа NoneType - т.е. None.
Иными словами, cursor.fetchone() вернул None, и конечно у None нельзя взять индекс.
Почему fetchone() вернул None? Потому что запрос SELECT не нашёл ни одной строки с подходящим значением photo!

Как это решить?
Вставить в таблицу строку с photo, а если не получится (так как такое photo уже есть), то обновить. Есть два способа.
1. Кодом. Проверить, что вернул fetchone(). Если None, то делаем INSERT. Если не None, то UPDATE.
2. Средствами БД, что обычно называется UPDATE/INSERT, или коротко UPSERT. Для sqlite это потребует примерно такого запроса:

INSERT INTO stars (photo, star) VALUES (ид фото, 1) ON CONFLICT (photo) DO UPDATE SET star = star + 1

Требование: столбец photo должен быть первичным ключом или хотя бы иметь уникальный индекс, иначе запрос просто будет добавлять дубликат строки.

ВАЖНО
f'SELECT star FROM stars WHERE photo = "{photo}"' - никогда так не делайте! Особенно если входные данные получены от пользователя. Это хороший способ заполучить SQL-инъекцию.
Используйте placeholders, по порядку:
cursor.execute('SELECT star FROM stars WHERE photo = ?', (photo, ) )
или по именам
cursor.execute('SELECT star FROM stars WHERE photo = :photo', {'photo' : photo} )
И удобнее и безопаснее.

Comments

[hey_umbrella]

Спасибо большое! Но мне почему-то пишет cursor.execute(f'INSERT INTO stars (photo, star) VALUES (photo, 1) ON CONFLICT (photo) DO UPDATE SET star = star + 1')
sqlite3.OperationalError: no such column: photo

Хотя он есть и какой я должен написать ид фото,а ид написать photo если он берется photo = call.message.text
Вообще смысл это функции,в том что человек оценивает фото,а фото это url,уже отправленная

[Vindicar]

> VALUES (photo, 1)
вместо photo тут должно быть конкретное значение id для фотографии.
Вставьте его с помощью placeholders, как я показывал.

[hey_umbrella]

Vindicar, Так?Я просто не очень понял

if call.data == 'stars':
            photo = call.message.text
            con = sqlite3.connect("database.db")
            cursor = con.cursor()
            cursor.execute(f'INSERT INTO stars (photo, star) VALUES (photo, 1) ON CONFLICT (photo) DO UPDATE SET star = star + 1')
            con.commit()
            cursor.execute('SELECT star FROM stars WHERE photo = :photo', {'photo' : photo} )
            star = cursor.fetchone()
            con.commit()
            markup = telebot.types.InlineKeyboardMarkup()
            markup.add(telebot.types.InlineKeyboardButton(text=f'Рейтинг - ⭐{star}', callback_data="stars"))

[Vindicar]

hey_umbrella, я заметил. Повторю.
> VALUES (photo, 1)
вместо photo тут должно быть конкретное значение id для фотографии.
А у тебя так и осталось. Должно быть
cursor.execute('INSERT INTO stars (photo, star) VALUES (:photo, 1) ON CONFLICT (photo) DO UPDATE SET star = star + 1', {'photo': photo})

И скинь запрос CREATE TABLE, которым таблицу stars создаёшь - я подозреваю, у тебя столбец photo допускает неуникальные значения.

[hey_umbrella]

Vindicar, я создал table через эту прогу
CREATE TABLE "stars" (
"photo" TEXT,
"star" INTEGER
);

И теперь вот такая ошибка cursor.execute('INSERT INTO stars (photo, star) VALUES (:photo, 1) ON CONFLICT (photo) DO UPDATE SET star = star + 1', {'photo': photo})
sqlite3.OperationalError: ON CONFLICT clause does not match any PRIMARY KEY or UNIQUE constraint

[Vindicar]

Про что я и говорил. У тебя в таблице stars разрешены записи с одинаковым значением photo. Это имеет смысл в твоей задаче? Я думаю, нет. Если это не так, то поясни почему.
Поэтому таблицу надо пересоздать, сделать photo первичным ключом (PRIMARY KEY) - тогда нельзя будет иметь две или более записей с одинаковым значением photo. Как это сделать - читай в документации по sqlite.
И серьёзно, выучи уже основы SQL - понятие первичного ключа, понятие внешнего ключа, понятие индекса - там не так уж и много. Иначе будешь на каждом шагу спотыкаться при работе с БД.

[hey_umbrella]

Vindicar, понял,спасибо!