phpからsql の insertエラー

本での勉強が一周し、ほんの少しアレンジに挑戦している初心者です。
予約テーブルへの登録ができません。エラーの意味も引数がなんとかという意味もよく分かりません。
3時間くらい色々調べてもわからなく心が折れそうなので何が原因なのか教えていただきたいです。

$pdo = new PDO　～～　は消してあります。

コードif (isset($_POST)){
  
//$id = $_SESSION['customer']['kaiin_id']; //顧客マスタに登録済の会員ID
//$date =  $_REQUEST['day'];  //予約した時間
//$time =  $_REQUEST['time0']; //半角スペース(&nbsp;)が入った日時文字列  "2022-03-25&nbsp;17:00"
// echo $date;
echo '<br>';
// echo $time;
echo '<br>';

                                                        
$sql = $pdo->prepare('INSERT INTO reserve values(null.?.?.?)');

$sql->execute($_SESSION['customer']['kaiin_id'],$_REQUEST['day'],$_REQUEST['time0']);

//$sql->execute($id,$date,$time);　ただ短くしてみた

}

エラー文
Fatal error: Uncaught ArgumentCountError: PDOStatement::execute() expects at most 1 argument, 3 given in

executeの行に問題がある様なのですがわかりません。

maisumakun

2022/03/08 14:49

> executeの行に問題がある様なのですがわかりません。英語を和訳してみましょう。

111ttt

2022/03/08 15:18

すみません、いつもまず和訳するようにしているんですが execute()は最大で1つの引数を期待し、３つは…… 意味がわかりません。 js php のエラーは分かるようになってきたんですが、sqlのエラーはまだ理解できません

ebal4

2022/03/08 16:05

SQLのエラーではなくPHPのエラーですね

111ttt

2022/03/10 04:07

そうなんですね、それも分からないとは…重症です

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

回答1件

ベストアンサー

$sql->execute($_SESSION['customer']['kaiin_id'],$_REQUEST['day'],$_REQUEST['time0']);
↓
$sql->execute([$_SESSION['customer']['kaiin_id'],$_REQUEST['day'],$_REQUEST['time0']]);
とすればとりあえずこのエラーは解消するかと思います。

少し冗長に書くと以下の様にも書くことが出来ます。

PHP
1$reserve_values = [$_SESSION['customer']['kaiin_id'],$_REQUEST['day'],$_REQUEST['time0']];
2$sql->execute($reserve_values);
3

引数を配列にまとめてPDOStatement::execute()に渡している訳です。

（その後、インサート分のエラーが出ると思われるのでそちらはそちらで解決してみてください。.と,は違うものです）

原因

Fatal error: Uncaught ArgumentCountError: PDOStatement::execute() expects at most 1 argument, 3 given in
は許されている引数（１つ）より多くの引数（３つ）を与えているというPHPのエラーです。

$sql->execute($_SESSION['customer']['kaiin_id'],$_REQUEST['day'],$_REQUEST['time0']);
を見ると、$_SESSION['customer']['kaiin_id'],$_REQUEST['day'],$_REQUEST['time0']と、三つの引数を与えています。
PDOStatement::execute()は一つの配列を引数として受け取るメソッドなので、単に三つの引数を与えるとエラーが発生します。

調べ方

全部のメソッドの仕様を暗記する必要は無く、不明なメソッドや関数があったら
PHPマニュアル（公式リファレンス）で調べるのが最も確実です。
（分かりやすい風の残念なページも多いので、まずは公式リファレンスで調べるのがベストです）
PDOStatement::execute() site:www.php.netで検索すると
PHPマニュアル(公式リファレンス）の該当ページにたどり着くので、パラメータの部分を確認してみてください。


パラメータ ¶
params
実行される SQL 文の中のバインドパラメータと同数の要素からなる、 値の配列。すべての値は PDO::PARAM_STR として扱われます。

ひとつのパラメータに対して複数の値をバインドすることはできません。 例えば、IN() 句の中のひとつのパラメータに対して 2 つの値をバインドすることはできません。

指定した数よりも多い値をバインドすることはできません。 params のキーが PDO::prepare() で指定した SQL にある数より多い場合は、 ステートメントが失敗してエラーが発生します。

と記述があります。
今回のエラーは
実行される SQL 文の中のバインドパラメータと同数の要素からなる、値の配列。のところを無視して引数を三つ与えていることが原因だと分かります。

また、サンプルコードもよく読んでみましょう。
今回のケースだと

例3 位置を指定した値を伴うプリペアドステートメントの実行

PHP
1<?php
2/* 入力値の配列を伴うプリペアドステートメントの実行 */
3$calories = 150;
4$colour = 'red';
5$sth = $dbh->prepare('SELECT name, colour, calories
6    FROM fruit
7    WHERE calories < ? AND colour = ?');
8$sth->execute(array($calories, $colour));
9?>
10