JavaRush/Помощь по Java задачам/Не проходит валидацию

36 уровень

05.02.2018
1316просмотров
15комментариев

Не проходит валидацию

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

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

Решен

Ковыряюсь недели две. Если сначала было понятно, что вывод неправильный, то сейчас я уже вообще не втыкаю. Валидатор не принимает два последних пункта. Кто может подсказать тесты, на которых точно будет видно, что вывод правильный и скинуть примеры? Ну или может прям сразу увидит, что у меня не так. Всё писал сам, кроме рюкзака. Пробовал тягать рюкзак от разных людей, эффекта не было. Глаз уже дёргается. Забить и пройти мимо задачи не могу - не так устроен. Заранее спасибо. Вывод с тремя заказами по одной Fish:

Fish, Steak, Soup, Juice, Water
Введите название блюда для заказа:
Fish
Fish, Steak, Soup, Juice, Water
Введите название блюда для заказа:
exit
Your order: [Fish] of Tablet{number=5}
Start cooking - Your order: [Fish] of Tablet{number=5}, cooking time 25min
Your order: [Fish] of Tablet{number=5} was cooked by Amigo
13 is displaying... 4600, 7666
6 is displaying... 2506, 13922
5 is displaying... 2506, 13922
1 is displaying... 1523, 8461
12 is displaying... 1500, 12500
8 is displaying... 500, 4166
4 is displaying... 99, 825
Fish, Steak, Soup, Juice, Water
Введите название блюда для заказа:
Fish
Fish, Steak, Soup, Juice, Water
Введите название блюда для заказа:
exit
Your order: [Fish] of Tablet{number=5}
Start cooking - Your order: [Fish] of Tablet{number=5}, cooking time 25min
Your order: [Fish] of Tablet{number=5} was cooked by Amigo
13 is displaying... 4600, 7666
6 is displaying... 2506, 13922
5 is displaying... 2506, 13922
1 is displaying... 1523, 8461
12 is displaying... 1500, 12500
7 is displaying... 400, 2222
2 is displaying... 5, 83
Fish, Steak, Soup, Juice, Water
Введите название блюда для заказа:
Fish
Fish, Steak, Soup, Juice, Water
Введите название блюда для заказа:
exit
Your order: [Fish] of Tablet{number=5}
Start cooking - Your order: [Fish] of Tablet{number=5}, cooking time 25min
Your order: [Fish] of Tablet{number=5} was cooked by Amigo
6 is displaying... 2506, 13922
5 is displaying... 2506, 13922
1 is displaying... 1523, 8461
12 is displaying... 1500, 12500
10 is displaying... 400, 2222
11 is displaying... 350, 1944
4 is displaying... 99, 825
3 is displaying... 99, 550
2 is displaying... 5, 83
05-02-2018 - 352.62
Total - 352.62
05-02-2018
Amigo - 75 min

Вывод с добавленными fakedata (напрямую в мапы перед выводом)

Fish, Steak, Soup, Juice, Water
Введите название блюда для заказа:
Fish
Fish, Steak, Soup, Juice, Water
Введите название блюда для заказа:
exit
Your order: [Fish] of Tablet{number=5}
Start cooking - Your order: [Fish] of Tablet{number=5}, cooking time 25min
Your order: [Fish] of Tablet{number=5} was cooked by Amigo
13 is displaying... 4600, 7666
6 is displaying... 2506, 13922
5 is displaying... 2506, 13922
1 is displaying... 1523, 8461
12 is displaying... 1500, 12500
8 is displaying... 500, 4166
4 is displaying... 99, 825
Fish, Steak, Soup, Juice, Water
Введите название блюда для заказа:
Fish
Fish, Steak, Soup, Juice, Water
Введите название блюда для заказа:
exit
Your order: [Fish] of Tablet{number=5}
Start cooking - Your order: [Fish] of Tablet{number=5}, cooking time 25min
Your order: [Fish] of Tablet{number=5} was cooked by Amigo
13 is displaying... 4600, 7666
6 is displaying... 2506, 13922
5 is displaying... 2506, 13922
1 is displaying... 1523, 8461
12 is displaying... 1500, 12500
7 is displaying... 400, 2222
2 is displaying... 5, 83
Fish, Steak, Soup, Juice, Water
Введите название блюда для заказа:
Fish
Fish, Steak, Soup, Juice, Water
Введите название блюда для заказа:
exit
Your order: [Fish] of Tablet{number=5}
Start cooking - Your order: [Fish] of Tablet{number=5}, cooking time 25min
Your order: [Fish] of Tablet{number=5} was cooked by Amigo
6 is displaying... 2506, 13922
5 is displaying... 2506, 13922
1 is displaying... 1523, 8461
12 is displaying... 1500, 12500
10 is displaying... 400, 2222
11 is displaying... 350, 1944
4 is displaying... 99, 825
3 is displaying... 99, 550
2 is displaying... 5, 83
15-02-2018 - 47.12
12-02-2018 - 435.12
05-02-2018 - 352.62
04-02-2018 - 145.12
03-02-2018 - 145.12
01-02-2018 - 45.12
Total - 1170.22
05-02-2018
Amigo - 65 min
Smigo - 235 min
03-02-2018
Amigo - 25 min
Bmigo - 15 min
Smigo - 235 min
ymigo - 5 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(5); Cook firstCook = new Cook("Amigo"); StatisticManager.getInstance().register(firstCook); Waiter firstWaiter = new Waiter(); tablet.addObserver(firstCook); firstCook.addObserver(firstWaiter); tablet.createOrder(); tablet.createOrder(); tablet.createOrder(); DirectorTablet directorTablet = new DirectorTablet(); directorTablet.printActiveVideoSet(); directorTablet.printAdvertisementProfit(); directorTablet.printArchivedVideoSet(); directorTablet.printCookWorkloading(); } }

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

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

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

