うまくインサートができません

前提・実現したいこと

ここに質問の内容を詳しく書いてください。

登録画面を作成しています。記入→確認→ログインの順番で作っているのですがエラーも出ることなくページのリダイレクトはできています　しかし登録後サーバーを確認してもインサートされていません。

ページの先頭でini_set('display_errors',1);としているのですがHTMLのほうではエラーのメッセージは出ません。

何処が原因がわからず質問させていただきました。

回答よろしくお願いします。
■■な機能を実装中に以下のエラーメッセージが発生しました。

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

エラーメッセージ

該当のソースコード

session_start();
ini_set('display_errors',1);

require "function.php";
require "document.php";


 if (!isset($_SESSION['join'])) {
   header('Location:index.php');
   exit();
 }
// singupがクリックされたら
 if(isset($_POST['signup'])) {
   // 登録処理を実行

  $username = mysqli_real_escape_string($dbh,$_SESSION['join']['name']);
  $mail = mysqli_real_escape_string($dbh,$_SESSION['join']['mail']);
  $password = mysqli_real_escape_string($dbh,sha1($_SESSION['join']['pass']));
  $image = mysqli_real_escape_string($dbh,$_SESSION['join']['image']);

  $query = 'INSERT INTO members(name,email,password,picture) VALUES(?,?,?,?)';
  $stmt = $dbh->prepare($query);
  $data[] = $username;
  $data[] = $mail;
  $data[] = $password;
  $data[] = $image;
  $stmt->execute($data);

 $dbh=null;

  header('Location:thanks.php');
  exit();
}


?>

<!DOCTYPE html>
<html lang="ja">
  <head>
    <meta charset="utf-8">
    <title>gymgym</title>
    <link rel="stylesheet" href="styles.css">
  </head>
  <body>
    <header>
      <p>会員登録</p>
    </header>
    <div class="container">
      <form class="_form2" action="" method="post" enctype="multipart/form-data">
        <h1></h1>
        <dl class="">
          <p>ニックネーム  <span class="required"></p>
            <?= h($_SESSION['join']['name']); ?>
          <p>メールアドレス  <span class="required"></p>
              <?= h($_SESSION['join']['mail']); ?>
          <p>パスワード  <span class="required"><p/>
              <span>表示されません</span> <br>
            <?php
            echo ('<img src="./picture/'.$_SESSION['join']['image'].'">')
            ?>
            <br>
              <a href="join_index.php?action=rewrite">&laquo;&nbsp;書き直す</a>
              <input type="submit"  value="登録する" name="signup" class="confirmation">
          </dl>

        </div>
      </form>

  </body>
</html>

試したこと

ini_set('display_errors',1);でエラーのを視ようとしましたがHTMLのほうで何も出ずどこが原因かわからない

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

ここにより詳細な情報を記載してください。

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

回答3件

ベストアンサー

ini_set('display_errors',1);としているのですがHTMLのほうではエラーのメッセージは出ません。

エラーが出てないのなら出ませんし、リダイレクトしているならその過程で処理中断するほどの（Fatalレベル）エラーが出てなければそのままリダイレクトするでしょう。

「thanks.php」は単に「完了ページ」なのでしょうから、単体確認のためにそのリダイレクトはちゃんと機能が動くまでコメントアウトしておきましょう。

php
1//header('Location:thanks.php');

また、DB接続時の処理の基本として、try-catchは入れておきましょう。
特にDBはPHPからすると外部の仕組みなので、その外部の仕組みから返ってくる情報を正確にキャッチしてあげる必要があります。
mysqliもexecptionがあります。 ->mysqli_sql_exception

今回の記述だけだと接続がmysqliなのかPDOなのかが分かりません。
（mysqli_real_escape_string()に引数入れていることからmysqliだろうなとは思いますが）
DB接続関連の問題でしたら、接続部分のコードも記載するようにしてください。

※それによってキャッチするexceptionも違ってきます。

リダイレクトを止めて、try-catchを入れればとりあえず起きている問題は分かるはずです。

あとはinsertしようとしている情報が全て想定通りに来ているかデバッグ確認してください。

蛇足：
既に回答にありますが、prepareで？パラメータにより値をバインドしているのであれば
そのバインド時にSQLエスケープが行われますので、自前で「mysqli_real_escape_string」を導入する必要はありません。

投稿2019/07/08 03:00

編集2019/07/08 03:07

m.ts10806

総合スコア80850

yamamu

2019/07/08 09:30

大変丁寧な回答ありがとうございます。コメントアウトしエラー内容が分かったので無事に動作しましたありがとうございます！。

m.ts10806

2019/07/08 09:32

私は考え方やデバッグ方法のみしか回答していませんので「実際に意図通りの対応がされたか」は今いただいたコメントだけでは分かりません。「結局どのように解決したか」具体的にコメントいただけませんか？コードを伴うようであれば質問本文に追記していただいて構いません。 https://teratail.com/help/question-tips#questionTips4-2

yamamu

2019/07/08 13:33

headerをコメント化したところデータベースの接続をPDOで行っていたのにもエスケープをmysqliで処理していました。エスケープは接続の方法にあわせて行うのですね　単純なミスで申し訳ないです。詳しく書いていただきありがとうございます。

m.ts10806

2019/07/09 00:07

解決されたようで何よりです。

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

回答を読まないでベストアンサーするのは止めましょう。

投稿2019/07/07 06:08

Orlofsky

総合スコア16415

yamamu

2019/07/07 06:15

回答ありがとうございます　前回の質問に答えていただきましてありがとうございます。あの質問はページがリダイレクトしない原因が特定できず質問しましたがリダイレクトに関しては自己解決したため、質問を終わらせるために回答をくださった方にべエストアンサーをつけました。不快な思いをさせてしまい申し訳ございません。もしよろしければ上記の質問の回答をしていただけないでしょうか何卒よろしくお願いします。

Orlofsky

2019/07/07 06:17

わたしの回答を読む気がないようなので無理でしょう。

yamamu

2019/07/07 06:27

次回以降は自己解決したときの解決方法を記載する欄に書きたいと思います。焦っていて自己解決の欄を見落としていました回答ありがとうございました

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

おそらくPDOの質問だと思います。mysqliで処理したいわけではないですよね？
mysqli_real_escape_stringはmysqliライブラリが読まれたときに
使用可能になります、加えてPDOは投入文字をエスケープする必要がないので
同関数を利用する必要はありません

投稿2019/07/08 00:28