pythonでsqlの書き方について

Question

前提・実現したいこと

プログラミング初心者です。
pythonでsqlのデータと変数の中身を複数比較したいのですがSyntaxErroで動きません。
どのような書き方をすれば良いのでしょうか？

発生している問題・エラーメッセージ

SyntaxError: invalid syntax

該当のソースコード

sql = "SELECT * FROM table1 WHERE FLG = 0 AND mail_ad = '%s'" % address " AND mail_date < '%d'" % newdate

Comments

[Orlofsky]

table1の CREATE TABLE文を https://teratail.com/help#about-markdown の[コードを入力]を使って載せると適切なコメントが付き易いです。

[m.ts10806]

質問編集画面タイトル横にある「初心者アイコン」をご活用ください。「初心者」と質問で書くよりも伝わりますし、質問一覧に表示されるのでわかりやすくなります。

Answer 1

データベースがわからないので、MySQLと仮定して回答します。
質問文にはデータベース、使用ドライバを書いてくださいな。

sql = "SELECT * FROM table1 WHERE FLG = 0 AND mail_ad = '%s'" % address " AND mail_date < '%d'" % newdate

1,SQLインジェクションを防止するためにSQLを文字列連結するのではなく、プレースホルダを使用してくださいな。

sql = 'SELECT * FROM table1 WHERE FLG = 0 AND mail_ad = %s AND mail_date < %s'
cursor.execute(sql, (address, int(newdate)))

◇参考情報
MySQL Connector/PythonからSQLを投げる

2,ORDER BY句を指定しない時SELECT文は結果順序を保証しません。
ORDER BYはできるだけ付けてくださいな。

3,データベースやカーソルのclose忘れを防ぐためにcontextlib#closingを使用してくださいな。

データベース:MySQL
ドライバ:mysql.connector
のサンプルコードです、ご参考まで。

# -*- coding: utf-8 -*-
from mysql.connector import connect
from contextlib import closing

def main() -> None:
    config = {
        'host': '127.0.0.1',
        'database': 'XXXXXXX',
        'user': 'XXXXXX',
        'password': 'XXXXXX',
        'charset' : 'utf8'
    }
    with closing(connect(**config)) as conn:
        with closing(conn.cursor()) as cursor:
            address = "bbbbbbbbb"
            newdate = 20180706
            sql = 'SELECT * FROM table1 WHERE FLG = 0 AND mail_ad = %s AND mail_date < %s'
            cursor.execute(sql, (address, newdate))
            for row in cursor.fetchall():
                print(row)

if __name__ == "__main__":
    main()

Answer 2

SQLをロクに勉強しないでプログラムがまともに動作しないのに納品して、当然なことですが、お客様に納品を拒否される開発会社もあります。まず、SQL(今回はSELECT文)を実行してエラーにならないかを確認してからPythonに組み込みましょう。
SQLの中で使っているテーブルはCREATE TABLEが必要です。参考

同じSQLでも方言が大きいですから、どのデータベースを使うのかをタグで明示した方が適切なコメントが付き易いです。
SQLの観点から Oracle Database, PostgreSQL, MySQL の特徴を整理しよう！

データベースによっても直接SQLを実行する方法が違います。様々なDBでSQLをバッチ実行するには

Answer 3

%記法を使っているようですが、もっと詳しく使い方を調べてください。

https://qiita.com/takahiro_itazuri/items/e585b46d096036bc837f

Comments

[megune]

すみません。よくわからず使っておりました。
mail_dateとnewdateには8桁の数字が入っております。
どのように記述すればよいのでしょうか？

[mather]

えっと、 mail_date って質問内容にはないんですが、どこから来たんですか？
サンプルとしてaddress, newdateの具体的な値を決めて、あなたの欲しいSQL文（完成形）を具体的に書いてください。
これらの情報がないと答えられません。