update の処理が deleteの処理になってしまう

こんにちは。

PHP&MYSQL入門の
TODO LISTを作るためのコードを参照して、
自分なりにUPDATE処理を書いたのですが、それがDELETEの動きをして、データベース上からも消えてしまいます。
一体何が原因ですか？

↓書き換え前のコード（上記のサイトにまるまる書いてあったコード）

lang
1<?php
2    $pdo = new PDO("mysql:host=localhost;dbname=mydb;charset=utf8","root","", [PDO::ATTR_ERRMODE => PDO::ERRMODE_WARNING]);
3
4 if(isset($_POST['submit']) ){
5        $name = $_POST['name'];
6        $sth = $pdo->prepare("INSERT INTO todos (name) VALUES (:name)");
7        $sth->bindValue(':name', $name, PDO::PARAM_STR);
8        $sth->execute();
9    }elseif(isset($_POST['delete'])){
10        $id = $_POST['id'];
11        $sth = $pdo->prepare("delete from todos where id = :id");
12        $sth->bindValue(':id', $id, PDO::PARAM_INT);
13        $sth->execute();
14    }
15?>
16
17<!DOCTYPE HTML>
18<html lang="ja">
19<head>
20    <title>Todo List</title>
21    <link rel="stylesheet" href="//fonts.googleapis.com/css?family=Roboto:300,300italic,700,700italic">
22    <link rel="stylesheet" href="//cdn.rawgit.com/necolas/normalize.css/master/normalize.css">
23    <link rel="stylesheet" href="//cdn.rawgit.com/milligram/milligram/master/dist/milligram.min.css">
24</head>
25
26<body class="container">
27    <h1>Todo List</h1>
28    <form method="post" action="">
29        <input type="text" name="name" value="">
30        <input type="submit" name="submit" value="Add">
31    </form>
32    <h2>Current Todos</h2>
33    <table class="table table-striped">
34        <therad><th>Task</th><th></th></therad>
35        <tbody>
36<?php
37    $sth = $pdo->prepare("SELECT * FROM todos ORDER BY id DESC");
38    $sth->execute();
39    
40    foreach($sth as $row) {
41?>
42            <tr>
43                <td><?= htmlspecialchars($row['name']) ?></td>
44                <td>
45                    <form method="POST">
46                        <button type="submit" name="delete">Delete</button>
47                        <input type="hidden" name="id" value="<?= $row['id'] ?>">
48                        <input type="hidden" name="delete" value="true">
49                    </form>
50                </td>
51            </tr>
52<?php
53    }
54?>
55        </tbody>
56    </table>
57</body>
58</html>

↓書き換え後のコード（私がさらにupdate処理だけかいたコード）

lang
1<?php
2    $pdo = new PDO("mysql:host=localhost;dbname=mydb;charset=utf8","root","", [PDO::ATTR_ERRMODE => PDO::ERRMODE_WARNING]);
3
4    if(isset($_POST['submit']) ){
5        $name = $_POST['name'];
6        $sth = $pdo->prepare("INSERT INTO todos (name) VALUES (:name)");
7        $sth->bindValue(':name', $name, PDO::PARAM_STR);
8        $sth->execute();
9    }elseif(isset($_POST['delete'])){
10        $id = $_POST['id'];
11        $sth = $pdo->prepare("delete from todos where id = :id");
12        $sth->bindValue(':id', $id, PDO::PARAM_INT);
13        $sth->execute();
14    }　　　　　　　　　　　
15/// 書いた処理1
16elseif(isset($_POST['update'])){
17
18        $sql = 'update todos set name = :name where id = :id';
19        $sth= $pdo->prepare($sql);
20        $add = $_POST['add'];
21       
22        $sth->bindParam(':name', $add, PDO::PARAM_STR);
23        $sth->execute();
24        $id = $_POST['id'];
25        $sth->bindValue(':id', $id, PDO::PARAM_INT);
26        
27    }
28?>
29
30<!DOCTYPE HTML>
31<html lang="ja">
32<head>
33    <title>Todo List</title>
34    <link rel="stylesheet" href="//fonts.googleapis.com/css?family=Roboto:300,300italic,700,700italic">
35    <link rel="stylesheet" href="//cdn.rawgit.com/necolas/normalize.css/master/normalize.css">
36    <link rel="stylesheet" href="//cdn.rawgit.com/milligram/milligram/master/dist/milligram.min.css">
37</head>
38
39<body class="container">
40    <h1>Todo List</h1>
41    <form method="post" action="">
42        <input type="text" name="name" value="">
43        <input type="submit" name="submit" value="Add">
44    </form>
45    <h2>Current Todos</h2>
46    <table class="table table-striped">
47        <therad><th>Task</th><th></th></therad>
48        <tbody>
49<?php
50    $sth = $pdo->prepare("SELECT * FROM todos ORDER BY id DESC");
51    $sth->execute();
52    
53    foreach($sth as $row) {
54?>
55            <tr>
56                <td><?= htmlspecialchars($row['name']) ?></td>
57                <td>
58                    <form method="POST">
59                        <button type="submit" name="delete">Delete</button>
60                        <button type="submit" name="update">update</button>
61                        <input type="hidden" name="id" value="<?= $row['id'] ?>">
62                  ///書いた処理２
63                            <input type="text" name="add" value="">
64                            <input type="hidden" name="id" value="<?= $row['id'] ?>">
65
66                  
67                    </form>
68                </td>
69            </tr>
70<?php
71    }
72?>
73        </tbody>
74    </table>
75</body>
76</html>

いま、独学で勉強しているので、みにくい部分があるかもしれません。すみません。
もし応えてくださることがあれば、とてもうれしいです。

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

回答2件

submitというnameのパラメータ投げてないですよね？

投稿2020/03/12 00:26

yambejp

総合スコア116724

yambejp

2020/03/12 00:26

<?PHP $submit=filter_input(INPUT_POST,"submit"); if(!is_null($submit)){ switch($submit){ case "delete"; print "del"; break; case "update"; print "upd"; break; default: print "xxx"; break; } } ?> <form method="POST"> <button type="submit" name="submit" value="delete">Delete</button> <button type="submit" name="submit" value="update">update</button> <button type="submit" name="submit" value="">other</button> </form>