Java カプセル化応用問題

Question

---

文房具クラスを追加する
StationaryShop を実行すると実行結果の通りになるように、Stationary クラスを追加しなさい

また、追加にともない Amount クラスにも必要な修正を行うこと

実行結果
三菱ジェットストリーム が 3 本あります
MONO消しゴム が 2 本あります
金額は 562 円です

public class StationaryShop2 {

public static void main(String[] args) {
    Amount myCart = new Amount(0);
    
    Stationary ballPen = new Stationary(
        "三菱ジェットストリーム", new Amount(120), 3);
    
    Stationary eraser = new Stationary(
            "MONO消しゴム", new Amount(80), 2);
    myCart = myCart.add(ballPen.calcTotalAmount());
    myCart = myCart.add(eraser.calcTotalAmount());
    myCart = myCart.addTax();
    myCart.showValue();
  }
}

この続きのコードを記述せよ

という課題に取り組んでいるのですが、上手く実行結果が出ず、現在手詰まり状態です。

僕自身では

---

public class StationaryShop2 {

    public static void main(String[] args) {
        Amount myCart = new Amount(0);

        Stationary ballPen = new Stationary("三菱ジェットストリーム", new Amount(120), 3);
        Stationary eraser = new Stationary("MONO消しゴム", new Amount(80), 2);

        myCart = myCart.add(ballPen.calcTotalAmount());
        myCart = myCart.add(eraser.calcTotalAmount());
        myCart = myCart.addTax();
        myCart.showValue();
    }
}


final class Stationary {
    
    private String name;
    private int quantity;

    public Stationary(String name , Amount value , int quantity) {
        this.name = name;
        this.quantity = quantity;
    }
        

    public Stationary calcTotalAmount() {
        return new Stationary(name, quantity); 
    }
}
　
final class Amount { 

    private final int value;

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

    public Amount add(Amount augend) {
        return new Amount(value + augend.value);
    }

    public Amount addTax() {
        return new Amount((int)(value * 1.08));
    }

        
    public void showValue() {
        System.out.println(name + " が " + quantity + " 本あります");
        System.out.println("金額は " + value + " 円です");
    }  
}

というところまではいったのですが、nameとquantityをどう返せばいいのか、どう出力すればいいのかわからず、現在進むことができずに詰んでおります。
僕の勉強不足が原因であり、厳しいお言葉も覚悟してますが、よければ教えていただけると幸いです…

Comments

[BeatStar]

まず、ちゃんとmarkdownを使ってコードを提示してください。
ヒント: https://teratail.com/help/question-tips

[dodox86]

なぜ、各所でfinalを付けているのでしょうか。また、先のご質問
[Java カプセル化の問題]https://teratail.com/questions/9hb0ldg0g7qvjz
でいただいていた各回答の内容は理解し、消化できていますか？

[RiaFeed]

問題自体が酷い…

[BeatStar]

質問は編集できるので編集してください。お願いします。

[parabellum]

すいません修正しました
このサイトにまだ慣れてないのでご容赦ください

[BeatStar]

この本来の問題はどこまで提示されていたのでしょうか？

例えば「mainメソッドしか提示されていないけど、それ以外は自分の力で組んだ」のか、「今提示されているものが問題文の全てで、自分では何も考えていない」のか、「(一つ前のやつで)自分で調べてみたが何もわからなかった」なのか…

二つ目や三つ目なら問題が悪すぎる、一つ目なら「ちょっと惜しい」かなと。

[parabellum]

提示されていたのは、”問題文”、”実行結果の回答”、”メインメソッド”のみですね…今回の問題はひとつ前の問題の発展型で、Amountクラスはその問題で書いたものでした。そして今回はメインメソッドが前回の問題から少し追記され、Stationaryの追加とAmountの改修、というのが今回の問題です…

[BeatStar]

忘れていました。
Amountの本来のコードはありますか？
そのコードから手をつけたのが質問にあるAmountでしょうか？
それともそのまま？

[parabellum]

’’’
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 + " 円です");
}
}
’’’
前回の問題で書いたAmountはこうなってます。

[BeatStar]

つまり、showValueメソッドだけ違うってことですね。

[parabellum]

あ、ごめんなさい。一つ課題のメインメソッドはこうなってます。

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();
}
}

Answer 1

あえてコードなしで説明するとこんな感じ？

Amountクラスには

• getValueメソッドなんかを追加してvalueフィールドの値を他のクラスから取得できるようにする

Stationaryクラスは

• (mainメソッド変更禁止なら)コンストラクタで商品名と本数をprint
• コンストラクタに渡されてるAmount型のオブジェクトもフィールドを追加して代入
• calcTotalAmountメソッドはAmountのvalue(getValue)と本数を基に計算した結果をAmount型のコンストラクタに渡してインスタンス化し、そのインスタンスを返す

Comments

[parabellum]

//25日目 クラスを追加しよう

public class StationaryShop2 {
public static void main(String[] args) {
Amount myCart = new Amount(0);

Stationary ballPen = new Stationary("三菱ジェットストリーム", new Amount(120), 3);
Stationary eraser = new Stationary("MONO消しゴム", new Amount(80), 2);

myCart = myCart.add(ballPen.calcTotalAmount());
myCart = myCart.add(eraser.calcTotalAmount());
myCart = myCart.addTax();
myCart.showValue();
}
}

