Duplicate branch in "switch"

Ну я подумал, что это цикл. А, понял. Этого куска тут просто нет ) Вставлю ниже.

ppublic static void main(String[] args) throws Exception { //ввести с консоли несколько ключей (строк), пункт 7 BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); String key = ""; Movie movie = null; while (!(key == null)) { key = reader.readLine(); movie = MovieFactory.getMovie(key); switch(key) { case "soapOpera" : System.out.println(movie.getClass().getSimpleName()); continue; case "cartoon" : System.out.println(movie.getClass().getSimpleName()); continue; case "thriller" : System.out.println(movie.getClass().getSimpleName()); continue; default: key = null; // System.out.println(movie.getClass().getSimpleName()); }

Я знаю, что можно без свича. Пробую решать задачи по разному, а то потом забуду все, что учил до этого. Да и читаемость кода со свичом мне нравиться.

1. У тебя key никогда не будет null, ты его проинитил как "". 2. Вообще неясна логика применения тут switch. В методе main() тебе лишь нужно в цикле делать

....  {
               movie = MovieFactory.getMovie(key);
               System.out.println(movie.getClass().getSimpleName());
           }
           else {

Вывод будет зависеть от того, объект какого из наследников Movie ты занесёшь в переменную movie внутри метода movie = MovieFactory.getMovie(key); Кстати, как раз внутри getMovie() и можно было бы применить свитч, но если я правильно помню, там уже часть была реализована через if, и предлагалось сделать по аналогии.

Как это key не будут null? Я же его меняю в консоле? И как тогда у меня программа выходит из цикла? Я знаю, что можно было проще решить задачу, хотелось как нить по своему написать. Вставил свич, но из него выбивает после каждого кейса. Вставил в него continue, все работает, но идея ругается на дубликат. Т.е. меня интересует не как можно эту задачу по другому решить, а что можно сделать вместо дубликата, что бы не выкидывало из программы после каждой строчки.

break ставить не пробовал?

А Идея ругается, потому что у тебя несколько case с одинаковым содержимым. Это записывается так:

switch(key) {
    case "soapOpera" :
    case "cartoon" :
    case "thriller" :
        System.out.println(movie.getClass().getSimpleName());
        break;
    default:
        key = null;
}

Во, спс! Такого почему то ни где не видел ). Теперь код одинаково исполняется если break или continue