PHP 以前にログインしているユーザーを会員ページにリダイレクトする機能を実装したい

実現したいこと

タイトルの通り、「以前にログインしているユーザーを会員ページにリダイレクトする機能」を実装したいです。

現状

現在phpでwebサービスを開発中でアカウント作成機能とログイン機能が完成したのですが何か物足りない思いタイトルのような機能を作成しようと思いました。
自分でも考えたて実装したり調べたりしたのですが上手くいかずで質問させていただいた次第です。

該当のソースコード

php
1// login_exec.php(ログイン機能の処理ファイル)
2
3$pdo = db_connect();
4
5if(isset($_SESSION['user_id'])) {
6    header("Location: menupost.php");
7} else {
8    echo 'ログインしてください';
9}
10//ログイン処理
11function Login() {
12    // エラーがあったら表示
13    if (isset($_GET['error'])) {
14        echo $_GET['error'];
15    }
16    if(isset($_POST['login'])) {
17        $email = $_POST['email'];
18        $password = $_POST['password'];
19        if(!filter_var($email, FILTER_VALIDATE_EMAIL)) {
20            header('Location: login.php?error=入力された値が不正です。');
21            return false;
22        }
23        
24        //DB内でPOSTされたメールアドレスを検索
25        try {
26            $stmt = $pdo->prepare('SELECT * FROM users WHERE email = ?');
27            $stmt->execute([$email]);
28            $row = $stmt->fetch(PDO::FETCH_ASSOC);
29        } catch(\Exception $e) {
30            echo $e->getMessage() . PHP_EOL;
31        }
32        
33        //emailがDB内に存在しているか確認
34        if(!isset($row['email'])) {
35            header('Location: login.php?error=メールアドレス又はパスワードが間違っています。');
36            return false;
37        }
38        
39        //パスワード確認後にsessionにメールアドレスを渡す
40        if(password_verify($password, $row['password'])) {
41            session_regenerate_id(true); //session_idを新しく生成し、置き換える
42            $_SESSION['user_id'] = $row['user_id'];
43            $user_id = $_SESSION['user_id'];
44            ?>
45        <div class="alert alert-primary" role="alert">ログインに成功しました。</div>
46        <?php
47        header("Location: menupost.php");
48    } else {
49        ?>
50            <!-- <div class="alert alert-danger" role="alert"></div> -->
51        <?php
52        header("Location: login.php?error=メールアドレス又はパスワードが間違っています。");
53        return false;
54    }
55    }
56}
57
58Login();

html
1<!-- login.tpl(ログイン機能の表示ファイル) -->
2
3<form action="login_exec.php" method="POST">
4    <div class="emailbox">
5        <p class="forminfo">メールアドレス</p>
6        <input type="email" name="email">
7    </div><br><br>
8
9    <div class="passbox">
10        <p class="forminfo">パスワード</p>
11        <input type="password" name="password">
12    </div><br><br>
13
14    <button type="submit" class="accbtn" name="login">ログインする</button>
15</form>
16<div class="signUpbox">
17    <p>アカウントをお持ちでない方はこちらから</p>
18    <a href="signup.php">アカウントを作成する</a>
19</div>
20
21<footer>
22
23</footer>

試したこと

if(isset($_SESSION['user_id'])) {
    header("Location: menupost.php");
} else {
    echo 'ログインしてください';
}

こちらの処理でsessionからuser_idが確認できたら会員ページにリダイレクトする処理を考えたのですが失敗しました。

以上です。
わかる方がいらしたらご回答をお願いしたいです。

yambejp

2020/08/28 10:27

セッションが残ってたらリダイレクトする必要がないのでは？

kotouharuto

2020/08/28 11:21

あ、確かにそうですね。もう少し自分で確かめてみます。

退会済みユーザー

2020/08/28 20:26

本題と関係ないのですが、header 関数は、すべての出力の前にコールするようにしてください。多分、提示されているコードでも動くと思いますが、推奨される記述ではありません。 https://www.php.net/manual/ja/function.header.php https://qiita.com/chinka/items/d63a4d0f2c4e5e794284

kotouharuto

2020/08/28 23:23

te2jiさんわざわざありがとございます。参考にさせていただきます。