Как же решить эту задачу?

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

Обсуждается

Если кому интересно, то начало тут: прошлое обсуждение (закрыл этот вопрос, потому как код не обновился, смысла там что-то писать нет) Переписал полностью весь код с 5, 6, 7 шага. (QLQuery inteface), выделил утилитные методы в отдельные классы. Методы для работы с запросом в QueryUtils.class методы для работы с логами в LogsUtils.class. Методы сделал статическими. Изменил логику работы метода LogParser.execute(String query) и получил +3 пункта "ОК". А непонятки остались. То есть, если рассуждать логически то: 1. Если проходят последние два пункта, а в особенности последний пункт, значит запрос разбирается верно, и алгоритм выборки данных по основной части запроса работает, условие из запроса парсится верно, метод возвращает верный набор данных 2. Если проходит хоть одно из всех прочих условий, значит верно парсятся и даты запрашиваемого промежутка. Алгоритм работы то на все возможные наборы полей запроса тоже один. То есть либо он по разному тестирует разные наборы <Запрашиваемое поле> for <Условие> = <Значение> and date between <промежуток времени>, либо он требует от одного алгоритма выбора и фильтрации данных разные результаты. Вопрос знатокам: А че надо то ему вообще? Проверял ручками что смог - все вроде верно работает. Понять не могу, как проверяет валидатор и что проверяет. Где он ошибку находит? Что должен вернуть метод, если для какого-то запроса не окажется данных. Например, логическая ошибка запроса вроде: "get status for date = "01.01.2015 0:00:00" and date between "01.01.2019 0:00:00" and "01.01.2020 0:00:00"" - null, пустой список? Фильтровать по датам интервала включительно или же нет?

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

Пример запроса:
get ip for user = "Eduard Petrovich Morozko" and date between "11.12.2013 0:00:00" and "03.01.2014 23:59:59".

Ожидаемый результат:
Set<String> с записями: 127.0.0.1 и 146.34.15.5.

Общий формат запроса:
get field1 for field2 = "value1" and date between "after" and "before"
Дополнительным параметром может быть только интервал дат, который нас интересует.

Поддержка старых форматов запросов должна сохраниться.

Требования:

Вызов метода execute с параметром "get ip for user = "[any_user]" and date between "[after]" and "[before]"" должен возвращать множество уникальных IP адресов, с которых работал пользователь с именем [any_user] в период между датами [after] и [before].
Вызов метода execute с параметром "get ip for date = "[any_date]" and date between "[after]" and "[before]"" должен возвращать множество уникальных IP адресов, события с которых произведены в указанное время [any_date] в период между датами [after] и [before].
Вызов метода execute с параметром "get ip for event = "[any_event]" and date between "[after]" and "[before]"" должен возвращать множество уникальных IP адресов, у которых событие равно [any_event] в период между датами [after] и [before].
Вызов метода execute с параметром "get ip for status = "[any_status]" and date between "[after]" and "[before]"" должен возвращать множество уникальных IP адресов, события с которых закончились со статусом [any_status] в период между датами [after] и [before].
Вызов метода execute с параметром "get user for ip = "[any_ip]" and date between "[after]" and "[before]"" должен возвращать множество уникальных пользователей, которые работали с IP адреса [any_ip] в период между датами [after] и [before].
Вызов метода execute с параметром "get user for date = "[any_date]" and date between "[after]" and "[before]"" должен возвращать множество уникальных пользователей, которые произвели любое действие в указанное время [any_date] в период между датами [after] и [before].
Вызов метода execute с параметром "get user for event = "[any_event]" and date between "[after]" and "[before]"" должен возвращать множество уникальных пользователей, у которых событие равно [any_event] в период между датами [after] и [before].
Вызов метода execute с параметром "get user for status = "[any_status]" and date between "[after]" and "[before]"" должен возвращать множество уникальных пользователей, у которых статус равен [any_status] в период между датами [after] и [before].
Вызов метода execute с параметром "get date for ip = "[any_ip]" and date between "[after]" and "[before]"" должен возвращать множество уникальных дат, за которые с IP адреса [any_ip] произведено любое действие в период между датами [after] и [before].
Вызов метода execute с параметром "get date for user = "[any_user]" and date between "[after]" and "[before]"" должен возвращать множество уникальных дат, за которые пользователь [any_user] произвел любое действие в период между датами [after] и [before].
Вызов метода execute с параметром "get date for event = "[any_event]" and date between "[after]" and "[before]"" должен возвращать множество уникальных дат, за которые произошло событие равно [any_event] в период между датами [after] и [before].
Вызов метода execute с параметром "get date for status = "[any_status]" and date between "[after]" and "[before]"" должен возвращать множество уникальных дат, за которые произошло любое событие со статусом [any_status] в период между датами [after] и [before].
Вызов метода execute с параметром "get event for ip = "[any_ip]" and date between "[after]" and "[before]"" должен возвращать множество уникальных событий, которые произошли с IP адреса [any_ip] в период между датами [after] и [before].
Вызов метода execute с параметром "get event for user = "[any_user]" and date between "[after]" and "[before]"" должен возвращать множество уникальных событий, которые произвел пользователь [any_user] в период между датами [after] и [before].
Вызов метода execute с параметром "get event for date = "[any_date]" and date between "[after]" and "[before]"" должен возвращать множество уникальных событий, которые произошли во время [any_date] в период между датами [after] и [before].
Вызов метода execute с параметром "get event for status = "[any_status]" and date between "[after]" and "[before]"" должен возвращать множество уникальных событий, которые завершены со статусом [any_status] в период между датами [after] и [before].
Вызов метода execute с параметром "get status for ip = "[any_ip]" and date between "[after]" and "[before]"" должен возвращать множество уникальных статусов, которые произошли с IP адреса [any_ip] в период между датами [after] и [before].
Вызов метода execute с параметром "get status for user = "[any_user]" and date between "[after]" and "[before]"" должен возвращать множество уникальных статусов, которые произвел пользователь [any_user] в период между датами [after] и [before].
Вызов метода execute с параметром "get status for date = "[any_date]" and date between "[after]" and "[before]"" должен возвращать множество уникальных статусов, которые произошли во время [any_date] в период между датами [after] и [before].
Вызов метода execute с параметром "get status for event = "[any_event]" and date between "[after]" and "[before]"" должен возвращать множество уникальных статусов, у которых событие равно [any_event] в период между датами [after] и [before].
Поддержка формата запросов из задания 5 должна сохраниться.
Поддержка формата запросов из задания 6 должна сохраниться.

