pythonでMysqlを利用する場合にwhere句で日付（timestamp）を使いたいがエラーになる

Question

pythonでMySqlを動かしているのですが、単純なことでつまずいてしまいました。

import MySQLdb
import time
import datetime


connection = MySQLdb.connect(host='localhost', user='hoge', passwd='passw0rd', db='test', charset='utf8')
cursor = connection.cursor()

ts = time.time()
timestamp = datetime.datetime.fromtimestamp(ts).strftime('%Y-%m-%d %H:%M:%S')

cursor.execute("DELETE FROM testtbl WHERE time < %s",timestamp)

みたいな感じで使いたいのですが、最後の行がうまく動きません。

cursor.execute("DELETE FROM testtbl WHERE time < %s",timestamp)

File "/usr/lib/python2.7/dist-packages/MySQLdb/cursors.py", line 210, in execute
query = query % args
TypeError: not all arguments converted during string formatting

ちなみに

cursor.execute("DELETE FROM testtbl WHERE time < '2017-10-02 23:30:00'")
だと動きますし

print timestamp とやっても'2017-10-02 23:30:00'
と表示されますし

いったい何がダメなんでしょうか。
もちろん　データベースtesttblテーブルのtimeはタイムスタンプ型です。

宜しくお願いします。

Answer 1

TypeError: not all arguments converted during string formatting
ですと「文字列の書式整形で全部の引数が変換対象になっていない」とのような意味なので、
%s に対応するものが無いようです。

cursor.execute("DELETE FROM testtbl WHERE time < '%s'" % timestamp)

こうです。いったん分けて、

s = "DELETE FROM testtbl WHERE time < '%s'" % timestamp
cursor.execute(s)

とすると確認しやすいです。

追記
"DELETE FROM testtbl WHERE time < %s", timestamp
ですとカンマで区切られてしまうので、文字列の配列になってしまい、%s が放置されてしまう
と言うのがそもそもの原因です。

Comments

[primetimefuture]

解決できました。ありがとうございました。

Answer 2

("...where time < %s",(timestamp))
のようにtimestampを()でくるんでtupleにする

("...where time <　'%s'",timestamp)
のように%sを''でくるむ
のどちらかでかいけつしませんかね？

後者のような気もしますが

追記

("...where time < %s",(timestamp,))
こうするのが正解っぽい。

"query include %s" % arg
みたいなことしてると、 argに不正な文字列が仕込まれたときにSQL Injectionが起きる可能性があるので
cursor.execute("query include %s", args)
を利用したほうが安全なことが多いです。
(cursor.execute内部で argsを安全なものに変換してくれる可能性が高いため)
PEPによると
argsはシーケンスかマップなので、
[timestamp]と配列にするか(timestamp,)とtupleにする必要があったっぽいですね。

Comments

[primetimefuture]

ありがとうございました。どうやらカンマで区切るのがだめだったみたいです。%だったみたいです。
WHERE time < '%s' " % timestamp)