MySQLのシングルクオート記載方法

Question

            String a=item.getSpringCode();
            String b=item.getSpringName();
            String c=item.getSpringText();
                       
            String sql = "INSERT INTO ITEM ( SpringCode,SpringName,SpringText) VALUES ('" + a+ "'" ","  "'" b+ "'" ","  c+ ")" ;

上記コードの ('" + a+ "'" "," "'" b+ "'" "," c+ ")" ;の部分でエラーがでます。
どこがいけないのか間違いが見つけられません。
insertでのMySQLシングルクオートを連携して記載したいのですが、教えていただきたいです。

Answer 1

そんなことで悩まずにPreparedStatementを使いましょう。

Answer 2

どこがいけないのか間違いが見つけられません。

文字列合成でSQLを組み立てること自体が、（ほとんどの場合に）不適当です。

プリペアードステートメントを使って、構文と値を別個に組み立てるようにするのが妥当です。

Answer 3

どこがいけないのか間違いが見つけられません。

Javaとしての文字列連結が正しく出来てないのは見つけられなくないのでは・・・。

('" + a+ "'" "," "'" b+ "'" "," c+ ")"`

SQLでもDBアクセスの問題じゃなくJavaの基礎部分です。

で、なくても既に指摘がある通りプリペアドステートメントで組み立てるべきです。
prepareStatementの使用

Answer 4

ちゃんとエラーメッセージを読みましょう。
多分、こんなものが出ると思います。

Main.java:6: error: ';' expected
        String sql = "INSERT INTO ITEM ( SpringCode,SpringName,SpringText) VALUES ('" + a+ "'" ","  "'" b+ "'" ","  c+ ")" ;
                                                                                              ^
Main.java:6: error: not a statement
        String sql = "INSERT INTO ITEM ( SpringCode,SpringName,SpringText) VALUES ('" + a+ "'" ","  "'" b+ "'" ","  c+ ")" ;
                                                                                                         ^
Main.java:6: error: ';' expected
        String sql = "INSERT INTO ITEM ( SpringCode,SpringName,SpringText) VALUES ('" + a+ "'" ","  "'" b+ "'" ","  c+ ")" ;
                                                                                                              ^
Main.java:6: error: not a statement
        String sql = "INSERT INTO ITEM ( SpringCode,SpringName,SpringText) VALUES ('" + a+ "'" ","  "'" b+ "'" ","  c+ ")" ;
                                                                                                                     ^
4 errors

さらに、Javaですから文法エラーは^（ハット）で場所を示してくれているハズです。
文字列の連結したいのに足りないものがありますよね。

最後に、今回程度であれば再現は簡単なので気まぐれで回答しますが、
質問文にエラーメッセージは載せてくれると回答がもっとスムーズになります。

追記

他の回答を見て思い出しました。確かにこの場合はPreparedStatementを使うべきですね。

インタフェースPreparedStatement
https://docs.oracle.com/javase/jp/15/docs/api/java.sql/java/sql/PreparedStatement.html

Comments

[neonemo]

いやいや、ちょっと待ってください。
私の回答がベストってString連結でパラメータを埋め込んでSQLを作る方法を採用したって事でしょうか？
自分だけで使うようなアプリで使うならいざ知らず、大勢が使うようなものではよろしい実装ではありません。
せめて代表的なリスクは把握しておいてください。

IPA　独立行政法人情報処理推進機構
「安全なSQLの呼び出し方」を公開
https://www.ipa.go.jp/security/vuln/press/201003_websecurity_sql.html

[m.ts10806]

フィードバックは欲しいですね。
https://teratail.com/help/question-tips#questionTips4-2