新規会員登録後にそのままログインさせたい

前提・実現したいこと

新規でデータベースに会員情報を登録し
再度ログインせずにそのまま
ログインした状態にしたいです。

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

データベースへ情報の登録はされるものの
最終的にはデータベースへの接続エラーとなります。

該当のソースコード

現在の会員登録のコード

<!DOCTYPE html>
<html lang="ja">
<head>
<meta charset="utf-8">
<meta name="viewport" content="width=device-width, initial-scale=1.0">
<title>会員登録画面</title>
</head>
    
<body>
    
<header>
<h1 class="daimei">FREEDOM RECORDS</h1>    
<p id="target">menu</p>
</header>
<warapper>
<main>

<?php
    try{

require_once("../common/common.php");

$post = sanitize($_POST);

$name = $post["name"];
$address = $post["address"];
$tel = $post["tel"];
$email = $post["email"];
$pass = $post["pass"];
        
$pass = md5($pass);
        
$dsn = "mysql:host=localhost;dbname=shop;charset=utf8";
$user = "root";
$password = "";
$dbh = new PDO($dsn, $user, $password);
$dbh -> setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
        
$sql = "SELECT email FROM menber WHERE1";
$stmt = $dbh -> prepare($sql);
$stmt -> execute();
        
while(true) {
    $rec = $stmt -> fetch(PDO::FETCH_ASSOC);
    if(empty($rec) === true) {
        break;
    }
    $mail[] = $rec["email"];
}
        
if(in_array($email, $mail) === true) {
    print "すでに使われているmailアドレスです。<br><br>";
    print "<a href='../shop/shop_list.php'>トップへ戻る</a>";
    $dbh = null;
} else {   
$sql = "INSERT INTO menber(name, email, address, tel, password) VALUES(?,?,?,?,?)";
$stmt = $dbh -> prepare($sql);
$data[] = $name;
$data[] = $email;
$data[] = $address;
$data[] = $tel;
$data[] = $pass;
$stmt -> execute($data);
        
$dbh = null;
        
 
print "登録完了しました。<br><br>";
print "<a href='../shop/shop_list.php'>トップへ戻る</a>";
}
}
catch(Exception $e) {
   print "只今障害が発生しております。";
   print "a href='menber_login.php'>ログインページへ戻る</a>";
   exit();
}
?>

</main>

</warapper>

<script src="../main.js"></script>
<script src="../anime.min.js"></script>
</body>
</html>

試したこと

下記のようにSELECT文を追加

$sql = "INSERT INTO menber(name, email, address, tel, password) VALUES(?,?,?,?,?)";

$stmt = $dbh -> prepare($sql);
$data[] = $name;
$data[] = $email;
$data[] = $address;
$data[] = $tel;
$data[] = $pass;
$stmt -> execute($data);

$sql = "SELECT code, name FROM menber WHERE email=? AND password=?";
$stmt = $dbh -> prepare($sql);
$data[] = $name;
$data[] = $email;
$data[] = $address;
$data[] = $tel;
$data[] = $pass;
$data;
$stmt -> execute($data);

$dbh = null;

ご回答宜しくお願い致します。

m.ts10806

2021/08/30 12:43

>データベースへの接続エラーとなります。でしたら、「登録後にログイン状態に」以前の問題では？いずれにしてもエラーハンドリングしてください。 https://www.php.net/manual/ja/pdo.error-handling.php

shunta80

2021/08/30 12:53

select文を記載しなければ正常に処理されます。上記のコードの書き方は間違えてはいないのでしょうか？

m.ts10806

2021/08/30 20:14

セッションも使ってない状態でどうして「正常に処理される」のか分からないですが、DBはあくまでPHPとは外部の仕組みなのでエラーを正確に取得しようと思ったらエラーハンドリングするしかありません。どこで問題が起きているか、間違っているかを問題切り分けするためのエラーハンドリングです。闇雲に答えだけ求めても解決はしないため、まずは切り分けてそこから次の手を考えましょうと言う話をしています。

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

回答3件

自己解決

本件につきまして
seesionstartを使用することで解決できました。

ご回答いただきありがとうございました。

投稿2021/08/31 16:32

shunta80

総合スコア96

前提

最終的にはデータベースへの接続エラーとなります。

と質問にありますが、これは恐らく誤解です。

多分、

PHP
1
2catch(Exception $e) {
3   print "只今障害が発生しております。";
4   print "a href='menber_login.php'>ログインページへ戻る</a>";
5   exit();
6}
7

で表示しているメッセージが出るのでデータベースへの接続エラーだと解釈されていると思うのですが、そうでは無い場合があります。

上記実装は例外の詳細が一切分からず、デバッグも出来ないのでかなり悪い実装です。（一部初学者向けの教材に上記のような実装がありますが、その場合は教材自体を変更した方が良いと断言できるくらいには悪い実装です）

業務では例外の内容をログに取りつつ、エンドユーザー向けのエラーメッセージを表示したいところですが、学習段階であれば

PHP
1
2catch(Exception $e) {
3   echo $e->getMessage();
4   echo $e->getTraceAsString();
5   die();
6   
7/*
8   print "只今障害が発生しております。";
9   print "a href='menber_login.php'>ログインページへ戻る</a>";
10   exit();
11*/
12}
13

等として、例外の内容を把握しましょう。

PHPマニュアル PDO エラーおよびエラー処理
も一度しっかりと読む＆サンプルコードを実行してみることを強くお勧めします。

$e->getMessage()の内容を読むことで、具体的に発生しているエラーを把握することが出来ます。

回答

新規でデータベースに会員情報を登録し再度ログインせずにそのままログインした状態にしたいです。

menber_login.phpもしくはそのフォームの送信先でログインを処理している部分があるはずなので、そこから処理を持ってくる（出来れば関数にして共通処理にする）などの工夫が必要です。
まずはログイン処理がどの様に行われているかを把握するところから始めてみて下さい。

投稿2021/08/31 04:20

tanat

総合スコア18716

参考になるか分かりませんが回答させていただきます。
最初に、気になった点と改善したほうがいい点がありましたので回答させていただきます・
１．テーブル名がmenberとなっていますが英単語的にはmemberです。（私もたまに間違えます..（笑））
２．

PHP
1$pass = md5($pass);

もしかしてパスワードをmd5で暗号化？していますか・
これは危険なのでpassword_hash関数を利用してください。
md5やsha1は高速で効率の良いハッシュのために設計されたもので現在は非推奨となっています。
ちなみにpassword_hash関数で生成したパスワードとFormに入力されたものを比較する際は
password_verify関数を利用します。詳細はPHPのマニュアルなどを見てみてください。

３．値が入力されていますか？
Formで値が空だった場合のエラー処理を書いていますか？
またHTMLのFormでのname属性とPHP側の名前はあっていますか？
Formから渡した値がすべてPHPに受け渡されているか確認してください。

投稿2021/08/31 00:03