JavaRush/Помощь по Java задачам/Появляется ошибка в import или time out

39 уровень

23.06.2021
412просмотров
16комментариев

Появляется ошибка в import или time out

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

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

Решен

Добрый день. Если отправляю задачу на проверку из любого класса в пакете ad, выпадает ошибка компиляции

Ошибка в файле com/javarush/task/task27/task2712/big17/ad/AdvertisementManager.java в строке :  3
Не могу найти описание класса "ConsoleHelper" в packageе "ru.rush.task.task27.task2712.big17". Возможно вы забыли его импортировать (указать в import)

Все импорты верные. Пересоздание пакета не помогает. Пересоздание ConsoleHelper тоже Отправка на проверку из любого другого класса заканчивается time out. Подскажите, пожалуйста, как можно решить эту проблему?

Реализуем третий и четвертый пункт статистики - список активных и неактивных роликов.
Для этого проще использовать доступ к хранилищу рекламных роликов - класс AdvertisementStorage.
1. В пакете ad создай StatisticAdvertisementManager, который будет предоставлять информацию из AdvertisementStorage в нужном нам виде.
Сделай его синглтоном.

2. В классе StatisticAdvertisementManager создай и проинициализируй поле типа AdvertisementStorage.

3. В StatisticAdvertisementManager создай два (или один) метода (придумать самостоятельно), которые из хранилища AdvertisementStorage достанут все необходимые данные - соответственно список активных и неактивных рекламных роликов.
Активным роликом считается тот, у которого есть минимум один доступный показ.
Неактивным роликом считается тот, у которого количество показов равно 0.

4. Реализуй логику методов printActiveVideoSet и printArchivedVideoSet в классе DirectorTablet.
Используй методы/метод, созданные в предыдущем пункте.
Сортировать по имени видео-ролика в алфавитном порядке без учета регистра.
Сначала английские, потом русские.

Пример вывода для printActiveVideoSet (имя ролика - оставшееся количество показов ролика):
First Video - 100 Second video - 10 Third Video - 2 четвертое видео - 4

Пример вывода для printArchivedVideoSet (имена роликов с количеством показов 0):
Second video Third Video четвертое видео

Требования:

Класс StatisticAdvertisementManager должен быть синглтоном.
Метод printActiveVideoSet в классе DirectorTablet должен быть реализован в соответствии с условием задачи.
Метод printArchivedVideoSet в классе DirectorTablet должен быть реализован в соответствии с условием задачи.

package com.javarush.task.task27.task2712; import com.javarush.task.task27.task2712.kitchen.Cook; import com.javarush.task.task27.task2712.kitchen.Waiter; public class Restaurant { public static void main(String[] args) { Tablet tablet = new Tablet(1); Cook amigo = new Cook("Amigo"); tablet.addObserver(amigo); Waiter waiter = new Waiter(); amigo.addObserver(waiter); tablet.createOrder(); DirectorTablet directorTablet = new DirectorTablet(); directorTablet.printAdvertisementProfit(); directorTablet.printCookWorkloading(); directorTablet.printActiveVideoSet(); directorTablet.printArchivedVideoSet(); } }

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

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

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

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

Master

24 июня 2021, 13:25

for (int i = availableVideos.size() - 1; i >= 0 ; i--) {
            List tmpVideoList = new ArrayList<>(availableVideos);
            tmpVideoList.remove(i);
            makeOptimalVideoLists(tmpVideoList);
        }

Диана Чиганцева

Уровень 39

25 июня 2021, 06:18

Подскажите, пожалуйста, чем цикл с конца лучше?

Павел Безумный учёный

Expert

24 июня 2021, 10:00

Тайм-аут во время валидации возникает из-за ошибки в алгоритме подбора роликов для показа. В методе makeOptimalVideoLists() перед рекурсивным вызовом добавьте следующую проверку:
if (calculateTotalDurationOneListOfVideo(tmpVideoList) <= timeSeconds) { makeOptimalVideoLists(tmpVideoList); }
Это поможет значительно сократить количество проверяемых наборов, если хранилище storage будет более объёмным (например, 15-20 роликов). После этого заполните хранилище следующим набором роликов:

videos.add(new Advertisement(someContent, "Video 1", 500, 10, 5 * 60));
videos.add(new Advertisement(someContent, "Video 2", 700, 10, 7 * 60));
videos.add(new Advertisement(someContent, "Video 3", 500, 10, 5 * 60));
videos.add(new Advertisement(someContent, "Video 4", 1500, 10, 15 * 60));
videos.add(new Advertisement(someContent, "Video 5", 700, 10, 7 * 60));
videos.add(new Advertisement(someContent, "Video 6", 100, 10, 1 * 60));

