【PHP】mysqli_stmt_bind_paramのタイプ文字長と引数の数が一致しないエラー

レコードごとに写真データ(blob)を３つ含む一般的なデータが有り、各写真データはnullableです。これらの登録データを「変更するページ」のコードにおいて、困っています。

ユーザーが現在の登録画像から変更するためにinputで新たな画像を選択すればそれにあわせてSQL文とmysqli_stmt_bind_paramの第２パラメーターのタイプの文字列を変更する形にしています。

行き詰まったのは mysqli_stmt_bind_param の第３引数以降の部分です。固定部分はいいのですが、３つの写真についてはユーザーによる変更（inputタグからの入力）がある場合とない場合で動的に渡す引数を切り替えないといけなくなりますね。このパラメーターはmixed型なので配列でもOKだと思い以下のようなコードにしたのですが、タイプを示す文字の数とバインドする変数の数が違うというエラーになります。各変数の実際の出力を確認したのですが、数は一致しています。SQL文内の？の数とバインドする変数の数は一致しています。どうしてこのエラーが出るのかを知りたいです。

ただ、もっと言えば、そもそも私のこの処理全体についても、もっとスマートなやり方とかあれば教えていただきたいです

Fatal error: Uncaught ArgumentCountError: The number of elements in the type definition string must match the number of bind variables in /****/modify.php:111

修正ページのHTML（一部）

HTML
1<form action=""  enctype="multipart/form-data" method="post">
2<!-- 略（画像以外の項目のinput群） -->
3  <tr>
4    <td>画像１を選択</td>
5  </tr>
6  <tr>
7    <td>
8      <input id="fileinput" type="file" name="image[]" onchange="previewImage(this);" accept="image/*">
9      <p><!-- 修正のため選択した画像または登録済画像のプレビュー -->
10          Preview:<br><img id="preview" src="image.php?id=<?= $result['id']; ?>&file=1" style="max-width:200px;">
11      </p>
12    </td>
13  </tr>
14  <tr>
15    <td>画像２を選択</td>
16  </tr>
17  <tr>
18    <td>
19      <input id="fileinput" type="file" name="image[]" onchange="previewImage2(this);" accept="image/*">
20      <p><!-- 修正のため選択した画像または登録済画像のプレビュー-->
21          Preview:<br><img id="preview2" src="image.php?id=<?= $result['id']; ?>&file=2" style="max-width:200px;">
22      </p>
23    </td>
24  </tr>
25  <tr>
26    <td>画像３を選択</td>
27  </tr>
28  <tr>
29    <td>
30      <input id="fileinput" type="file" name="image[]" onchange="previewImage3(this);" accept="image/*">
31      <p><!-- 修正のため選択した画像または登録済画像のプレビュー-->
32          Preview:<br><img id="preview3" src="image.php?id=<?= $result['id']; ?>&file=3" style="max-width:200px;">
33      </p>
34    </td>
35  </tr>
36 </table>
37
38  <input type="submit" value="修正する" />
39  <!-- 対象レコードのidを渡す-->
40  <input type="hidden" name="id" value="<?= $result['id']; ?>">
41</form>

