returnステートメントってなんですか？

Question

硬貨の入れ物のクラスCoinCaseです。```java
class CoinCase{
//硬貨のそれぞれの枚数
private int FiveHundred = 0;
private int Hundred = 0;
private int Fifty = 0;
private int Ten = 0;
private int Five = 0;
private int One = 0;

//xは硬貨の種類、yは硬貨を入れる枚数
void AddCoins(int x,int y){
switch(x){
case 500:FiveHundred += y;break;
case 100:Hundred += y;break;
case 50:Fifty += y;break;
case 10:Ten += y;break;
case 5:Five += y;break;
case 1:One += y;break;
}
}

//xで指定した硬貨が何枚あるかを返す
int GetCount(int x){
switch(x){
case 500:return FiveHundred;
case 100:return Hundred;
case 50:return Fifty;
case 10:return Ten;
case 5:return Five;
case 1:return One;
}
return 0;//ここです。
}

//総額を返す
int GetAmount(){
return (FiveHundred500)+(Hundred100)+(Fifty50)+(Ten10)+(Five*5)+(One);
}

}

メソッドGetCountの「return 0」というのがわかりません。この部分を書かないとeclipseで「このメソッドはint 型の結果を戻す必要があります。」「returnステートメントを追加します。」と表示されます。これってどういうことですか？ switch文内のreturnで結果を戻せてはいないのですか？
それぞれのreturnのあとでbreakをいれると「到達不能コード」と出ますし…

Answer 1

switchで見ているxが、ブロック中のいずれのcaseにも合致しなかった場合、
何もreturnしないままswitch文を抜けてしまいます。
コンパイル段階でこのメソッドにこれらの値しか渡らないことは保証できません。
返り値をintで指定している以上、何も値を返さずに(または、例外を発生させずに)メソッドを終了することはできないので、エラーになってしまうのです。

この仕様だったらMapを使ったほうが簡潔に書ける気がします。

class CoinCase{
    private Map<Integer, Integer> coins;

    public CoinCase(){
        coins = new HashMap<Integer, Integer>();
        coins.put(1, 0);
        coins.put(5, 0);
        coins.put(10, 0);
        coins.put(50, 0);
        coins.put(100, 0);
        coins.put(500, 0);
    }

    void addCoins(int x, int y){
        if(!coins.containsKey(x)) return;  //あるいは例外をスロー
        coins.put(x, coins.get(x) + y);
    }

    int getCount(int x){
        Integer temp = coins.get(x);
        return temp != null ? temp.intValue() : 0;
        //Java8なら次の1文で可能
        //return coins.getOrDefault(x, 0);
    }

    int getAmount(){
        //Java7以前
        int result = 0;
        for(Map.Entry<Integer, Integer> e : coins.entrySet()){
            result += e.getKey() * e.getValue();
        }
        return result;

        //Java8ならこう書ける
        //return coins.entrySet().stream()
        //            .mapToInt(x -> x.getKey() * x.getValue())
        //            .sum();
    }
}

Comments

[退会済みユーザー]

おお。いつもありがとうございます。参考にさせていただきます。

Answer 2

caseの中のbreakは、case文から抜けるという意味です。
returnはそこに到達した時点でそのメソッドから抜けて値を返します。
なので、returnの後ろにbreakを入れても
「returnで返っちゃうからbreakは絶対に実行されないよ」
と警告しているのですね。

addCoinsもaddCoins(999,999)みたいに呼び出されたらどのcaseにも当てはまらないです。
でも「無視して良い」内容なので何も記述がないのです。

個人的には例え面倒でも、case文の中にreturnを羅列せずに済ます方法を取りたいですねー。

Answer 3

caseに該当しない場合の処理がないため、switch外でのreturnが必要になります。

たとえば、GetCountの引数として、1000が与えられた等の場合、
どのcaseにも該当しないため、returnしなくなってしまいます。

実際にそのクラスを使う側で1,5,10,50,100,500以外を渡さないと決めていたとしても、
ロジックとしてはreturnしないパターンがあるため、エラーとなります。

Answer 4

int xとして、たとえば385とか、switchにない値が来た時のために必要です。

もしそんな値が来ないというのであれば、そもそも型に汎用のintではなくて硬貨の種類を決めたEnumを使う、あるいはどうしてもintでなければいけないというのであれば、条件を外れたところでは例外を投げるようにしても大丈夫です。