php このコードの書き方はあっているでしょうか

Question

progateのPHP学習コース２をやっています。
今やっている課題は、料理注文サイトを作ることを目指しています。
注文したJUICE、COFFEE、CURRY、PASTAの４つの商品の個数と金額を出し、
値を受け取り小計金額をだす作業です。
今回、理解できない部分のコードがあるので そのコードがなぜこの記述なのか
教えていただけないでしょうか。

<?php
class Menu {
  private $name;
  private $price;
  private $image;
  private $orderCount = 0;
  
  public function __construct($name, $price, $image) {
    $this->name = $name;
    $this->price = $price;
    $this->image = $image;
  }
  
  public function hello() {
    echo '私は'.$this->name.'です';
  }
  
  public function getName() {
    return $this->name;
  }
  
  public function getImage() {
    return $this->image;
  }
  
  public function getOrderCount() {
    return $this->orderCount;
  }
  
  public function setOrderCount($orderCount) {
    $this->orderCount = $orderCount;
  }
  
  public function getTaxIncludedPrice() {
    return floor($this->price * 1.08);
  }
  
  public function getTotalPrice() {
    return $this->getTaxIncludedPrice() * $this->orderCount;
  }
  
}
?>

現在「getTotalPriceメソッドを定義してください」の部分を取り組んでおり
回答は上記の書き方なのですが、getTotalPriceメソッドの中で
$this->getTaxIncludedPrice()　と　$this->orderCount　を掛けています。

ここで思ったのですが、片方は
$this->でgetTaxIncludedPrice()メソッドなのですが
片方が$this->orderCount　という　getOrderCount()メソッドの戻り値になっています。
この場合、getOrderCount()というメソッドがあるので

public function getTotalPrice(){
    return $this->getTaxIncludedPrice() * $this->getOrderCount();

でもいいのでは？と思いました。
実際にこちらに変えてみると、表示内容は同じでした。
片方がメソッドなのになぜもう片方はメソッドを使っていないのかがわからない状態です。
この記述の片方がメソッドで、片方がメソッドの中身（$this->orderCount）である理由は何でしょうか。
よろしくお願いいたします。

Comments

[m.ts10806]

コードやエラーはマークダウンのcode機能を利用してご提示ください。
https://teratail.com/questions/238564

[monoymd]

失礼しました。```を使うと反映されるのですね。初めての投稿のためお手数をおかけいたしました。
確認のほどよろしくお願いいたします。

Answer 1

今の状態であれば$this->orderCountでも$this->getOrderCount();でもどちらでも同じデータにアクセスすることになるため、結果的には同じかなと思います。

ただしおそらく$this->getOrderCount();は外部からこのインスタンスが保持するorderCountを得るための「ゲッタ」というニュアンスが強いと思うので、私もそういう意味では違いがわかりやすくなるように内部で計算処理では$this->orderCountを使う方が良さそうだとは思います。

また、場合によっては外側からゲッタを経由して値を取得する際にインスタンス内部の値を加工して返すという場面も結構多く、その場合であれば$this->orderCountと$this->getOrderCount();では結果が異なるため適切に使い分ける必要が出てきます。

Comments

[monoymd]

回答ありがとうございます。

＞ただしおそらく$this->getOrderCount();は外部からこのインスタンスが保持するorderCountを得るため＞の「ゲッタ」というニュアンスが強いと思うので、私もそういう意味では違いがわかりやすくなるように内＞部で計算処理では$this->orderCountを使う方が良さそうだとは思います。

$this->getOrderCount();　を使うとゲッタとして動いているように見えてしまうので、ここではゲッタというよりも　$this->orderCount　として　orderCountの値を使っている、という意味になるようにしたほうが良いということで合っていますでしょうか。

[tsuyo_244]

そうですね、クラス内部でインスタンス変数を参照するときにあえてゲッタを経由する必要はないので、ダイレクトに参照しにいった方がわかりやすいと感じます。

もう少し詳しく書くと、ゲッタの本来の役割として外から勝手に変更させないようprivateにしたインスタンス変数に対し、変更はさせないけど値の読み取りだけは許可したい時などにその手段を提供する用途で作成するものなので、それをあえてクラス内部で使う必要がないかなという感じです

[monoymd]

なるほど、理解できました。詳細、迅速な回答いただきありがとうございました。