このSQL構文の何が違いますか??

Question

はじめまして。
現在以下のようなコード(PHP,MySQL)を実装しているのですが、上手くいきません。

ちなみにfavoritesテーブルは、
id(プライマリー) user_id item_id
で構成されています。

<?php 
$user_id = $_POST["input1"];
$item_id = $_POST["input2"];

$dsn='mysql:dbname=略;host=略;charset=utf8';
$user='略';
$password='略';
$dbh=new PDO($dsn,$user,$password);
$dbh->setAttribute(PDO::ATTR_ERRMODE,PDO::ERRMODE_EXCEPTION);

//$user_id,$user_itemが同じrowを見つける
 $sql0='SELECT id FROM favorites WHERE user_id = {$user_id} AND item_id = {$item_id}';
        $stmt0=$dbh->prepare($sql0);
        $stmt0->execute();
		$rec=$stmt0->fetch(PDO::FETCH_ASSOC);
		

if(isset($rec['id'])){

//rowがあった場合は消す
$sql2="DELETE FROM favorites WHERE id =".$rec['id'];
$stmt2=$dbh->prepare($sql2);

$stmt2->execute();


}else{

//rowがない場合は新しく生成する
$sql='INSERT INTO favorites (user_id,item_id) VALUES (?,?)';
$stmt=$dbh->prepare($sql);
$data[]=$user_id;
$data[]=$item_id;

$stmt->execute($data);

}

$dbh=null;

?>

何かお分かりの方は是非ともご教授ください。
よろしくお願い申し上げます。

Comments

[lightwill]

どううまくいっていないのかわからないのですが、
エラーメッセージとかがあれば追記して下さい。

Answer 1

どんなエラーでていて、どこがうまくいっていないのでしょうか?
おそらく
$sql0='SELECT id FROM favorites WHERE user_id = {$user_id} AND item_id = {$item_id}';
ここだと推察されますが、シングルクオート(')で囲まれている部分には変数を入れても変数として認識してくれません。
変数として認識させるためには、文字列をダブルクオート(")で囲ったほうがいいので、
$sql0="SELECT id FROM favorites WHERE user_id = {$user_id} AND item_id = {$item_id}";
ただし、これは実際あまりいい実装ではありません。(セキュリティー的に。SQLインジェクションのリスクをはらむので)
なので、INSERT文でやっていることと同様に
$sql0='SELECT id FROM favorites WHERE user_id = ? AND item_id = ?';
$stmt0=$dbh->prepare($sql0);
$data[]=$user_id;
$data[]=$item_id;
$stmt0->execute($data);
このようにすればいいと思います。

Answer 2

恐らくなのですが、最初のSELECT文がシングルクォートで括られているせいでuser_idとitem_idが変数として使えないのではないのかと思います。文字列の中で変数を使用する場合は、ダブルクォーテーションで括らないと出来なかったはずです。

Comments

[Shinog]

ご回答ありがとうございます！
ダブルクォーテーションで括ることで無事解決出来ました!