SQL構文が間違えている？Where句を追加しただけなのにうまく取れない…。

Question

			sql = "SELECT c._id, c.name, c.type1, c.type2, c.publisher, c.author, c.english, c.official1, c.official2, c.official3, count(c._id), count(l.generation_id) " +
					"FROM collection AS c " +
					"INNER JOIN collection_generation AS cm ON c._id=cm.generation_id" +
					"LEFT JOIN( SELECT DISTINCT log.generation_id FROM log) AS l " +
					"ON cm.generation_id=l.generation_id" +
					"GROUP BY c._id;";

というSQLに、「type1」に「国」という文字が入っているカラムに限定して抽出した結果がほしいので

			sql = "SELECT c._id, c.name, c.type1, c.type2, c.publisher, c.author, c.english, c.official1, c.official2, c.official3, count(c._id), count(l.generation_id) " +
					"FROM collection AS c " +
           "WHERE type1 = " + getQuery() + " " +
					"INNER JOIN collection_generation AS cm ON c._id=cm.generation_id" +
					"LEFT JOIN( SELECT DISTINCT log.generation_id FROM log) AS l " +
					"ON cm.generation_id=l.generation_id" +
					"GROUP BY c._id;";

"WHERE type1 = " + getQuery() + " " +

という一文を追加したところ、SQLのエラーが発生します。
SQL構文が間違えているんでしょうか？

ちなみに、Androidアプリの開発中で、SQLiteを使用し、「rawQuery(sql,null)」というメソットの引数として与える構文です。

間違えてる点をご指摘頂きたいです。
よろしくお願いします。

Answer 1

JOINした結果に対して条件を付加するのであれば、下記のように最後にWHERE句を追加しなければいけません。

sql = "SELECT c._id, c.name, c.type1, c.type2, c.publisher, c.author, c.english, c.official1, c.official2, c.official3, count(c._id), count(l.generation_id) " +
                    "FROM collection AS c " +
                    "INNER JOIN collection_generation AS cm ON c._id=cm.generation_id " +
                    "LEFT JOIN( SELECT DISTINCT log.generation_id FROM log) AS l " +
                    "ON cm.generation_id=l.generation_id " +
                    "GROUP BY c._id " +
           "WHERE c.type1 = " + getQuery()

collectionテーブルに条件を追加したものをJOINしたいのであれば、下記のように副問い合わせにする必要があります。

sql = "SELECT * from " +
			"(select _id, name, type1, type2, publisher, author, english, official1, official2, official3, count(c._id), count(l.generation_id) " +
                    "FROM collection " +
           		    "WHERE type1 = " + getQuery() + ") AS c " +
                    "INNER JOIN collection_generation AS cm ON c._id=cm.generation_id" +
                    "LEFT JOIN( SELECT DISTINCT log.generation_id FROM log) AS l " +
                    "ON cm.generation_id=l.generation_id" +
                    "GROUP BY c._id;";

どちらも同じ結果になると思いますが、仕様にあわせて何をしたいのかを明示的に選んでみてください。
動作検証はしていないので、エラーになったら適宜修正してください。

Comments

[SmartBuzz]

副問い合わせの結果、排出したクエリは
SELECT * from (SELECT c._id, c.name, c.type1, c.type2, c.publisher, c.author, c.english, c.official1, c.official2, c.official3, count(c._id), count(l.generation_id) FROM collection WHERE c.type1='国') AS c INNER JOIN collection_generation AS cm ON c._id=cm.collection_id LEFT JOIN( SELECT DISTINCT log.generation_id FROM log) AS l ON cm.generation_id=l.generation_id GROUP BY c._id;
となりました。
しかし、エラーです…。

この書き方としては、先に抽出するデータに削っておいてから、joinするという流れなのでしょうか？

[ttyp03]

その通りです。
collectionテーブルのレコード数が膨大なら、こちらの方が処理速度は速くなると思います。

[SmartBuzz]

確かに処理速度を気にする必要がありますね。
アドバイス頂いた副問い合わせを今回の実装に入れてみます。

ありがとうございます！

Answer 2

Where句の位置が違うと思います。
また、getQuery()の値が文字列の場合はシングルでくくる必要もあるかと。。。

