1

常に arg1 の値を返す理由がわかりません。ウェイトコンバーターを作成しています。

public double convert(double arg1,int arg2,int arg3) {
    // arg1 = amount, arg2 = from, arg3 = to
    double milligram = 1;
    double gram = 1000;
    double ounce = 28349.5;
    double pound = 453592;
    double answer = 0;
    switch(arg2) {
    case 0: switch(arg3) { // if milligram
            case 0: answer = (arg1 * milligram) / milligram;
            case 1: answer = (arg1 * milligram) / gram;
            case 2: answer = (arg1 * milligram) / ounce;
            case 3: answer = (arg1 * milligram) / pound;
    }
    case 1: switch(arg3) { // if gram
            case 0: answer = (arg1 * gram) / milligram;
            case 1: answer = (arg1 * gram) / gram;
            case 2: answer = (arg1 * gram) / ounce;
            case 3: answer = (arg1 * gram) / pound;
    }
    case 2: switch(arg3) { // if ounce
            case 0: answer = (arg1 * ounce) / milligram;
            case 1: answer = (arg1 * ounce) / gram;
            case 2: answer = (arg1 * ounce) / ounce;
            case 3: answer = (arg1 * ounce) / pound;
    }
    case 3: switch(arg3) { // if pound
            case 0: answer = (arg1 * pound) / milligram;
            case 1: answer = (arg1 * pound) / gram;
            case 2: answer = (arg1 * pound) / ounce;
            case 3: answer = (arg1 * pound) / pound;
    }
    } // end arg2 switch
    return answer;
}

私は論理のどこかを台無しにしましたが、どこにあるのかわかりません。どんな助けでも大歓迎です。

4

5 に答える 5

9

breakステートメントがありません:

case 0: 
   answer = (arg1 * milligram) / milligram;
   break;
   ...
于 2012-10-06T18:46:05.157 に答える
2

このようにすべてのケースを書き直します

case 2: switch(arg3) { // if ounce
        case 0: answer = (arg1 * ounce) / milligram;break;
        case 1: answer = (arg1 * ounce) / gram;break;
        case 2: answer = (arg1 * ounce) / ounce;break;
        case 3: answer = (arg1 * ounce) / pound;break;
于 2012-10-06T18:47:20.403 に答える
0

ネストされた case ステートメントは非常に読みにくく、デバッグが困難です。代わりに、必要な機能をメソッド呼び出しにカプセル化することをお勧めします。

そうは言ってbreakも、スイッチのどこにもステートメントはありません。ケースに関係なく、最後のケースまでフォールスルーします (ケースの底に等しい答えを設定します)。

于 2012-10-06T18:46:14.083 に答える
0

breakswitchステートメントで使用する必要があります。そうしないと、結果が損なわれる可能性があります

于 2012-10-06T18:47:27.590 に答える
0

using 以外breakに、 usingreturnステートメントはうまく機能するだけでなく、ケースをすり抜けることも防ぎます。

case 2: switch(arg3) { // if ounce
    case 0: return (arg1 * milligram) / milligram;
于 2012-10-06T18:52:28.270 に答える