JavaRush/Помощь по Java задачам/Не принимает пункты с фильтром по дате "get ... for date ...

40 уровень

03.03.2018
1119просмотров
20комментариев

Не принимает пункты с фильтром по дате "get ... for date = "[any_date]"" - номера 2, 6, 15, 19

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

Решен

Прошел курс, осталась только эта долбанная задача из всех. Сил уже нет на этот валидатор... Пробовал по-всякому, написал дуракоупорную конструкцию для проверки синтаксиса query (метод checkQuery) - ни в какую. Даты сравнивал и как String, и как Date (через compareTo) - тоже не нравится.

Давай добавим поддержку параметра запроса в наш QL.

Примеры запросов с параметром:
1) get ip for user = "Vasya"
2) get user for event = "DONE_TASK"
3) get event for date = "03.01.2014 03:45:23"

Общий формат запроса с параметром:
get field1 for field2 = "value1"
Где: field1 - одно из полей: ip, user, date, event или status;
field2 - одно из полей: ip, user, date, event или status;
value1 - значение поля field2.

Алгоритм обработки запроса следующий: просматриваем записи в логе, если поле field2 имеет значение value1, то добавляем поле field1 в множество, которое затем будет возвращено методом execute.

Пример:
Вызов метода execute("get event for date = "30.01.2014 12:56:22″") должен вернуть Set<Event>, содержащий только одно событие SOLVE_TASK. Какая именно задача решалась возвращать не нужно.

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

Требования:

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

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

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

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

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

Парьев Дмитрий

Уровень 40

4 марта 2018, 18:10

Во первых у вас как-то очень сложно парсится дата, может там ошибка? вот в одну строку: Date date = new SimpleDateFormat("d.M.y H:m:s").parse(values[2]); И в record.java замените SimpleDateFormat("d.M.y H:m:s") Да и везде в общем-то тоже. А главная ошибка здесь - строка 386:

if (methodResult.equals(queryValue)) {

У вас входные данные могут быть такие "get event for date = "03.01.2014 03:45:23"" А в логе такие ".... 3.1.2014 3:45:23 ..." Видите разницу? А вы сравниваете не как даты, а как строки! Вот вам метод для сравнения дат:

boolean isDate(Date after, Date before, Date date) {
    return (after == null || date.after(after))
              && (before == null || date.before(before));
}

Вадим

Уровень 40

4 марта 2018, 19:30

Спасибо за подсказку с SimpleDateFormat. Я пробовал сравнивать и как строки, и как дату (второе конечно же логичнее, но в комментариях писали про строковое сравнение). Сейчас попробовал еще раз - не принимает. Единственное, что я не очень понял что это за метод такой Вы написали. Точнее как он мне сейчас поможет... Это, я так понимаю, для формирования выборки по определенному периоду? Но мне сейчас это не надо, наверно это 7я задача. На данном этапе, для запроса блаблабла for date = "..." как по мне, достаточно сравнить timestamp'ы двух дат: date1.getTime() == date2.getTime() P.S.: а что, заапложенный к вопросу код нельзя обновлять? Мой уже столько изменений претерпел...)

Парьев Дмитрий

Уровень 40

4 марта 2018, 22:13

Этот метод я с самого начала использовал, просто в данном случае after = before. Попробуйте такой тест:

System.out.println(logParser.execute("get event for date = \"3.1.2014 3:45:23\""));
System.out.println(logParser.execute("get event for date = \"03.01.2014 3:45:23\""));
System.out.println(logParser.execute("get event for date = \"\""));
System.out.println(logParser.execute("get date for date = \"29.2.2028 05:4:7\""));
System.out.println(logParser.execute("get user for date = \"29.02.2028 05:4:07\""));

Что выведет ваша программа? Должно:

[LOGIN]
[LOGIN]
[]
[Tue Feb 29 05:04:07 EET 2028]
[Amigo]

Вадим

Уровень 40

5 марта 2018, 05:59

Подправил обработку неверного query - поменял возврат с null на пустой HashSet. Теперь вывод один-в-один. Валидатор снова не принял. Уже не знаю - может ли это быть как-то связано с типом ОС или ее локализацией? Вроде ж пофиг должно быть... Уже 26 попыток) я скоро стану как тот чувачок с ноутом и кофе на картинке валидатора) ща ток кофе заварю...)

Парьев Дмитрий

Уровень 40

5 марта 2018, 09:30

Если query битый, то null, а если правильный но с нулевым value, то что будет, то будет Вот еще пара тестов:

System.out.println(logParser.execute("get user for =date = \"29.02.2028 05:4:07\""));
System.out.println(logParser.execute("get =user for date = \"29.02.2028 05:4:07\""));
System.out.println(logParser.execute("get date for date = \"25.2.2028 05:4:7\""));

[Amigo, Vasya Pupkin, Eduard Petrovich Morozko]
null
[]

Я использовал такой обработчик query:

Set<Object> set = new HashSet<>();
       if (query == null || query.isEmpty()) return set;

       Pattern pattern = Pattern.compile("get (ip|user|date|event|status)"
               + "( for (ip|user|date|event|status) = \"(.*?)\")?");
       Matcher matcher = pattern.matcher(query);
       String field1 = null;
       String field2 = null;
       String value = null;
       String after = null;
       String before = null;
       if (matcher.find()) {
           field1 = matcher.group(1);
           field2 = matcher.group(3);
           value = matcher.group(4);
       }
       if (field1 == null) return null;
       if (field2 == null) field2 = field1;

