エラーの理由がわかりません。

出ているエラーコードは、下記の通りです。
Parse error: syntax error, unexpected '$sql' (T_VARIABLE) in C:\xampp\htdocs\update.php on line 6

このSQL文のどこに問題があるのでしょうか？
6行目
update table名 set workday=?, workgroup=?, personal=?, product=?, createcnt=?, unit=?, bestbydata=?, remarks=?, worktime=?, breaktime=? where id=?

edit.php(特に問題なし)
1<head>
2  <link rel="stylesheet" href="css/edit.css" type="text/css">
3  <link href="https://fonts.googleapis.com/css?family=M+PLUS+1p&display=swap" rel="stylesheet">
4  <title>編集画面</title>
5</head>
6<?php
7$pdo = new PDO('mysql:host=localhost;dbname=workinfo;charset=utf8','root','root');
8?>
9<table border ="1" class="table">
10<tr>
11<th>ID</th>
12<th>作業日</th>
13<th>グループ</th>
14<th>担当者</th>
15<th>製造製品</th>
16<th>生産数</th>
17<th>単位</th>
18<th>賞味期限</th>
19<th>備考</th>
20<th>就業時間</th>
21<th>休憩時間</th>
22</tr>
23<?php foreach ($pdo ->query('SELECT * FROM ogaki')as $row){ // where workdayで抽出条件を設定する
24  echo '<tr>';
25  echo '<form action="update.php" method="post">';
26  echo '<input type="hidden" name="id" value="',$row['id'],'">';
27  echo '<td>',$row['id'],'</td>';
28  echo '<td>','<input type ="date" name="workday" value="',$row['workday'],'">','</td>';//dateを使うなら、SQLの前でreplace
29  echo '<td><input type ="text" name="workgroup" value="',$row['workgroup'],'"></td>';
30  echo '<td><input type ="text" name="personal" value="',$row['personal'],'"></td>';
31  echo '<td><input type ="text" name="product" value="',$row['product'],'"></td>';
32  echo '<td><input type ="text" name="createcnt" value="',$row['createcnt'],'"></td>';
33  echo '<td><input type ="text" name="unit" value="',$row['unit'],'"></td>';
34  echo '<td><input type ="text" name="bestbydata" value="',$row['bestbydata'],'"></td>';
35  echo '<td><input type ="text" name="remarks" value="',$row['remarks'],'"></td>';
36  echo '<td><input type ="text" name="worktime" value="',$row['worktime'],'"></td>';
37  echo '<td><input type ="text" name="breaktime" value="',$row['breaktime'],'"></td>';
38  echo '<td><input type="submit" value="更新"></td>';
39  echo '</form>';
40  echo '</tr>';
41}
42?>
43</table>
44

update.php
1<?php
2$pdo = new PDO('mysql:host=localhost;dbname=workinfo;charset=utf8','root','root');
3
4var_dump()
5//IDを基に各項目の値を更新する
6$sql = $pdo -> prepare('update ogaki set workday=?, workgroup=?, personal=?, product=?, createcnt=?, unit=?, bestbydata=?, remarks=?, worktime=?, breaktime=? where id=?');
7if (empty($_REQUEST['workday'])){
8  echo '日付はYYYY-MM-DDの形で入力してください。';
9}elseif(!preg_match('/[0-9]+/',$_REQUEST['createcnt'])){
10    echo '生産数は整数で入力してください。';
11}elseif($sql -> execute($_REQUEST['workday'],$_REQUEST['workgroup'],$_REQUEST['personal'],$_REQUEST['product']
12,$_REQUEST['createcnt'],$_REQUEST['unit'],$_REQUEST['bestbydata'],$_REQUEST['remarks'],$_REQUEST['worktime'],
13$_REQUEST['breaktime'],$_REQUEST['id'])
14){
15  echo '更新に成功';
16}else{
17  echo '更新に失敗';
18}
19?>
20

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

回答3件

php
1var_dump()

↓

php
1var_dump();

投稿2019/12/20 09:28

退会済みユーザー