sql = "SELECT c._id, c.name, c.type1, c.type2, c.publisher, c.author, c.english, c.official1, c.official2, c.official3, count(c._id), count(l.generation_id) " +
      " FROM collection AS c " +
      " INNER JOIN collection_generation AS cm ON c._id=cm.generation_id" +
      " LEFT JOIN( SELECT DISTINCT log.generation_id FROM log) AS l " +
      " ON cm.generation_id=l.generation_id" +
      " WHERE type1 = '" + getQuery() + "' " +
      " GROUP BY c._id;";

Comments

[SmartBuzz]

確かに・・・。
シングルトンを失念していました。

再度テストしてみたら、このシングルトンが原因でした！

ですので、今回のベストアンサーにさせて頂きました。
ありがとうございます！

[roast_chicken]

どのようなエラーになっているのでしょうか？
sqlをログ出力してみて、直接データベースにSQLを発行してみたらいかがでしょうか？

[SmartBuzz]

失礼しました。
コメント編集させて頂きました。

[nakit]

シングルクォート
シングルクォーテーション
と
シングルトンとは全く別の単語ですのでお気をつけて。

Answer 3

KaedeKazaneさん指摘のスペースなしで連結している部分も問題ですが、Where句の挿入位置も問題です。
Where句はFrom/Joinの後、GroupByの前に記述します。

投入予定のSQL文が目的の動作をするかは、可能であればプログラム中で実行する前に（簡易的な内容でもいいので）事前にSQL文単体でDBに投入して動作確認しておくことをお勧めします。

Comments

[SmartBuzz]

今回、Android のSQLiteで開発しており、MySQLのような管理画面でSQL文の単体テストができず、困っておりました…。
しかし、今回は自分の単純な記載ミスが原因でした。。

これも、単体テストをしていればすぐ発見できた内容ですね。
まさしくアドバイス頂いた通りだと思います。

ありがとうございました。

Answer 4

連結部分にスペースがない箇所がいくつかあるので、そこが繋がって構文エラーになっているんだと思います。
例えば下記の場所とか。

"INNER JOIN collection_generation AS cm ON c._id=cm.generation_id" +
"LEFT JOIN( SELECT DISTINCT log.generation_id FROM log) AS l "
↓
"INNER JOIN collection_generation AS cm ON c._id=cm.generation_idLEFT JOIN( SELECT DISTINCT log.generation_id FROM log) AS l "

追記

失礼しました。
上記の場所はエラーが発生するようになった変更とは関係ありませんでしたね。

SELECT文の構文は以下のようになります。

SELECT 抽出項目 FROM 抽出対象の表 WHERE 抽出条件 (ORDER句等)

JOIN(表の結合)はFROM句で対象となる表を結合するのでFROM句の一部となります。
そのため、WHERE句はJOINの後に記述します。

sql = "SELECT c._id, c.name, c.type1, c.type2, c.publisher, c.author, c.english, c.official1, c.official2, c.official3, count(c._id), count(l.generation_id) " +
                    "FROM collection AS c " +
                    "INNER JOIN collection_generation AS cm ON c._id=cm.generation_id" +
                    "LEFT JOIN( SELECT DISTINCT log.generation_id FROM log) AS l " +
                    "ON cm.generation_id=l.generation_id" +
                    //結合してから条件を指定
                    "WHERE type1 = " + getQuery() + " " +
                    "GROUP BY c._id;";

Comments

[SmartBuzz]

丁寧に、SELECT構文の解説をして頂きありがとうございます。

今回のエラーの起因のひとつであるご指摘かつ丁寧に書いて頂き大変わかりやすい解説でした。
こちらもベストアンサーとさせて頂きたいのですが、完全に忘れていたシングルトンの指摘を今回のベストアンサーとさせて頂きました。

ありがとうございました。

[KaedeKazane]

ちなみに、「'」はシングルクォーテーション（シングルクォート）ですよ。
「シングルトン」は、インスタンスが1つしか生成されないことを保証する設計を表す別の言葉です。

[SmartBuzz]

'
はシングルトンではなく、シングルクォーテーションなんですね…

度々のご指摘ありがとうございます。基礎勉強中の初心者ですので、質問事態も不明確な質問をしてしまいますが、今後ともご回答のほど宜しくお願いいたします。