戻り値の扱いについて

Question

前提・実現したいこと

今金額を計算するプログラムを作っているのですがなぜかエラーになってしまいます。
やりたいことは、コマンドライン引数の値を足して合計が1000円より大きく5000円以下なら-10％し、
5000円より大きいのであれば-20％するといったものが作りたいです。

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

Exception in thread "main" java.lang.Error: Unresolved compilation problems: 
	代入の左側は変数でなければなりません
	構文エラーがあります。"AssignmentOperator Expression" を挿入して Assignment を完了してください
	構文エラーがあります。";" を挿入して Statement を完了してください

	at Sample4.discount(Sample4.java:36)
	at Sample4.main(Sample4.java:16)

該当のソースコード

public class Sample4 {

	public static void main(String[] args) {
		
		int total = 0 ;
		
		for(int i = 0 ; i < args.length ; i++) {
			
			total = total + Integer.parseInt(args[i]) ;
			
		}
		
		int totalTax = calcTax(total) ;
		
		int payment = discount(totalTax) ;　　　　　　　　　//16行目
		
		System.out.println("合計金額 :" + payment) ;
		

	}
	
	static int calcTax(int total) {
		
		double calcTax = total * 1.08 ;
		
		return (int)calcTax ;
	}
	
	static int discount(int totalTax) {　　　　　　　　　　　　 //30行目
		
		if(totalTax > 1000 & totalTax <= 5000) {
			
		double	discount = (totalTax - 1000) * 0.9 ;
		
		return (int)discount ;
			
		}else if(totalTax > 5000){
			
			double discount = (totalTax - 5000) * 0.8 ;
			
			return (int)discount ;
		}
	}

}

試したこと

かれこれ30分以上見返していて、if節の中にreturn文があるからなのかなー、と思ったりしてるのですがreturn文を
うまく外に出してあげる方法も思いつきません。
いろいろ試していて一部のコードを変えるたび、問題がなかったところでもエラーが起きたりしたので
問題が検出されているところ以外でもなにかおかしなところがあるかもしれません。
どなたかデバッグをお願いいたします。

Answer 1

税抜価格から割引したあと、8% 課税するならこんな感じですね。

public class Sample4 {

    public static void main(String[] args) {
        int total = 0;

        for (String arg : args) {
            total = total + Integer.parseInt(arg);
        }

        int discount = discount(total);
        int payment = calcTax(discount);

        System.out.println("合計金額 :" + payment);
    }

    static int calcTax(int total) {
        double calcTax = total * 1.08;

        return (int)calcTax;
    }

    static int discount(int total) {

        double discount;

        if(total > 1000 && total <= 5000) {
            discount = total * 0.9;
        }else if(total > 5000){
            discount = total * 0.8;
        }else {
            discount =  total;
        }

        return (int)discount;
    }
}

Comments

[NEKOTABETAI]

回答ありがとうございます。この方法が一番よさそうです。

Answer 2

keicha_hrsさんの回答にありますが

discount()のif〜else ifのどちらの条件にも合致しなかった場合の戻り値がないからではありませんか？

ですね。
つまり、割引していないケースがreturnされていません。

なんか気になったのですが、割引って税込み価格でやります？
※合計1000円の場合って、割引なしで1080円にするのでは？
2000円の10%引きって、1800円+税じゃないの？
※実際に表示される金額が・・・
仕様の提示が間違ってるのかな・・・

	static int discount(int totalTax) {

		if (totalTax > 1000 & totalTax <= 5000) {

			double discount = (totalTax - 1000) * 0.9;

			return (int) discount;

		} else if (totalTax > 5000) {

			double discount = (totalTax - 5000) * 0.8;

			return (int) discount;
		}

		return 0;
	}

Comments

[momon-ga]

おっと、discountは割引額じゃなくて割引後の金額か。
returnは0でなくて、totalTaxですね。

[NEKOTABETAI]

回答ありがとうございます。確かに割引してないときの戻り値がないのが原因でした。
金額の合計が確かにおかしいですね。見直してみます。

Answer 3

discount()のif〜else ifのどちらの条件にも合致しなかった場合の戻り値がないからではありませんか？

それと、

        if(totalTax > 1000 && totalTax <= 5000) {

のように&&にしなければ期待する結果は得られないでしょう。

Comments

[NEKOTABETAI]

回答ありがとうございます。＆＆が「AかつB」という事でしょうか

[keicha_hrs]

そうです・・・と言いたいところだけど、この場合はどちらでも良かったのか。そこは申し訳ないです。

ところで、discount()メソッドについてなんですが、

double pay1 = (totalTax - 1000) * 0.9 ;
や
double pay2 = (totalTax - 5000) * 0.8 ;
という計算をされていますが、この1000引いたり5000引いたりする計算は必要なのでしょうか？また、税込価格に対して計算するという条件なのですか？

今の式だと、例えば3000円のものがあったら、課税した3240円から1000円引いた2240円に0.9倍した2016円が答えになってしまいますが・・・これではおかしいですよね？

[keicha_hrs]

momon-gaさんのおっしゃるように、割引計算を課税前の価格に対して行うのか、課税後の価格に対して行うのかという仕様の再確認も必要そうですね。

[Marie]

本筋ではないですが、&&と&の違いについて少々。

A && B
AがFalseならBは評価せずにFalseを返します。(短絡演算)
A & B
Aの評価値によらず、必ずBも評価します。

つまり、使い分けとしては、
Bの内部で副作用を与えない場合には&&を使い、
Bの内部で副作用を与える場合には&を使います。

&&を使うべき場面で&を使用すると、結果は変わりませんが、無駄な計算が発生することになるため、どちらでも良いわけではありません。

[NEKOTABETAI]

@keicha_hrsさん回答ありがとうございます。もう一度仕様を見直してみます。

[NEKOTABETAI]

@Marieさん補足説明ありがとうございます。今回の場合だと＆＆が適切だったのですね。

Answer 4

関数名と同名の変数が定義されてます。
これダメです

そして、同名の変数を複数箇所で定義してますが、これもダメです

Comments

[NEKOTABETAI]

```Java

public class Sample4 {

public static void main(String[] args) {

int total = 0 ;

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

total = total + Integer.parseInt(args[i]) ;

}

int totalTax = calcTax(total) ;

int payment = discount(totalTax) ;

System.out.println("合計金額 :" + payment) ;

}

static int calcTax(int total) {

double calcTax = total * 1.08 ;

return (int)calcTax ;
}

static int discount(int totalTax) {

if(totalTax > 1000 & totalTax <= 5000) {

double pay1 = (totalTax - 1000) * 0.9 ;

return (int)pay1 ;

}else if(totalTax > 5000){

double pay2 = (totalTax - 5000) * 0.8 ;

return (int)pay2 ;
}
}

}
```

直してみましたがこういう事であっていますか？
あとこれでも戻り値がかえってなさそうです。

[momon-ga]

低評価しました。
>関数名と同名の変数が定義されてます。
>これダメです

そんなことないでしょ。

> そして、同名の変数を複数箇所で定義してますが、これもダメです
ダメではありますが、今回のケースは関係ない（ブロックが違うので定義できる）

[kazuma-s]

> 関数名と同名の変数が定義されてます。これダメです
そんなことはありません。
public static void main(String[]args) { int main = 23; System.out.println(main); }

[NEKOTABETAI]

皆さん回答ありがとうございます。なんか同じ名前使ったら大変そうですから
次から気を付けます。