Почему не создается таблица в БД?

Question

[TemaKam]

вообще не могу понять, что не так
сотнями способами уже пробовал и все равно что-то не нравится

name = self.task_name.toPlainText().replace('\n', '')
query = """/*!40101 SET @saved_cs_client = @@character_set_client */;
DROP TABLE IF EXISTS %s;
CREATE TABLE %s 
(id int(11) NOT NULL AUTO_INCREMENT,
account varchar(255) NOT NULL,
token varchar(255) NOT NULL,
proxy varchar(255) NOT NULL,
status varchar(255) NOT NULL,
ReasonBlock varchar(255) NOT NULL,
Provider varchar(255) NOT NULL,
idUser varchar(255) NOT NULL,
activ varchar(10) NOT NULL,
FIO varchar(255) NOT NULL,
Sex varchar(255) NOT NULL,
Age int(11) NOT NULL,
Country varchar(255) NOT NULL,
City varchar(255) NOT NULL,
Friends int(11) NOT NULL,
Followers int(11) NOT NULL,
frfol int(11) NOT NULL,
grouplinks varchar(255) NOT NULL,
groupaudit int(11) NOT NULL,
avatar varchar(10) NOT NULL,
oldProvider varchar(255),
oldDate varchar(255),
PRIMARY KEY (id))
ENGINE=InnoDB DEFAULT CHARSET=utf8mb4; 
/*!40101 SET character_set_client = @saved_cs_client */;"""
cursor.execute(query, [name, name])

(1064, "You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'SET character_set_client = utf8 */;\nDROP TABLE IF EXISTS 'Перечек';\nCREAT' at line 2")

(1064, "You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'SET character_set_client = utf8 */;\nDROP TABLE IF EXISTS Перечек;\nCREATE ' at line 2")

(1064, "You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'DROP TABLE IF EXISTS 'Перечек';\nCREATE TABLE 'Перечек' \n(id int(11' at line 2")

(1064, "You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'SET character_set_client = utf8mb4 */;\nDROP TABLE IF EXISTS 'Перечек';\nCR' at line 2")

(1064, "You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'DROP TABLE IF EXISTS 'Перечек';\nCREATE TABLE 'Перечек' \n(id int(11' at line 2")

UPD
причем при другом запросе ошибок нет
query = f"DROP TABLE IF EXISTS {name};"

Answer 1

[galaxy]

Нельзя имена таблиц (и колонок) передавать в запрос как параметры. Используйте хотя бы str.format()

query = """/*!40101 SET @saved_cs_client = @@character_set_client */;
DROP TABLE IF EXISTS {};
CREATE TABLE {} 
...
""".format(name, name)
cursor.execute(query)

Comments

[TemaKam]

sql-инъекции не действуют на такое?

[TemaKam]

и f-строки тоже пробовал, все равно ошибка

[TemaKam]

как и с format

(1064, "You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'DROP TABLE IF EXISTS Перечек;\nCREATE TABLE Перечек \n(id int(11) NO' at line 2")

[TemaKam]

Dr. Bacon, зависит от условий, там же вон первой строкой имя берется с формы
ну и явно вписывал, все равно ошибка

(1064, "You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'DROP TABLE IF EXISTS Перечек;\nCREATE TABLE Перечек \n(id int(11) NO' at line 2")

[TemaKam]

Dr. Bacon, у меня суть такая, что на отдельную задачу создается своя таблица, удаляется задача - удаляется таблица
может и по-другому сделаю потом, но ошибка то все равно непонятно в чем сейчас

[TemaKam]

Dr. Bacon, для задачи не 1 запись в таблице, а рандомное, мб тысячи, мб только 100

UPD
в другом месте запрос (правда тоже на удаление) срабатывает
query = f"DROP TABLE IF EXISTS {name};"
проблема какая-то именно с созданием

[TemaKam]

Dr. Bacon, ну тогда объясни, как разделять задачи в общей таблице, когда количество строк рандомное на каждую задачу и может добавлять и удаляться?

[TemaKam]

Dr. Bacon, ладно, логично. Но все же может с текущей ошибкой как-то выяснить? создание таблицы из кода все равно может пригодиться

[galaxy]

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

If multi is set to True, execute() is able to execute multiple statements specified in the operation string.

operation = 'SELECT 1; INSERT INTO t1 VALUES (); SELECT 2'
for result in cursor.execute(operation, multi=True):
  if result.with_rows:

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

[TemaKam]

galaxy, cursor.execute(query, multi=True)

execute() got an unexpected keyword argument 'multi'

а куда ж параметр вставлять, если некуда?

spoiler

метод execute

[galaxy]

Это другой DB api, в pymysql, как ниже написали, можно флаг поставить.
А еще лучше - разбейте на несколько запросов.

[AUser0]

Один SQL-запрос за раз, названия таблиц и столбцов заключить в обратный апостроф `.
И character set сохранять-восстанавливать нет смысла, тем более комментариями, которые в чистом SQL-запросе - ни к селу, ни к городу. Вы же не MYSQL dump восстанавливаете, ну?!

[TemaKam]

AUser0, из дампа просто как раз брал запрос на создание

Answer 2

[o5a]

Основная проблема в наличии нескольких команд в одном SQL запросе. По умолчанию execute может исполнить только одну команду. Т.е. нужно разбить исполнение на несколько запусков execute.

cursor.execute("DROP TABLE ...")
cursor.execute("CREATE TABLE IF EXISTS ... ")

Другой вариант - использовать флаги:
https://stackoverflow.com/questions/58544640/pymys...

Но и как уже заметили, создавать и удалять таблицы - плохой дизайн. Лучше разделять задачи или еще что там нужно на уровне полей самих таблиц, которые фиксированы.