Помогите добрые люди решит эту проблему! getIPsForUser(String, Date, Date)

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

Обсуждается

все работает отлично но валик зациклился на getIPsForUser(String, Date, Date). У меня и проверка файлов ".log" вроде бы корректно реализовано, но ошибка "null по null" для пользователя getIPsForUser(String, Date, Date) все равно не хочет отстать.

Сегодня мы напишем парсер логов.

Лог файл имеет следующий формат:
ip username date event status

Где:
ip - ip адрес с которого пользователь произвел событие.
user - имя пользователя (одно или несколько слов разделенные пробелами).
date - дата события в формате day.month.year hour:minute:second.
event - одно из событий:
LOGIN - пользователь залогинился,
DOWNLOAD_PLUGIN - пользователь скачал плагин,
WRITE_MESSAGE - пользователь отправил сообщение,
SOLVE_TASK - пользователь попытался решить задачу,
DONE_TASK - пользователь решил задачу.
Для событий SOLVE_TASK и DONE_TASK существует дополнительный параметр,
который указывается через пробел, это номер задачи.
status - статус:
OK - событие выполнилось успешно,
FAILED - событие не выполнилось,
ERROR - произошла ошибка.

Пример строки из лог файла:
"146.34.15.5 Eduard Petrovich Morozko 05.01.2021 20:22:55 DONE_TASK 48 FAILED".
Записи внутри лог файла не обязательно упорядочены по дате, события могли произойти и быть записаны в лог в разной последовательности.
Все параметры разделены табуляцией ("\t").

Класс, который будет отвечать за парсинг логов называется LogParser.
1.1. Добавь в класс LogParser конструктор с параметром Path logDir, где logDir - директория с логами (логов может быть несколько, все они должны иметь расширение log).
1.2. Реализуй интерфейс IPQuery у класса LogParser:
1.2.1. Метод getNumberOfUniqueIPs(Date after, Date before) должен возвращать количество уникальных IP адресов за выбранный период. Здесь и далее, если в методе есть параметры Date after и Date before, то нужно возвратить данные касающиеся только данного периода (включая даты after и before).
Если параметр after равен null, то нужно обработать все записи, у которых дата меньше или равна before.
Если параметр before равен null, то нужно обработать все записи, у которых дата больше или равна after.
Если и after, и before равны null, то нужно обработать абсолютно все записи (без фильтрации по дате).
1.2.2. Метод getUniqueIPs() должен возвращать множество, содержащее все не повторяющиеся IP. Тип в котором будем хранить IP будет String.
1.2.3. Метод getIPsForUser() должен возвращать IP, с которых работал переданный пользователь.
1.2.4. Метод getIPsForEvent() должен возвращать IP, с которых было произведено переданное событие.
1.2.5. Метод getIPsForStatus() должен возвращать IP, события с которых закончилось переданным статусом.

Реализацию метода main() можешь менять по своему усмотрению.

Требования:

В классе LogParser должен быть создан конструктор public LogParser(Path logDir).
Класс LogParser должен поддерживать интерфейс IPQuery.
Метод getNumberOfUniqueIPs(Date, Date) должен возвращать количество уникальных IP адресов за выбранный период.
Метод getUniqueIPs(Date, Date) класса LogParser должен возвращать множество, содержащее все не повторяющиеся IP адреса за выбранный период.
Метод getIPsForUser(String, Date, Date) класса LogParser должен возвращать IP адреса, с которых работал переданный пользователь за выбранный период.
Метод getIPsForEvent(Event, Date, Date) класса LogParser должен возвращать IP адреса, с которых было произведено переданное событие за выбранный период.
Метод getIPsForStatus(Status, Date, Date) класса LogParser должен возвращать IP адреса, события с которых закончилось переданным статусом за выбранный период.

package com.javarush.task.task39.task3913; import java.nio.file.Paths; import java.util.Date; public class Solution { public static void main(String[] args) { LogParser logParser = new LogParser(Paths.get("c:/logs/")); System.out.println(logParser.getNumberOfUniqueIPs(null, new Date())); System.out.println(logParser.getIPsForUser("Amigo", null, null)); } }

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

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

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

Jan

Уровень 47

7 мая 2022, 14:19

Добрый день! Думаю что-то не так в строках 111-112. return (after == null || !after.after(logDate)) && (before == null || !before.before(logDate)); Если after будет null, то получается сравнение "null.after(Date)", а так нельзя. Можно присвоить в случае after == null {after = new Date(0);} А для before нулевого - дата с максимальным значением Long. И потом уже сравнивать.

Nikola

Уровень 41

7 мая 2022, 14:58

if(after == null) after = new Date(0);
        if(before == null) before = new Date(Long.MAX_VALUE);

        return logDate.after(after) &&
                logDate.before(before);

Не помогло, ту же ошибку выдает

Jan

Уровень 47

7 мая 2022, 15:41

Может ему 72я строчка не нравится? .filter(usr -> usr.getUser().equals(user)) Не представляю что может быть не так. Может так попробовать .filter(usr -> usr.getUser().compareTo(user)==0) Закомментированный код без стримов тоже не работает сейчас?

Jan

Уровень 47

7 мая 2022, 15:49

Ещё мне не нравится, что метод private Set<MyLogClass> filterLogList(Date after, Date before) возвращает Set, а не List. Мне кажется в таком случае теряются "одинаковые логи". Теоретически юзер мог сделать одно и то же действие в один промежуток времени с разных Ip адресов. А в случае Set эта информация будет потеряна.

Nikola

Уровень 41

7 мая 2022, 16:14

Да и так пробовал :))

Nikola

Уровень 41

7 мая 2022, 16:25

Это просто жесть, все пробовал уже что вы подсказали но этот валик уперся как осел на этой ошибке :)))

Nikola

Уровень 41

7 мая 2022, 16:27

Спасибо за потраченное время