When refactoring if-else if-else statements containing a break statement into a switch construct, the break label gets misplaced:

Code before refactoring:

public class SwitchRefactoringError {
    static void test(int t) {
        int r=0;
        while(true) {
            if (t==1) {
                r=10;
            } else if (t==2) {
                r=9;
            } else {
                r=8;
                break;  // Terminates the while loop
            }
        }
    }
}


Code after applying the proposed refactoring into a switch statement:


public class SwitchRefactoringError {
    static void test(int t) {
        int r=0;
        OUTER:
        while (true) {
            switch (t) {
                case 1:
                    r=10;
                    break;
                case 2:
                    r=9;
                    break;
                default:
                    r=8;
                    break OUTER; // Terminates the while loop. NOPE it restarts
            }
        }
    }
}


Ihmo, the label should be placed *after* the while loop, not before.

Warm regards
Ivo