Пункт про время на экран

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

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

Архивный

вроде все уже перепробовал, в упор не вижу ошибку и еще, он не реагирует на exit, и дело не в методе getAllDishesForOrder(), readString пробовал разные варианты из комментов и прочего, не берет

Сейчас повар готовит блюда моментально. На самом деле приготовление занимает какое-то время.
Нам нужно вычислить время приготовления всего заказа, чтобы потом подбирать рекламные ролики для этого времени.

1. Предположим, что нам известно время приготовления каждого блюда в минутах. Захардкодим его в классе Dish.
1.1. Измени создание элементов enum - FISH(25), STEAK(30), SOUP(15), JUICE(5), WATER(3);
1.2. Создай поле private int duration с геттером.
Чтобы создать геттер, нажми Alt+Insert и выбери пункт Getter. Далее выбери имя поля и нажми OK(Enter).
1.3. Создай конструктор Dish(int duration), в котором инициализируй поле duration. Чтобы создать конструктор, нажми Alt+Insert и выбери пункт Constructor. Далее выбери имя поля и нажми OK(Enter).

2. Добавим возможность посчитать продолжительность приготовления всего заказа.
Куда его добавить???
2.1. Если в класс Cook, то повар сможет считать продолжительность приготовления заказа.
Чтобы другие классы могли получить эту информацию, необходимо будет получить доступ к объекту Cook.
2.2. Если в класс Order, то имея доступ к заказу всегда можно узнать продолжительность приготовления.
2.3. Выбери правильное место из п.2.1. и п.2.2. и добавьте метод public int getTotalCookingTime(), который посчитает суммарное время приготовления всех блюд в заказе.
2.4. Добавим вывод в консоль этой информации. Пусть теперь информация о заказе выводится в таком виде:
Your order: [SOUP, JUICE, WATER] of Tablet{number=5}, cooking time 23min

Наведем красоту:
3. Запустим приложение и сразу введем 'exit'. Вывод получился не очень красивым:
Start cooking - , cooking time 0min was cooked by Amigo
Сделай так, чтобы если в заказе нет блюд, он не отправлялся повару. Найди это место и реализуй логику.
Для этого также в классе Order создай вспомогательный метод public boolean isEmpty(), который будет определять, есть ли какие либо блюда в заказе.

Требования:

Поле duration в enum Dish должно быть приватным и иметь публичный геттер.
Время приготовления блюд должно соответствовать условию задачи.
В классе Order должен быть создан метод public int getTotalCookingTime возвращающий количество времени требующееся на приготовление текущего заказа.
Помимо списка блюд в заказе, на экран также должно быть выведено время приготовления заказа.
Метод isEmpty в классе Order должен возвращать true, если заказ пуст.
Если в заказе отсутствуют блюда, он не должен быть отправлен повару.

package com.javarush.task.task27.task2712; import com.javarush.task.task27.task2712.kitchen.Cook; import com.javarush.task.task27.task2712.kitchen.Waiter; import java.io.IOException; public class Restaurant { public static void main(String[] args) throws IOException { Tablet tablet = new Tablet(0); tablet.createOrder(); tablet.createOrder(); tablet.createOrder(); tablet.createOrder(); Cook vasa = new Cook("Vasa"); tablet.addObserver(vasa); vasa.addObserver(new Waiter()); } }

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

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

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

Александр Дорофеев Backend Developer в Ай-Теко

Expert

14 мая 2019, 15:42

public static List<Dish> getAllDishesForOrder() throws Exception Вот почему все так любят саму операцию чтения вставлять с условие. Мало того, что она там не всегда корректно работает, так еще и читается не очень понятно. И не ожидаешь ее там увидеть - это условие, а не действие. Открывай предыдущий пункт и переписывай так, как требуют. Не знаю, как там приняли, но написано некорректно, это точно. Насколько я помню, меню там должно выводиться еще раз после принятия элемента заказа. Я сам делал через do-while, потому что первый раз выводить меню надо в любом случае.

Артем Divertitto Senior Android-разработчи в United Tech

Master

14 мая 2019, 06:52

Метод toString класса Order не правильно реализован

Александр Дорофеев Backend Developer в Ай-Теко

Expert

11 мая 2019, 21:49

полезный

Надо дописать нужный текст в метод toString() класса Order

Evgeny Samuylov

Уровень 32

12 мая 2019, 13:58

Но зачем, ведь я его написал в классе Cook

Evgeny Samuylov

Уровень 32

12 мая 2019, 13:59

ConsoleHelper.writeMessage("Start cooking - "
                + order.toString() + ", cooking time "
                + order.getTotalCookingTime() + "min");
        setChanged();
        notifyObservers(order);

я конкретно вот об этом

Александр Дорофеев Backend Developer в Ай-Теко

Expert

12 мая 2019, 14:19

полезный

А зачем нужны лишние надписи, если программа в каком то варианте начнет вызывать у класса Cook этот метод? Общее правило - абстракция и минимальная связность. Каждый элемент должен быть максимально автономен, а тут ты хочешь одну работу разбросать по 2 методам, которые вообще то должны выполнять свою работу. Давай еще все классы в 1 файл свалим и будем с этим работать.

Evgeny Samuylov

Уровень 32

21 мая 2019, 19:52

@Override
    public String toString() {
        if (dishes.isEmpty()) {
            return "";
        } else {
            return "Your order: "
                    + dishes + " of "
                    + tablet + ", cooking time " + getTotalCookingTime() + "min" ;
        }
    }

Вынес, ошибка сохранилась

Александр Дорофеев Backend Developer в Ай-Теко

Expert

21 мая 2019, 19:55

public static List<Dish> getAllDishesForOrder() throws Exception переписывай