PHP「localhost でリダイレクトが繰り返し行われました」と表示されてしまいます

Question

現在、webアプリを作成するために練習用としてログイン機能を作成しようとしているのですが、登録したemailとpasswordを入力すると、localhost でリダイレクトが繰り返し行われました。と表示されてしまいます。
こちらを解決して、ログイン機能を完成させたいです。

該当のコード

//ログインページです(index.php)
ob_start();
session_start();
if(isset($_SESSION['user']) != "") {
    header("Location: home.php");
}
include_once 'dbconnect.php';

if(isset($_POST['login'])) {

    $email = $mysqli->real_escape_string($_POST['email']);
    $password = $mysqli->real_escape_string($_POST['password']);
  
    // クエリの実行
    $query = "SELECT * FROM users WHERE email='$email'";
    $result = $mysqli->query($query);
    if (!$result) {
      print('クエリーが失敗しました。' . $mysqli->error);
      $mysqli->close();
      exit();
    }
  
    // パスワード(暗号化済み）とユーザーIDの取り出し
    while ($row = $result->fetch_assoc()) {
      $db_hashed_pwd = $row['password'];
      $user_id = $row['user_id'];
    }
  
    // データベースの切断
    $result->close();
  
    // ハッシュ化されたパスワードがマッチするかどうかを確認
    if (password_verify($password, $db_hashed_pwd)) {
      $_SESSION['user'] = $user_id;
      header("Location: home.php");
      exit;
    } else { ?>
      <div class="alert alert-danger" role="alert">メールアドレスとパスワードが一致しません。</div>
    <?php }
}

//ログイン情報確認ページ(home.php)
session_start();
include_once 'dbconnect.php';---
if(isset($_SESSION['user'])) {
    header("Location: index.php");
}

$query = "SELECT * FROM users WHERE user_id=" .$_SESSION['user']. "";
$result = $mysqli->query($query);

$result = $mysqli->query($query);
if(!$result) {
    print ('クエリーが失敗しました' . $mysqli->error);
    $mysqli->close();
    exit();
}

while($row = $result->fetch_assoc()) {
    $username = $row['username'];
    $email = $row['email'];
}

$result->close();

試したこと

Cookieを削除したのですが、もう一度ログイン情報を入力したら同じ状況になったため、効果がなかったです。

---

以上です。
わかる方がいらしたらご回答よろしくお願いします。

Answer 1

true != "" はtrue
false != "" はfalse
を返すので、
if(isset($_SESSION['user']) != "")とif(isset($_SESSION['user']))は同じ値を返します。
（issetはtrueもしくはfalseのみを返すため）
そのため、片方がtrueだともう片方も必然的にtrueになり、リダイレクトを繰り返していると思われます。

Comments

[kotouharuto]

ご回答ありがとうございます。
よく考えたら確かにそうですね！もう少ししっかりと考えてコードを書こうと思います。
参考にさせていただいてコードを修正したら解決できました。

Answer 2

if(isset($_SESSION['user'])) {

「定義されていればtrue」

if(isset($_SESSION['user']) != "") {

「issetの結果が空じゃなければtrue」

これだと違いがほとんどないのでは？
userがあるかぎり、永遠に行ったり来たりしてます。

---

ログアウト時はdestroyするとして、セッション自体がnullかどうかで判定した方が良いかと思います。

Comments

[kotouharuto]

ご回答ありがとうございます。
確かにそうですね！参考になります。
そうさせていただきました、スムーズに解決できてよかったです。

[m.ts10806]

個々の値を確認する癖をつけられたら如何でしょうか。
デバッグすれば自ずと見えてきます。逆にデバッグできないと、問題解決力は育ちません。