PHPでデータベースを更新したいが、更新できない。

前提・実現したいこと

投稿の編集機能を実装したい。

発生している問題・エラーメッセージ

プログラミング初心者です。
上記のように投稿の編集ページを作成しているのですが、下記のように投稿の編集がうまくいっておりません。

・formが送信されているはずだが、データベースのデータをUPDATEできない。
（データが更新されないまま、fromタグのアクション属性で指定しているlist.phpへページ遷移してしまう。）

実際の画面はこちらです。
queryメソッドでidに紐づいたデータの読み込みまではできています。

いろいろネット上で調べてみたものの、うまく解決できませんでした。
わかる方がいらっしゃれば、ご教授いただけると幸いです。

よろしくお願いいたします。

該当のソースコード

php
1<?php
2
3session_start();
4require('../../app/dbconnect.php');
5
6if(isset($_SESSION['id']) && $_SESSION['time'] + 86400 > time()) {
7  $_SESSION['time'] = time();
8
9  // list.phpからGETメソッドでデータ送信がうまくいっているかをチェック
10  if(!empty($_GET['book_id'])) {
11    $book_id = htmlspecialchars($_GET['book_id'], ENT_QUOTES);
12    
13    // データベースからデータを読み込む
14    $book = $db->query("SELECT * FROM books WHERE id = $book_id");
15    if($book) {
16      $book_data = $book->fetch(); //連想配列で取得
17    } else {
18      header('Location: list.php');
19    }
20  }
21
22  // 以下、データ編集
23
24  // POST形式でbook_idが送信されているかどうか
25  if(!empty($_POST['book_id'])) {
26    $book_id = print(htmlspecialchars($_POST['book_id']));
27
28    //画像アップのエラー確認
29    $fileName = $_FILES['image']['name'];
30    if(!empty($fileName)) {
31      $ext = substr($fileName, -3);
32      if($ext != 'jpg' && $ext != 'png' && $ext != 'gif') {
33        $error['image'] = 'type';
34      }
35    }
36
37    //エラーがなければデータを更新
38    if(empty($error)) {
39
40      //画像ファイルの保存処理
41      $image = date('YmdHis') . $_FILES['image']['name'];
42      move_uploaded_file($_FILES['image']['tmp_name'], '../book_image/' . $image);
43      $_POST['image'] = $image;
44      $_SESSION['image'] = $image;
45
46      // データベースのデータを更新
47      $stmt = $db->prepare("UPDATE books SET title=?, text=?, image=? WHERE id=$book_id");
48      $stmt->execute(array(
49        $_POST['title'],
50        $_POST['text'],
51        $_POST['image']
52      ));
53    }
54  }
55
56} else {
57  header('Location: ../login.php');
58}
59
60?>
61
62<!DOCTYPE html>
63<html lang="en">
64<head>
65  <meta charset="UTF-8">
66  <meta http-equiv="X-UA-Compatible" content="IE=edge">
67  <meta name="viewport" content="width=device-width, initial-scale=1.0">
68  <title>本　編集ページ</title>
69</head>
70<body>
71  <div class="global-container">
72    <form action="list.php" method="post" enctype="multipart/form-data">
73      <input type="hidden" name="book_id" value="<?php print(htmlspecialchars($book_data['id'], ENT_QUOTES)); ?>">
74      <header class="header">
75        <div class="header__inner">
76          <div class="header__inner__cancel">
77            <a href="list.php">
78              <div class="header__inner__cancel__btn">キャンセル</div>
79            </a>
80          </div>
81          <div class="header__inner__save">
82            <input type="submit" value="保存">
83          </div>
84        </div>
85      </header>
86      <section class="contents">
87        <div class="content">
88          <div class="content__image">
89            <p>本の画像</p>
90            <img src="../book_image/<?php print(htmlspecialchars($book_data['image'], ENT_QUOTES)); ?>" width=100>
91            <input type="file" name="image">
92            <?php if($error['image'] === 'type'): ?>
93              <p class="error">*画像は「.jpg」「.png」「.gif」のいずれかを指定して下さい。</p>
94            <?php endif; ?>
95          </div>
96          <div class="content__title">
97            <p>本のタイトル</p>
98            <input type="text" name="title" maxlength="255" value="<?php print(htmlspecialchars($book_data['title'], ENT_QUOTES)); ?>">
99          </div>
100          <div class="content__text">
101            <p>こだわりポイント</p>
102            <textarea name="text" maxlength="500" id="" cols="30" rows="10"><?php print(htmlspecialchars($book_data['text'], ENT_QUOTES)); ?></textarea>
103          </div>
104        </div>
105      </section>
106    </form>
107  </div>
108</body>
109</html>

