Java 複数入力する金種計算について

Question

###前提・実現したいこと
Java勉強中の初心者です。言葉足らずな所がありましたら教えて頂けたらと思います。

現在、入力された複数の金額の通貨毎の合計枚数を算出するというプログラムを組んでいます。
金額を複数入力し"0"を入力されたときに次の処理へ進み枚数計算処理をし出力するという流れまでは出来たのですが、結果がどうしても最後に入力された金額の枚数しか表示されません。

見直してコードもいじったりしてみたのですが、どうしても上手くいきません。
どこの処理がおかしいのか教えていただきたいです。
宜しくお願いします

###該当のソースコード

import java.util.Scanner;
import java.util.*;
class Maisu {
	
	//お金の種類の配列と枚数を格納する空配列を作る
	static int[] moneyList = {10000,5000,1000,500,100,50,10,5,1};
	static int[] emptyMoney= new int[moneyList.length];
	
	public static void main(String[] args) {
		//いくつ金額が入力されるか分からないのでListを使う
		List<Integer> list = new ArrayList<Integer>();
		
		//複数金額入力部、複数計算部、結果出力部各メソッドを呼ぶ
		multipleInput(list);
		multipleCalculate(list);
		output();
	}
	
	//複数金額入力部メソッド
	//入力された金額が0でなければリストへ追加
	//0であればbreakする
	static void multipleInput(List<Integer> list){
		while(true){
			int okane = moneyInput();
			if(okane == 0){
				break;
			}else{
				list.add(okane);
			}
		}
	}
	
	//金額入力部メソッド
	static int moneyInput(){
	System.out.println("金額を入力してください");
	Scanner sc = new Scanner(System.in);
	int money;	
		
		//return文が実行されたときループを抜ける
		while(true){
			String okane=sc.next();
			//整数ではない金額(小数点など)が入力されたときにエラーを表示する処理
			try{
				money = Integer.parseInt(okane);
				
			}catch(NumberFormatException e){
				System.out.println("正しい金額を入力してください");
				continue;
				
			}
			//入力された金額が1円未満だった場合は再入力を促す
			if(money <0){
				System.out.println("正しい金額を入力してください");
			}
				return money;
		}
	}
	
	//複数計算部メソッド
	static void multipleCalculate(List<Integer> list){
		//格納された金額の数だけfor文で回す
		for(int n=0;n<list.size();n++){
			//計算部メソッドを呼ぶ
			calculate(list,n);
		}
	}
	
	//計算部メソッド
	static void calculate(List<Integer> list,int x){
		int money;
		
		money = list.get(x);
		
		for(int j=0; j<moneyList.length;j++){
			emptyMoney[j] = money / moneyList[j];
			money %= moneyList[j];
			
		}
	}
	
	//計算結果出力部メソッド
	static void output(){
		for(int x=0; x < moneyList.length; x++){
			if(0 < emptyMoney[x]){
				System.out.println(moneyList[x]+"円\t"+emptyMoney[x]+"枚");
			}
		}
	}
}

Answer 1

初心者にしてはきれいなソースですね。

for(int n=0;n<list.size();n++){
    //計算部メソッドを呼ぶ
    calculate(list,n);
}

↑によって、calculateメソッドが何回か呼ばれていますが、calculateメソッドの中の

    for(int j=0; j<moneyList.length;j++){
        emptyMoney[j] = money / moneyList[j];

の部分で、前回計算した emptyMoney[j] を踏みつぶしちゃってます。

Comments

[minty]

matobaaさんとngsvxさんの回答を見て一度メソッド毎に分解して考え直した結果
何故処理が上手くいかないかを理解することが出来ました。(最終的に合計金額を出す処理を
加えることによって上手く動作しました...。)
参考書を元に綺麗に見やすいコーディングを心がけているので綺麗なソースと言っていただけて
本当に嬉しいです。今後も頑張って勉強を続けていこうと思います。
ありがとうございました。

Answer 2

原因については、既に回答がありますのでいいかと思います。

ただ、通常、金種計算の際は、全体だけではなく、個別の情報も必要なことがほとんどです。
ですので、個別の算出を行って、最後にそれを合計するという作りにしておけば、
わかりやすいのではと考えます。

ついでに、結果論になりますが、そうしておけば今回の間違いもなかったのではないかと。