JavaRush/Помощь по задачам/Помогите понять что хочет валидатор

41 уровень

29.01.2021
406просмотров
22комментариев

Помогите понять что хочет валидатор

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

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

Решен

Я не могу понять что не устраивает валидатор. Как уже не изгалялся. Даже втупую копировал их решение. Ничего не помогает. Второй пункт не проходит никак , а первый 50 на 50 скачет. Кто решил эту задачу скажите что хочет валидатор?

Реализуем третий и четвертый пункт статистики - список активных и неактивных роликов.
Для этого проще использовать доступ к хранилищу рекламных роликов - класс 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); DirectorTablet directorTablet = new DirectorTablet(); Cook cook = new Cook("Vis"); Waiter waiter = new Waiter(); cook.addObserver(waiter); tablet.addObserver(cook); tablet.createOrder(); directorTablet.printActiveVideoSet(); directorTablet.printAdvertisementProfit(); directorTablet.printArchivedVideoSet(); directorTablet.printCookWorkloading(); } }

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

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

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

Виталий Embedded Developer в Germanstandard

2 февраля 2021, 09:11

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

hidden #2322530

Уровень 41

29 января 2021, 12:32

private static StatisticAdvertisementManager INSTANCE;

сделай сразу инициализацию. тебя никто не просил тут делать ленивую. есть ещё такой метод:

return o1.getName().compareToIgnoreCase(o2.getName());

alex_us

Уровень 41

29 января 2021, 12:41

это уже перепробовано - не работает. 2 пункт не проходит, а первый через раз

hidden #2322530

Уровень 41

29 января 2021, 12:43

что именно перепробовано

alex_us

Уровень 41

29 января 2021, 12:44

и реализация синглтона не ленивая и метод compareToIgnoreCase

hidden #2322530

Уровень 41

29 января 2021, 12:44

ConsoleHelper.writeMessage("");

а эту строку ты зачем выводишь на экран?

alex_us

Уровень 41

29 января 2021, 12:46

это для удобства чтения при тестировании . Чтобы разделялись выводы методов

hidden #2322530

Уровень 41

29 января 2021, 12:47

какой актуальный код метода?

alex_us

Уровень 41

29 января 2021, 12:50

а как обновить прикрепленный код?

hidden #2322530

Уровень 41

29 января 2021, 12:51

через редактирование вопроса вроде. галочку перещёлкнуть туда сюда, и подкрепится последний вариант.

alex_us

Уровень 41

29 января 2021, 12:52

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

import java.util.ArrayList;
import java.util.List;

public class StatisticAdvertisementManager {
    private static final StatisticAdvertisementManager INSTANCE = new StatisticAdvertisementManager();
    private  AdvertisementStorage storage = AdvertisementStorage.getInstance();

    private StatisticAdvertisementManager() {}

    public static StatisticAdvertisementManager getInstance() {
        return INSTANCE;
    }

    public List<Advertisement> getActiveVideos() {
        List<Advertisement> resultList = new ArrayList<>();

        for (Advertisement ad : storage.list()) {
            if (ad.getHits() > 0)
                resultList.add(ad);
        }

        return resultList;
    }

    public List<Advertisement> getNoActiveVideos() {
        List<Advertisement> resultList = new ArrayList<>();

        for (Advertisement ad : storage.list()) {
            if (ad.getHits() == 0)
                resultList.add(ad);
        }

        return resultList;
    }
}

alex_us

Уровень 41

29 января 2021, 12:52

public void printActiveVideoSet() {
        StatisticAdvertisementManager adManager = StatisticAdvertisementManager.getInstance();
        List<Advertisement> activeVideoList = adManager.getActiveVideos();

        Collections.sort(activeVideoList, new Comparator<Advertisement>() {
            @Override
            public int compare(Advertisement o1, Advertisement o2) {
                return o1.getName().compareToIgnoreCase(o2.getName());
            }
        });

        for (Advertisement ad : activeVideoList) {
            ConsoleHelper.writeMessage(ad.getName() + " - " + ad.getHits());
        }

    }

    public void printArchivedVideoSet() {
        StatisticAdvertisementManager adManager = StatisticAdvertisementManager.getInstance();
        List<Advertisement> noActiveVideoList = adManager.getNoActiveVideos();

        Collections.sort(noActiveVideoList, new Comparator<Advertisement>() {
            @Override
            public int compare(Advertisement o1, Advertisement o2) {
                return o1.getName().compareToIgnoreCase(o2.getName());
            }
        });

        for (Advertisement ad : noActiveVideoList) {
            ConsoleHelper.writeMessage(ad.getName());
        }

    }

alex_us

Уровень 41

29 января 2021, 13:07

это все раздражает сил нет. Хоть бери и втупую их код со всех классов копируй

hidden #2322530

Уровень 41

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

public class AdvertisementStorage {
    private static AdvertisementStorage instance = new AdvertisementStorage();

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

alex_us

Уровень 41

29 января 2021, 13:11

не проходит и главное все так просто и прямолинейно вроде и при тестировании выводит вроде правильно все

alex_us

Уровень 41

29 января 2021, 13:19

короче я втупую скопировал их классы . При выводе ничего не поменялось. Ни на грамм. Зато прошло валидацию. Тошнит)

hidden #2322530

Уровень 41

29 января 2021, 13:20

ну так неинтересно)

alex_us

Уровень 41

29 января 2021, 13:25

да тошнит и отбивает желание) я подобавлял видео в хранилище с hits = 0 все выводит четко в алфавитном порядке и рус после англ. Что им еще надо?) а так это может привести к тому что из ненависти я на пару дней выйду из строя)) такое уже бывало пару раз

Анатолий

Уровень 41

30 января 2021, 18:44

Привет, а можешь написать код обоих методов (printActiveVideoSet() и второго) из обоих классов (DirectorTablet и StatisticAdvertisementManager), которые прошли валидацию по 2 пункту, бьюсь над этим тоже...

alex_us

Уровень 41

30 января 2021, 18:48

public void printActiveVideoSet() {
        List<Advertisement> videoSet = StatisticAdvertisementManager.getInstance().getVideoSet(true);
        Collections.sort(videoSet, new Comparator<Advertisement>() {
            @Override
            public int compare(Advertisement o1, Advertisement o2) {
                return o1.getName().toLowerCase().compareTo(o2.getName().toLowerCase());
            }
        });

        for (Advertisement advertisement : videoSet) {
            System.out.println(advertisement.getName() + " - " + advertisement.getHits());
        }
    }

    public void printArchivedVideoSet() {
        List<Advertisement> videoSet = StatisticAdvertisementManager.getInstance().getVideoSet(false);
        Collections.sort(videoSet, new Comparator<Advertisement>() {
            @Override
            public int compare(Advertisement o1, Advertisement o2) {
                return o1.getName().toLowerCase().compareTo(o2.getName().toLowerCase());
            }
        });

        for (Advertisement advertisement : videoSet) {
            System.out.println(advertisement.getName());
        }
    }

alex_us

Уровень 41

30 января 2021, 18:49

public List<Advertisement> getVideoSet(boolean isActive) {
        List<Advertisement> result = new ArrayList<>();
        for (Advertisement advertisement : storage.list()) {
            if (!isActive ^ advertisement.isActive()) {
                result.add(advertisement);
            }
        }
        return result;
    }

alex_us

Уровень 41

30 января 2021, 18:49

это решение в некоторых местах довольно криво выглядит, можно лучше. Но только такое прошло