0

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

Мой код:

public class CollectToMap {
    public static Map<Integer, Integer> collect(Stream<Integer> data) {
        Map<Integer, Integer> result = Set.of(data).stream().collect(Collectors.toMap(k -> k, v -> v));

        return result;
    } 
}

Мой тест:

public class CollectToMapTest {
    @Test
    public void test() {
        List<Integer> data = List.of(2, 3);
        Map<Integer, Integer> expect = Map.of(
                2, 4,
                3, 9
        );
        Map<Integer, Integer> result = CollectToMap.collect(data.stream());
        assertEquals(expect, result);
    }
}

В чем моя ошибка?

4
  • @Danil Apsadikov, лучше не форматировать так код, что все символы > слетают: ru.stackoverflow.com/review/suggested-edits/780201
    – insolor
    9 сен 2022 в 12:26
  • @insolor Хорошо, заметил только после отправки ) 9 сен 2022 в 12:29
  • У вас повторяется одна и та же подобная ошибка: вы зачем-то оборачиваете каждый раз входной список/стрим в отдельную коллекцию, для которой снова применяете .stream(). Таким образом Set.of(data).stream() возвращает Stream<Stream<Integer>>. Представленный код вообще не компилируется, возникает ошибка: error: incompatible types: inference variable K has incompatible bounds 9 сен 2022 в 20:34
  • что нужно исправить в коде чтобы код компилировался?
    – Dr.Joe
    10 сен 2022 в 13:22

1 ответ 1

1

На сколько я понял, вы забыли v * v

public static Map<Integer, Integer> collect(Stream<Integer> data) {
    return data.collect(Collectors.toMap(k -> k, v -> v * v));
}
4
  • нет. дело не в этом коде - v * v
    – Dr.Joe
    9 сен 2022 в 13:11
  • Тогда не совсем понял в чем вопрос. Я не много изменил код и ваш тест мой код прошел. Уточните проблему 9 сен 2022 в 16:53
  • @DanilApsadikov, вы радикально изменили код, убрав излишнее оборачивание входного стрима в Set при помощи Set.of, поэтому ваш вариант успешно скомпилировался и прошел тест. 9 сен 2022 в 20:38
  • @DanilApsadikov все разобрался. Спасибо!
    – Dr.Joe
    11 сен 2022 в 14:08

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.