PHPプログラムでのクラス、オブジェクトに関する質問（初心者です）

Question

前提・実現したいこと

PHP初心者です。
PHPのクラスに関する質問です。
以下のAuthefiticationsクラスでUserDataクラスのゲッターを呼び出したいので、UserDataクラスのオブジェクトを生成したいです。

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

IDEのエディタ上にエラーが表示されてしまいます。（プログラム実行はしていません）

Unexpected: assign　//オブジェクト生成コード上にでてきます。

該当のソースコード

<?php
namespace todo;
use todo\UserData;

class Authentications
{
    private $leaf = false;
    $user = new UserData();//①フィールド上に宣言したが×

    public function __construct(){
        $user = new UserData();//②コンストラクタに宣言したが使用できない
    }
    public function confirmID($user_id){
        if($user_id==$user->getID()){
            $this->leaf= true;
        }
        return $this->leaf;
    }

    public function confirmPass($password){
        if($password==$user->getPass()){
            $this->leaf= true;
        }
        return $this->leaf;
    }
}

<?php
namespace todo;

class UserData
{
    private $user_id = "php";
    private $password = "Docker";

    public function getID(){
        return $this->user_id;
    }

    public function getPASS(){
        return $this->password;
    }

}

試したこと

Authefications.phpの①②

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

PHP7.2
Laravelとかは使ってません。
フレームワークなしです。

Comments

[tanat]

IDEの警告を見ることは大事ですが、
まずはPHPを実行してエラーメッセージを確認するようにして下さい。

[退会済みユーザー]

class Authenticationsの$userってなに？

[退会済みユーザー]

質問の件名に、わざわざ初心者ですとか書く必要はありません。初心者であることを示す若葉マークみたいな機能の方を使ってください。

[m.ts10806]

質問はいつでも編集できるので下記だけでもご対応ください
・(初心者です)を外す
・初心者マークをつける

Answer 1

class Authenticationsの$userの扱いが雑すぎてこまる。
そのくせ、$this->leafとか書けているのに、
同じことをuserも$this->userとすればいいだけなのに、
どうして。

    public function __construct(){
        $user = new UserData();//②コンストラクタに宣言したが使用できない
    }

これ、$userがローカル変数扱いされてて、
コンストラクタを抜けると$userは揮発してしまうんだが。
メンバー変数じゃないから、消えちゃうし参照もできなくなる。

そうならないよう、メンバ変数として定義する必要があって、
ローカル変数へのアクセスと区別して保持しないといけない。

class Authentications
{
    private $leaf = false;
    private $user;

    public function __construct(){
        $this->user = new UserData();  // メンバー変数にUserData()のインスタンスを保持
    }

    public function confirmID($user_id){
        if ($user_id == $this->user->getID()) {  // ほんとうは == よりも === とするべきか
            $this->leaf = true;
        }
        return $this->leaf;
    }

    public function confirmPass($password){
        if ($password == $this->user->getPass()) {  // ほんとうは == よりも === とするべきか
            $this->leaf = true;
        }
        return $this->leaf;
    }
}

とかなるだろうか。（机上のコードなので実行してないけども。）

文法上の間違いではないけど、適度に空白をつけて見やすさを改善しないと
コードへの理解も深まらないとわたしは思う。

ところで、confirmID()の結果とconfirmPass()の結果が
同じメンバー変数$this->leafを上書きしてていいのかい？

Comments

[yuri0906]

回答とご意見ありがとうございます。もう少し文法チェックしてから質問します。

Answer 2

Authenticationsクラスで
private $user = null;
など宣言しておいて、コンストラクタ内で
$this->user=new UserData();
とかにしてみては？

Answer 3

ざっとしか見てないですけど、色々問題ありそうです＾＾；

とりあえず、質問の箇所は以下で解決できるかと

プロパティ

プロパティを定義するには public、protected あるいは private のいずれかのキーワードのあとに、オプションの型宣言を続け、 そして通常の変数の宣言を続けます。 宣言時に初期値を設定することもできますが、初期値は定数値でなければなりません。 つまり、コンパイル時に評価可能な値でなければならず、 実行時の情報がないと評価できない値であってはいけないということです。

---

なんかよくわからん説明になったので、とりあえず動くってレベルのコードを追記
やりたいのは、UserData インスタンスをプロパティで持ちたいってことなんですかね？

<?php
class Authentications
{
    private $leaf = false;

    public function __construct(){
        $this->user = new UserData();
    }
    public function confirmID($user_id){
        if($user_id===$this->user->getID()){
            $this->leaf= true;
        }
        return $this->leaf;
    }
}

class UserData
{
    private $user_id = "php";
    private $password = "Docker";

    public function getID(){
        return $this->user_id;
    }
}

$a = new Authentications();
var_dump($a->confirmID("php"));

Comments

[退会済みユーザー]

$this->userの宣言がない？

[退会済みユーザー]

動くだけのコードだからねぇ＾＾；
そもそも命名規則から見直したい。