JavaRush/Помощь по Java задачам/Программа работает в Идее. Что м.б не так?

44 уровень

25.09.2022
358просмотров
22комментариев

Программа работает в Идее. Что м.б не так?

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

Решен

Компилятор ругается:

Убедись, что при вводе с консоли слова user создается экземпляр класса User

Вывод в консоль по результатам выполнения программы такой:

I usually just live.
I usually do nothing.
I usually write code.
It's a wonderful life!

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

Ввести [в цикле] с клавиатуры несколько строк (ключей).
Строки (ключи) могут быть такими: "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 { public static void live() { System.out.println("I usually just live."); } } class Loser implements Person { public static void doNothing() { System.out.println("I usually do nothing."); } } class Coder implements Person { public static void writeCode() { System.out.println("I usually write code."); } } class Proger implements Person { public static void enjoy() { System.out.println("It's a wonderful life!"); } } }

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

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

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

Гофф

Уровень 20

Master

25 сентября 2022, 14:08

1. (!(key.equals("user")) вызовет ошибку, если key окажется null, лучше будет ("user".equals(key)) 2. дефолт в первом свитче не нужен, ты перд свитчём проверил, что значение - одно из четырёх, дефолт заведомо никогда не сработает. Но оптимальнее было бы как раз наоборот - убрать эту проверку перед свитчём, тогда бы вот дефолт пригодился, можно было бы в дефолт просто засунуть break, что бы выйти из цикла в соответсвии с условиями. 3. Метод doWork надо расписывать в каждом подклассе отдельно, а не один для всех в классе Solution.

Grock

Уровень 44

25 сентября 2022, 14:17

Спасибо за ответ. По п. 1. В Идее ошибки нет. Код пускай и корявый, но рабочий. Но спасибо за совет. По п. 2. Верно. Поправлю. По п. 3. Метод doWork по условиям задачи должен определять тип объекта класса и вызывать метод этого класса, указанный в задаче. Поэтому расписывать doWork нужно только в классе Solution.

Гофф

Уровень 20

Master

25 сентября 2022, 14:23

Я расписывал в каждом классе, что логично, ибо это свойство именно класса. Грубо говоря, оно должно работать не только из Solution, а из любого другого класса тоже = логика работы должна быть расписана не в Solution, а в самих классах.

class Proger implements Person {
    @Override
    public void doWork() {
        System.out.println("It's a wonderful life!");
    }
}

Но ничто не вечно, возможно, они изменили условия задачи и хотят именно нелогичное решение, такое здесь бывает сплошь и рядом...

Grock

Уровень 44

25 сентября 2022, 14:26

Думаю, да. Видимо изменились условия. Сейчас там это выглядит так:

class Proger implements Person {
    public void enjoy() {
        System.out.println("It's a wonderful life!");
    }
}

Т.е. создан метод enjoy(), который и нужно вызвать.

Гофф

Уровень 20

Master

25 сентября 2022, 14:33

Тогда сорри, что запутал тебя. Хот не удивлюсь, если они условия изменили, а проверку валидатора - нет :-D

Grock

Уровень 44

25 сентября 2022, 14:38

Ничего. Спасибо за совет)

Гофф

Уровень 20

Master

25 сентября 2022, 14:42

А если так? if (person instanceof Person.User) ((Person.User) person).live();

Гофф

Уровень 20

Master

25 сентября 2022, 14:43

Там по условию метод live (и остальные) нужно вызвать именно у объекта, то есть по схеме объект.метод().

Grock

Уровень 44

25 сентября 2022, 14:53

Во втором варианте задачи (см. код здесь) я как раз обращался к методу через объект, но все равно. Не получилось:

public static void doWork(Person person) {
        switch (person.getClass().getSimpleName()) {
                case "User": person.live(); break;
                case "Loser": person.doNothing(); break;
                case "Coder": person.writeCode(); break;
                case "Proger": person.enjoy(); break;
                default: throw new IllegalStateException("Unexpected value");
            }
    }

Grock

Уровень 44

25 сентября 2022, 14:54

А что это за конструкция такая:

((Person.User) person).live();

