php-これよりもクエリで変数を管理するためのより良い方法はありますか？

解決策：

クエリは安全なので、インジェクション攻撃に関するすべての冗談は無視してください。

一連の条件が増え続けると予想される場合は、 if-elseif-elseやcase-switchではなくルックアップ配列をお勧めします。彼らはあなたのスクリプトを肥大化させることになります。このようなことを考えてください...

 $conditions = [
    1 => "< 3",
    2 => "> 10",
];

if (empty($_POST['condition']) || !isset($conditions[$_POST['condition']])) {
    // write default behavior
} else {
    // use $conditions[$_POST['condition']] in your query
}
 Copy code

このデータ構造とプロセスにより、一連の条件をクリーン、簡潔、効率的に拡張できます。

解決策：

はい、もっと良い方法があります。プリペアドステートメントを使用してSQLインジェクションを防ぐだけでなく、同じデータを使用してフロントエンドで選択を生成したり、クエリを作成したりできます。たとえば、この関数を見てください。

 public function getFilterOptions(): array
{
    return [
        ['name' => 'Age >30', 'comparator' => '>', 'comparable' => 30],
        ['name' => 'Age until 21', 'comparator' => '<=', 'comparable' => 21],
    ];
}
 Copy code

次に、このメソッドを使用して、フロントエンドでフィルター選択を生成できます。

 echo '<select name="query_filter">';
foreach (getFilterOptions() as $key => $option) {
    echo '<option value="' . $key . '">' . $option['name'] . '</option>';
}
echo '</select>';
 Copy code

質問者が使用しているテンプレートシステムがわからないため、このコードについてはご容赦ください。例としてプレーンPHPを使用するのが理にかなっています。

 $optionIndex = $_POST['query_filter'];
// TODO: you should check first if this index exists and handle errors appropriately
$option = getFilterOptions()[$optionIndex];

// building the query will be easy then...
// note: it is not possible to dynamically bind operators of a query
$stmt = $pdo->prepare("SELECT * FROM persons WHERE age $option['comparator'] :comparable");
$stmt->execute($option);
// or explicit: $stmt->execute(['comparable' => $option['comparable']]);
// using PDO is actually not really necessary here as the options are hard coded
// and not user-given, but it is best practice anyway...
 Copy code