[Core, Task 1411 - User, Loser, Coder and Proger] Подскажите, пожалуйста, где зарыта @ - не проходит валидация по классу

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

Обсуждается

Валидация выдает: "Для каждой корректной(user, loser, coder, proger) введенной строки должен быть вызван метод doWork с соответствующим объектом класса Person в качестве параметра. Убедись, что при вводе с консоли слова user, создается экземпляр класса User" (c)

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

Ввести [в цикле] с клавиатуры несколько строк (ключей).
Строки (ключи) могут быть такими: "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!"); } } }

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

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

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

Ksenia VolkovaJava Developer в DXC

Master

17 июня 2020, 18:38

doWork должен вызываться по мере ввода строк: ввел строку, создал объект, вызвал метод. (комментарии в исходном коде задачи не случайно написаны)

kde.intro Team Lead

17 июня 2020, 18:44

Так тоже не работало: до текущего варианта была именно такая реализация -

...
public static void main(String[] args) throws Exception {
    BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
    //тут цикл по чтению ключей, пункт 1
    while (true) {
        //создаем объект, пункт 2
        String typeOfPerson = reader.readLine();
        long numberOfMatchingTypes = kindOfMen.stream().filter(d -> d.getTypeOfMan().equals(typeOfPerson)).count();
        if (numberOfMatchingTypes == 1) {
            kindOfMen k = kindOfMen.valueOf(typeOfPerson.toUpperCase());
            Person person = k.createPerson();
            doWork(person); //вызываем doWork
        } else {
            break;
        }
    }
}
...

(@__@)

Уровень 39

17 июня 2020, 19:29

мое решение донельзя простое:

while(true){
  key = reader.readLine();
  if(key.equals("user")) person = new Person.User();
  else if(key.equals("loser")) person = new Person.Loser();
  else if(key.equals("coder")) person = new Person.Coder();
  else if(key.equals("proger")) person = new Person.Proger();
  else return;

  doWork(person); //вызываем doWork
}

но вариант с enum оч заинтересовал. правда я минут 30 как пытаюсь понять логику, но не выходит. можно расписать её более детально? можно личкой. буду благодарен

Wladyslaw Java Developer

Master

17 июня 2020, 19:39

Что такое key?

kindOfMen k = kindOfMen.valueOf(key.toUpperCase());

И что такое typeOfPerson тогда?

String typeOfPerson = reader.readLine();

kde.intro Team Lead

17 июня 2020, 19:43

Ошибка копи-паста - поправил в предыдущем сообщении :)

Ksenia VolkovaJava Developer в DXC

Master

17 июня 2020, 19:46

Зачем Solution реализует Person? Тебе вообще твой код самому нравится? С многочисленными повторами, ненужными return...

kde.intro Team Lead

17 июня 2020, 19:55

Предложите варианты правильных практик - с удовольствием выслушаю, возьму на заметку и буду признателен.

Ksenia VolkovaJava Developer в DXC

Master

17 июня 2020, 20:11

Полезная стратегия - не усложнять код там, где это не требуется. Например, тут использование enum только усложняет и перегружает код. Если уж не хотелось решать совсем просто (как у (@__@), например), можно было бы вынести создание объектов в "фабрику".

(@__@)

Уровень 39

18 июня 2020, 03:42

С точки зрения поддержки кода в будущем, мой вариант терпит фиаско. Если потребуется внести изменение в список, то придется лезть в код и дописывать логику. Потому очень хотелось вынести "user", "loser", "coder", "proger" в отдельный список и пробежать по нему циклом. И, в случае необходимости, вносить изменения в список, а не алгоритм. Увы, адекватного решения, на момент прохождения, я не нашел.. А тут enum, вроде как оч подходит для этого.. но пока мне непонятен псевдокод для примера

list {   //список ключ : значение
  "user" : User;
  ...
  "proger" : Proger;
}

foreach (el  : list){
  if(el.key == reader.string) person = new Person.[ el.value ];
}
doWork(person);

и если я правильно понял kde.intro пытался сделать нечто похожее если подобное в java вообще возможно :)