[PHP]クラス内でのスコープについて

Question

PHP学習に伴ってオブジェクト指向やクラス作成についても調べています

前提
調査/学習の過程のクラスでコンストラクタ、ゲッター、セッターを調べていました
例題コードを読み解こうとして以下のようなコードを見ていたんですが

class Hoge {
    private $name = '';                  //name(1)
    public function __construct($name) { //name(2)
        $this->setName($name);           //name(3)
    }
    public function setName($name) {     //name(4)
        //受け取り変数の検証
        $this->name = $name; 　　　　　　　//name(5)(6)
    }
    public function getName() {
        return $this->name;              //name(7)
    }
}

$name多すぎですorz

質問(1)
やっている事は
・コンストラクタでクラス外から引数$nameを受け取る
・セッターで引数$nameをプロパティ$nameにセットする
・ゲッターでプロパティ$nameを戻り値として渡す
と捉えて間違えていませんでしょうか

質問(2)
(上記質問が正しいと踏まえて)
コードに補足コメントとして番号を振りましたが、
・引数$Nameは(2)(3)(4)
・プロパティ$nameは(1)(5)(7)
・戻り値としての変数は宣言していない(プロパティをそのまま返している)
と解釈して間違えていませんでしょうか

質問(3)
(上記質問が正しいと踏まえて)

class Hoge {
    public $argname ;                            //宣言不要？
    private $name = '';                          //name(1)
    public function __construct($argname) {      //name(2)
        $this->setName($argname);                //name(3)
    }

    public function setName($argname) {          //name(4)
        //受取変数検証
        $this->name = $argname;                  //name(5)(6)
    }

    public function getName() {
        return $this->name;                      //name(7)
    }
}

と、意味毎に変数名を分けるのはPHPのコードを書く際のお行儀としておかしい所ありますか？

一つのスレにいくつも質問を書いてすみません
「書いて通ればそれが正義」で突っ走ると、後々矯正不能なゲテモノコード作成者になりそうで基礎段階での確認ばかりをしています

よろしくお願いします

Answer 1

質問(1)
やっている事は
・コンストラクタでクラス外から引数$nameを受け取る
・セッターで引数$nameをプロパティ$nameにセットする
・ゲッターでプロパティ$nameを戻り値として渡す
と捉えて間違えていませんでしょうか

正しいです。

質問(2)
(上記質問が正しいと踏まえて)
コードに補足コメントとして番号を振りましたが、
・引数$Nameは(2)(3)(4)
・プロパティ$nameは(1)(5)(7)
・戻り値としての変数は宣言していない(プロパティをそのまま返している)
と解釈して間違えていませんでしょうか

正しいです。

質問(3)
(上記質問が正しいと踏まえて)

class Hoge {
public $argname ; //宣言不要？
private $name = ''; //name(1)
public function __construct($argname) { //name(2)
$this->setName($argname); //name(3)
}

public function setName($argname) {          //name(4)
    //受取変数検証
    $this->name = $argname;                  //name(5)(6)
}

public function getName() {
    return $this->name;                      //name(7)
}

}

と、意味毎に変数名を分けるのはPHPのコードを書く際のお行儀としておかしい所ありますか？

「意味毎に変数名を分ける」こと自体は特におかしなところはありませんが、
そもそも$this->name と $name はそもそも別物なので、
敢えてさらに別名$argnameを付けることには悪影響の方が大きいのでは無いかなと感じました。

Comments

[imprezza]

ご回答ありがとうございます
考え違いをしていないと判り一安心しています
今までの言語でやっていた癖で、同じメソッド内でスコープが違う事で別な$nameがあるってのに違和感ありまして…
しかも引数の宣言無くて「おまえ何者？」となって混乱に拍車をかけていました
PHPでは変数宣言は無くてもOKで、型も柔軟という特徴を理解していますが、ついつい「引数であること」「何の型変数なのか」を変数名に入れて長い変数にしてしまい長い変数名がゾロゾロ並ぶ不気味なコードを書いています
少しこの辺りは改めようかと思っています
ありがとうございました