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

Question

[vinegarnnl]

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()
    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:
            return self.cursor.execute("INSERT INTO 'users' (user_id) VALUES (?);", (user_id,))
            self.connection.commit()

@bot.message_handler(commands = ['start', 'help'])
def welcome(message):
    if not db.user_exists(message.chat.id):
        db.add_user(message.chat.id)

все импортировано, ошибок не выдает, но и в бд не записывает данные.

Comments

[Ипатьев]

о кстати, return я не заметил

[Дмитрий]

Если с кавычками так сложно, то не пишите их в имена полей

Answer 1

[Ипатьев]

Мужчина, вы издеваетесь?
Я вам уже раз пять повторил ответ на этот вопрос:

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

Comments

[vinegarnnl]

так self.connection.commit() стоит, с кавычками все нормально, я не понимаю что мне надо еше сделать.

[Ипатьев]

с кавычками всё ненормально.
снова разуваем глаза и смотрим

[vinegarnnl]

Ипатьев, поменял ковычки, тот же результат.
P.S. return то же убрал, ничего не изменилось

Answer 2

[Vindicar]

def user_exists(self, user_id):
    with self.connection:
        ...
def add_user(self, user_id):
    with self.connection:

По выходу из блока with соединение закроется. Второй блок with его уже не переоткроет.
В таких ситуациях блок with использовать не надо.

Answer 3

[Алексей С.]

Как то так:

result = self.cursor.execute("Select * FROM users WHERE user_id = ?", (user_id,)).fetchall()

Comments

[mayton2019]

+1

В этих ботах есть логгирование? ХЗ. Ну пускай автор еще добавит следующее

result = self.cursor.execute("Select * FROM users WHERE user_id = ?", (user_id,)).fetchall()
print("length of result = ", len(result))

Ну и insert подправить соотвественно с этими ненужными кавычками.