Не проходит по 2 пункту

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

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

Решен

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

Реализуем третий и четвертый пункт статистики - список активных и неактивных роликов.
Для этого проще использовать доступ к хранилищу рекламных роликов - класс 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.Dish; import com.javarush.task.task27.task2712.kitchen.Order; 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_1 = new Tablet(1); Cook cook = new Cook("John"); tablet_1.addObserver(cook); Waiter waiter = new Waiter(); cook.addObserver(waiter); tablet_1.createOrder(); tablet_1.createOrder(); /* tablet_1.createOrder(); tablet_1.createOrder();*/ DirectorTablet directorTablet = new DirectorTablet(); directorTablet.printAdvertisementProfit(); directorTablet.printCookWorkloading(); directorTablet.printActiveVideoSet(); directorTablet.printArchivedVideoSet(); } }

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

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

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

И. Ж.

Уровень 41

1 июня 2023, 18:03

Аналогичная ситуация, второй пункт тоже не проходил. Использовал ваши наработки, добавил ваше дополнение, и тоже все за раз прокатило. Благодарю, так хрен бы разобрался!

Igor

Уровень 38

4 ноября 2022, 13:15

Разобрался самостоятельно. Как оказалось, конкретно в этом задании ошибки не было, да и вообще её не было, просто у валидатора есть скрытый подтекст, который нужно было реализовать еще в 10 задании, которое я уже прошёл))) На правильную мысль натолкнул комментарий Павел в одном из вопросов к задаче. Пришлось допилить метод processVideos(). И так при определении оптимального набора видеороликов (задача 10) может возникнуть ситуация, когда существует несколько вариантов набора видео-роликов: - с одинаковой суммой денег, полученной от показов; - с одинаковым суммарным временем; - с одинаковым количеством роликов. Как показывает практика, валидатор в подобных случаях отдаёт предпочтение плейлисту, в котором больше роликов, расположенных ближе к началу в исходном хранилище. Вероятно, это связано с реализацией "жадного" алгоритма, отбрасывающего лишние ролики с "хвоста". Я решил эту задачу следующим образом. Написал метод в AdvertisementManager, который определяет сумму индексов всех элементов в начальном списке.

public int bestIndex(List<Advertisement> videos) {
        int sumIndex = 0;
        List<Advertisement> storage = AdvertisementStorage.getInstance().list();
        for (Advertisement ad : videos) {
            if (storage.contains(ad)) {
                sumIndex = storage.indexOf(ad) + sumIndex;
            }
        }
        return sumIndex;
    }

Igor

Уровень 38

4 ноября 2022, 13:17

А потом добавил еще один вариант перебора листов в checkSet(List<Advertisement> videos)

public void checkSet(List<Advertisement> videos) {
        if (bestVideos == null) {
            if (calcDurationPlaying(videos) <= timeSeconds) {
                bestVideos = videos;
                bestPrice = calcMaxSumOfAmountPerOneDisplaying(videos);
            }
        } else {
            int calcTime = calcDurationPlaying(videos);
            int calcTimeBest = calcDurationPlaying(bestVideos);
            long calcPrice = calcMaxSumOfAmountPerOneDisplaying(videos);
            if (calcTime <= timeSeconds) {
                if (calcPrice > bestPrice) {
                    bestVideos = videos;
                    bestPrice = calcPrice;
                } else {
                    if (calcPrice == bestPrice) {
                        if (calcTime > calcTimeBest) {
                            bestVideos = videos;
                        } else {
                            if (calcTime == calcTimeBest) {
                                if (videos.size() < bestVideos.size()) {
                                    bestVideos = videos;
                                } else {
                                    if (videos.size() == bestVideos.size()) {
                                        if (bestIndex(videos) < bestIndex(bestVideos)) {
                                            bestVideos = videos;
                                        }
                                    }
                                }
                            }
                        }
                    }
                }
            }
        }
    }

Igor

Уровень 38

4 ноября 2022, 13:23

ТЕСТ

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

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

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

А раньше было:

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

Алексей

Уровень 35

16 ноября 2022, 21:43

Спасибо, сделал Collections.revese перед тем как отправлять в метод с рекурсией и все пропустило. Надеюсь эту дичь пофиксят!