JavaRush/Помощь по Java задачам/кажется проблема с выводом

41 уровень

21.01.2021
354просмотров
8комментариев

кажется проблема с выводом

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

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

Решен

Решил отредактировать вопрос, до этого было пару догадок почему не проходит валидация, но я вроде как успешно их разрешил. Догадки были следующие: 1) проблемы с выводом (три лишних раза). Данная проблема описана ниже и успешно решена 2) проблема с исключение. Полностью скопировал метод processVideo() (а вернее даже класс, чтобы наверняка) и посмотрел вывод в консоль. Обнаружил, что не бросается исключение если список заказов пуст, поэтому добавил в свой procesVideo() проверку на ноль, если сумарное время рекламы равно нулю

public void processVideos() {
        if (storage.list().isEmpty() || storage.list() == null) {
            StatisticManager.getInstance().register(new NoAvailableVideoEventDataRow(timeSeconds));
            throw new NoVideoAvailableException();
        }
        if (sortList == null) setSortList(); //если лист не инициализирован, то инициализируем
        if (sortList.isEmpty()) {
            if (timeForAdv == 0) throw new NoVideoAvailableException(); //добавил вот здесь
            sortListForShow();
            sysOut();
            return; //если лист уже пуст, то выходим из метода
        }
        if (sortList.get(0).getHits() > 0 &&
                ((sortList.get(0).getDuration() + timeForAdv) <= timeSeconds)) { //если у данного ролика есть показа, то начинае проверку
            timeForAdv = timeForAdv + sortList.get(0).getDuration(); //увеличиваем сумарное время показа рекламы
            amount = amount + sortList.get(0).getAmountPerOneDisplaying(); //увеличиваем сумарную стомость показа всех роликов
            listForShowing.add(sortList.get(0));
        }
        sortList.remove(0);
        if (timeForAdv == timeSeconds) {
            if (timeForAdv == 0) throw new NoVideoAvailableException(); // добавил вот здесь
            sortListForShow();
            sysOut();
            return; //если время показа уже заполнено, то выходим из метода
        }
        processVideos();
    }

3) психанул еще раз и полностью поменял свой класс с методом procesVideo() на класс из готового решения для прохождения валидации... и все равно не проходит валидацию... что на самом деле радует, так как получается, что дело не в этом методе. Буду дальше искать проблему.. Проблема ниже уже решена, оставил просто для истории: не могу понять почему такой вывод: Почему при создании первого заказа выводится три лишних раза "Your order: [FISH] of Tablet{number=1}, cooking time 25min was cooked by Amigo" Скорее всего получается дело в этом, поэтому и не проходит валидацию. Ноги начинают расти в методе public Order createOrder() в классе Tablet в строке 33, потом перекидывает в класс Cook в метод update() и затем в строке 29 перекидывает в класс Waiter в метод update()

FISH, STEAK, SOUP, JUICE, WATER
Выберите блюдо, для выход из меня выбора введите "exit"
FISH
exit
Start cooking - Your order: [FISH] of Tablet{number=1}, cooking time 25min
Your order: [FISH] of Tablet{number=1}, cooking time 25min was cooked by Amigo
Your order: [FISH] of Tablet{number=1}, cooking time 25min was cooked by Amigo
Your order: [FISH] of Tablet{number=1}, cooking time 25min was cooked by Amigo
Third Video is displaying... 200, 333
First Video is displaying... 50, 277
FISH, STEAK, SOUP, JUICE, WATER
Выберите блюдо, для выход из меня выбора введите "exit"
FISH
exit
Start cooking - Your order: [FISH] of Tablet{number=2}, cooking time 25min
Third Video is displaying... 200, 333
First Video is displaying... 50, 277
FISH, STEAK, SOUP, JUICE, WATER
Выберите блюдо, для выход из меня выбора введите "exit"
FISH
exit
Start cooking - Your order: [FISH] of Tablet{number=3}, cooking time 25min
First Video is displaying... 50, 277
Second Video is displaying... 10, 11
21-Jan-2021 - 560.00
Total - 560.00
21-Jan-2021
Amigo - 25 min
Ivan - 25 min
Vasya - 25 min

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

1. В классе VideoSelectedEventDataRow создай геттер для поля amount.
В классе CookedOrderEventDataRow создай геттер для поля cookName.

2. В StatisticManager создай метод (придумать самостоятельно), который из хранилища достанет все данные, относящиеся к отображению рекламы, и посчитает общую прибыль за каждый день.
Дополнительно добавь вспомогательный метод get в класс хранилища, чтобы получить доступ к данным.

3. Начнем с логики метода printAdvertisementProfit в классе DirectorTablet.
Используя метод из предыдущего пункта вывести в консоль в убывающем порядке даты и суммы.
Для каждой даты из хранилища событий, для которой есть показанная реклама, должна выводится сумма прибыли за показы рекламы для этой даты.
В конце вывести слово Total и общую сумму.

Пример:
14-Jul-2013 - 2.50 13-Jul-2013 - 1.02 12-Jul-2013 - 543.98 Total - 547.50

