JavaでArrayList版の計算結果が出ない。

###前提・実現したいこと
プログラミング初心者で自分一人ではできていませんが、宜しくお願いします。
ここに質問したいことを詳細に書いてください
javaで買い物データとして、商品名、単価、数量を最大１０組まで入力。
入力が終わったら、データリストの明細を表示し、合計金額を表示する。
というのを作ったのですが、合計金額が出ずに困っています。

###発生している問題・エラーメッセージ

エラーは出ませんが、合計金額が出ません。

###該当のソースコード
import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStreamReader;
import java.util.ArrayList;
import java.util.List;

public class ShoppingCalculation4_5 {

private static final int MaxSize = 10;
public static void main(String[] args) throws IOException {
	BufferedReader br = new BufferedReader(new InputStreamReader(System.in));

	
	List<String> goods = new ArrayList<>(MaxSize);
	List<Long> prices = new ArrayList<>(MaxSize);
	List<Long> quantities = new ArrayList<>(MaxSize);
	
	System.out.println("(入力は１０個まで、または単価か数量に「-1」入力で終了");

	try{
	for (int i = 0; i < MaxSize; i++) {
		System.out.println(String.format("\n========== 商品データの入力 %d ==========",i + 1));

		System.out.print("商品名 --> ");
		goods.add(readName (br));

		System.out.print("単 価 --> ");
		prices.add(readNumber (br));

		System.out.print("数 量 --> ");
		quantities.add(readNumber (br));
		

		if (prices.equals("-1") || quantities.equals("-1")) {
			break;
		}

	}
	
	System.out.println("\n==========");
	System.out.println("【明細】");

	long sum = 0;
    for (int i = 0; i < goods.size(); i++) {
        sum += Amount(goods.get(i), prices.get(i), quantities.get(i));
    }
    System.out.println("- - - - - - - - - - - - - - - - - - - - -");
    System.out.println(String.format("合計; %d 円\n税込み; %d 円", sum, (long) (sum * 1.08)));
} catch (IllegalArgumentException e) {
    System.out.println(e.getMessage());
} catch (Exception e) {
    System.out.println("入力の受付を終了しました。");
}
	
	}

private static long Amount(String goods, Long prices, Long quantities) {
	long amount = prices * quantities;
    System.out.println(
       String.format("【%s】; %d × %d = %d 円", goods, prices, quantities, amount));
    return amount;
}

private static Long readNumber(BufferedReader br){
	 try {
            long num = Long.parseLong(br.readLine());
            if (num == -1) throw new TerminateException();
            if (num < 0) throw new IllegalArgumentException();
            return num;
        } catch (TerminateException e) {
            throw e;
        } catch (Exception e) {
            System.out.println("整数値を入力してください");
            return readNumber(br); // 再帰
        }
	        }
private static String readName(BufferedReader br) {
       try {
            String name = br.readLine();
            if (name == null || name.isEmpty()) throw new IllegalArgumentException();
            return name;
        } catch (Exception e) {
            System.out.println("名称は必須入力です");
            return readName(br); // 再帰
        }
}
   private static class TerminateException extends RuntimeException {
        private static final long serialVersionUID = 1L;
    }
}

###試したこと

###補足情報(言語/FW/ツール等のバージョンなど)
Java
eclipse

abs123

2016/05/27 09:39

入力値、期待する出力値、現状の出力値、を出来れば追加してください。

HogeAnimalLover

2016/05/27 12:05

何も表示されないなら途中でprintlnでもしてどこまで進んでいるか確認すればいいですよ

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

回答3件

計算結果のところは既に回答がついていますので、それ以外の点について。

catch文の例外は無視しないようにしましょう。
catch文で例外を「握りつぶす」と、どこで例外がスローされて処理が中断したのかが分かりにくくなります。

mainメソッドのcatch文だけは、下のようにしてスタックトレースが出力されるようにしておくほうが良いです。
そうすれば、どこでその例外が発生して処理が中断したのかが追いやすくなります。

lang
1} catch (IllegalArgumentException e) {
2    e.printStackTrace();
3    System.out.println(e.getMessage()); 
4} catch (Exception e) { 
5    e.printStackTrace();
6}

...

それと、A-pZさんの書かれているように、処理を中断するために例外をスローするのは良くない作法です。
ではどうすれば良いのかというと、ちょっとだけ説明すると、

（Optional使う例なのでJava8限定）
例えば、readNumberの戻り値にOptionalLongを使って例外の代わりにOptionalLong.empty()を返し、呼び出し元でreturnなどで終了処理するのが良いです。

投稿2016/05/27 11:15