package com.javarush.task.task39.task3913; public enum Event { LOGIN, DOWNLOAD_PLUGIN, WRITE_MESSAGE, SOLVE_TASK, DONE_TASK }

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

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

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

Артем DivertittoSenior Android-разработчи в United Tech

Master

21 апреля 2021, 08:35

Из того кода что выше, явно я нашел две ошибки, вот тут я подправил два метода

public static String dateAfter(String query) {
       Matcher m = Pattern.compile("(and date between) \"((\\d{1,2}\\.){2}\\d{1,6} (\\d{1,2}:?){3})\"").matcher(query);
       if (m.find()) {
           return m.group(2);
       }
       return null;

   }

   public static String dateBefore(String query) {
       Matcher m = Pattern.compile("(and date between) \"((\\d{1,2}\\.){2}\\d{1,6} (\\d{1,2}:?){3})\" and \"((\\d{1,2}\\.){2}\\d{1,6} (\\d{1,2}:?){3})\"").matcher(query);
       if (m.find()) {
           return m.group(5);
       }
       return null;
   }

Evgeny Kuznetsov

Уровень 39

22 апреля 2021, 04:24

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

"(and date between) \"((\\d{1,2}\\.){2}\\d{1,6} (\\d{1,2}:?){3})\""

вы поставили квантификатор от одного до шести символов. Эта часть отвечает за шаблон года в запросе: get ip for user = "Eduard Petrovich Morozko" and date between "11.12.2013 0:00:00" and "03.01.2014 23:59:59". А шестизначных годов не будет как минимум 97 979 лет =))). Со вторым выражением вообще что-то я не пойму, к чему все это? Ну можно было тогда не писать второй блок шаблона для даты, можно было просто поставить квантификатор и все. Зачем два раза писать одно и тоже? Я бы не сказал, что тут были какие-то ошибки, просто выражения работали по другому. Не могли бы вы пояснить, где конкретно была ошибка, каким-то образом влияющая на результат работы регулярного выражения и программы в целом?

Owpk

Уровень 2

18 апреля 2021, 17:38

"[any_date]" and date between "[after]" ... - строго между двумя датами, то есть не >=, а просто >, и можешь это

.filter(log -> ((getDate(log, df).getTime()) >= (after != null ? after.getTime() : 0))
                        && ((getDate(log, df).getTime()) <= (before != null ? before.getTime() : maxDate)))

заменить на это

private Predicate<Log> getFilterByDate(Date date1, Date date2) {
     return log -> ((getDate(log, df).getTime()) >= (after != null ? after.getTime() : 0))
                        && ((getDate(log, df).getTime()) <= (before != null ? before.getTime() : maxDate)));
}

@Override
    public Set<String> getIPsForEvent(Event event, Date after, Date beforet) {
//       if (event == null) {
//           throw new NullPointerException();
//        }
        long maxDate = getMaxDate();
        final Set<String> set;
        set = logs.stream()
                .filter(getFilterByDate(afger, before))
                .map(LogsUtils::getIp)
                .collect(Collectors.toSet());

        return set;
    }

Evgeny Kuznetsov

Уровень 39

18 апреля 2021, 18:30

Эти методы из прошлых заданий, они не актуальны. Их нет смысла редактировать. Они ни как не влияют на 7-ю задачу. И, к слову, все эти методы прошли в свое время валидатор, не смотря на то, что там фильтруются даты включительно Спасибо за совет, запомню, что так делать можно.