実行結果がnullになる。

Question

Detail.javaが、

package jp.co.bbreak.sokusen._2._1;

import java.math.BigDecimal;
public class Detail{private String itemName;
private BigDecimal amount;
public String getItemName(){return itemName;}
public void setItemName(String itemName)
{this.itemName = itemName;}
public BigDecimal getAmount(){
return amount;}
public void setAmount(BigDecimal amount){
this.amount = amount;
}

public Object お米5kg() {
// TODO 自動生成されたメソッド・スタブ
return null;

}
}

で、
package jp.co.bbreak.sokusen._2._1;

import java.math.BigDecimal;

public class PrintReceipt1 {
public static void main(String[] args){
//明細データの作成
Detail detail = new Detail();
detail.setItemName("お米5kg");
detail.setAmount(new BigDecimal(1980));

	//明細データの加工
	StringBuilder sb = new StringBuilder();
	sb.append(detail.お米5kg());
	sb.append("       ");
	sb.append(1980);
	sb.append("円");
	
	//明細の出力
	System.out.println(sb.toString());
}

}

これを実行すると、
null 1980円
となります。
ちゃんとした実行結果になる方法を教えてください。

Comments

[dit.]

teratailにはコードを見やすくするためのしくみがあります。質問にコードを記載するときは、https://teratail.com/help/question-tips#questionTips3-5-1を参考に、コードブロックにしてください。コード部分を選択して<code>ボタンを押し、「ここに言語を入力」をjavaにしてください。質問するときのヒントhttps://teratail.com/help/question-tipsに一度目を通しておくといいです。

Answer 1

全体的に書き換えてみました。
Detail クラスのメソッドの整理をします。
それをつかって お米5kgの 要素の作成と表示をします。
また、Integer より大きい数を扱えているかも試しています。

Detail

package jp.co.bbreak.sokusen._2._1;

import java.math.BigDecimal;

public class Detail {
    private String itemName;
    private BigDecimal amount; // 円

    public Detail(String itemName, BigDecimal amount) {
        this.itemName = itemName;
        this.amount = amount;
    }
    public Detail(String itemName) {
        this(itemName, BigDecimal.ZERO);
    }

    public void setItemName(String itemName) {
        this.itemName = itemName;
    }
    public String getItemName() {
        return itemName;
    }
    public void setAmount(BigDecimal amount) {
        this.amount = amount;
    }
    public BigDecimal getAmount() {
        return amount;
    }
    
    @Override
    public String toString() {
        return String.format("%s\t%s円", itemName, amount.toString());
    }
}

PrintReceipt1.java

package jp.co.bbreak.sokusen._2._1;

import java.math.BigDecimal;

public class PrintReceipt1 {
    public static void main(String[] args) {
        // 明細データの作成
        Detail detail = new Detail("お米5kg", new BigDecimal(1980));
        // 明細の出力
        System.out.println(detail);

        // 詳細の加工
        detail.setItemName("お米10kg");
        detail.setAmount(new BigDecimal(2500));
        // 明細の出力
        System.out.println(detail);

        detail = new Detail("お米2kg");
        System.out.println(detail);

        // detail.setAmount(new BigDecimal(900));
        BigDecimal v = new BigDecimal(Integer.MAX_VALUE);
        detail.setAmount(v.multiply(v).multiply(v));
        System.out.println(detail);
        System.out.println(String.format("名前: %s, 値段: %s円", detail.getItemName(), detail.getAmount()));
    }
}

実行例

メモ：
getAmount（）の上のコードでの実装はあまり良くないです。理由と修正方法を考えてみてください。

Answer 2

//明細データの加工
StringBuilder sb = new StringBuilder();
// sb.append(detail.お米5kg()); 以下に変更する
sb.append(detail.getItemName());
sb.append("       ");
sb.append(1980);
sb.append("円");

Answer 3

何を出したいのか分かりませんが、detail.getItemName()を使えばいいんじゃないですか？

Answer 4

そりゃ、「nullを返す」って言ってるメソッド使ってるんだからnullが帰ってくるに決まってるじゃないですか。
そのメソッドの返り値を返したいものに変えれば解決しますよ。

Answer 5

detail.お米5kg()がnullを返す作りになっているので、
書かれた通りプログラムは動いていますよ。(=ちゃんとした実行結果になっています)

まあ、下記メソッドを直せば望む結果になるのではないでしょうか

public Object お米5kg() {
  // TODO 自動生成されたメソッド・スタブ
  return null;
}