自動販売機のお釣りプログラム

Question

前提・実現したいこと

Java学習初心者です。
自動販売機のお釣りのプログラムを作っています。
コマンドライン引数に "1 5 10 50 100 500 777 580" と記入したときに、

警告：1円は使えません。
警告：5円は使えません。
777は硬貨として適切な値ではありません。
80円のお釣りです。ありがとうございました。

と表示させたいです。
コマンドライン引数の一番最後(580)が購入したものの値段、
1 5 10 50 100 500 777 が投入した金額です。

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

コマンドプロンプトに、

警告：1円玉は使えません
警告：5円玉は使えません
777は硬貨として適切な値ではありません
580は硬貨として適切な値ではありません
1130円のお釣りです。ありがとうございました

と出てきてしまします。

アドバイス、ヒントなど頂きたいです。
よろしくお願いします。

該当のソースコード

class Jihan2 {
	public static void main(String[] args) {

		int total = 0;
		int change = Integer.parseInt(args[7]);
	
		
		for (int i = 0; i < args.length; i++) {
			
			int coins = Integer.parseInt(args[i]);
			
			if (coins == 1 || coins == 5) {

				System.out.println("警告：" + coins + "円玉は使えません");

			} else if (coins == 10 || coins == 50 || coins == 100 || coins == 500) {

				total += coins;
				change = total - change ;

				
			} else {

				System.out.println(coins + "は硬貨として適切な値ではありません");

			}
		}
		
		System.out.println(change + "円のお釣りです。ありがとうございました"); 

	}
}

補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Comments

[退会済みユーザー]

仕様定義をちゃんと日本語で記述すると良いですよ。
＊現状では緩すぎます。ごく一部の仕様しか定義されていません。
それが出来ればあとは条件分岐としてコードに落とすだけの作業になります。

[tomato]

アドバイスありがとうございます！！
仕様定義をちゃんと日本語で記述するとはどういうことでしょうか？
"前提・実現したいこと" をもっと詳細にしたほうがいいということですか？

[YT0014]

硬貨として有効な数値を全部挙げる、うち使用可能な効果を全部挙げる、引数の数は固定or可変、など、箇条書きで構わないので、できる限り矛盾なく、明確な文言で作成し、質問に追記してください。

Answer 1

totalが10+50+100+500 = 660 、changeが580で、
total-changが、80になると想像しているかもしれませんが。

} else if (coins == 10 || coins == 50 || coins == 100 || coins == 500) {
    total += coins;
    change = total - change ;

をcoinsが10のときに通ると、

total = 10、change = -570
になります。

つぎのループでは、

total = 30、change = 30 - (-570)
となります。

というわけで、

totalが10+50+100+500 = 660 、changeが580

には、なりません。(totalの方は660ですが、changeの方が580でない）

対応は、changeをつどつど変更せずにループを抜けてtotalが決まったあとに

System.out.println((total - change)+ "円のお釣りです。ありがとうございました");

と、するか

changeの計算にchange自身を使わないようにする。
※そもそも、お釣り（change)に対して、商品金額を設定することに違和感がある

int change = 0;
int price = Integer.parseInt(args[args.length -1]);
中略
} else if (coins == 10 || coins == 50 || coins == 100 || coins == 500) {

    change = total - price;

みたいな感じで、変数を別に用意すれば、

System.out.println("総額" + price + "円です。" + total + "円うけとりましたので、");
System.out.println(change + "円のお釣りです。ありがとうございました");

※これも最初の対応のように、chageの計算をループの中でせずに以下のように書くことはできます。

System.out.println("総額" + change + "円です。" + total + "円うけとりましたので、");
System.out.println((total - change) + "円のお釣りです。ありがとうございました");

Comments

[tomato]

"※そもそも、お釣り（change)に対して、商品金額を設定することに違和感がある"
確かに。。。と思いました。
"1130円" もどうして、この数字が出てきたのかわからなかったので教えていただいて助かりました。
とても分かりやすく教えていただきありがとうございます！

Answer 2

アドバイス、ヒントなど頂きたいです。

と言うことですので。まず、main(String[] args)のargs.lengthは、コマンドライン引数の数を表します。これは良いですね。コマンドライン引数に1 5 10 50 100 500 777 580と指定したら、8が得られる、と言うことです。で、 Javaでの配列の添え字は0始まりですから、int change = Integer.parseInt(args[7])は一番最後の580、購入金額が得られます。これも良いですね。

で、コマンドライン引数のargs[0]からargs[7]の一つ前、0～6間をループさせたいのでしょうが、以下のfor文の条件式で正しいでしょうか。よく考えましょう。

for (int i = 0; i < args.length; i++) {

更に、正当な硬貨と判断した時の処理で、

} else if (coins == 10 || coins == 50 || coins == 100 || coins == 500) {

    total += coins;
    change = total - change ;

となっていますが、change = total - change;とは？ 硬貨を投入総額totalに加算するたびにこの処理をするとどういう意味になるか考えてみましょう。あと少しだと思いますので、がんばってください。

Comments

[tomato]

ご指摘いただいたところ再度、考えてみます！
ありがとうございます！

Answer 3

int change = Integer.parseInt(args[7]);

配列の添字は0から始まります。
args配列の最初の要素はargs[0]、2番目の要素はargs[1]……といった感じです。ということは7番目の要素はargs[6]です。