コマンドライン引数で受取った値をfor文の中で偶奇わけしたい

Question

ケーキの名前と個数をコマンドラインから受け取り計算して合計を示すケーキ屋さんのプログラムを作りたいです。

例えば、コマンドラインに「ショートケーキ 2 モンブラン 1」と打った時、ショートケーキの値段×2+モンブランの値段×1で合計を算出できるようになってほしいです。

わたしはコマンドラインで受取る配列の番号に着目し、偶数(0,2,4,...)ならケーキの種類、奇数(1,3,5,..)ならケーキの個数で、奇数と偶数をワンセットで([0][1],[2][3],...)繰り返し処理を行えば良いと考え、以下のようなプログラムを書きました。

class Cakeshop{
	public static void main(String[] args){
		int cake = 0 ; //cakeの初期化
		int sum = 0 ; //sumの初期化
		int each_price = 0 ; //each_priceの初期化
		
		for( int i=0 ; i<args.length/2 ; i++ ){
			//コマンドラインに打ち込まれたケーキ名によってそのケーキの値段を決める
			if( args[2*i]=="ショートケーキ"){
				cake = 320 ;
			}else if( args[2*i]=="モンブラン" ){
				cake = 350 ;
			}else if( args[2*i]=="チョコレートケーキ" ){
				cake = 370 ;
			}else if( args[2*i]=="いちごのタルト" ){
				cake = 400 ;
			}else{
				cake = 300 ;
			}
			
			each_price = cake * Integer.parseInt(args[2*i+1]) ;　//各ケーキの値段×個数から出た値
			sum = sum + each_price ;　//各ケーキの値段×個数から出た値を加算していき、合計を求める
		}
		
		System.out.println( "合計金額は" + sum + "円です。" ) ;
	}
}

すると、エラーは出てきませんでしたが、全てどのケーキも300円として計算されてしまいます...。場合分けの方法が間違ってるのでしょうか。

Answer 1

コマンドラインで受取る配列の番号に着目し、偶数(0,2,4,...)ならケーキの種類、奇数(1,3,5,..)ならケーキの個数で

ということですので, もう一工夫されると良いのではと思いました.
それは for ループの変数 i の扱いについてです.

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

という書き方によって i が 0 から[種類・個数の組]の数(=args.length/2)分ループしていますが, これによって配列の番号に用いる場合に i を 2 倍する必要が生じています.

これを

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

としますと, i が 0 から 2,4,6,... と常に偶数を指すことになり, [種類・個数の組]の"種類"の配列の番号を指すようになります.
ですので, args[2*i] は args[i] となり, args[2*i+1] は args[i+1] となって, i を 2 倍する必要が無くなります.

Comments

[chimo]

なるほど！iの進め方を変えるんですね！！iをゼロから一つ飛ばしで進ませることで、偶数のみをさすようにするという画期的な方法があるとは思いませんでした！！メモして、以降きちんとシステムの中でつかえるように心がけます！！とても勉強になりました！コメントありがとうございました！

[jimbe]

ただ, この変更は正常処理時は動作はオリジナルと変わりませんが, ある異常時には異なります.
それは, [種類・個数の組]がちゃんと入力されなかった時, すなわち, 個数を入力し忘れた等で args.length が奇数だった時です.
「ショートケーキ 2 モンブラン」とモンブランの個数を入力し忘れた時, オリジナルではショートケーキ 2個の計算だけして結果が表示されますが, 変更した方ではエラー(例外)が表示されます.
これはどちらが正しいというよりは仕様としてどちらにするかという問題で, 件では練習問題のようでしたので正常時のみ動作が一致することで良しとしています.

[chimo]

なるほど…いわれてみるまで気づきませんでしたが、そういうエラーも起こってしまうのですね。とても勉強になりました！ありがとうございます！！

Answer 2

if( args[2i]==ショートケーキ){

2iというキーワードが意味不明です
もしかして、2*i のつもりなのかな？

---

Cakeshop.java:7: error: cannot find symbol

for( i=0 ; i<args.length/2 ; i++ ){
^

iという変数が定義されてないってことですね
関数の最初にint i; をいれるか、
for(int i=0 ; i<args.length/2 ; i++ ){
とするか、ですね

＃エラーメッセージを読んで、なんと書いてあるのか読めるようにしよう

---

文字列と文字列の比較 - Stringクラスを使った文字列処理 - Java入門

文字列の比較は==ではできません
if( args[2*i].equals("ショートケーキ")){
とする必要がありますね

Comments

[chimo]

そうです！！そこが問題だったんですかね…数学の感覚でいてました…修正してもう一度コンパイルしてみます。

[chimo]

修正してみましたがやはりエラーが出てしまいます・・・。

[y_waiwai]

なら、質問を編集して、そのエラーメッセージを提示しよう

一つエラー潰してもそれで終わりじゃありませんよｗ

[chimo]

あ、ごめんなさい！質問編集して新たにでたエラーも追記しています。回答していただいたのに追記のお伝えが遅れてしまい申し訳ないです！！

[y_waiwai]

回答に追記しました

[chimo]

回答の追記ありがとうございます。iにintを付け加えて、また自分でもエラーを読んで気づけたこととしてはモンブランやショートケーキを""で囲んでいなかったのでそこも付け加えてコンパイルしてみた結果、コンパイルは通りましたが、全てのケーキが300円として計算されてしまいます...。一応質問にあげたコードも編集しております。

[y_waiwai]

回答に追記しました

[chimo]

あああ…equals忘れてました！！ありがとうございます！！！