Почему не записываются данные в бд SQLITE?

Question

[mrcoldurfuck]

import sqlite3
import sys
import pathlib

script_dir = pathlib.Path(sys.argv[0]).parent
db_file = script_dir / 'database.db'
conn = sqlite3.connect(db_file)

class Database:
    def __init__(self, db_file):
        self.connection = sqlite3.connect('db_file', check_same_thread=False)
        self.cursor = self.connection.cursor()
        db = self.connection
        c = db.cursor()
        # c.execute("""CREATE TABLE users (
        #     id      INTEGER PRIMARY KEY AUTOINCREMENT,
        #     user_id INTEGER NOT NULL,
        #     money   INTEGER NOT NULL
        #                     DEFAULT (0)
        # );""")

    def user_exists(self, user_id):
        with self.connection:
            result = self.cursor.execute("Select * FROM 'users' WHERE 'user_id' = ?", (user_id,)).fetchall()
            return bool(len(result))
    def add_user(self, user_id):
        with self.connection:
            self.cursor.execute("INSERT INTO 'users' ('user_id') VALUES (?)", (user_id,))
    def user_money(self, user_id):
        with self.connection:
            result = self.cursor.execute("SELECT 'money' FROM 'users' WHERE 'user_id' = ?", (user_id,)).fetchmany(1)
            return int(result[0][0])
    def set_money(self, user_id, money):
        with self.connection:
            return self.cursor.execute("UPDATE 'users' SET 'money' = ? WHERE 'user_id' = ?", (money, user_id))

db = sqlite3.connect(db_file)

db.commit()
db.close()

при нажатие на кнопку для вывода кол-ва денег выдает ошибку: in user_money
return int(result[0][0])
IndexError: list index out of range
Данные в бд не появляются, не могу понять в чем проблема.
P.S. создание таблицы закомментировано, потому что она уже создана

Comments

[Kvason]

return int(result[0])

[mrcoldurfuck]

Kvason, не помогло

[Kvason]

mrcoldurfuck, что у тебя возвращает этот запрос, выведи его принтом

Answer 1

[Ипатьев]

После запросов, выполняющих изменения в БД, надо вызывать
self.connection.commit()

И имена колонок и таблиц нельзя брать в одинарные кавычки. Потому что в одинарных кавычках пишутся строки.
И если даже иногда sqlite и смотрит сквозь пальцы на такую ошибку (когда она может понять по контексту, что имеется в виду), это не значит, что так будет везти всегда.
Строка 'money' никогда не будет равна числу.

Comments

[mrcoldurfuck]

def user_exists(self, user_id):
        with self.connection:
            result = self.cursor.execute("Select * FROM 'users' WHERE 'user_id' = ?", (user_id,)).fetchall()
            return bool(len(result))
    def add_user(self, user_id):
        with self.connection:
            self.cursor.execute("INSERT INTO 'users' ('user_id') VALUES (?)", (user_id,))
           <i> self.connection.commit()</i>
    def user_money(self, user_id):
        with self.connection:
            result = self.cursor.execute("SELECT 'money' FROM 'users' WHERE 'user_id' = ?", (user_id,)).fetchmany(1)
            return int(result[0][0])
    def set_money(self, user_id, money):
        with self.connection:
            return self.cursor.execute("UPDATE 'users' SET 'money' = ? WHERE 'user_id' = ?", (money, user_id))
           <i> self.connection.commit()</i>

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

[Ипатьев]

имена колонок и таблиц нельзя брать в одинарные кавычки. Потому что в одинарных кавычках пишутся строки.
И если даже иногда sqlite и смотрит сквозь пальцы на такую ошибку (когда она может понять по контексту, что имеется в виду), это не значит, что так будет везти всегда.
Строка 'money' никогда не будет равна числу.

[mrcoldurfuck]

Ипатьев, я идиот, спасибо большое, простите за мою грубость.

[Ипатьев]

Я и сам лажанулся. Думал что во всех запросах будут ошибки, но оказывается insert сработает с одинарными, и имя таблицы в кавычках тоже тоже проблем не вызывает. Интересное поведение, я такого никогда раньше не видел.

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

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

Дополню любопытный нюанс:
Вообще-то в sqlite по-умолчанию автокоммит. Но вот именно в реализации в python в библиотеке автокоммит по-умолчанию выключен.

[o5a]

mrcoldurfuck, если в названиях таблиц/полей не используете верхний регистр, то кавычки там вообще не нужны, чтобы себя не путать, просто пишите их обычным текстом, т.е.
SELECT money FROM users WHERE user_id = ?

Кавычки (именно двойные - ") или ` используются из-за того, что SQL регистрозависим, и если явно обзывать таблицу в верхнем регистре, то обращаться нужно четко по имени, для этого и используются кавычки.

Т.е. если создать таблицу
CREATE TABLE "Users"

То при попытке обратиться
SELECT FROM users
получим ошибку, т.к. таблицы "users" нет, надо будет писать
SELECT FROM "Users"

[Ипатьев]

o5a, вы сейчас какую-то ерунду написали.
В-первых, SQL регистроНЕзависим, хотя он здесь вообще не при чем
Во-вторых, идентификаторы в SQLite, о которых здесь идёт речь, тоже регистроНЕзависимы. Имя таблицы можно писать в любом регистре, с кавычками или без - без разницы.
В-третьих, кавычки нужны совсем для другого - для идентификаторов, в которых могут встречаться пробелы, или которые совпадают с ключевыми словами

[o5a]

Ипатьев, В SQL как правило указанные без кавычек названия идентификаторов автоматом приводятся к одному регистру, и так уже запрашиваются в базе (например, в Postgres - к нижнему, в Oracle - к верхнему).
Поэтому если в Pg создать таблицу
create table UsERs
то в базе она создастся как users и обращаться к ней надо соответствующе:
или
select * from "users"
или же без кавычек, тогда не важен регистр (т.к. запрос названия без кавычек автоматом конвертирует в "базовый" регистр, хотя бы так
select * from UsERs
И при этом нельзя к ней обратиться по явному верхнему регистру, т.к. по факту таблица именно "users".
select * from "UsERs"
вызовет ошибку

Но если таблица создана с явным указанием регистра
create table "UsERs"
То обращаться к ней без кавычек уже не получится, только
select * from "UsERs"

Про SQLite я честно не уточнял, если так, значит он действительно регистронезависим, и к нему это не применяется. Я скорее говорил про общие принципы SQL баз.

Answer 2

[PavelMos]

Надо проверить что делает функция записи "INSERT INTO 'users' ('user_id') VALUES (?)", (user_id,)) пользователя и функцию записи денег
После нее если базу посмотреть через менеджер, в таблице изменения отображаются ?