Q&A
コードはマークダウンのcode機能を利用してご提示ください。
前提・実現したいこと
プログラミング学習歴1か月の超初学者です。
progateで学習中に以下のサンプルコードを見て
理解できない点があったので質問させていただきます。
php
1<?php 2class Menu { 3 private $name; 4 private $orderCount = 0; 5 6 public function getName() { 7 return $this->name; 8 } 9 10 public function getOrderCount() { 11 return $this->orderCount; 12 } 13 14 public function setOrderCount($orderCount) { 15 $this->orderCount = $orderCount; 16 } 17} 18 $menus = new Menu(); 19?> 20 21<!DOCTYPE html> 22<html> 23(中略) 24 <div class="order-wrapper"> 25 <h2>注文内容確認</h2> 26 <?php foreach ($menus as $menu): ?> 27 **<?php $orderCount = $_POST[$menu->getName()] ?>** 28 <p class="order-amount"> 29 <?php echo $menu->getName(); ?> 30 <?php echo $orderCount;?> 31 個 32 </p> 33 <?php endforeach ?> 34 </div> 35</body> 36</html>
Menuクラス内で$orderCountをprivateで指定しているにも関わらず、
クラス外から<?php $orderCount = $_POST[$menu->getName()] ?>と
セッターを介さずに値を変更できるのは何故でしょうか？
上記を<?php $menu->setOrderCount($_POST[$menu->getName()])?>としたところ不正解でした。
またgetName()はどこにも定義されていません。定義しているコードをご提示ください。
get～～なので変更している箇所はないようにも見受けられますが、どういう意味で書かれているのでしょうか。
大変失礼いたしました。
code機能を利用し、書き直しました。
また、getName()を定義している箇所を記載いたしました。
質問と無関係なコードを削除した際に
誤って一緒に消してしまっていたようです・・・
classの{ 閉じてませんが、どこまで実際のコードなのでしょうか。
構文エラーが出ます。
回答者には提示されているコードが全てなので、エラーで問題があるわけではないのでしたら
なるべくエラーの出ないコードを提示していただけると、余計なところに気を回さなくて済みますし、
的確な回答を得られます。
度々すいません。
次回から気を付けます・・・
ご指摘ありがとうございました！
回答2件
0
ベストアンサー
Menuクラス内で$orderCountをprivateで指定しているにも関わらず、
クラス外から<?php $orderCount = $_POST[$menu->getName()] ?>と
セッターを介さずに値を変更できるのは何故でしょうか？
変更できていません。クラス外の$orderCountは、プロパティとは無関係な、単なる変数です。
投稿2019/12/07 04:21
編集2019/12/07 04:35
総合スコア145963
ご回答いただきありがとうございます。
クラス内のプロパティを変更しているわけではなく、
インスタンス化した$menus内の$orderCountに値を代入しているだけだから
セッターは不要ということでしょうか？
> インスタンス化した$menus内の$orderCountに値を代入しているだけだから
セッターは不要ということでしょうか？
いえ、違います。$orderCountは$menusとも関係ない、単なるローカル変数です。名前がたまたまプロパティと一致しているだけです。
ご回答いただきありがとうございました。
たまたまプロパティと名前が一致しているだけとのことで、
クラス外の$orderCountの名前を変えても、問題ないことが確認できました。
0
$orderCount = $_POST[$menu->getName()]
上記、ローカル変数に代入してるだけですので
MenuクラスのプロパティorderCountの値は変わっていません。
画面出力のために$orderCountという変数で受け取っているだけですね。
名前が紛らわしいのであれば、なるべく別の変数名をつけると良いです。
MenuクラスのプロパティorderCountの値を変更しようと思ったら
setOrderCountメソッドを通す必要があります。
ただ、このコードだけだと、同じくprivateのnameに何も設定されてませんよね。
初期化もされてませんので別のエラーがでるように思いますけど・・。
投稿2019/12/07 04:45
編集2019/12/07 04:47総合スコア80875
あなたの回答
tips
プレビュー
