Python-сообщество

agryn · Дек. 19, 2012 23:01:30

У меня есть масив id list_id = по которым нужно удалить данные в таблице
при запуске такого кода:

list_id = [1,2,3,4,5,]
....
cur.execute(""" 
                DELETE 
                FROM `mbpjs_tst_jglist_companies`
                WHERE id in %s
                """, tuple(list_id))

Выдает ошибку типа синтаксическая ошибка. Как правильно сделать запрос по спику id в масиве

——

RUSSIAN WAR CRIMES IN UKRAINE

Отредактировано agryn (Дек. 19, 2012 23:02:32)

py.user.next · Дек. 20, 2012 04:02:32

>>> """ %s """ % '(1, 2, 3)'
' (1, 2, 3) '
>>>

JOHN_16 · Дек. 20, 2012 04:09:08

py.user.next
не не не, вы что советуете - если такое подставит в реальный код, то SQL-инъекция будет доступна наверняка. Здесь %s это не питоновская подстановка, а MySQL - она призвана экранировать символы для предотвращения атак.

agryn
попробуйте так (или вообще без tuple - но это не уверен):

cur.execute(""" 
                DELETE 
                FROM `mbpjs_tst_jglist_companies`
                WHERE id in %s
                """, (tuple(list_id), ))

_________________________________________________________________________________
полезный блог о python john16blog.blogspot.com

Отредактировано JOHN_16 (Дек. 20, 2012 04:11:15)

py.user.next · Дек. 20, 2012 08:05:22

JOHN_16
Здесь %s это не питоновская подстановка

я не знал

sank · Дек. 20, 2012 09:00:26

Вроде так надо:

list_id = [1,2,3,4,5,]
....
cur.execute(""" 
                DELETE 
                FROM `mbpjs_tst_jglist_companies`
                WHERE id in (%s)
                """, (','.join(list_id),))

Skivbard · Янв. 11, 2013 12:10:34

sank
Не-не-не, это плохой паттерн, как и лепка строки выше
JOHN_16 прав.
Не знаю как в мускуле, но вот, что говорится в документации по постгресу на этот счет:
http://initd.org/psycopg/docs/usage.html

Warning Never, never, NEVER use Python string concatenation (+) or string parameters interpolation (%) to pass variables to a SQL query string. Not even at gunpoint.

sank · Янв. 11, 2013 12:22:32

Ну тогда банально без параметров сделать так:

list_id = [1,2,3,4,5,]
....
cur.execute(""" 
                DELETE 
                FROM `mbpjs_tst_jglist_companies`
                WHERE id in (%s)
                """ % ','.join(list_id))

либо циклом по одному, если list_id сожержит слишком много элементов. А то может база не скушать …

Skivbard · Янв. 11, 2013 12:38:28

list_id = list(range(1000))
cur.execute(""" 
                DELETE 
                FROM `mbpjs_tst_jglist_companies`
                WHERE id in %s
                """, (tuple(list_id), ))

Вот так вполне себе удачно скушает. Базе по большему счету по барабану какой там длинны запрос.
А вот это:

list_id = [1,2,3,4,5,]
....
cur.execute(""" 
                DELETE 
                FROM `mbpjs_tst_jglist_companies`
                WHERE id in (%s)
                """ % ','.join(list_id))

те же яйца, только в профиль и даже хуже, потому что лепка строки, а не передача аргумента.

Python-сообщество

Уведомления

#1 Дек. 19, 2012 23:01:30

MySQL WHERE id in %s

#2 Дек. 20, 2012 04:02:32

MySQL WHERE id in %s

#3 Дек. 20, 2012 04:09:08

MySQL WHERE id in %s

#4 Дек. 20, 2012 08:05:22

MySQL WHERE id in %s

#5 Дек. 20, 2012 09:00:26

MySQL WHERE id in %s

#6 Янв. 11, 2013 12:10:34

MySQL WHERE id in %s

#7 Янв. 11, 2013 12:22:32

MySQL WHERE id in %s

#8 Янв. 11, 2013 12:38:28

MySQL WHERE id in %s

Board footer