MySQLのプレースホルダの書き方について

Question

前提・実現したいこと

PDOの勉強をしている初心者です。

MySQLを使用して、DBに入っているデータをDatepickerを使用し、日付で絞り込みをしたく
下記のコードの書き方をしていました。

変数をシングルクォーテーションで囲んでおり、
プレースホルダの使用を指摘され調べていましたが、書き方がいまいち分からず絞り込みができません。
間違っている部分を教えていただきたいです。よろしくお願いいたします。

該当のソースコード

<div id="datepicker">
 <input type="text" name="date_start">
 <input type="text" name="date_end">
</div>

$start = $_POST['date_start'];
$end = $_POST['date_end'];

$sql = "SELECT * FROM news WHERE created_at BETWEEN '$start' AND '$end'";

試したこと

<div id="datepicker">
 <input type="text" name="date_start">
 <input type="text" name="date_end">
</div>

$start = $_POST['date_start'];
$end = $_POST['date_end'];

$sql = "SELECT * FROM news WHERE created_at BETWEEN start = ? AND end = ?";

$stmt->bindValue(1, $start, PDO::PARAM_STR);
$stmt->bindValue(2, $end, PDO::PARAM_STR);

$stmt = $pdo->prepare($sql);
$stmt->execute(array($start,$end));

Answer 1

日付をバリデートするかどうかにもよりますがこんな感じ

<?PHP
$start = filter_input(INPUT_POST,'date_start');
$end = filter_input(INPUT_POST,'date_end');
$sql ="SELECT * FROM news WHERE 1 ";
if(!is_null($start) and !is_null($end) and $start!=="" and $end!==""){
  $sql.="and created_at BETWEEN start = ? AND end = ?";
  $data[]=$start;
  $data[]=$end;
}
print $sql;
try{
  $dsn = 'mysql:host=localhost; dbname=test1;charset=utf8;';
  $pdo = new PDO($dsn, $user,$password);
  $pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION); 
  $stmt = $pdo->prepare($sql);
  $stmt->execute($data);
  $rows=$stmt->fetchAll(PDO::FETCH_ASSOC);
  print_r($rows);
}catch(PDOException $e){
  die($e->getMessage());
}
?>
<form method="post">
start:<input type="text" name="date_start"><br>
end:<input type="text" name="date_end"><br>
<input type="submit" value="go">
</form>

Comments

[mi-17]

とても分かりやすく、回答していただき有難うございました！

Answer 2

「日付を示す文字列」を与えるという意味で、
SQL文上では文字列データから日付データに変換するような工夫があると良いかも。
https://dev.mysql.com/doc/refman/5.6/ja/cast-functions.html
にあるような「CAST('2000-01-01' AS DATE)」のようなことを想定して
CAST(? AS DATE)としてプレースホルダを置くとか。

$startや$endに入っている文字列が、SQLの文法上日付文字列と認識できない形だったりすると
クエリーの実行でエラーになる可能性があります。
例えば「20180724」みたいな区切り文字列を使ってないとか。

Comments

[mi-17]

CAST(? AS DATE)などを使う方法もあるのですね！
丁寧に有難うございます！

Answer 3

提示されたコードに特におかしな点はございません。
文法的な問題はありませんので、意図した値が投げられているのかなどを、var_dump() などで確認しましょう。

あくまで、提示されたコードから読み取れる部分でのことです。
接続されていないとか、レコードセットの結果表示されていないとかそういった他の問題があるかどうかの可能性排除しての話です。

Comments

[mi-17]

確認してみます、有難うございます！