Pavlo Plynko Java-разработчик в CodeGym

Expert

7 февраля 2018, 10:54

При трех заказах на одну Water у тебя должно вывести: Total - 25.05 Сейчас выводит: Total - 75.18 Соотв. не: 5 is displaying... 2506, 13922 а: 5 is displaying... 835, 4638

Alexey Sverbeev

Уровень 36

8 февраля 2018, 07:45

спасибо, буду проверять

Alexey Sverbeev

Уровень 36

8 февраля 2018, 08:02

Что-то я не очень понял. Если у меня 3 заказа по 25.06, то сложить эти результаты ведь логично? И что не так с 5 is displaying? Если её initialAmount 2506, почему она должна стать 835? Может я в принципе не понимаю логику задания и что из-под меня хочет система?

Pavlo Plynko Java-разработчик в CodeGym

Expert

8 февраля 2018, 08:22

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

Alexey Sverbeev

Уровень 36

8 февраля 2018, 12:04

То есть получается, что initialAmount - это общий бюджет на рекламу, а hits - количество показов на эти деньги. Мне, как рекламщику, была непонятна эта схема и я её (машинально) переделал под себя. Теперь разобрался, спасибо. Но валидатору этого пока что мало. Вывод теперь соответствует:

Your order: [Water] of Tablet{number=5}
Start cooking - Your order: [Water] of Tablet{number=5}, cooking time 3min
Your order: [Water] of Tablet{number=5} was cooked by Amigo
5 is displaying... 835, 4638
Your order: [Water] of Tablet{number=5}
Start cooking - Your order: [Water] of Tablet{number=5}, cooking time 3min
Your order: [Water] of Tablet{number=5} was cooked by Amigo
5 is displaying... 835, 4638
Your order: [Water] of Tablet{number=5}
Start cooking - Your order: [Water] of Tablet{number=5}, cooking time 3min
Your order: [Water] of Tablet{number=5} was cooked by Amigo
5 is displaying... 835, 4638
08-02-2018 - 25.05
Total - 25.05
08-02-2018
Amigo - 9 min

C 3 Fish:

//оставил только последний из-за лимита символов в комментарии
Your order: [Fish] of Tablet{number=5}
Start cooking - Your order: [Fish] of Tablet{number=5}, cooking time 25min
Your order: [Fish] of Tablet{number=5} was cooked by Amigo
5 is displaying... 835, 4638
6 is displaying... 626, 3477
1 is displaying... 507, 2816
12 is displaying... 150, 1250
10 is displaying... 200, 1111
11 is displaying... 3, 16
3 is displaying... 49, 272
4 is displaying... 9, 75
08-02-2018 - 121.69
Total - 121.69
08-02-2018
Amigo - 75 min

Pavlo Plynko Java-разработчик в CodeGym

Expert

8 февраля 2018, 12:20

решение

А набор роликов остался такой же ?

private AdvertisementStorage() {
    Object someContent = new Object();
    add(new Advertisement(someContent, "1", 1523, 3, 3 * 60));
    add(new Advertisement(someContent, "2", 5, 2, 1 * 60));
    add(new Advertisement(someContent, "3", 99, 2, 3 * 60));
    add(new Advertisement(someContent, "4", 99, 10, 2 * 60));
    add(new Advertisement(someContent, "5", 2506, 3, 3 * 60));
    add(new Advertisement(someContent, "6", 2506, 3, 3 * 60));
    add(new Advertisement(someContent, "7", 400, 1, 3 * 60));
    add(new Advertisement(someContent, "8", 500, 1, 2 * 60));
    add(new Advertisement(someContent, "10", 400, 2, 3 * 60));
    add(new Advertisement(someContent, "11", 350, 100, 3 * 60)); // 3 min
    add(new Advertisement(someContent, "12", 1500, 10, 2 * 60)); //15 min
    add(new Advertisement(someContent, "13", 4600, 2, 10 * 60));   //10 min
}

