почему 2 и 3 условия не проходят?

Вопрос по задаче

User, Loser, Coder and Proger

Java Core, 4 уровень, 8 лекция

Обсуждается

все работает идеально же

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

Ввести [в цикле] с клавиатуры несколько строк (ключей).
Строки (ключи) могут быть такими: "user", "loser", "coder", "proger".
Ввод окончен, когда строка не совпадает ни с одной из выше указанных.
Для каждой введенной строки нужно:
1. Создать соответствующий объект [см. Person.java], например, для строки "user" нужно создать объект класса User.
2. Передать этот объект в метод doWork.
Написать реализацию метода doWork, который:
1. Вызывает метод live() у переданного объекта, если этот объект (person) имеет тип User.
2. Вызывает метод doNothing(), если person имеет тип Loser.
3. Вызывает метод writeCode(), если person имеет тип Coder.
4. Вызывает метод enjoy(), если person имеет тип Proger.

Требования:

Метод main должен считывать строки с клавиатуры.
Метод main должен прекращать считывать строки с клавиатуры, как только введенная строка не совпадает с одной из ожидаемых (user, loser, coder, proger).
Для каждой корректной (user, loser, coder, proger) введенной строки должен быть вызван метод doWork с соответствующим объектом класса Person в качестве параметра.
В классе Solution должен быть реализован метод doWork с одним параметром типа Person.
Метод doWork должен вызывать метод live() у переданного объекта, если этот объект имеет тип User.
Метод doWork должен вызывать метод doNothing() у переданного объекта, если этот объект имеет тип Loser.
Метод doWork должен вызывать метод writeCode() у переданного объекта, если этот объект имеет тип Coder.
Метод doWork должен вызывать метод enjoy() у переданного объекта, если этот объект имеет тип Proger.

package com.javarush.task.task14.task1411; public interface Person { class User implements Person { void live() { System.out.println("I usually just live."); } } class Loser implements Person { void doNothing() { System.out.println("I usually do nothing."); } } class Coder implements Person { void writeCode() { System.out.println("I usually write code."); } } class Proger implements Person { void enjoy() { System.out.println("It's a wonderful life!"); } } }

Комментарии (4)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

Денис Enterprise Java Developer

3 марта 2024, 15:38

Во первых, выучи ключевое слово break; Во вторых, зачем так сложно то? Это не пример задачи на полиморфизм) Достаточно сделать свич по строке, создают нужную персону и вызвать у нее метод... все. Да и если говорить о полиморфизме, то он реализоваться должен иначе, например в интерфейсе Person объявляется метод doWork, который определяется уже в классах этот интерфейс реализующих. Выгода в этом от того что ты можешь не кастовать конкретные инстансы в их классы, а просто запустить один метод у родительского типа.

List<Person> list = List.of(/* some objects */);
list.forEach(Person::doWork);

александр пьянов

Уровень 39

3 марта 2024, 18:55

тогда вопросы: как еще прервать из switch цикл? doWork статический метод, значит весь метод с реализацией засовывается в интерфейс? Еще почему в List<Person> не могу добавлять обекты классов реализующих интерфейс, ввдается ошибка, что non-static variable this cannot be referenced from a static concext.

Денис Enterprise Java Developer

3 марта 2024, 19:39

Ошибку я думаю ты и погуглить можешь, ну или перевести, там все достаточно понятно написано, очевидно никакого отношения к типу объектов эта ошибка не имеет. Вопрос про свичи и циклы не очень понял, но полагаю тебя интересует что-то такое:

public static void main(String[] args) {
    Scanner scanner = new Scanner(System.in);
    boolean shouldRun = true;
    while (shouldRun && scanner.hasNextLine()) {
        switch (scanner.nextLine()) {
            case "user":
                new Person.User().live();
                break;
            case "loser":
                new Person.Loser().doNothing();
                break;
            case "coder":
                new Person.Coder().writeCode();
                break;
            case "proger":
                new Person.Proger().enjoy();
                break;
            default:
                shouldRun = false;
        }
    }
}

Денис Enterprise Java Developer

3 марта 2024, 19:45

Если говорить о подходе с интерфейсным методом можно было бы сделать вот так:

public static void main(String[] args) {
    Scanner scanner = new Scanner(System.in);
    boolean shouldRun = true;
    List<Person> personList = new ArrayList<>();
    while (shouldRun && scanner.hasNextLine()) {
        switch (scanner.nextLine()) {
            case "user":
                personList.add(new Person.User());
                break;
            case "loser":
                personList.add(new Person.Loser());
                break;
            case "coder":
                personList.add(new Person.Coder());
                break;
            case "proger":
                personList.add(new Person.Proger());
                break;
            default:
                shouldRun = false;
        }
    }
    personList.forEach(Person::doJob);
}

Соответственно в каждом классе метод просто переименовывается в doJob(); Но тут сама задача не про это потому всей красоты подхода не видно :)