In this post about SQLite, aaronasterling told me that
cmd = "attach \"%s\" as toMerge" % "b.db"
: is wrongcmd = 'attach "{0}" as toMerge'.format("b.db")
: is correctcmd = "attach ? as toMerge"; cursor.execute(cmd, ('b.db', ))
: is right thing
But, I've thought the first and second are the same. What are the differences between those three?
You should use
'
instead of"
, so you don't have to escape.You used the old formatting strings that are deprecated.
This uses the new format string feature from newer Python versions that should be used instead of the old one if possible.
This one omits string formatting completely and uses a SQLite feature instead, so this is the right way to do it.
Big advantage: no risk of SQL injection
The first and second produce the same result, but the second method is prefered for formatting strings in newer versions of Python.
However the third is the better approach here because it uses parameters instead of manipulating strings. This is both faster and safer.
Because it is not being escaped. If you replaced the b.db with user input, it would leave you vulnerable to SQL injection.