4. Реализуем второй пункт статистики - загрузка (рабочее время) повара, сгруппировать по дням.
В StatisticManager создай метод (придумать самостоятельно), который из хранилища достанет все данные, относящиеся к работе повара, и посчитает общую продолжительность работы для каждого повара отдельно.

5. Реализуем логику метода printCookWorkloading в классе DirectorTablet.
Используя метод из предыдущего пункта вывести в консоль в убывающем порядке даты, имена поваров и время работы в минутах (округлить в большую сторону).
Для каждой даты из хранилища событий, для которой есть запись о работе повара, должна выводится продолжительность работы в минутах для этой даты.
Если повар не работал в какой-то из дней, то с пустыми данными его НЕ выводить (см. 13-Jul-2013)
Поваров сортировать по имени

Пример:
14-Jul-2013 Ivanov - 60 min Petrov - 35 min 13-Jul-2013 Ivanov - 129 min 12-Jul-2013 Ivanov - 6 min Petrov - 5 min

6. Обрати внимание, что при выводе даты, название месяца должно быть на английском (при форматировании можно передать параметр Locale.ENGLISH).

Требования:

В классе VideoSelectedEventDataRow должен быть создан геттер для поля amount.
В классе CookedOrderEventDataRow должен быть создан геттер для поля cookName.
Метод printAdvertisementProfit класса DirectorTablet должен быть реализован в соответствии с условием задачи.
Метод printCookWorkloading класса DirectorTablet должен быть реализован в соответствии с условием задачи.

package com.javarush.task.task27.task2712; import com.javarush.task.task27.task2712.kitchen.Cook; import com.javarush.task.task27.task2712.kitchen.Waiter; import com.javarush.task.task27.task2712.statistic.StatisticManager; public class Restaurant { public static void main(String[] args) { Tablet tablet = new Tablet(1); Cook cook = new Cook("Amigo"); Waiter waiter = new Waiter(); tablet.addObserver(cook); cook.addObserver(waiter); Tablet tablet2 = new Tablet(2); Cook cook2 = new Cook("Ivan"); Waiter waiter2 = new Waiter(); tablet2.addObserver(cook2); cook.addObserver(waiter2); Tablet tablet3 = new Tablet(3); Cook cook3 = new Cook("Vasya"); Waiter waiter3 = new Waiter(); tablet3.addObserver(cook3); cook.addObserver(waiter3); tablet.createOrder(); tablet2.createOrder(); tablet3.createOrder(); tablet.createOrder(); DirectorTablet directorTablet = new DirectorTablet(); directorTablet.printAdvertisementProfit(); directorTablet.printCookWorkloading(); directorTablet.printActiveVideoSet(); directorTablet.printArchivedVideoSet(); } }

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

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

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

Илья Backend Developer в СберТех

22 января 2021, 11:52

проблему вроде как нашел, чуть позже отпишусь

Илья Backend Developer в СберТех

22 января 2021, 13:39

решение

необходимо было копейки преобразовать в рубли, просто разделив на 100 полученное значение из мапы

Илья Backend Developer в СберТех

22 января 2021, 09:36

ап теме, отредактировал вопрос!

Илья Backend Developer в СберТех

22 января 2021, 08:14

Ап теме, решение до сих пор не нашел(((((

Илья Backend Developer в СберТех

22 января 2021, 08:09

достало не понимать в чем дело, психанул и скопировал полность класс AdvertisementManager, чтобы посмотреть каким должен быть вывод. Увидел что если список заказов пуст, то выводится лог по NoAvailableExc. А у меня не выводится. Почему-то метод processVideo() этого менеджера не бросает это исключение, если список заказов пуст. Вот пытаюсь разобраться

Илья Backend Developer в СберТех

22 января 2021, 08:13

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

public void processVideos() {
        if (storage.list().isEmpty() || storage.list() == null || timeSeconds == 0) {
            StatisticManager.getInstance().register(new NoAvailableVideoEventDataRow(timeSeconds));
            throw new NoVideoAvailableException();
        }

hidden #2322530

Уровень 41

21 января 2021, 19:25

это должно быть всё в одном try

try {
    order = new Order(this);
} catch (IOException e) {
    logger.log(Level.SEVERE, "Console is unavailable.");
}
setChanged();
if (!order.isEmpty()) {
    notifyObservers(order);
}
try {
    new AdvertisementManager(order.getTotalCookingTime() * 60).
            processVideos();
} catch (NoVideoAvailableException e) {
    logger.log(Level.INFO, "No video is available for the order " + order);
}

ошибка

cook.addObserver(waiter);
cook.addObserver(waiter2);
cook.addObserver(waiter3);

Илья Backend Developer в СберТех

21 января 2021, 20:00

спасибо, теперь понял почему три строчки выводятся, НО..... валидацию все равно не проходит

Метод printAdvertisementProfit класса DirectorTablet должен быть реализован в соответствии с условием задачи.

не могу понять в чем проблема