PS: хорошо бы увидеть актуальный ваш код.

Вадим

Уровень 40

5 марта 2018, 09:34

А разве query

get user for =date = "29.02.2028 05:4:07"

не считается битым?

Парьев Дмитрий

Уровень 40

5 марта 2018, 09:38

Вроде битый, но у меня воспринимается как отсутствие field2 и value

Вадим

Уровень 40

5 марта 2018, 09:43

Да в конце-концов, я так понимаю, что нет разницы как обрабатывать невалидный запрос. Если верить ответу в соседнем вопросе, тестирование проходит только на валидных.

Парьев Дмитрий

Уровень 40

5 марта 2018, 09:59

Если только валидные, то проблема остается в сравнении дат. Ну раз так, тогда, могу лишь рассказать как у меня идет проверка даты: в записи лога, храню дату как Date а сравниваю их так value == null || logRecord.date.equals(new SimpleDateFormat("d.M.y H:m:s").parse(value)) А еще попробуйте закоментировать все "e.printStackTrace();" и все лишние выводы в консоль.

Парьев Дмитрий

Уровень 40

5 марта 2018, 10:27

решение

А вот еще, мы же в будущем) System.out.println(logParser.execute("get user for date = \"29.02.20028 05:4:07\"")); И в файле example.log добавьте запись "120.120.120.122 Amigo 29.2.20028 5:4:7 SOLVE_TASK 18 OK" Все валидно, но у вас не пройдет.

Вадим

Уровень 40

5 марта 2018, 10:41

О ДАААААА!!!! 29 попыток, Карл!

Вы решили задачу лучше, чем 1% учеников.
Вам удалось ее решить с 29 попытки.

Хаха всего-то нужно было добавить запятую в регексе :D после этого все даты с 4-х и более значными годами стали приниматься. До этого в паттерне даты года брались только 4-значные:

...\\d{4}...

Спасибо огромнейшее за помощь!

Парьев Дмитрий

Уровень 40

5 марта 2018, 14:08

Мои поздравления!)

Анастасия

Уровень 40

3 марта 2018, 20:28

посмотрела парсинг лога, вы разделяете по табу? String[] values = log.split("\t"); Посмотрела свой парсинг лога, я разделяю по пробелу: private Pattern pattern = Pattern.compile("(? < ip > [\\d]+.[\\d]+.[\\d]+.[\\d]+)\\s(? < user > [a-zA-Z ]+)\\s(? < date > [\\d]+.[\\d]+.[\\d]+ [\\d]+:[\\d]+:[\\d]+)\\s(? < event > [\\w]+)\\s?((?<taskNumber>[\\d]+)|)\\s(? < status > [\\w]+)"); может у автотеста валидатора какая-то фигня с пробелами-табами в датах...

Вадим

Уровень 40

3 марта 2018, 22:06

Странно, а как это оно так у Вас работает? В логе ж поля разделены табами, а пробелы только между ивентом и таском, и в дате. Как бы там ни было, я попробовал со сплитом по пробелу и выгреб эксепшн) в отчаянии даже отправил это валидатору - не принял)

Вадим

Уровень 40

4 марта 2018, 07:47

Утром на свежую голову еще раз перечитал и понял что вчера сделал неправильно) Сейчас попробовал Ваш способ сплита - у меня работает, валидатор не принимает те же пункты

Анастасия

Уровень 40

3 марта 2018, 09:04

судя по отмеченным пунктам проблема как раз в том, что не работает сравнение дат, все пункты посыпались, где for date = "[any_date]"" могу точно сказать, что даты надо в файле искать по полному совпадению, точно так же, как и имя пользователя, и айпи, и все что угодно. То есть поиск полностью работает по String.equals вне зависимости от того, какой queryField А вот результат важно отдавать именно в правильном типе, например, сет дат должен быть именно сетом дат, но я вижу у вас с этим порядок..

Вадим

Уровень 40

3 марта 2018, 09:23

Строки и сравниваю... Нашел одну ошибку: не работало при запросе даты в формате 4.5.2012 6:8:9. Исправил, всё равно валидатор не принял.

Анастасия

Уровень 40

3 марта 2018, 09:45

не должно быть разницы никакой в том, какой формат даты Если в логе написана дата в виде 4.5.2012 6:8:9 то, запрашивая дату в квери в виде "4.5.2012 6:8:" у вас должна быть найдена запись Сранение строго строковое

Анастасия

Уровень 40

3 марта 2018, 10:20

get event for date = "03.01.2014 03:45:23" --- LOGIN эта штука при этом работает?

Вадим

Уровень 40

3 марта 2018, 13:25

Да, работает, на такой запрос выводит [LOGIN] Уже даже добавил строковое поле в класс Record, в которое пихаю дату как она записана в логе. Т.е. избегаю двойной конвертации даты при сравнении в execute: String > Date > String. Так тоже не принимает... ВАЛИДАТОР, ГДЕ ЖЕ ТЕБЕ ПОЧЕСАТЬ ЧТОБЫ ТЫ УСПОКОИЛСЯ???