Java カプセル化の問題

Question

Java初心者です。カプセル化の問題を解いています。
問題がわからなくて苦戦しています。
調べてもなかなか解けませんでした…
よければ解答例を教えていただけると幸いです。

問題文↓
金額クラスを作る
StationaryShop を実行すると実行結果の通りになるように、Amount クラスを作成しなさい

※消費税率は 8% とする

このコードの内容は変えてはいけない
ーーーーーーーーーーーーーーーーーーーーーーーーー
public class StationaryShop {

public static void main(String[] args) {

    Amount myCart = new Amount(0);
    Amount ballPen = new Amount(120);
    Amount eraser = new Amount(80);

    myCart = myCart.add(ballPen);
    myCart = myCart.add(eraser);

    myCart = myCart.addTax();
    myCart.showValue();
}

}
ーーーーーーーーーーーーーーーーーーーーーーーーー

これより先のAmountクラスを作る

僕はここまで書いて訳が分からなくなってしまいました。
class Amount {
private int myCart;
private Amount ballPen;
private Amount eraser;

public Amount(int myCart) {
    this.myCart = myCart;
}

public Amount add(Amount ballPen) {
    myCart = ballPen;
    return ballPen;
}

public Amount addTax(int myCart) {
    myCart = (int) (myCart * 1.08);
    return myCart;
}

public void showValue() {

    System.out.println("金額は " + myCart + " 円です");

}

よければご教授お願いします。

Comments

[Zuishin]

> 調べてもなかなか解けませんでした…

調べるからです。
復習したら解けます。

[usekay]

カートと商品を同じクラスにするのは微妙な気がする…そもそも問題が悪いですね…

[ozwk]

System.out.println("金額は 216 円です");
としてあとはコンパイル通るように書けば問題の条件を満たせるのでそういう意味でも問題が悪いですね

[dodox86]

こちらの質問、質問者さんに限りませんが、コード全体ではなくメソッドだけをマークダウンの記法にするのが何か最近、流行っているのでしょうかね。そんな風に教えられたのでしょうか。

[parabellum]

4人の方もありがとうございました！！
僕の理解力の低さも相まってなかなか難しい問題でした…

[jimbe]

> メソッドだけをマークダウンの記法にする
想像ですが、コード＝実行する部分…という認識なのかもしれませんね。

[ozwk]

> dodox86さん

いつからかはわかりませんが、
スペース4つでコードブロック判定になるようになってます

[jimbe]

> スペース4つでコードブロック
私も知りませんでした。
ということは皆さん意図せずたまたまブロックになってたということですね。

[dodox86]

@ozwkさん

コメントいただいていたのに気が付きませんでした。すみません、そして情報どうもありがとうございます。なるほど、それで合点がいきました。

Answer 1

実行例が不明ですが、おそらく Amount クラスのインスタンスは数値一つだけ持っていれば良さそうです。

private Amount ballPen;
private Amount eraser;

この2行は消しましょう。

private int myCart;

これは、金額クラスの値ですから private int price; などのわかりやすい名前にしましょう。

add() や addTax() が返り値を持つのは妙な設計の気がしますが、設問の意図ではこれらは this を返すべきなのでしょう。

Comments

[parabellum]

すいません、実行例を書き忘れました…
実行例は　
金額は 216 円です
となります！
めちゃくちゃ助かりました、ありがとうございます！！

[usekay]

myCartも Amountなのでそれではだめなような。

[jimbe]

> add() や addTax() が返り値を持つ
Amount は不変ということでしょう。

[int32_t]

> Amouont は不変ということでしょう。

なるほど。それなら少し納得ですね。

であれば、たとえば addTax() は
return new Amount((int)(this.price * 1.08));
という感じですね。

[usekay]

まあ計算できりゃなんでもいいので問題としてはそれでいいのでしょうけどオブジェクト指向の学習を目的とした問題としては微妙だなと…

Answer 2

Amount を「不変」(一度作ったら値が変わらないモノ)とすると思います。

public class StationaryShop {
  public static void main(String[] args) {
    Amount myCart = new Amount(0);
    Amount ballPen = new Amount(120);
    Amount eraser = new Amount(80);

    myCart = myCart.add(ballPen);
    myCart = myCart.add(eraser);

    myCart = myCart.addTax();
    myCart.showValue();
  }
}

final class Amount {
  private final int value;

  Amount(int value) {
    this.value = value;
  }
  Amount add(Amount augend) {
    return new Amount(value + augend.value);
  }
  Amount addTax() {
    return new Amount((int)(value * 1.08));
  }
  void showValue() {
    System.out.println("金額は " + value + " 円です");
  }  
}

実行結果

金額は 216 円です

Comments

[parabellum]

詳しいコードまでありがとうございます！！
なるほど、「不変」とするのですね…
正直どれだけひねっても思いつきませんでした…
ここで折れずに勉強頑張ります！！

[jimbe]

この問題に必ず必要というわけではありませんが、 int32_t さんが回答で書かれている「add() や addTax() が返り値を持つのは妙な設計の気がします」という所で、可変なオブジェクトでは自身が変わ(れ)るので返り値は不要です(有ってイケナイわけではありません)が、不変なクラスオブジェクトでは自身が変わ[れ/ら]ないで新しいオブジェクトを返しますので、不変のほうが自然に感じます。( String や BigDecimal ( https://docs.oracle.com/javase/jp/8/docs/api/java/math/BigDecimal.html ) 等が同様です。 )
「不変性」や「Immutable」(インミュータブル)、対になる「mutable」(ミュータブル)等をお調べになってみてください。

[parabellum]

なるほど…BigDecimalや「不変性」や「Immutable」(インミュータブル)、対になる「mutable」(ミュータブル)等はまだ勉強していない部分なので頑張ってみます！！
色々教えてくださって助かります！！