PHP｜商品を内税と外税に条件分岐させたい

前提・実現したいこと

商品の税をフラグで内税と外税にif文で条件分岐させたいです。

こちら（https://teratail.com/questions/183532）を参考にある程度は、当方の条件に合うように作成いたしました。

$included = falseとあり、この部分がフラグということは分かりますが、
それをどのような式でtrue or falseに分岐させるのかが、いまいちよく分かりません。

ソースコード

php
1<?php
2	class Item {
3		private $taxRate = 0.08;
4		//public $price;
5    	//public $included = false;
6
7    	public function __construct ( int $price, $included = false ) {
8        	$this->price = $price;
9        	$this->included = $included;
10    	}
11
12		public function tax():int {
13			if ( $included ) { // true = 内税
14				return $this->price * $this->taxRate;
15			} else { // false = 外税
16				return $this->price + ( $this->price * $this->taxRate );
17			}
18		}
19	}
20	$item = new Item( 100 );// ( "価格", "flag" )
21	echo $item->tax().PHP_EOL;
22?>

解答例や解説、アドバイスをいただけますと幸いです。
よろしくお願いいたします。

行動規範の内容に同意します

回答2件

もしかすると、クラスというものがよく分からないということでしょうか。
クラスにはメンバ変数と呼ばれる変数を持つことができます。
記載したソースコードでいうと、private $taxRate = 0.08; はメンバ変数にあたります。
メンバ変数はクラス内の関数で共有しています。
例えばコンストラクタ（__construct 関数のことです）で設定した値を、tax 関数で使う事ができます。

内税と外税にif文で条件分岐とのことですので、
コンストラクタで、メンバ変数の included に設定し、
tax 関数でメンバ変数の inclueded を使用して判定すれば良いかと思います。
コンストラクタは記載通りでよいかと思いますが、
tax 関数に記載してあるは if ( $included ) { だと、ローカル変数になってしまうので
$this-> を付けてメンバ変数を指すようにする必要があります。

多分、気がついているかもしれませんが、Item クラスで、

//public $included = false;

とメンバ変数がコメントアウトしてあるので、// を消してあげる必要があります。

投稿2020/01/30 04:18

QZX

総合スコア44

退会済みユーザー

2020/01/30 05:08

To: QZXさんありがとうございます。 $this->を見落としてました。メンバ変数を持たせても変化がなかったので、コメントアウトしてしまっていました、、、

行動規範の内容に同意します

ベストアンサー

newのときに第２引数にt/f送るだけでは

追記

php
1<?php
2    class Item {
3        private $taxRate = 0.08;
4        //public $price;  //コメントアウトはずす
5        //public $included = false;//コメントアウトはずす
6
7        public function __construct ( int $price, $included = false ) {
8            $this->price = $price;
9            $this->included = $included;
10        }
11
12        public function tax():int {
13            if ( $included ) { // true = 内税    $this->included  に変更
14                return $this->price * $this->taxRate;
15            } else { // false = 外税
16                return $this->price + ( $this->price * $this->taxRate );
17            }
18        }
19    }