回答編集履歴

追記

2019/02/14 03:24

投稿

m.ts10806

score213

test CHANGED Viewed

@@ -11,3 +11,215 @@
 原因は全く一緒ではないにしろ、原因の探し方は同じだと思います。
+------
+コメントしたとおり、提示コードでは同エラーは起きません。
+ただ、インデントがかなり荒れていて読みづらいので、整理したうえで、気になるところをコメントしてみました。
+```php
+<?php
+$named=$_POST['content'];
+$commented=$_POST['content2'];
+$deleted=$_POST['number'];
+$edited=$_POST['edit'];
+$editnumber=$_POST['content3']; //mts10806 取得してるけどどこにも使ってない
+$passed=$_POST['pass'];
+$pass2=$_POST['pass2'];
+$pass3=$_POST['pass3'];
+$timed = date('Y/n/d H:i:s');
+// データベース接続
+$dsn = 'mysql:dbname=test;host=localhost';
+$user = 'root';
+$password = '';
+$pdo = new PDO($dsn, $user, $password, array(
+    PDO::ATTR_ERRMODE => PDO::ERRMODE_WARNING
+));
+//mts10806 DBに関する処理を行うときはきちんとtry-catchしてException(今回はPDOException)を捕捉するようにしましょう
+// テーブル作成
+$sql = "CREATE TABLE IF NOT EXISTS mission_4_1" . "(" . "id INT auto_increment," . "name char(32)," . "comment TEXT," . "time DATETIME," . "pass INT," . "primary key(id)" . ");";
+$stmt = $pdo->query($sql);
+// テーブル作成確認
+$sql1 = 'SHOW TABLES';
+$result = $pdo->query($sql1);
+foreach ($result as $row) {
+    echo $row[0];
+    echo '<br>';
+}
+echo "<hr>";
+// テーブル 中身 確認
+$sql2 = 'SHOW CREATE TABLE mission_4_1';
+$result = $pdo->query($sql2);
+foreach ($result as $row1) {
+    echo $row1[1];
+}
+echo "<hr>";
+$sql3 = $pdo->prepare("INSERT INTO mission_4_1(id,name,comment,time,pass) VALUES(:id,:name,:comment,:time,:pass)");
+//mts10806 auto_increment設定しているならinsertにidは基本不要
+$sql3->bindParam(':name', $name, PDO::PARAM_STR);
+$sql3->bindParam(':comment', $comment, PDO::PARAM_STR);
+$sql3->bindValue(':time', $time, PDO::PARAM_STR);
+$sql3->bindValue(':pass', $pass, PDO::PARAM_INT);
+//mts10806 どっちかに統一を。bindValueの方がのぞましい。
+$name = $named; //mts10806 ここで定義してもbindParam,BindValueで使うことにはならない
+$comment = $commented;
+$time = $timed;
+$pass = $passed;
+$sql3->execute(); //mts10806 SQLで用意したパラメータとセットしたパラメータの数があわないのでエラーでますよ。
+if (! empty($pass3)) {
+    $sql7 = "SELECT*FROM mission_4_1 WHERE id=$edited";
+    $stmt = $pdo->query($sql7);
+    foreach ($stmt as $row) {
+        if ($row['id'] == $edited and $row['pass'] == $pass3) {
+            $id = "$edited";
+            $name = "$named";
+            $comment = "$commented";
+            $sql4 = 'update mission_4_1 set name=:name,comment=:comment where id=:id';
+            $stmt = $pdo->prepare($sql4);
+            $stmt->bindParam(':name', $name, PDO::PARAM_STR);
+            $stmt->bindParam(':comment', $comment, PDO::PARAM_STR);
+            $stmt->bindParam(':id', $id, PDO::PARAM_INT);
+            $stmt->execute();
+        }
+    }
+} elseif (! empty($pass2)) {
+    $sql8 = "SELECT*FROM mission_4_1 WHERE id=$deleted";
+    $stmt = $pdo->query($sql8);
+    foreach ($stmt as $row) {
+        if ($row['id'] == $deleted) {
+            $id = "$deleted";
+            $sql5 = 'delete from mission_4_1 where id=:id';
+            $stmt = $pdo->prepare($sql5);
+            $stmt->bindParam(':id', $id, PDO::PARAM_INT);
+            $stmt->execute();
+        }
+    }
+}
+$sql6 = 'SELECT*FROM mission_4_1';
+$stmt = $pdo->query($sql6);
+$results = $stmt->fetchAll();
+foreach ($results as $row2) {
+    // $rowの中にはテーブルのカラム名が入る
+    echo $row2['id'] . ',';
+    echo $row2['name'] . ',';
+    echo $row2['comment'] . ',';
+    echo $row2['time'] . '<br>';
+}
+?>
+```

追記

2019/02/14 03:07

投稿

m.ts10806

score213

test CHANGED Viewed

