PDOを使ったSQL文のエラー(HY093)

Question

みなさんこんばんは,rui3718と申します。
この度初めてご質問を投稿させていただきます。

環境
データベース：MySQL ver5.0
PHP 5.4

まずはコードを見ていただくのが早いかと思います。

$sql = "SELECT `id`,`title` FROM `posts` WHERE ";
foreach($tags AS $tag){
	$like = "`tag` LIKE '%{$tag}%'";
	$sql .= $like;
}
$sql .= " ORDER BY created DESC LIMIT 0,8";
$stmt = $no_name->pdo->prepare($sql);
$stmt->execute();
var_dump($stmt->errorCode());

PDOを使ってPHPからMySQLへアクセスをしているのですが、
SQL文を動的に作り上げていく際にエラーが発生してしまい、
原因が突き止められないため、皆様のお力をお借り出来たら、
と思いこの質問を投稿させていただきます。

$stmtにつきましてはvar_dump()したところ、以下の様なコードが出力されているので、
SQL文に関してはしっかりと発行されており、また、このSQL文をサーバー側で直接実行したところ、問題なく情報を受け取ることが出来ました。

object(PDOStatement)#6 (1) { ["queryString"]=> string(139) "SELECT `id`,`title` FROM `posts` WHERE `tag` LIKE '%ねこ%' ORDER BY created DESC LIMIT 0,8" } 

PDOのエラーコードに関しましては、パラメータの数不足を表す、HY093のコードが出力されております。

しかし、このSQL文ではbindParam、bindValueを使っていませんし、そもそもプレースホルダーを用いていません。

調べましても似たような状況の方へ当たりませんでした。

この問題の解決策、もしくは、代替策など心当たりがあるかたおりましたら、ご教示願いたいです。

長々と失礼いたしました。

Answer 1

とりあえず、SQL文のエラーなのか、PDOのエラーなのか問題の切り分けが必要だと思います。
みた感じ、foreachで繰り返されたときに「AND」か「OR」が必要じゃないでしょうか。

Comments

[rui3718]

回答ありがとうございます。
現在、最も簡単なテスト環境で実行しておりますので、
foreachの繰り返しが一度しか起きていないので、「AND」、「OR」を
SQL文に含ませる必要はないかと思い処理を加えておりません。

また本文中にありまますようにSQL文自体は生成されている文を
実際にMySQLで実行したところ、問題なくレコードを参照できております。

実際にPDOExceptionも以下のように吐き出されております。

PDOStatement::execute(): SQLSTATE[HY093]: Invalid parameter number: no parameters were bound

Answer 2

SQLの構文エラーがあるように見受けられます。空白がいたるところで不足していると思います。

Comments

[rui3718]

ご指摘いただいた通り、空白が足りなかったみたいです。
結合する際にスペースを加えるのを失念していたみたいです。
しかし、コードを修正しましたが、エラーは消えません…。
どうすればよいのでしょうか。

[TaMaMhyu]

SQL自体についてはyutamaruさんの回答の件もありますが、プレースホルダがないのにprepareを使うことが原因ではないでしょうか？

[TaMaMhyu]

話がそれますが、$tagについてはプレースホルダを使用すべきだと思います。

[rui3718]

$tagの数が不定のため、$tagの数だけ、ループしてSQL文に条件を加えようと思って、今回のプログラムを書きました。
プレースホルダーにするにはあらかじめ個数が決まっていないと、SQL文が書けないかと思っていたのですが、個数をカウントしてその数だけWHERE句を生成して、後々数をbindするという事でしょうか・・・＞＜

[TaMaMhyu]

`?`によるプレースホルダであれば、SQLに`?`を順次結合と同時に、$tagをexecuteに渡す配列として継ぎ足していけば可能だと思います。
名前付きプレースホルダの場合は連番をつけて、SQLには連番付きで順次結合と同時に、$tagをプレースホルダの名前による連想配列を作ってからbindすればどうにでもなるかと思いますが。

[TaMaMhyu]

http://qiita.com/mpyw/items/b00b72c5c95aac573b71
この記事がPHPからMySQLを使用する情報がよくまとまっていると思います。
ついでに、LIKE句に関する注意点が書かれているので、参考にされると良いかと思います。

[TaMaMhyu]

名前付きプレースホルダでもexecuteに連想配列を渡せばできますね。
文字列以外の形式でバインドをしたい場合は無理ですが…