Помогите плз, не проходит по 3му пункту

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

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

Решен

кончились идеи( "Класс TestOrder должен быть потомком класса Order"

Сейчас у нас один планшет и 1 повар.
Давай создадим несколько планшетов, которые будут рандомно генерировать заказы, и сделаем два повара.

1. В классе Restaurant создай ПРИВАТНУЮ статическую константу int ORDER_CREATING_INTERVAL = 100.

2. В отдельном классе создай таск (Runnable) RandomOrderGeneratorTask. Этот таск должен:
2.1. Хранить список всех планшетов.
2.2. Используя Math.random выбирать случайный планшет.
2.3. У RandomOrderGeneratorTask должен быть только один единственный метод.
2.4. Генерировать случайный заказ каждые ORDER_CREATING_INTERVAL миллисекунд для планшета из п.2.2 (не печатай стек-трейс).
Сейчас заказ создается в методе createOrder в классе Tablet.
В классе Tablet создай метод void createTestOrder() с похожей функциональностью, который будет случайным образом генерировать заказ со случайными блюдами не общаясь с реальным человеком.
Список планшетов и интервал передай в конструкторе RandomOrderGeneratorTask.

Подсказка:
а) создай класс TestOrder - наследник Order - в пакете родителя;
б) в классе Order создай protected метод initDishes(), в котором инициализируй dishes. Вызови этот метод в конструкторе;
в) проверь, что в классе Order поле dishes имеет модификатор доступа protected;
г) переопредели initDishes в классе-наследнике TestOrder: проинициализируй поле dishes пустым списком и заполни его случайным набором блюд;
д) вместо создания объекта Order в методе createTestOrder() класса Tablet, создавай объект класса TestOrder.
Остальной функционал метода createTestOrder оставь таким же как в createOrder.

3. Отрефакторь методы createTestOrder() и createOrder(): в одном из методов выдели код, который повторяется в обоих методах, и нажми Ctrl+Alt+M, введи любое имя метода и нажми ОК. IDEA предложит заменить этот код во втором методе, подтверди.

Требования:

В классе Restaurant должна быть создана приватная статическая константа ORDER_CREATING_INTERVAL типа int со значением 100.
В классе Order должен быть создан метод initDishes инициализирующий dishes.
В конструкторе класса Order должен быть вызван метод initDishes.
Класс TestOrder должен быть реализован в соответствии с условием задачи.
Класс RandomOrderGeneratorTask должен быть реализован в соответствии с условием задачи.

package com.javarush.task.task27.task2712; import com.javarush.task.task27.task2712.ad.AdvertisementManager; import com.javarush.task.task27.task2712.ad.NoVideoAvailableException; import com.javarush.task.task27.task2712.kitchen.Order; import java.util.Observable; import java.io.IOException; import java.util.logging.Level; import java.util.logging.Logger; import static java.util.logging.Level.SEVERE; public class Tablet extends Observable { final int number; static Logger logger = Logger.getLogger(Tablet.class.getName()); public Tablet(int number) { this.number = number; } public Order createOrder() { try { Order order = new Order(this); return getOrder(order); } catch (IOException e) { logger.log(SEVERE, "Console is unavailable."); return null; } catch (Exception e) { return null; } } @Override public String toString() { return String.format("Tablet{number=%d}", number); } public void createTestOrder(){ try { getOrder(new TestOrder(this)); } catch (IOException e) { logger.log(SEVERE, "Console is unavailable."); } } private Order getOrder(Order order) { if (!order.isEmpty()) { ConsoleHelper.writeMessage(order.toString()); try { (new AdvertisementManager(order.getTotalCookingTime()*60)).processVideos(); } catch (NoVideoAvailableException e) { logger.log(Level.INFO, "No video is available for the order " + order); // StatisticManager.getInstance().register(new NoAvailableVideoEventDataRow(order.getTotalCookingTime())); } setChanged(); notifyObservers(order); return order; } else return null; } }

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

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

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

hidden #228406

Уровень 41

18 января 2019, 23:14

public class TestOrder extends Order {

    public TestOrder(Tablet tablet) throws IOException {
        super(tablet);
    }

    @Override
    protected void initDishes() throws IOException {
        //Определим случайное количество блюд от 1 до общего количества в меню :)
        int countRandomDish = (int)(1 + (Math.random() * Dish.values().length));
        dishes = new ArrayList<>();
        while (countRandomDish > 0)
        {
            //Выбираем случайное блюдо
            int indexRandom = (int)(Math.random() * Dish.values().length);
            //Помещаем его в список
            dishes.add(Dish.values()[indexRandom]);
            countRandomDish--;
        }
    }
}

Victoria Sedletskaya

Уровень 38

18 января 2019, 23:27

спасибо. Скопировала полностью, не хочет принимать :-( видимо в чем-то другом дело

hidden #228406

Уровень 41

19 января 2019, 02:20

public Order createOrder() {
    try {
        Order order = new Order(this);
        return getOrder(order);
    }
    catch (IOException e) {
        logger.log(SEVERE, "Console is unavailable.");
        return null;
    } catch (Exception e) {
        return null;
    }
}

public void createTestOrder(){
    try {

        getOrder(new TestOrder(this));

    }
    catch (IOException e) {
        logger.log(SEVERE, "Console is unavailable.");
    }

}

Два этих метода должны быть одинаковы судя по заданию, но возвращать экземпляты разных классов. У Вас же метод createTestOrder вообще ничего не возвращает в отличии от createOrder. Попробуйте в эту сторону посмотреть.

Victoria Sedletskaya

Уровень 38

19 января 2019, 12:08

в условии сказано "В классе Tablet создай метод void createTestOrder()" но я попробовала возвращать тоже Order. не пропускает

hidden #228406

Уровень 41

19 января 2019, 22:42

решение

Только сейчас обратил внимание, что у Вас класс TestOrder не в пакете kitchen. переместите его в пакет kitchen, туда же где и основной класс Order. В итоге у Вас в имопортах, будет класс Table, а не классы Dish и Order.

package com.javarush.task.task27.task2712.kitchen;

import com.javarush.task.task27.task2712.Tablet;

import java.io.IOException;
import java.util.ArrayList;

public class TestOrder extends Order {

    public TestOrder(Tablet tablet) throws IOException {
        super(tablet);
    }

    @Override
    protected void initDishes() throws IOException {
        //Определим случайное количество блюд от 1 до общего количества в меню :)
        int countRandomDish = (int)(1 + (Math.random() * Dish.values().length));
        dishes = new ArrayList<>();
        while (countRandomDish > 0)
        {
            //Выбираем случайное блюдо
            int indexRandom = (int)(Math.random() * Dish.values().length);
            //Помещаем его в список
            dishes.add(Dish.values()[indexRandom]);
            countRandomDish--;
        }
    }
}

Victoria Sedletskaya

Уровень 38

20 января 2019, 13:59

действительно.. только сейчас вдумалась в условие "а) создай класс TestOrder - наследник Order - в пакете родителя." спасибо большое похоже именно в этом было дело. потому что после переноса не принял но изменил рекомендацию ментора на "в методе CreateTestrOrder должен быть создан объект TestOrder" вернула тип возвращаемого значения CreateTestrOrder на void , и с 22 попытки приняло спасибо, без Вас бы я бросила эту задачу

hidden #228406

Уровень 41

20 января 2019, 22:22

Удачи :)