@@ -11,221 +11,3 @@
 原因は全く一緒ではないにしろ、原因の探し方は同じだと思います。
-------
-コメントしたとおり、提示コードでは同エラーは起きません。
-ただ、インデントがかなり荒れていて読みづらいので、整理したうえで、気になるところをコメントしてみました。
-```php
-<?php
-$named=$_POST['content'];
-$commented=$_POST['content2'];
-$deleted=$_POST['number'];
-$edited=$_POST['edit'];
-$editnumber=$_POST['content3']; //mts10806 取得してるけどどこにも使ってない
-$passed=$_POST['pass'];
-$pass2=$_POST['pass2'];
-$pass3=$_POST['pass3'];
-$timed = date('Y/n/d H:i:s');
-// データベース接続
-$dsn = 'mysql:dbname=test;host=localhost';
-$user = 'root';
-$password = '';
-$pdo = new PDO($dsn, $user, $password, array(
-    PDO::ATTR_ERRMODE => PDO::ERRMODE_WARNING
-));
-//mts10806 DBに関する処理を行うときはきちんとtry-catchしてException(今回はPDOException)を捕捉するようにしましょう
-// テーブル作成
-$sql = "CREATE TABLE IF NOT EXISTS mission_4_1" . "(" . "id INT auto_increment," . "name char(32)," . "comment TEXT," . "time DATETIME," . "pass INT," . "primary key(id)" . ");";
-$stmt = $pdo->query($sql);
-// テーブル作成確認
-$sql1 = 'SHOW TABLES';
-$result = $pdo->query($sql1);
-foreach ($result as $row) {
-    echo $row[0];
-    echo '<br>';
-}
-echo "<hr>";
-// テーブル 中身 確認
-$sql2 = 'SHOW CREATE TABLE mission_4_1';
-$result = $pdo->query($sql2);
-foreach ($result as $row1) {
-    echo $row1[1];
-}
-echo "<hr>";
-//mts10806 変数名や関数名にマジックナンバーは厳禁。何を保管しているものか、何をするものかわからない。prepareの返りはstatementなのでsqlという変数名もナンセンスです。
-$sql3 = $pdo->prepare("INSERT INTO mission_4_1(id,name,comment,time,pass) VALUES(:id,:name,:comment,:time,:pass)");
-//mts10806 auto_increment設定しているならinsertにidは基本不要
-$sql3->bindParam(':name', $name, PDO::PARAM_STR);
-$sql3->bindParam(':comment', $comment, PDO::PARAM_STR);
-$sql3->bindValue(':time', $time, PDO::PARAM_STR);
-$sql3->bindValue(':pass', $pass, PDO::PARAM_INT);
-//mts10806 どっちかに統一を。bindValueの方がのぞましい。
-$name = $named; //mts10806 ここで定義してもbindParam,BindValueで使うことにはならない
-$comment = $commented;
-$time = $timed;
-$pass = $passed;
-$sql3->execute(); //mts10806 SQLで用意したパラメータとセットしたパラメータの数があわないのでエラーでますよ。
-if (! empty($pass3)) {
-    $sql7 = "SELECT*FROM mission_4_1 WHERE id=$edited";
-    $stmt = $pdo->query($sql7);
-    foreach ($stmt as $row) {
-        if ($row['id'] == $edited and $row['pass'] == $pass3) {
-            $id = "$edited";
-            $name = "$named";
-            $comment = "$commented";
-            $sql4 = 'update mission_4_1 set name=:name,comment=:comment where id=:id';
-            $stmt = $pdo->prepare($sql4);
-            $stmt->bindParam(':name', $name, PDO::PARAM_STR);
-            $stmt->bindParam(':comment', $comment, PDO::PARAM_STR);
-            $stmt->bindParam(':id', $id, PDO::PARAM_INT);
-            $stmt->execute();
-        }
-    }
-} elseif (! empty($pass2)) {
-//mts10806 なんでこの前までbindしてきてるのになぜここでやらない・・
-    $sql8 = "SELECT*FROM mission_4_1 WHERE id=$deleted";
-    $stmt = $pdo->query($sql8);
-    foreach ($stmt as $row) {
-        if ($row['id'] == $deleted) {
-            $id = "$deleted";
-            $sql5 = 'delete from mission_4_1 where id=:id';
-            $stmt = $pdo->prepare($sql5);
-            $stmt->bindParam(':id', $id, PDO::PARAM_INT);
-            $stmt->execute();
-        }
-    }
-}
-$sql6 = 'SELECT*FROM mission_4_1';
-$stmt = $pdo->query($sql6);
-$results = $stmt->fetchAll();
-foreach ($results as $row2) {
-    // $rowの中にはテーブルのカラム名が入る
-    echo $row2['id'] . ',';
-    echo $row2['name'] . ',';
-    echo $row2['comment'] . ',';
-    echo $row2['time'] . '<br>';
-}
-?>
-```