POSTで取得した数だけプレースホルダを設置したい

Question

＜開発環境＞
xampp:Windows 8.0.3
VSCode：ver-1.54.3

＜相談したいこと＞
POSTで値を受け取った数だけプレースホルダを設置したいと考え、以下のコードを書きました。

if(isset($_POST["List"])) {
　　$Lists = $_POST["List"];       
} else {
    echo 'アクセスエラー';
}

//配列の要素数だけパラメータを設置
$str = str_repeat(', ?', (count($Lists)-1));

$stmt = $pdo->prepare (
         　　　　　"SELECT * FROM 名簿 M 
                      INNER JOIN 受入先マスタ U ON S.引受先 = U.No 
                      INNER JOIN 国マスタ K ON S.国籍 = K.code
                    WHERE M.氏名 IN (?" . $str . ");");


for ($i=1; $i < count($Lists); $i++) {
          $stmt->bindValue($i, $Lists[$i-1], PDO::PARAM_STR);
        }
           
$stmt->execute();

例えば３つ値を取得した場合、
SQLの実行文は

SELECT * FROM 名簿 M 
　　INNER JOIN 受入先マスタ U ON S.引受先 = U.No 
　　INNER JOIN 国マスタ K ON S.国籍 = K.code 
WHERE M.氏名 IN (?, ?, ?);

と出力されるので問題がなさそうに感じますが、エラーがでます。

SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; 
check the manual that corresponds to your MariaDB server version for the right syntax to use near '', 
?, ?')' at line 4

どこを見落としているのかが分からず相談させていただきました。
よろしくお願いします。

＜試したこと＞
PHPでSQL文中に変数を展開するにはどうすればよいのでしょうか
https://teratail.com/questions/96690

SQLSTATE[42000]: Syntax error or access violation: が発生するが原因がわからない
https://teratail.com/questions/60425

サイトを参考にして確認してみました。

Answer 1

bindValueで$iが1から始まっているので指定する要素数が１個減ってませんか？

for ($i=1; $i < count($Lists); $i++) {
↓↓↓
for ($i=1; $i <= count($Lists); $i++) {

Comments

[on73]

SQL文ばかり見ていたため、そこを精査していませんでした…！
修正できました、ご指摘ありがとうございます！

[yambejp]

Listの渡し方のイメージができてないかもしれません
name="List[]"で渡していたら
PHP側では空文字も検査の対象になりそうです。
こういう質問のときはHTMLもあわせて書いたほうがよいでしょう

[on73]

合わせてのご指摘ありがとうございます。

ご指摘の通り
name="List[]"で渡しています。
ここでは追記しませんが、List[]に空文字は入らないようなコードになっています。

今後POSTやGET周りの質問をするときにはHTMLの記載も合わせてするようにします！

[on73]

ありがとうございました！

[yambejp]

流れ的にはこうですね
<?PHP
$list=filter_input(INPUT_POST,"List",FILTER_DEFAULT,FILTER_REQUIRE_ARRAY);
$sql="select * from tbl where 1 ";
$datas=[];
if(!is_null($list)){
$sql.="and name in (".implode(",",array_fill(0,count($list),"?")).") ";
$datas=array_merge($datas,$list);
}
print $sql.";<br>";
print_r($datas);
?>
<script>
document.addEventListener('change',()=>{
document.querySelectorAll('[name="List[]"]').forEach(x=>{
x.disabled=!x.previousElementSibling.checked;
});
});
</script>
<form method="post">
<label><input type="checkbox" class="list"><input type="text" name="List[]" disabled required></label><br>
<label><input type="checkbox" class="list"><input type="text" name="List[]" disabled required></label><br>
<label><input type="checkbox" class="list"><input type="text" name="List[]" disabled required></label><br>
<input type="submit" value="send">
</form>

[on73]

さっそく実行してみました。

すっきりしたコードで勉強になります、ありがとうございます。

Answer 2

生成したプレースホルダ数に対し、バインド数が一致していないのではないでしょうか？

if(isset($_POST["List"])) {
　　$Lists = $_POST["List"];       
} else {
    echo 'アクセスエラー';
}

//配列の要素数だけパラメータを設置
$str = str_repeat(', ?', (count($Lists)-1));

$stmt = $pdo->prepare (
         　　　　　"SELECT * FROM 名簿 M 
                      INNER JOIN 受入先マスタ U ON S.引受先 = U.No 
                      INNER JOIN 国マスタ K ON S.国籍 = K.code
                    WHERE M.氏名 IN (?" . $str . ");");


//これだと2回しかバインドされないのでは？
for ($i=1; $i < count($Lists); $i++) {
          $stmt->bindValue($i, $Lists[$i-1], PDO::PARAM_STR);
        }

$stmt->execute();

ちなみに自分だと動的なプレースホルダ数生成はこのようにして作っています。

$ary_placeholder = array_fill(0,count($Lists),"?"); //？だけの配列を生成
$str_placeholder = implode(",",$ary_placeholder); //文字列にする

$stmt = $pdo->prepare (
         　　　　　"SELECT * FROM 名簿 M 
                      INNER JOIN 受入先マスタ U ON S.引受先 = U.No 
                      INNER JOIN 国マスタ K ON S.国籍 = K.code
                    WHERE M.氏名 IN ({$str_placeholder});");

こうすれば、いちいち最後の配列の個数に悩むことはないと思いますし、ループ文で気を遣う（普通に処理したらよい）こともないです。

Comments

[on73]

2回しかバインドされない…！
SQL文ばかり見ていたのでそこを考え直していませんでした、ご指摘ありがとうございます。

コードもたいへん参考になります、ありがとうございます！