class Stationary {

private String name;
private int quantity;
private int value;

public Stationary(String name , Amount value , int quantity) {
this.name = name;
this.quantity = quantity;
}

public void showValue() {
System.out.println(name + " が " + quantity + " 本あります");
}

public void setValue(int value) {
this.value = value;
}

public Amount calcTotalAmount() {
return new Amount(value*quantity);
}
}

class Amount {

private int value;

public int getValue() {
return this.value;
}

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

public Amount add(Amount augend) {
return new Amount(value + augend.value);
}

public Amount addTax() {
return new Amount((int)(value * 1.08));
}

public void showValue() {
System.out.println("金額は " + value + " 円です");
}
}

ほかに3つほど回答が来てましたが、貴方様のやり方で試してみました。
ひとまずここまでできたのですが、合ってますでしょうか？
・コンストラクタに渡されてるAmount型のオブジェクトもフィールドを追加して代入
というのがよくわからず…理解力なくてすいません…

[RiaFeed]

showValueがどこからも呼ばれていないからprintされないし
valueはint型でもいいですが、コンストラクタ引数のamountから値を取得していないから多分0のままですよ

[parabellum]

なるほど…
色々ありがとうございます、もう少し思索してみます

Answer 2

System.out.println(name + " が " + quantity + " 本あります");

を Amount の showValue に入れていますが、 name も quantity も Amount にありません。
そもそも Amount クラスは金額を表すだけで、その詳細(内訳)を保持する役割は無い設計と思われます。
ですので 実行結果の

三菱ジェットストリーム が 3 本あります
MONO消しゴム が 2 本あります
金額は 562 円です

のうち、最後の合計金額以外は Stationary クラスが表示するのが妥当でしょう。
Stationary オブジェクトに実行が移るタイミングはコンストラクタと calcTotalAmount メソッドですが、 calcTotalAmount は計算が主ですので表示するのは違う気がします。
以上から、 System.out.println(name + " が " + quantity + " 本あります"); は Stationary のコンストラクタで行うと良いように思います。

もちろん、 Amount の修正も OK ということですので、設計を変えて (Amount という名前ながらも ) イロイロ載せて解決する方法もあります。その為のコーディング要素を勉強されているかは不明ですが。

私が思ったのはこんな風です。

class Stationary {
  private String name;
  private Amount unitprice;
  private int quantity;

  public Stationary(String name, Amount unitprice, int quantity) {
    this.name = name;
    this.unitprice = unitprice;
    this.quantity = quantity;
    showValue();
  }
  public Amount calcTotalAmount() {
    return unitprice.multiply(quantity);
  }
  public void showValue() {
    System.out.println(name + " が " + quantity + " 本あります");
  }
}

final class Amount {
  private final int value;

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

Comments

[parabellum]

返信遅れてすみません、前回、今回と回答ありがとうございます！
金額以外の実行はどこで表示するか迷っていましたが、この回答で納得できました！
正直まだまだ理解できてないところが多いですが…

Answer 3

public static void main(String[] args) {
    Amount myCart = new Amount(0);

    Stationary ballPen = new Stationary("三菱ジェットストリーム", new Amount(120), 3);
    Stationary eraser = new Stationary("MONO消しゴム", new Amount(80), 2);

    myCart = myCart.add(ballPen.calcTotalAmount());
    myCart = myCart.add(eraser.calcTotalAmount());
    myCart = myCart.addTax();
    myCart.showValue();
}

コレ↑がコンパイルできるのであれば、Stationary#calcTotalAmount() は Amount を返さねばならない。
であるなら、コレ↓がおかしい。

public Stationary calcTotalAmount() { // なんで Stationary を返すん?
    return new Stationary(name, quantity); 
}

Comments

[parabellum]

返信遅れてごめんなさい、回答ありがとうございます！
そこは一応なんとなくですが理解はできました！

Answer 4

コツ1：処理の流れを読む

プログラミングはやらためったに描くものではありません。処理の流れを記述するのです。
そう、料理のレシピのように。

コードを読むコツは「一行レベルで、その行が何をしているかを考えながら読む」です。

    // Amountのオブジェクトに0を渡して生成
    Amount myCart = new Amount(0);
    // ボールペンの生成
    Stationary ballPen = new Stationary(
        "三菱ジェットストリーム", new Amount(120), 3);
    // けしごむの生成
    Stationary eraser = new Stationary(
            "MONO消しゴム", new Amount(80), 2);
    // ボールペンの合計金額を計算して
    // カートに入れる?
    myCart = myCart.add(ballPen.calcTotalAmount());
     // 同じくけしごむも。
    myCart = myCart.add(eraser.calcTotalAmount());
    // 消費税分を足して
    myCart = myCart.addTax();
    // 結果を出力(カート側が)
    myCart.showValue();

となりますね。あとはこれに出てくるmyCartやらなんやらの実装ですね。

（ちょっと中断します）

Comments

[parabellum]

なるほど…料理のレシピのようにとは言いえて妙ですね。
一行レベルで、その行が何をしているかを考えながら読むというのは心がけてみます