First: You're missing break;
in your switch statement for each of the cases, except case 4 (not sure if it's by design).
Second: Choice is int
, your cases are char
s. Change one of them, so that both of them match. You're never finding a match for your case, so you just keep on looping.