PHP
1  $id = $_REQUEST['id'];
2  $title   = $_REQUEST['title'];
3  $resolution = $_REQUEST['resolution'];
4  $detail  = $_REQUEST['detail'];
5  $schoolyear  = $_REQUEST['schoolyear'];
6  $subject  = $_REQUEST['subject'];
7  $keyword  = $_REQUEST['keyword'];
8  $source  = $_REQUEST['source'];
9  $app = $_REQUEST['app'];
10  $params = [$title,$resolution,$detail,$schoolyear,$subject,$keyword,$source,$app];
11  $sqlBase = "UPDATE jirei SET title = ?, resolution = ?, detail = ?, schoolyear = ?, subject = ?, keyword = ?, source = ?, app = ?";
12  $columnP = "ssssssss"; // 最後につけるidの分のsは最後でつける
13  $addSQL = "";
14
15  if ($_FILES['image']['name'][0] !== "") {
16    $filename = $_FILES['image']['name'][0];
17    $filetype = $_FILES['image']['type'][0];
18    $filecontent = file_get_contents($_FILES['image']['tmp_name'][0]);
19    $size = $_FILES['image']['size'][0];
20    $columnP = $columnP."sssi";
21    $addSQL = $addSQL.", `image_name` = ?, `image_type` = ?, `image_content` = ?, `image_size` = ?";
22    $params = array_merge($params ,[$filename,$filetype,$filecontent,$size]);
23  }
24  if ($_FILES['image']['name'][1] !== "") {
25    $filename2 = $_FILES['image']['name'][1];
26    $filetype2 = $_FILES['image']['type'][1];
27    $filecontent2 = file_get_contents($_FILES['image']['tmp_name'][1]);
28    $size2 = $_FILES['image']['size'][1];
29    $columnP = $columnP."sssi";
30    $addSQL = $addSQL.", `image_name2` = ?, `image_type2` = ?, `image_content2` = ?, `image_size2` = ?";
31    $params = array_merge($params ,[$filename2,$filetype2,$filecontent2,$size2]);
32  }
33  if ($_FILES['image']['name'][2] !== "") {
34    $filename3 = $_FILES['image']['name'][2];
35    $filetype3 = $_FILES['image']['type'][2];
36    $filecontent3 = file_get_contents($_FILES['image']['tmp_name'][2]);
37    $size3 = $_FILES['image']['size'][2];
38    $columnP = $columnP."sssi";
39    $addSQL = $addSQL.", `image_name3` = ?, `image_type3` = ?, `image_content3` = ?, `image_size3` = ?";
40    $params = array_merge($params ,[$filename3,$filetype3,$filecontent3,$size3]);
41  }
42  if ($_FILES['image']['name'][3] !== "") {
43    $filename4 = $_FILES['image']['name'][3];
44    $filetype4 = $_FILES['image']['type'][3];
45    $filecontent4 = file_get_contents($_FILES['image']['tmp_name'][3]);
46    $size4 = $_FILES['image']['size'][3];
47    $columnP = $columnP."sssi";
48    $addSQL = $addSQL.", `image_name4` = ?, `image_type4` = ?, `image_content4` = ?, `image_size4` = ?";
49    $params = array_merge($params ,[$filename4,$filetype4,$filecontent4,$size4]);
50  }
51
52  if($title == '' || $resolution == '' || $detail == '' || $schoolyear == '' || $subject == '' || $keyword == ''){
53    $error_flag = true;
54  }
55
56  $sqlFinal = $sqlBase.$addSQL." WHERE id = ?;";
57  $columnP = $columnP.'s';
58  $params[] = $id;
59  // echo $columnP.'<br>'.'<br>';　各変数を確認
60  // echo $sqlFinal.'<br>'.'<br>';
61  // echo count($params);
62  // exit();
63
64  if(!$error_flag){
65    if ($stmt = mysqli_prepare($link, $sqlFinal)) {
66       mysqli_stmt_bind_param($stmt, $columnP, $params);
67       mysqli_stmt_execute($stmt);
68       $stmt_error = mysqli_stmt_error($stmt);
69       if($stmt_error !== ""){
70         var_dump($stmt_error);
71         exit();
72       }
73       mysqli_stmt_close($stmt);
74    } else {
75      echo 'dame';
76      exit();
77    }
78  }

行動規範の内容に同意します

回答1件

ベストアンサー

mysqli_stmt_bind_param($stmt, $columnP, $params);

PHP: mysqli_stmt::bind_param - Manual

手続き型のmysqli_stmt_bind_paramはパラメータを引数として羅列しないと行けないですが理解されていますか？
汎用性をあげるためにはcall_user_func_array() などで工夫が必要です

投稿2022/05/09 08:17

yambejp

総合スコア114829

Fujiman

2022/05/09 09:03

いえ、理解していませんでした。ここで配列を展開しないといけないのですね。展開するのをやってみます。ありがとうございます！

Fujiman

2022/05/26 02:48

すみません、解決したつもりだったのですが、解決していませんでした。 mysqli_stmt_bind_para関数に渡すパラメーターをimplodeするとイケルと思ったのですが相変わらず数が合わないというエラーになります。 $columnP,の文字数と$paramsの要素数は一致しています。 mysqli_stmt_bind_param($stmt, $columnP, implode(',', $params)); これだとどうしていけないのでしょうか？

行動規範の内容に同意します