PHP｜コンストラクタを設定しないで呼び出したい

Question

前提・実現したいこと

前回（http://teratail.com/questions/238438）のものを、
クラスにコンストラクタを設定しないで呼び出したいのですが、Arrayとなってしまいします。
var_dumpでNULLになっているということは、値をうまく変数に送れていないのでしょうか？
それとも、呼び出し方が間違っているのでしょうか？

ソースコード

<?php
    class Item {
        private $taxRate = 0.08;
        public $price;
        public $included = false;

        public function tax( int $price, $included = false ) {
            $this->taxRate = $taxRate;
            $this->price = $price;
            $this->included = $included;

            if ( $this->included ) { // true = 内税
                $array = [
                    'tax' => $this->price * $this->taxRate,
                    'price' => $this->price - ( $this->price * $this->taxRate )
                ];
                return $array;
            } else { // false = 外税
                $array = [
                    'tax' => $this->price * $this->taxRate,
                    'price' => $this->price
                ];
                return $array;
            }
        }
    }

    $item = new Item();

    echo $item->tax( 100, true ).PHP_EOL; // true = 内税
    echo $item->tax( 100, false ).PHP_EOL; // false = 外税
    var_dump($item->tax);
?>

エラーメッセージ

Array
Array
NULL

アドバイスなど、いただけますと幸いです。
よろしくお願いいたします。

Comments

[kei344]

URLは省略せずマークダウンで提示してください。

[退会済みユーザー]

To: kei344
修正しました。

Answer 1

メンバ変数にこだわる意味がちょっとわかりません
コンストラクタはなくても問題ないです

Comments

[yambejp]

<?php
class Item {
private $taxRate = 0.08;
public function tax($price, $included = false ) {
if($included ){
$array = [
'tax' => $price * $this->taxRate,
'price' => $price - ( $price * $this->taxRate )
];
} else { // false = 外税
$array = [
'tax' => $price * $this->taxRate,
'price' => $price
];
}
return $array;
}
}
$item = new Item();
echo $item->tax( 100, true )["price"].PHP_EOL; // true = 内税
echo $item->tax( 100, false )["price"].PHP_EOL; // false = 外税
?>

[退会済みユーザー]

ありがとうございます。
メンバ変数にこだわっているわけではないです、、

Answer 2

なんか色々おかしいですが、質問内容にだけ回答します。

間違い

$this->$taxRate = 0.08;
echo $item->tax( 100, true ).PHP_EOL; // true = 内税
echo $item->tax( 100, false ).PHP_EOL; // false = 外税

修正候補

$this->taxRate = 0.08;
var_dump( $item->tax( 100, true ));
var_dump( $item->tax( 100, false ));

Comments

[退会済みユーザー]

ありがとうございます。
$this->$taxRate = 0.08;　は、見落としです、、

Answer 3

tax()の戻り値が配列なので、echoで中身を表示することは出来ません。

    $item = new Item();

    $price = $item->tax( 100, true );
    echo $price['tax'];
    echo $price['price'];

みたいにするか、そもそも配列で返すのではなくて数値で返すようにするかですね。

Comments

[退会済みユーザー]

ありがとうございます。
戻り値が配列だと、echoで表示できないのですね、、