Не могу понять причину

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

Java Multithreading, 9 уровень, 15 лекция

Архивный

List<Dish> allDishes = Arrays.asList(Dish.values());

String order;
        while (!(order = readString()).equals("exit")) {
            if(!allDishes.contains(order)) {
                writeMessage("Сожалеем, но в нашем меню такого блюда нет."

Почему не видит Fish в списке?

1. Мы много работаем с консолью. Пора создать единую точку взаимодействия.
Создай класс ConsoleHelper с единственным BufferedReader, через который будем работать с консолью.
Запомни, этот класс не хранит никаких данных и состояний, поэтому все методы будут статическими.
Создай в нем три метода:
- writeMessage(String message) - для вывода message в консоль
- String readString() - для чтения строки с консоли
- List<Dish> getAllDishesForOrder() - просит пользователя выбрать блюдо и добавляет его в список.
Выведи список всех блюд и попроси пользователя ввести строку - название блюда.
Введенное 'exit' означает завершение заказа.
В случае, если введенное блюдо не представлено в меню, выведи сообщение о том, что такого блюда нет и продолжи формирование заказа.
Исключения ввода-вывода бросай выше, на этом уровне не понятно, что с ними делать.

2. Вернемся к классу Order: в нем есть ссылка на планшет, и еще есть список выбранных блюд.
Инициализируй список dishes в конструкторе, вызвав метод getAllDishesForOrder из ConsoleHelper.

3. Перепиши метод toString в классе Order. Пусть он возвращает пустую строку, если нет блюд в заказе, иначе результат должен быть таким, как в примере (используй toString объекта tablet):
Your order: [JUICE, FISH] of Tablet{number=5}
Для этого также измени метод toString в классе Tablet (внутри класса Tablet нажмите Alt+Insert -> toString()).
Пример возвращаемого значения:
Tablet{number=5}

4. У нас все завязано на работу с консолью. Однако, при возникновении исключений, наше приложение умрет.
Чтобы узнать причину - добавим в Tablet статическое поле logger типа java.util.logging.Logger, инициализированное именем класса (Logger.getLogger(Tablet.class.getName())).

5. В методе createOrder класса Tablet обработаем исключения ввода-вывода.
Запишем в лог "Console is unavailable.". Уровень лога - SEVERE - это самый серьезный уровень, мы не можем работать.
Также в методе createOrder класса Tablet должен быть создан новый заказ и выведен в консоль (используй ConsoleHelper).

6. Надо начинать тестировать наше приложение.
Добавьте в main создание планшета и создание четырех заказов.

Требования:

Метод writeMessage класса ConsoleHelper должен выводить полученную строку в консоль.
Метод readString класса ConsoleHelper должен возвращать строку считанную с консоли.
Метод getAllDishesForOrder класса ConsoleHelper должен возвращать список блюд выбранных пользователем.
В конструкторе класса Order список dishes должен быть инициализирован результатом работы метода getAllDishesForOrder.
Метод toString в классе Order должен возвращать содержимое заказа согласно условию задачи.
Поле logger в классе Tablet должно быть инициализировано с помощью метода Logger.getLogger() с именем текущего класса в качестве параметра.
В методе createOrder класса Tablet должен быть создан новый заказ, в качестве параметра передай текущий планшет.
В случае возникновения IOException в процессе создания заказа - в лог должно быть записано сообщение "Console is unavailable.", уровень - SEVERE.
В методе main класса Restaurant должен быть создан новый планшет и с его помощью созданы четыре новых заказа.
Метод toString в классе Tablet должен возвращать информацию о планшете (смотри пример в задании).

package com.javarush.task.task27.task2712; import com.javarush.task.task27.task2712.kitchen.Order; import java.io.IOException; import java.util.logging.Level; import java.util.logging.Logger; public class Tablet { private final int number; private static Logger logger = Logger.getLogger(Tablet.class.getName()); public Tablet(int number) { this.number = number; } public Order createOrder() { Order newOrder = null; try { newOrder = new Order(this); } catch (IOException e) { logger.log(Level.SEVERE, "Console is unavailable."); } return newOrder; } @Override public String toString() { return "Tablet{" + "number=" + number + '}'; } }

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

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

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

Денис Enterprise Java Developer

15 марта 2020, 22:16

решение

Привет, очень бегло прошелся по твоему коду и сравнивая со своим нашел несколько отличий. 1. Tablet.createOrder() у меня void, там идея просто создать новый объект, остальное крутиться начинает в конструкторе Order. 2. ConsoleHelper.getAllDishesForOrder() зачем то я цикл реализовывал через do while при том ввод пользователя я делал в do а потом просто проверял его под while на exit, еще в этом методе меня чуток смущает allDishes. Возможно мой подход и не очень верный, но проще и читаемее будет реализовать через Try-Catch, что то вроде

try {
                Dish dish = Dish.valueOf(userInput);
                dishes.add(dish);
                writeMessage("Successfully added, one more?");
            }

Ну и это... по картинке с дебаггером :) У тебя в allDIshes лежат Dish объекты, а сравниваешь ты со строкой. Не удивительно, что ее там не находит. 3. Order.toString() что то мне подсказывает, что dishes.toArray().toString() возвращает что то вроде [Ljava.lang.Object;@14ae5a5 Проверь верность вывода. Думаю, что если эти пункты попинать, то задачка с мертвой точки да сдвинется. Успехов.

Soros

Уровень 39

16 марта 2020, 06:39

Ок. Увидел ошибку попытки сравнения Dish и String. Согласен с решением через try. Метод Dish.valueOf(order) выбросит ошибку, если введённой строки энуме Dish нет. Только не могу этого увидеть в документации Enum javadoc Согласно документации, единственный статический метод valueOf принимает два параметра, а мы легко работаем с одним. И в документации ничего не говорится про ексепшен. И в документации нет метода values(). Может не ту документацию смотрю, или не правильно читаю?

Soros

Уровень 39

16 марта 2020, 06:46

С ексепшеном разобрался - RuntimeException, для которого не обязательно писать код обработки, наверное поэтому в javadoc про возможность их появления в методах и не упоминается. Но вот отсутствие в javadoc метода values() у энума - для меня пока загадка. И c параметрами метода valueOf() тоже пока не разобрался. Что можешь сказать?

Денис Enterprise Java Developer

16 марта 2020, 14:14

Ты немного не туда смотришь Enum != enum https://docs.oracle.com/javase/tutorial/java/javaOO/enum.html В данном случае Enum (по твоей ссылке) родительский (абстрактный) класс который еще и умеет работать с любыми потомками. Например будет работать Enum.valueOf(Dish.class, "Fish") ; равно как и Enum.valueOf(MyEnum.class, "MyValue");

public enum TestEnum {
    One,
    Two
}

public static void main(String[] args) {
    System.out.println(TestEnum.valueOf("One"));
    System.out.println(Enum.valueOf(TestEnum.class, "One"));
}

В свою очередь у каждого enum есть свои фишки. В том числе метод values()

Soros

Уровень 39

16 марта 2020, 14:26

Спасибо.