Задача не проходит.

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

Обсуждается

Уже даже скопировал готовое решение, но задача так и так не проходит. Я просто не знаю что делать.

Первая стратегия готова, пришло время ее протестить. Для этого:
6.1. Создай класс Solution, если ты не сделал это раньше.
6.2. Добавь в класс Solution реализации вспомогательных статических методов:
6.2.1. Set<Long> getIds(Shortener shortener, Set<String> strings). Этот метод должен для переданного множества строк возвращать множество идентификаторов. Идентификатор для каждой отдельной строки нужно получить, используя shortener.
6.2.2. Set<String> getStrings(Shortener shortener, Set<Long> keys). Метод будет возвращать множество строк, которое соответствует переданному множеству идентификаторов.
При реальном использовании Shortener, задача получить из множества строк множество идентификаторов и наоборот скорее всего не встретится, это нужно исключительно для тестирования.
6.2.3. testStrategy(StorageStrategy strategy, long elementsNumber). Метод будет тестировать работу переданной стратегии на определенном количестве элементов elementsNumber. Реализация метода должна:
6.2.3.1. Выводить имя класса стратегии. Имя не должно включать имя пакета.
6.2.3.2. Генерировать тестовое множество строк, используя Helper и заданное количество элементов elementsNumber.
6.2.3.3. Создавать объект типа Shortener, используя переданную стратегию.
6.2.3.4. Замерять и выводить время необходимое для отработки метода getIds для заданной стратегии и заданного множества элементов. Время вывести в миллисекундах. При замере времени работы метода можно пренебречь переключением процессора на другие потоки, временем, которое тратится на сам вызов, возврат значений и вызов методов получения времени (даты). Замер времени произведи с использованием объектов типа Date.
6.2.3.5. Замерять и выводить время необходимое для отработки метода getStrings для заданной стратегии и полученного в предыдущем пункте множества идентификаторов.
6.2.3.6. Сравнивать одинаковое ли содержимое множества строк, которое было сгенерировано и множества, которое было возвращено методом getStrings. Если множества одинаковы, то выведи "Тест пройден.", иначе "Тест не пройден.".
6.2.4. Добавь метод main(). Внутри метода протестируй стратегию HashMapStorageStrategy с помощью 10000 элементов.
6.3. Проверь, что программа работает и тест пройден.

Требования:

Метод getIds должен для переданного множества строк возвращать множество идентификаторов.
Метод getStrings должен для переданного множества идентификаторов возвращать множество строк.
Метод testStrategy должен выводить на экран имя класса стратегии.
Метод testStrategy должен генерировать тестовое множество строк используя метод generateRandomString класса Helper (количество элементов должно быть равно параметру elementsNumber).
Метод testStrategy должен создавать объект типа Shortener используя переданную стратегию.
Метод testStrategy должен замерять количество времени выполнения методов getIds и getStrings используя объекты типа Date.
Метод testStrategy должен сравнивать количество сгенерированных строк и строк в множестве полученном в результате работы метода getStrings и выводить на экран результат прохождения теста.
В методе main должен быть вызван метод testStrategy.

package com.javarush.task.task33.task3310; import com.javarush.task.task33.task3310.strategy.HashMapStorageStrategy; import com.javarush.task.task33.task3310.strategy.StorageStrategy; import java.util.Date; import java.util.HashSet; import java.util.Set; public class Solution { public static void main(String[] args) { long elementsNumber = 10000; testStrategy(new HashMapStorageStrategy(), elementsNumber); } public static void testStrategy(StorageStrategy strategy, long elementsNumber) { Helper.printMessage(strategy.getClass().getSimpleName() + ":"); Set<String> origStrings = new HashSet<>(); for (int i = 0; i < elementsNumber; ++i) { origStrings.add(Helper.generateRandomString()); } Shortener shortener = new Shortener(strategy); Date startTimestamp = new Date(); Set<Long> keys = getIds(shortener, origStrings); Date endTimestamp = new Date(); long time = endTimestamp.getTime() - startTimestamp.getTime(); Helper.printMessage("Время получения идентификаторов для " + elementsNumber + " строк: " + time); startTimestamp = new Date(); Set<String> strings = getStrings(shortener, keys); endTimestamp = new Date(); time = endTimestamp.getTime() - startTimestamp.getTime(); Helper.printMessage("Время получения строк для " + elementsNumber + " идентификаторов: " + time); if (origStrings.equals(strings)) Helper.printMessage("Тест пройден."); else Helper.printMessage("Тест не пройден."); Helper.printMessage(""); } public static Set<Long> getIds(Shortener shortener, Set<String> strings) { Set<Long> keys = new HashSet<>(); for (String s : strings) { keys.add(shortener.getId(s)); } return keys; } public static Set<String> getStrings(Shortener shortener, Set<Long> keys) { Set<String> strings = new HashSet<>(); for (Long k : keys) { strings.add(shortener.getString(k)); } return strings; } }

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

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

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

hidden #2322530

Уровень 41

19 октября 2020, 08:27

public synchronized Long getId(String string){
    //return null;
    return storageStrategy.getKey(string);
}

у тебя если такой записи в словаре нет, вернёт null.

@Override
public Long getKey(String value) {
    for(Long i : data.keySet()){
        if(data.get(i).equals(value)){
            return i;
        }
    }
    return null;
}

Set<String> origStrings = new HashSet<>();
for (int i = 0; i < elementsNumber; ++i) {
    origStrings.add(Helper.generateRandomString());
}
Set<Long> keys = getIds(shortener, origStrings);

а как бы таких строк еще там нет. вывод? у тебя будут всегда возвращаться null. насколько я помню задание, в методе:

public synchronized Long getId(String string){

надо вернуть ключ, если у него есть такой значение, а если такого элемента в словаре нет, тогда надо его создать и вернуть новый ключ. ЗЫ. на будущее, если валидатор пишет что ошибка в таком то методе, это значит, что ошибка может быть в любом другом методе, который вызывается в этом. как пример я указал выше.