Mysql.connector.errors.ProgrammingError: 1054 (42S22): Unknown column ' ' in 'where clause'. Как решить?

Question

[venera060]

функция принимает входящее сообщение:

def check_answer(message):

    db_work = mySQL()

    a = db_work.check_answers(message)
    print(a)
    db_work.close()

отправляем запрос в бд есть ли в таблице такое значение:

def check_answers(self, text):
        self.cursor.execute("SELECT * FROM answers WHERE text = " + str(text))
        data = self.cursor.fetchone()
        return data

на первые 2 числовых запроса она реагирует нормально. на третий агрится.
нужно сделать защиту от дурака:
если пользователь вбивает что-то рандомное, а не значения, которые есть в бд,
data должна просто вернуть 0, None, False. (что угодно)
Но он крашится и пишет что столбец не найден, когда это не столбец а значение в столбце

Answer 1

[Анатолий]

Поправьте

self.cursor.execute("SELECT * FROM answers WHERE text = " + str(text))

на

self.cursor.execute("SELECT * FROM answers WHERE text = '" + str(text) + "'")

Comments

[venera060]

Cпасибо!
Парюсь с этим вторые сутки. А можно узнать почему так следует писать?

[Анатолий]

venera060,
запрос к БД был :
SELECT * FROM answers WHERE text = Пекин
Для БД это значит, что условие выборки - значение поля text = значению поля Пекин.

Правильный запрос:
SELECT * FROM answers WHERE text = 'Пекин'
Для БД это значит, что условие выборки - значение поля text = строке 'Пекин'.

Т.е. строки в запросе обязательно нужно оборачивать в кавычки.

[MaxKozlov]

Вообще-то так тоже не надо писать.
А надо

self.cursor.execute("SELECT * FROM answers WHERE text = %s", (str(text)))

https://dev.mysql.com/doc/connector-python/en/conn...

[venera060]

MaxKozlov, в таком варианте он выдает ошибку

[venera060]

Анатолий, спасибо))

[MaxKozlov]

venera060, Я не могу за вас отладить ваш код.
Я дал ссылку на документацию, где есть примеры.
Но я опять же не могу быть уверен что вы используете именно эту библиотеку.

В любом случае, настоятельно рекомендуется использовать вызов с параметрами, а не генерировать запросы конкатенацией, иначе, рано или поздно вы получите в качестве text что-нибудь вроде ""; drop database answers" и будете плакать по своей базе

[o5a]

venera060,

в таком варианте он выдает ошибку

в том примере нужно добавить запятую, параметр ожидает данные в виде кортежа/списка
(str(text), )