Кто шарит в Stream API?

Решен

Ребят, кто знает, как данный метод зарефакторить таким образом, что бы результат был тот же, используя при этом только Stream API?

public Set<String> getUniqueIPs() {
    Set<String> strings = new HashSet<>();
    for (String line : logs) {
        Matcher matcher = pattern.matcher(line);
        if (matcher.find())
            strings.add(matcher.group("ip"));
    }
    return strings;
}

Используя стримы, я столкнулся с проблемой преобразования исходящей строки line из списка logs в matcher.group("ip"). Обьясню, вот такая конструкция stream добавляет в сет целый елемент из списка logs (то есть line), если он подпадает под паттерн. А нужно обрезать линию-елемент списка logs до matcher.group("ip")

public int getUniqueIPs() {
        return logs.stream()
                .filter(pattern.asPredicate())
                .collect(Collectors.toSet());
    }

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

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

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

JustinianJudge в Mega City One

Master

11 августа 2019, 22:02

решение

Я бы написал из серии:

return logs.stream()
                .map(pattern::matcher)
                .filter(Matcher::matches)
                .map(matcher -> matcher.group("ip"))
                .collect(Collectors.toSet());

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

Ярослав Java Developer

Master

12 августа 2019, 15:14

Да, решение красивее, я свое "на коленках" сочинил за минуту, отвлекаясь на другие дела :)

hidden #1281202

Уровень 41

12 августа 2019, 15:29

Вот это то что надо, отличное решение! Ссылки на методы вместе со стримами выглядят невероятно, спасибо большое. Насколько могу понять, в map операторе вызывается метод matcher(line) по переменной pattern и возвращается обьект Matcher, который переходит в оператор filter и нужен для вызова метода matches, так как он нестатический и без Matcher не может быть вызван. То есть, происходит то же самое:

Pattern pattern = Pattern.compile("example");
        Matcher matcher = pattern.matcher(line);
        matcher.matches();

Далее filter делает свою работу в matches и мы получаем только елементы logs, которые matches отметил true. Если где-то ошибся, прошу исправь, так как очень хочется понять, сколько же функционала спряталось за такой красотой 👏🏼

JustinianJudge в Mega City One

Master

12 августа 2019, 15:48

Да, где-то так. Обычный метод-референс:

list.forEach(System.out::println);

для каждого элемента списка вызвать метод println переменной out, просто вместо коллекции, в качестве саплаера выступает операции map и filter , результатом работы которых является стрим объектов для пакетной обработки элементов ну и терминальная операция коллект. Рекомендую очень Java 8 in Action, я сам еще не читал, но по лямбдах и стримах очень легко, просто и толково написано. А ты проверял, работает вообще? )

hidden #1281202

Уровень 41

12 августа 2019, 16:13

Да, я проверил твой вариант, он отлично работает у меня в методе getAllUsers из задания.

@Override
    public Set<String> getAllUsers() {
        return logs.stream()
                .map(pattern::matcher)
                .filter(Matcher::matches)
                .map(matcher -> matcher.group("name"))
                .collect(Collectors.toSet());
    }

Для всех остальных методов должны присутствовать проверки времени, юзеров и других составляющих. В этой задаче, у меня уже 400 строк только моего кода на 3 подзадаче и при этом все методы примерно по размеру того, который я просил зарефакторить. Возможно, в конце задачи я все методы переведу на стрим и посмотрим, сколько строк получится срезать.

hidden #1281202

Уровень 41

12 августа 2019, 16:27

@Override
    public Set<Event> getAllEvents(Date after, Date before) {
        return logs.stream()
                .map(pattern::matcher)
                .filter(Matcher::matches)
                .filter(matcher -> isDateInRange(dateParser(matcher.group("date")), after, before))
                .map(matcher -> matcher.group("event"))
                .map(this::getEvents)
                .collect(Collectors.toSet());
    }

Вот так теперь выглядит метод с проверкой на время и выводом обьектов класса Events. Читабельность понемногу теряется, если, например, попробовать сбить два фильтра вместе, и теперь этот метод больше по обьёму, чем обычный. Это только одна проверка на время, а в некоторых методах нужно ещё две минимум(

Ярослав Java Developer

Master

11 августа 2019, 19:09

полезный

strings.stream() .map(string -> { Matcher matcher = pattern.matcher(line); if (matcher.find()) return matcher.group("ip"); return null; }) .filter(Objects::nonNull) .collect(Collectors.toSet()); Как вариант. Но это скорее самый быстрый вариант, первый пришедший в голову. Еще есть вариант такой: return logs.stream() .filter(pattern.asPredicate()) .map(string -> { Matcher matcher = pattern.matcher(string); matcher.find(); // чтобы оно прошлось по строке return matcher.group("ip"); }) .collect(Collectors.toSet());

hidden #1281202

Уровень 41

11 августа 2019, 19:51

Решение хорошое, совсем не думал о анонимном классе в .map операторе и повторном matcher.find() в нём же, так как отталкивался от .filter(pattern.asPredicate()). На самом деле, искал изящное решение в 3-4 строки, так как судя по исходному методу, решения побольше исключают преимущества стримов в данном примере. Еще подожду - может кто-то знает, как написать что-то подобное. Если нет, то комментарий, на который я сейчас пишу ответ - решение)