Вопрос по logic, switch-statement, java – Проблема с использованием переключателя в Java

1

Я могу'не понимаю, почему этоs всегда возвращает значение 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;
}

Я перепутала где-то в моей логике, но яЯ не вижу, где. Любая помощь будет оценена.

@irrelephant Я предпочитаю использовать операторы switch, поскольку онимне легче понять. У меня нетЯ использовал их долгое время, поэтому я полностью забыл о необходимости разбивать каждый случай. dead beef
Я только предлагаю это, так как выможно было бы сократить код примерно с 30 строк до 2 :-) irrelephant
В этом случае было бы лучше использовать массивы и вообще убрать переключатели. Что-то вродеanswer = (arg1 * conv[arg2]) / conv[arg3]; irrelephant
Почему Дон'T вы просто назвать свои аргументы сумму, от и до? Brendan Long

Ваш Ответ

5   ответов
0

Кроме использованияbreak, с помощьюreturn заявления делает свое дело так же, как это также предотвращает падение через случаи.

case 2: switch(arg3) { // if ounce
    case 0: return (arg1 * milligram) / milligram;
0

ки. Это'Лучше было бы вместо этого инкапсулировать нужные вам функции в вызов метода.

Как говорится, тамнет нетbreak заявление в любом месте вашего коммутатора. Независимо от случая, он будет проходить до последнего случая (установив ответ, равный основанию случая).

9

Вам не хватаетbreak заявления:

case 0: 
   answer = (arg1 * milligram) / milligram;
   break;
   ...
Спасибо, это сработало. dead beef
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;
0

break в утверждениях переключателя, иначе ваш результат может получить detoriated

Похожие вопросы