ユーザー定義関数が上手くいかない

Question

現在PHPでPDO接続をしているのですがユーザー定義関数を作成してからのコードでORDER BY句で昇順にしたいのですが行えず詰まっております。
ページの表示自体は行えているのですが昇順表示になっていません。

下記のコードだと昇順表示できるのですが、その下のユーザー定義関数だと昇順化されません。
ORDER BYをいれる位置が合っていないのでしょうか。

Comments

[m.ts10806]

$sqlの中身をそのままご提示いただけますか？
;);　　だとおかしい感じになると思いますが

Answer 1

どうせやるならこんな感じで

<?PHP
$key_list=["key","board_id","message","password","created_at","updated_at"];;
$where=filter_input(INPUT_POST,"where",FILTER_DEFAULT,FILTER_REQUIRE_ARRAY);
$where=array_filter((array)$where,function($x) use($key_list){
  return in_array($x,$key_list)!==false;
},ARRAY_FILTER_USE_KEY);

if(count($where)>0){
  $sql="select * from tbl where 1 ";
  $data=[];
  foreach($where as $key=>$val){
    $sql.="and {$key}=? ";
    $data[]=$val;
  };
  $sql.= 'ORDER BY DESC LIMIT 0,10 ';
  print $sql.";<br>";
  print_r($data);
}

?>
<script>
window.addEventListener('DOMContentLoaded',()=>{
  [].forEach.call(document.querySelectorAll('#f1 [type=checkbox]'),x=>{
    x.addEventListener('change',e=>{
      var self=e.target;
      var target=self.closest('label').nextElementSibling;
      target.disabled=!self.checked;
    });
  });
});
</script>
<form method="post" id="f1">
<label><input type="checkbox" checked>key:</label><input type="text" name="where[key]" value="value"><br>
<label><input type="checkbox" checked>board_id:</label><input type="text" name="where[board_id]" value="12"><br>
<label><input type="checkbox" checked>message: </label><input type="text" name="where[message]" value="2"><br>
<label><input type="checkbox" checked>password:</label><input type="text" name="where[password]" value="2"><br>
<label><input type="checkbox" checked>created_at:</label><input type="text" name="where[created_at]" value="2"><br>
<label><input type="checkbox" checked>updated_at:</label><input type="text" name="where[updated_at]" value="2"><br>
<label><input type="checkbox" checked>dummy:</label><input type="text" name="where[dummy]" value="123"><br>
<input type="submit" value="go">
</form>

Answer 2

function getTable2($pdo, $table, $where = array())
{
    $sql = 'SELECT * FROM ' . $table;
    $where = '';  //ここで引数を上書きしている

    if (!empty($where)) //なのでここは常にempty()===trueとなり通らない

       $where .= ' ORDER BY id DESC LIMIT 0,10;);';     //だからifの中にあるここも通らない
       //通ったとしても　;);　という締め方ではSQL構文エラー ); が要らない

つまりこうなってる

function test($a){
    $a = 2;
    echo $a;    
}
test(1);
//2

Comments

[yuntama]

コードを修正してifの中を通るよう修正したのですが動かせまえん。
考えられる原因としてなにかアドバイスいただけたら幸いです。

[m.ts10806]

どんどんグチャグチャになっています。壊れていっています。
そもそも引数と同じ名称の変数を同じスコープ内で扱うことが悪手であるという指摘の回答なのです。
それに最も確認しなければならないことは「想定通りのSQLが出来上がっているか」です。
実行前にexitで止めて実行予定のSQLを画面出力してみれば一目瞭然です。
↓それに

[m.ts10806]

ORDER BY って検索条件ありなし関係ありますか？

[yuntama]

もう一度位置から見直してみます。
ORDER BYで検索してリストを昇順に表示するために使おうと思っています。

[m.ts10806]

でも引数受け入れが下記だと空配列もあるわけですから。「検索条件関係なくORDER BYを入れておく必要」はあると思います。要件や仕様は知りませんけど。
$where = array()