質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.35%
SQL

SQL(Structured Query Language)は、リレーショナルデータベース管理システム (RDBMS)のデータベース言語です。大きく分けて、データ定義言語(DDL)、データ操作言語(DML)、データ制御言語(DCL)の3つで構成されており、プログラム上でSQL文を生成して、RDBMSに命令を出し、RDBに必要なデータを格納できます。また、格納したデータを引き出すことも可能です。

Python

Pythonは、コードの読みやすさが特徴的なプログラミング言語の1つです。 強い型付け、動的型付けに対応しており、後方互換性がないバージョン2系とバージョン3系が使用されています。 商用製品の開発にも無料で使用でき、OSだけでなく仮想環境にも対応。Unicodeによる文字列操作をサポートしているため、日本語処理も標準で可能です。

Q&A

解決済

1回答

699閲覧

WHERE文に変数を入れたい

kazuki19950107

総合スコア3

SQL

SQL(Structured Query Language)は、リレーショナルデータベース管理システム (RDBMS)のデータベース言語です。大きく分けて、データ定義言語(DDL)、データ操作言語(DML)、データ制御言語(DCL)の3つで構成されており、プログラム上でSQL文を生成して、RDBMSに命令を出し、RDBに必要なデータを格納できます。また、格納したデータを引き出すことも可能です。

Python

Pythonは、コードの読みやすさが特徴的なプログラミング言語の1つです。 強い型付け、動的型付けに対応しており、後方互換性がないバージョン2系とバージョン3系が使用されています。 商用製品の開発にも無料で使用でき、OSだけでなく仮想環境にも対応。Unicodeによる文字列操作をサポートしているため、日本語処理も標準で可能です。

0グッド

0クリップ

投稿2020/06/29 15:36

WHERE文の条件としたいものを引数で取り、それを成型してwheresに格納する。
そのwheresを条件式としてSELECT文を実行したいが、SyntaxErrorが発生する。

python

1import psycopg2 2import config 3 4def reference(formMemberList): 5 dbusers = 'postgres' 6 dbnames = 'CRM' 7 passwords = 'taku0217' 8 conn = psycopg2.connect(" user=" + dbusers +" dbname=" + dbnames +" password=" + passwords) 9 10 cur = conn.cursor() 11 12 wheres = "" 13 for menber in formMemberList: 14 if(formMemberList.index(menber)!=0): 15 wheres += " AND " 16 wheres += menber 17 18 cur.execute("SELECT * FROM customer WHERE '%s'" %wheres) 19 20 rows = cur.fetchall() 21 22 cur.execute("COMMIT") 23 24 cur.close() 25 conn.close() 26 27 return rows 28 29formMemberList = ["name:'takumi'", "birth:'1995-02-17'"] 30rows = reference(formMemberList) 31

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

まずそうなところが2つあるようです。

結合部のシングルクオート

cur.execute("SELECT * FROM customer WHERE '%s'" %wheres)
この行で、 %s をシングルクオートで囲んでいるので、WHERE の後が ' で囲まれてしまいます。

formMemberListの中身

"name:'takumi'" という内容なので SQLにそのまま埋め込めません。
今回は、とりあえず : -> = に置換して形を合わせてみました。

蛇足

where のあとに "1 = 1" をとりあえず入れておくと、 AND を入れる入れないの判定が不要になって便利です。
サンプルコードではこれを使って楽をしています。

蛇足2

member が menber になっていますが、そのままにしてあります。

結論

必要な部分を切り出して最小限のコードを書きました。 下記ご参照下さい。
yymmtさん、ご指摘ありがとうございました。

def reference(formMemberList): wheres = "1 = 1" for menber in formMemberList: wheres += " AND " wheres += menber.replace(":", "=") print("SELECT * FROM customer WHERE %s" %wheres) return None formMemberList = ["name:'takumi'", "birth:'1995-02-17'"] rows = reference(formMemberList)

投稿2020/06/29 15:53

編集2020/06/29 16:15
YakumoSaki

総合スコア2027

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

yymmt

2020/06/29 16:01

微妙にメンバーリストも違っているので修正する必要があります(関数内でreplaceしようということなのかも知れませんが)。 formMemberList = ["name = 'takumi'", "birth = '1995-02-17'"]
YakumoSaki

2020/06/29 16:02

わ、本当だ。。 ちょっと回答書き直さないとですね。ありがとうございます。
kazuki19950107

2020/06/29 23:45

YakumoSaki様 yymmt様、ご丁寧に教えていただきまして、本当にありがとうございます。 ご指摘頂いた箇所を修正すると、正常に動作するようになりました。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.35%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問