Запустите программу и сделайте заказ FISH. В этом случае должен быть определён такой оптимальный набор видео:

Video 4 is displaying... 150, 166
Video 3 is displaying... 50, 166
Video 1 is displaying... 50, 166

Диана Чиганцева

Уровень 39

24 июня 2021, 12:21

Получается мне нужно полностью переработать алгоритм подбора роликов? Т.к. вставка этой проверки перед рекурсивным вызовом ломает весь текущий подбор. Например, набор Video 4, Video 3, Video 1 даже не подбирается. Печаль

Павел Безумный учёный

Expert

24 июня 2021, 12:33

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

Диана Чиганцева

Уровень 39

24 июня 2021, 12:42

К сожалению, снова time out

Павел Безумный учёный

Expert

24 июня 2021, 23:32

решение

В результате экспериментов с кодом удалось выяснить следующее. 1. Избежать тайм-аута можно, обернув рекурсивный вызов метода makeOptimalVideoLists() в условие:

if (calculateTotalPriceOneListOfVideo(availableVideos) >= bestPriceListOfVideo)
    makeOptimalVideoLists(tmpVideoList);
}

2. В методе checkVideo() перед добавлением плейлиста в общий список videoLists можно добавить дополнительную проверку:

if (totalPrice > bestPriceListOfVideo) {
    videoLists.clear();
}

Если найден плейлист с большей прибылью от показа, то все предыдущие составленные плейлисты можно не рассматривать (оптимизация). 3. В методе processVideos() перед выполнением поиска оптимального плейлиста создайте список-копию исходного хранилища роликов, "разверните" его при помощи метода Collections.reverse() и после этого передайте в метод makeOptimalVideoLists(). Такой манёвр изменяет порядок вывода равноценных роликов в плейлисте, и часто это помогает удовлетворить валидатор.

Диана Чиганцева

Уровень 39

25 июня 2021, 08:09

Ура! Получилось)) Всего одна проверка, о которой вы написали в п.1, очень сильно сократила кол-во рекурсивных вызовов. Эх, мне еще учиться и учиться)) Все советы очень помогли.Спасибо Вам большое за помощь и отзывчивость) Подскажите еще, пожалуйста, не будет лишней дополнительная проверка?

if (!videoLists.contains(tmpVideoList)) {
    makeOptimalVideoLists(tmpVideoList);
}

Если такой набор уже встречался и был добавлен, то не проверять снова. Она сокращает кол-во рекурсий еще немного.

Павел Безумный учёный

Expert

25 июня 2021, 10:35

Рад, что помог)) Хорошая идея насчёт исключения дублирующихся наборов, но такую проверку нужно делать в методе checkVideo() перед добавлением плейлиста в список videoLists. Если в данное условие обернуть вызов makeOptimalVideoLists(), то цепочка рекурсивных вызовов может прерваться преждевременно, и не все варианты плейлистов будут получены. Можно также в качестве накопительного хранилища videoLists использовать TreeSet, в который передать компаратор при создании. Тогда все плейлисты-дубликаты будут автоматически отсеиваться, а первый элемент множества будет оптимальным плейлистом.

Flexo Bending Unit #3370318

23 июня 2021, 15:59

Известный баг Когда задачу апрувите, вставайте на файл условия, с него надо апрувиться

Диана Чиганцева

Уровень 39

23 июня 2021, 17:05

С файла условия получаю timeout

Flexo Bending Unit #3370318

23 июня 2021, 17:09

значит, проблему дальше надо смотреть. бесконечные циклы, например. с consolehelper проблем больше нет, как видите)

Диана Чиганцева

Уровень 39

24 июня 2021, 06:55

Если не трудно, посмотрите, пожалуйста код. Может я чего-то не замечаю.

Flexo Bending Unit #3370318

24 июня 2021, 09:28

извините, не смогу - задача слишком объемная. обычно если я смотрю задачу, то копирую себе в IDE, тестирую... но тут слишком много файлов. притом, уверен, что я смогу запустить ваш код и ничего подозрительного не найду, валидация в этих задачах, она такая. сфокусируйтесь на изменениях с последней утверждённой задачи, почитайте комментарии под этой задачей, возможно, возникали похожие ситуации

Диана Чиганцева

Уровень 39

24 июня 2021, 09:46

В том-то и дело, что ни у кого не встретила проблем с timeout. Все равно спасибо за помощь)

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

Master

24 июня 2021, 11:59

timeout вознимает, так как слишком долго выполняется твой код из-за рекурсии