POSTで値を受け取ってからの並び替え

Question

①聞きたいこと
ドロップダウンで項目を選択後、並び替えボタンを押すと商品一覧の並び替えを行う方法
②起きている問題
商品は表示されていますが、並び替えができておりません。
③ソースコード

<?php
require('./dbconnect.php');

if ($_POST['filter'] === 'vibration') {
    $filter = 'vibration';
    $sort = $db->prepare('SELECT * FROM tool ORDER BY ?');
    $sort->bindParam(1, $filter);
    $sort->execute();
?>

 <?php
    foreach ($sort as $tool) {
    print('<div class="images col-md-3">');
    print('<div class="image_1">');
    print('<a href="./racket-' . $tool['number'] . '/racket_' . $tool['number'] . '.php?number=' . $tool['number'] . '">');
    print('<img src="images/racket' . $tool['number'] . '.jpg" alt="' . $tool['name'] . '" height="230" width="230">');
    print('<div>' . $tool['name'] . '<br>価格：' . $tool['price'] . '円(税込)<br>反発性：' . $tool['repulsion'] . '<br>振動特性：' . $tool['vibration'] . '</div>');
    print('</a>');
    print('</div>');
    print('</div>');
    } 
?>

<form method="post" action="">
  <select name="filter">
    <option value="select">選択してください</option>
    <option value="price">価格順</option>
    <option value="repulsion">反発性順</option>
    <option value="vibration">振動特性順</option>
  </select>
  <input type="submit" value="並び替え">
</form>

④問題について考えたこと
bindParamのところでうまくいっていないような気がします。

Answer 1

order byにはカラムが指定されるのでバインドできません。
バインドできるのは「値」のみ。
直接文字列として組み込んでください。
SQLインジェクション避けたいなら入力値をそのまま入れるのではなく
1なら○○など２なら▲▲、それ以外はデフォルト□□
など対応する設定値をプログラム側で用意し変換すると良いです（あくまで一案）

気がします

エラーハンドリングを適切に組み込んでおけば何が起きているか見えます。エラーが出ないのはエラーが出る仕組みにしていないからです。

Comments

[yuyamaa]

直接文字列を組み込んで作成しました。
ありがとうございました！

[m.ts10806]

でしたら、それで自己解決とするのはちょっと。

[yuyamaa]

すみません、やり方わからずに自己解決としてしまいました。申し訳ありません！
次回から気をつけます。

[m.ts10806]

「質問するときのヒント」にあるように「最も役に立った回答を」ベストアンサーに選びます。やり方が分からないことはないのでは。回答には解決済みにする、ベストアンサーにするボタンが出ているはずですし。
あと、「受付中に戻す」こともできるので、いわゆる「ベストアンサーの付け替え」もできます。

Answer 2

filterがvibrationではないときにどうしたいのでしょうか？

$filter=filter_input(INPUT_POST,'filter');
if($filter === 'vibration') {
  $res= $pdo->query('SELECT * FROM tool ORDER BY vibration');
}

追記

<?PHP
$filter=filter_input(INPUT_POST,'filter');
$sql='SELECT * FROM tool ';
if(in_array($filter,['price','vibration','repulsion'])){
  $sql.="order by `$filter` desc";
}
print $sql;
?>
<form method="post">
<label><input type="radio" name="filter" value="price">price</label>
<label><input type="radio" name="filter" value="vibration">vibration</label>
<label><input type="radio" name="filter" value="repulsion">repulsion</label><br>
<input type="submit" value="send">
</form>

Comments

[yuyamaa]

回答ありがとうございます。
valueがselectの時は通常の並び、priceの時は価格高い順、repulsionの時は反発性高い順に並び替えたいです。

[yambejp]

追記しておきました
そういうのはホワイトリストを使って合致したときorder byを
付加すればよいです

[yuyamaa]

ありがとうございます。こちらの方が効率的にプログラムできました。

Answer 3

if ($_POST['filter'] === 'price') {
    $sort = $db->query('SELECT * FROM tool ORDER BY price DESC');
    $sort->execute();
} elseif ($_POST['filter'] === 'vibration') {
    $sort = $db->query('SELECT * FROM tool ORDER BY vibration DESC');
    $sort->execute();
} elseif ($_POST['filter'] === 'repulsion') {
    $sort = $db->query('SELECT * FROM tool ORDER BY repulsion DESC');
    $sort->execute();
} else {
    $sort = $db->query('SELECT * FROM tool');
    $sort->execute();
};