Таймаут при проверке

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

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

Обсуждается

Валидатор ругается на таймаут. Код помощника, где есть цикл, менял 3 раза - эффект не измененн. Сама программа работает хорошо. В чём проблема?

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.Dish; import java.io.IOException; import java.util.List; import java.util.logging.Logger; import static com.javarush.task.task27.task2712.ConsoleHelper.getAllDishesForOrder; import static java.util.logging.Level.SEVERE; public class Tablet { private static Logger logger = Logger.getLogger(Tablet.class.getName()); private final int number; public Tablet(int number) { this.number = number; } public void createOrder() { try { List<Dish> dishList = getAllDishesForOrder(); } catch (IOException e) { logger.log(SEVERE, "Console is unavailable."); } } @Override public String toString() { return "Tablet{" + "number=" + number + '}'; } }

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

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

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

hidden #2460969

Уровень 2

15 августа 2021, 10:22

зачем здесь цикл?

public static String readString() throws IOException {
    while (true) {
            String buf = bis.readLine();
            if (buf != null)
                return buf;
    }
}

// Создаем static final карту
private static final Map<String, Dish> LOOKUP_MAP = new HashMap<>();

// Заполняем её всеми значениями
static {
    for (Dish env : values()) {
        LOOKUP_MAP.put(env.toString(), env);
    }
}

// Возвращаем Dish по строковому url
public static Dish getDishByString(String dish) {
    return LOOKUP_MAP.get(dish);
}

зачем это надо, если из enum можно также по строке получить объект. через valueOf() ?

Иван

Уровень 41

15 августа 2021, 11:47

Могу лишь очень частично согласиться. Метод readString взят целиком из прошлых уроков, там работало без проблем, и в тестовых прогонах ошибок не было. Вдруг Валя ругается. Непонятно. После замены проблема ушла. Касаемо второго замечания, то это, с моей точки зрения, гораздо более правильный метод позволяющий без перебора сравнивать литералы и enum. При многократном использовании даст ощутимый прирост по скорости без необходимости перебора полного списка.

hidden #2460969

Уровень 2

15 августа 2021, 11:49

что значит без перебора? есть встроенный метод valueOf() , который делает совершенно тоже самое, что ты тут написал. И так уж принято, если уже есть метод, который реализует желаемый тобой функционал, то надо использовать готовый метод, а не городить от себя.