Помогите понять

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

Решен

так работает и приняло Валидатором.

public static Map<String, Integer> countWords(ArrayList<String> list) {
        HashMap<String, Integer> result = new HashMap<String, Integer>();

        //напишите тут ваш код
        return list.stream().collect(Collectors.toMap(Function.identity(), o -> 1, (v1, v2) -> v1 + v2));


        // return result;
    }

А вот так

public static Map<String, Integer> countWords(ArrayList<String> list) {
        HashMap<String, Integer> result = new HashMap<String, Integer>();

        //напишите тут ваш код
        result = list.stream().collect(Collectors.toMap(Function.identity(), o -> 1, (v1, v2) -> v1 + v2));
        return result;
    }

Ошибка (v1, v2) -> v1 + v2 не хочет складывать Object

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

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

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

hidden #1607360

Уровень 25

26 марта 2018, 18:33

Приведение типов (Integer v1, Integer v2) -> v1 + v2 тоже результатов не дало :(

hidden #1607360

Уровень 25

26 марта 2018, 18:22

return list.stream().collect(Collectors.groupingBy(Function.identity(), Collectors.summingInt(o -> 1)));

вот так тоже работает

Ярослав Java Developer

Master

26 марта 2018, 18:10

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

Ярослав Java Developer

Master

26 марта 2018, 18:50

решение

Идея подсказывает, что можно сделать так:

Map<String, Integer> map = list.stream().collect(Collectors.toMap(Function.identity(), o -> 1, (Integer v1, Integer v2) -> v1+v2));

Если прописывать HashMap, то уже ругается. Почему - не могу разобраться. Я подозреваю, что toMap возвращает просто Map, потому первый пример и сработал, ведь HashMap реализует Map, потому по законам полиморфизма все хорошо, просто вместо какой-то конкретной реализации, метод возвращает только общий случай, работа на уровне интерфейсов. Вот что неясно - почему оно ругалось, что v1 и v2 объекты. Вот хороший ответ нашел: https://stackoverflow.com/questions/33724733/confused-by-java8-collectors-tomap , суть в том, что чтобы вернуть конкретную реализацию мапы, нужно передать еще четвертый параметр - ссылку на конструктор нужной мапы (например TreeMap::new). Собственно, я решил стереть Integer и оставить просто так: Map<String, Integer> s = list.stream().collect(Collectors.toMap(Function.identity(), o -> 1, (v1, v2) -> v1+v2)); Оно заработало. Похоже, что все дело было в том, что мы возвращали неправильную мапу. А вот и пример передачи 4-го параметра в метод: HashMap<String, Integer> result = list.stream().collect(Collectors.toMap(Function.identity(), o -> 1, (v1, v2) -> v1+v2, HashMap::new)); Тогда тоже все хорошо. P.S. Спасибо за вопрос, пока его разбирал около получаса примерно, успел познакомиться с коллектами в стримах :)

hidden #1607360

Уровень 25

26 марта 2018, 19:13

HashMap::new - вот он недостающий элемент. Дякую :)