А то у меня с 3 Fish последний отличается:

Your order: [Fish] of Tablet{number=5}, cooking time 25min
5 is displaying... 835, 4638
6 is displaying... 835, 4638
1 is displaying... 507, 2816
10 is displaying... 200, 1111
12 is displaying... 150, 1250
3 is displaying... 49, 272
4 is displaying... 9, 75
11 is displaying... 3, 16
Start cooking - Your order: [Fish] of Tablet{number=5}, cooking time 25min
Your order: [Fish] of Tablet{number=5}, cooking time 25min was cooked by Amigo
08-Feb-2018 - 127.96
Total - 127.96
08-Feb-2018
Amigo - 75 min

Alexey Sverbeev

Уровень 36

8 февраля 2018, 12:27

Да, у меня ещё в компараторе висел initialAmount(), заменил, но теперь по ходу какие-то проблемы с округлением, что ли.

Start cooking - Your order: [Fish] of Tablet{number=5}, cooking time 25min
Your order: [Fish] of Tablet{number=5} was cooked by Amigo
6 is displaying... 836, 4644
5 is displaying... 836, 4644
1 is displaying... 509, 2827
10 is displaying... 200, 1111
12 is displaying... 150, 1250
3 is displaying... 49, 272
4 is displaying... 9, 75
11 is displaying... 3, 16
08-02-2018 - 128.01
Total - 128.01
08-02-2018
Amigo - 75 min

Pavlo Plynko Java-разработчик в CodeGym

Expert

8 февраля 2018, 12:39

amountPerOneDisplaying = initialAmount / hits;

так считаешь?

Alexey Sverbeev

Уровень 36

8 февраля 2018, 12:48

нет, был Math.round, но ещё проблема была в revalidate. Убрал округление, убрал лишнюю строчку в revalidate, вывод такой:

Your order: [Fish] of Tablet{number=5}
Start cooking - Your order: [Fish] of Tablet{number=5}, cooking time 25min
Your order: [Fish] of Tablet{number=5} was cooked by Amigo
6 is displaying... 835, 4638
5 is displaying... 835, 4638
1 is displaying... 507, 2816
10 is displaying... 200, 1111
12 is displaying... 150, 1250
3 is displaying... 49, 272
4 is displaying... 9, 75
11 is displaying... 3, 16
08-02-2018 - 127.96
Total - 127.96
08-02-2018
Amigo - 75 min

Валидатор не принимает. Может еще дело в порядке вывода строк? Может я где-то не там notify ставлю?

Alexey Sverbeev

Уровень 36

8 февраля 2018, 13:06

Поставил processVideos до notify, строки теперь те же, валидатор все равно недоволен(

Your order: [Fish] of Tablet{number=5}
6 is displaying... 835, 4638
5 is displaying... 835, 4638
1 is displaying... 507, 2816
10 is displaying... 200, 1111
12 is displaying... 150, 1250
3 is displaying... 49, 272
4 is displaying... 9, 75
11 is displaying... 3, 16
Start cooking - Your order: [Fish] of Tablet{number=5}, cooking time 25min
Your order: [Fish] of Tablet{number=5} was cooked by Amigo
08-02-2018 - 127.96
Total - 127.96
08-02-2018
Amigo - 75 min

Pavlo Plynko Java-разработчик в CodeGym

Expert

8 февраля 2018, 13:10

Не принимает 3 и 4 требования?

Alexey Sverbeev

Уровень 36

8 февраля 2018, 13:12

Не принимало. Поиграл с pattern в формате даты и приняло (только в ней отличался вывод). Большое спасибо за помощь.

Вы решили задачу лучше, чем 5% учеников. Вам удалось ее решить с 48 попытки. Среднее количество попыток для этой задачи 14.95. Всего эту задачу решили 891 учеников.

Ivan Matsur

Уровень 40

10 июня 2018, 13:30

А ведь я когда писал паттерн даты, тоже подумал, что в примере может юзаться MMM, вместо MMMM, т.к. May состоит из 3-х символов... Но заюзал MMMM, потом долго искал ошибку - спасибо за напоминание :) Будущим покорителям: "правильный" формат месяца MMM, а не MMMM...

Наталья

Уровень 35

13 августа 2018, 11:41

Мил человек, спасибо тебе!!! Всё искала "фатальные ошибки" в логике своих методов (вали не принимал пп.3 и 4), а оказалось месяц надо выводить через "MMM"! =))))) рукалицо

NodeOne

Уровень 41

Expert

17 октября 2018, 15:29

респектос за настойчивость.