MovieFactory

while ((key = reader.readLine()) != null) - это по сути while(true)? И выход только через return?... Имхо лучше так:

while ((movie = MovieFactory.getMovie(reader.readLine())) != null){
            System.out.println(movie.getClass().getSimpleName());
        }

Почемуто не принимало решение

Movie movie = MovieFactory.getMovie(key);
            if(movie == null) break;
            System.out.println(movie.getClass().getSimpleName());

а в обратной последовательности приняло

Movie movie = MovieFactory.getMovie(key);
            if(movie != null) {
                System.out.println(movie.getClass().getSimpleName());
            } else {
                break;
            }

Вызов функции с хреновой строкой кидает исключение, связанное с тем что метод возвращает null Не парьтесь и оберните в try catch))

Я один через ArrayList решаю?)) Через валидатор еще не проверял, но работает правильно. Коплю по 20 решенных задач, потом проверяю в один день, собираю ачивки))

public class Solution {
    public static void main(String[] args) throws Exception {
        Scanner console = new Scanner(System.in);
        String movieString;
        ArrayList<String> moviesList = new ArrayList<>(Arrays.asList("cartoon","thriller","soapOpera"));
        ArrayList<String> addMoviesList = new ArrayList<>();
        while (true) {
            movieString = console.nextLine();
            if (moviesList.contains(movieString)) {
                addMoviesList.add(movieString);
            }
            if (!moviesList.contains(movieString)) {
                MovieFactory.getMovie(movieString);
                console.close();
                break;
            }
        }
        addMoviesList.forEach(movie -> MovieFactory.getMovie(movie));
    }

    static class MovieFactory {

        static Movie getMovie(String key) {
            Movie movie = null;

            if ("soapOpera".equals(key)) {
                movie = new SoapOpera();
                System.out.println(movie.getClass().getSimpleName());
            }
            if ("cartoon".equals(key)) {
                movie = new Cartoon();
                System.out.println(movie.getClass().getSimpleName());
            }
            if ("thriller".equals(key)) {
                movie = new Thriller();
                System.out.println(movie.getClass().getSimpleName());
            }
            return movie;
        }
    }

    static abstract class Movie {
    }

    static class SoapOpera extends Movie {
    }
    static class Cartoon extends Movie {

    }
    static class Thriller extends Movie {

    }
}

Мне одному хочется переломать ноги сказать спасибо тому кто писал условия задачи и комментарии внутри программы ?

не понимаю почему нул?

Интересная задача, но как же бесят эти вложенные статические классы.

Правильное решение вообще порадовало 😁

while ((key = reader.readLine()) != null) {
           Movie movie = MovieFactory.getMovie(key);
           if (movie == null) {
               return;
           }

Подсказка. Для некорректной строки не нужно выводить на экран значение. Нужно просто вызвать у нее метод MovieFactory.getMovie(некорректная строка);

Хочу тебя, коллега, предупредить. Валидатор не примет решение, если твоя прога при вводе "некорректной" строки, кидает NullPointerException. Рекомендую проверять корректность введенной строки не так, как ты себе это представляешь (в цикле while), а проверять тот объект, который будет создан на основании этой "некорректоной строки". Расшифровывать не буду ибо спойлеры.