補足情報（FW/ツールのバージョンなど）

開発環境はMAMPを使用しております。

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

回答3件

<form action="list.php" ～は、list.phpへデータを送りますが、
提示されているPHPコードは、list.phpなのでしょうか？

このedit.phpにアクセスするためには、そのsessionまわりを通過しないといけないようにしており、問題なくedit.phpにアクセスできるような形になっております。

『このedit.php』とあるので、提示したコードがedit.phpであれば、
<form action="edit.php" ～と記載するべきのように思います。

投稿2021/04/17 05:14

satokei

総合スコア1217

yuya_12345678

2021/04/18 12:39

ありがとうございます。この提示したコードは「edit.php」です。おっしゃる通り、フォームの送信先が間違っておりましたので、「edit.php」に変更して試してみました。これでうまくいくと思ったのですが、うまくいきませんでした。付け焼き刃の知識で考えてみたのですが、formをedit.phpに送信した時に、下記（データ読み込み）が先に実行されてしまっている関係で、データ更新がうまくいっていないのかもしれません。 if(!empty($_POST['book_id'])) { $book_id = print(htmlspecialchars($_POST['book_id'])); //画像アップのエラー確認 $fileName = $_FILES['image']['name']; if(!empty($fileName)) { $ext = substr($fileName, -3); if($ext != 'jpg' && $ext != 'png' && $ext != 'gif') { $error['image'] = 'type'; } } この場合、新しくupdate.phpのようなデータ更新専用のファイルを用意した方が良いのでしょうか、、、お手数おかけしますが、ご確認いただけますと幸いです。

satokei

2024/04/13 18:57 編集

エラーが発生していないか、どこまで期待通りに実行されているかなど、ご自身でデバッグをした方がよろしいかと思います。（あまりデバッグする気はしないです）あと別で気になったのは、 Update文で「$book_id」もバインド変数にした方がよいです。

yuya_12345678

2021/04/19 03:37

ありがとうございます。 $book_idもバインド変数に変更します。また、本事象が解決しました。エラーの原因は、「打ち間違え」でした。いろいろご質問にご対応いただき、誠にありがとうございました！

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

自己解決

本事象ですが、文字の打ち間違えによるエラーでした。
正しく記載した結果、問題なく動作しました。

また、本件から、１コードずつ正確に確認しながら進めることの大切さを、感じました。
今後は、1コードごとに確認しながら進めていきたいと思います。

いろいろと、ありがとうございました。

投稿2021/04/19 03:39

yuya_12345678

総合スコア0

ぱっと見で気になった箇所としては

if(isset($_SESSION['id']) && $_SESSION['time'] + 86400 > time()) {
  $_SESSION['time'] = time();

が最初はどうやってもくぐり抜けることができないんじゃないでしょうか？　まずはこの一文を外してチェックしてみたらどうでしょう。session周りがきちんと実装されていない気がしますし（$_SESSION['id']や$_SESSION['time']はどの辺で処理してるのでしょう）、sessionを使うなら実装はもうちょいしっかり考えないとダメな気がします。もしかしてこのソース外で処理しているなら他の原因かもしれませんのでごめんなさいです。

投稿2021/04/17 01:53

AbeTakashi

総合スコア4853

yuya_12345678

2021/04/17 02:07

ありがとうございます。 sessionに関しては、ログイン機能を別ページにて実装している関係で存在しています。 $_SESSION[‘id’]、$_SESSION[‘time’]は別ページで実装しています。このedit.phpにアクセスするためには、そのsessionまわりを通過しないといけないようにしており、問題なくedit.phpにアクセスできるような形になっております。ただ、sessionがデータベースへのアクセスに何かしらの影響があるのかどうかもチェックしてみたいので、おっしゃっていただいた形で確認してみます。ありがとうございます。一応、

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

あなたの回答