Гофф

Уровень 20

Master

25 сентября 2022, 14:54

полезный

Попробуй явно привести person к нужному типу ((Person.User) person).live();

Grock

Уровень 44

25 сентября 2022, 15:01

Все равно пишет:

Убедись, что при вводе с консоли слова user создается экземпляр класса User

И не выполнены те же условия. Вот какой код получился (может swich-case ему не нравится):

public static void doWork (Person person) {
            switch (person.getClass().getSimpleName()) {
                case "User": ((Person.User) person).live(); break;
                case "Loser": ((Person.Loser) person).doNothing(); break;
                case "Coder": ((Person.Coder) person).writeCode(); break;
                case "Proger": ((Person.Proger) person).enjoy(); break;
                default: throw new IllegalStateException("Unexpected value");
            }
    }

Grock

Уровень 44

25 сентября 2022, 15:08

Код сейчас такой (класс Solution):

package com.javarush.task.task14.task1411;

import java.io.BufferedReader;
import java.io.InputStreamReader;

/*
User, Loser, Coder and Proger
*/

public class Solution {
    public static void main(String[] args) throws Exception {
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        Person person = null;
        String key;

        while (true) {
            key = reader.readLine();
            if (!(key.equals("user")) &&
                !(key.equals("loser")) &&
                !(key.equals("coder")) &&
                !(key.equals("proger"))) { break; }

            switch (key) {
                case "user": person = new Person.User(); doWork(person); break;
                case "loser": person = new Person.Loser(); doWork(person); break;
                case "coder": person = new Person.Coder(); doWork(person); break;
                case "proger": person = new Person.Proger(); doWork(person); break;
                default: throw new IllegalStateException("Unexpected value");
        }
    }
}

    public static void doWork(Person person) {
        switch (person.getClass().getSimpleName()) {
                case "User": ((Person.User) person).live(); break;
                case "Loser": ((Person.Loser) person).doNothing(); break;
                case "Coder": ((Person.Coder) person).writeCode(); break;
                case "Proger": ((Person.Proger) person).enjoy(); break;
                default: throw new IllegalStateException("Unexpected value");
            }
    }
}

Grock

Уровень 44

25 сентября 2022, 15:09

И код интерфейса (Person):

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!");
        }
    }
}

Grock

Уровень 44

25 сентября 2022, 15:10

Но не принимает и те же "ругательства", что указаны изначально.

Grock

Уровень 44

25 сентября 2022, 15:46

В итоге компилятор просто хотел видеть instanceof. Также переделал код с помощью if else и решение принялось. При этом код короче стал.

Grock

Уровень 44

25 сентября 2022, 13:59

Так тоже не хочет:

switch (key) {
                case "user": person = new Person.User(); doWork(person); break;
                case "loser": person = new Person.Loser(); doWork(person); break;
                case "coder": person = new Person.Coder(); doWork(person); break;
                case "proger": person = new Person.Proger(); doWork(person); break;
                default: throw new IllegalStateException("Unexpected value");
        }

Гофф

Уровень 20

Master

25 сентября 2022, 14:15

3. Метод doWork надо расписывать в каждом подклассе отдельно, а не один для всех в классе Solution. И вяглядит примерно так: Свитч создаём person конец свитча person.doWork();

Гофф

Уровень 20

Master

25 сентября 2022, 14:19

А, и если отвечаешь, нажимай, пожалуйста, на "Ответить", а не добавляй новый комментарий. Иначе твой собеседник и не узнает, что ты что-то там написал.

Grock

Уровень 44

25 сентября 2022, 14:24

Спасибо за совет. Так действительно лучше будет. По п. 3 Не понял почему нужно в каждом классе расписывать его. По условиям задачи: В классе Solution должен быть реализован метод doWork с одним параметром типа Person.

Гофф

Уровень 20

Master

25 сентября 2022, 14:26

Возможно, изменились условия задачи, с тех пор, как я её решал. Я могу посмотреть только своё решение, оно было как я советовал. Те условия посмотреть не могу.

Grock

Уровень 44

25 сентября 2022, 14:40

Ладно, буду думать)