総合スコア0

このSQL文のどこに問題があるのでしょうか？

その上のvar_dump()に;がありません。

投稿2019/12/20 09:28

dice142

総合スコア5158

おす！

var_dump()

これの最後にセミコロンがねぇから、あとに続く$sqlまでずっと１文になっちまってるだけだぞ。

投稿2019/12/20 09:27

退会済みユーザー

総合スコア0

退会済みユーザー

2019/12/20 09:32

すいません。自分で書いておきながら、なんとも情けない質問となってしまいました。本来出ていたエラーはこちらでした。(そのエラーを見る為にvar_dump()を使用して付け忘れか、消し忘れてしまいました。) Warning: PDOStatement::execute() expects at most 1 parameter, 11 given in C:\xampp\htdocs\update.php on line 12 更新に失敗恐らく引数と項目数？があっていないというエラーであっておりますでしょうか？

退会済みユーザー

2019/12/20 09:35

おす！ execute()に直接パラメタ渡したらだめだと思うぞ。パラメタはbindParam()かbindValue()でバインドすんだ。 https://qiita.com/_dozen_/items/f6239cf1bdab6f8b0026

退会済みユーザー

2019/12/21 03:13

goku59さんありがとうございます。ご指摘の通りにコードを修正してみましたが、こんな感じですかね？元々見ている本ではexecute()の中にそのままパラメータが記載されていましたので、それに乗っ取って組んでいました。 $sql = "Update ogaki set (workday, workgroup, personal, product, createcnt, unit, bestbydata, remarks, worktime, breaktime)Values(?,?,?,?,?,?,?,?,?,?) where id=?"; $stmt= $pdo -> prepare($sql); $stmt -> bindParam(1,$_REQUEST['workday'],PDO::PARAM_STR);

退会済みユーザー

2019/12/21 03:32

上のもだめでしたので、これで組んでみましたがだめでした。 $sql = "UPDATE ogaki (workday, workgroup, personal, product, createcnt, unit, bestbydata, remarks, worktime, breaktime)Values(:workday, :workgroup, :personal, :product, :createcnt, :unit, :bestbydata, :remarks, :worktime, :breaktime) where id=? $stmt -> bindParam(':workday',$_REQUEST['workday'],PDO::PARAM_STR); これもだめでした。

退会済みユーザー

2019/12/21 03:37

$sql = "Update ogaki set (workday, workgroup, personal, product, createcnt, unit, bestbydata, remarks, worktime, breaktime)Values(?,?,?,?,?,?,?,?,?,?) where id=?"; $stmt -> bindParam(1,$_REQUEST['workday'],PDO::PARAM_STR); これだとエラーは出ませんが、更新は出来ていませんでした。

退会済みユーザー

2019/12/21 09:02

おす！あのよう、UPDATE文、そんな書き方できねぇんじゃねぇか？ SQLは方言あっから断言しずれえんだけんどもよう。今、どんなエラー出てんだ？

退会済みユーザー

2019/12/21 09:02

bindParamは、バインドするパラメタ、全部欠かなきゃダメだぞ。

退会済みユーザー

2019/12/21 09:07

ちなみにな、 Warning: PDOStatement::execute() expects at most 1 parameter, 11 given in これの意味は、「execute()は最大で1つの引数しか受け付けませんが11個渡されました」って書いてあってな、 execute()にバインドパラメタを渡す場合は配列１個にしなきゃなんねぇんだ。だけんども、そうやって渡すとどのプレースホルダとどの変数が対応してっか、実装してる本人でもわからなくなるから、１つ１つちゃんとバインドしなきゃだめじゃねぇかって書いたんだけんども teratailはアホばっか居っから意味がわからなくて低評価したバカが居るみてぇだ。

退会済みユーザー

2019/12/21 09:08

とりあえず、UPDATE文を治すのと、名前付きプレースホルダ使ってどのプレースホルダにどの変数をバインドしてっか、自分でわかるようにコード書いた方がいいと、おら、思うぞ。

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

あなたの回答