ログイン判定 クラス化について

Question

前提・実現したいこと

ログイン判定をクラス化しようと思っております
オブジェクト指向に関しては勉強したてのため、深く理解していない状況です

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

実際にクラスを作成してみたのですが"ログインされていません"が表示され、ログインできない状況です
コンストラクタにセッション変数の中身が渡せていないのでしょうか

該当のソースコード

<?php

class Login_Authentication
{
   private $session;

   public function __construct($session)
{
      $this->session = $session;
      $login = $this->session;
     
     if (empty($login)) {
       $_SESSION["FAILURE"] = "ログインされていません";
       header("Location: ログイン画面のURL");
       exit();
     }
  }
}

  session_start();
  session_regenerate_id(true);

  $Authentication = new Login_Authentication($_SESSION["NAME"]);

試したこと

クラスを使用せずに
if (isset($_SESSION["NAME"]))
{
を試してみたところログインできました

###変更後

<?php

class Login_Authentication
{
   private $session;

   public function __construct()
{
  　　session_start();
 　　 session_regenerate_id(true);
      $this->session = $_SESSION["NAME"]
      $login = $this->session;
     
     if (empty($login)) {
       $_SESSION["FAILURE"] = "ログインされていません";
       header("Location: ログイン画面のURL");
       exit();
     }
  }
}

  $Authentication = new Login_Authentication();

Comments

[m.ts10806]

$loginに何が設定されていますか？

[m.ts10806]

そもそも$_SESSION["NAME"]はどのように設定されるのでしょうか。

[mather]

$sessionがプロパティとコンストラクタの引数で同じ変数名なのが気になります。別の名前にしてみてはどうでしょうか？

[Company_]

$loginに関してはprivate $login;と定義しなければならないということでしょうか

[mather]

if (isset($_SESSION["NAME"])) ではなく if (empty($_SESSION["NAME"])) にしても動作しますか？

[Company_]

$_SESSION["NAME"]に関しては、どのように受け渡せばいいのか分かっていない状況です

[m.ts10806]

ひとまず引数で受け取ってる変数名をかえてみては

[m.ts10806]

受け渡すのではなく、値が設定されている場所の話です。今のコードだけだと定義されずに いきなり使われてます。というか$_SESSIONはスーパーグローバル変数なので正しく定義されてればどこからでも参照できるので引数で渡さなくても使えます。

[Company_]

変数名を変更してきました　public function __construct($test)

[Company_]

if (empty($_SESSION["NAME"]))で試したところログインできました

[mather]

コードを変更した場合、質問を更新しましょう。また変更したあとの実行結果はどうなりましたか？

[Company_]

質問を更新しました　変数名を変更しても実行結果は変わりませんでした

[Company_]

セッション変数を引数で渡さずにやってみましたが変わりませんでした

[m.ts10806]

code機能は何個でも関係なく利用できるので変更後のコードも同じようにコードブロックにしてください

[m.ts10806]

結局$_SESSION['name']はどこでどのように定義(情報が保管)されているのでしょうか？

[Company_]

ログイン成功時に、ログイン機能の方でセッション変数に情報が保管されるようにしております

[mather]

session_start() を毎回実行しているのも気になります。削除してみてはどうでしょうか。

[Company_]

承知しました　セッションについて、今一度勉強し直してみます

Answer 1

解決しました

そもそもセッション変数に値を渡せていませんでした

原因としましてはログイン画面で"ログインされていません"のセッション変数を消すために
$_SESSION = array();とsession_destroy();を記述していました

これから使用するセッション変数もクリアしてしまうことを知りませんでした

何とか無事にクラスを使用しログインできました

ご回答下さり有り難う御座いました

Comments

[m.ts10806]

個人的に気になったのですが、ご自身で組まれたコードではないのですか？
複数人で分担してるとか

[Company_]

私自身で作成したコードになります
クラスに関しては複数人で共有するために作成しました

[m.ts10806]

なるほど。
クラスにするにあたりまだちょっと連携が甘いところがあったというところですね。わかりました