Подскажите, что не так

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

Парсер логов (1)

Java Collections, 7 уровень, 15 лекция

Решен

Вроде, всё отрабатывает, как указано в задании.

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

Лог файл имеет следующий формат:
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; public enum Status { OK, FAILED, ERROR }

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

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

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

Денис Першин

Уровень 41

28 ноября 2022, 10:54

В регулярном выражении исправил: вместо пробела между параметрами поставил - или сколько-нибудь пробелов, или табуляция

(\\s+|\\t)

. В проверке файлов добавил уменьшение регистра букв (вдруг расширение файла будет .LOG) -

file.toString().toLowerCase().ensWith(".log")

Но все-равно не прошло. ХЗ... В итоге, пришлось зачитить.

Денис Першин

Уровень 41

27 ноября 2022, 15:51

public static void main(String[] args) {
    LogParser logParser = new LogParser(Paths.get("C:\\Users\\user\\IdeaProjects\\JavaRush\\src\\com\\javarush\\task\\task39\\task3913\\logs"));
    System.out.println(logParser.getNumberOfUniqueIPs(null, null));
    System.out.println("------------------------");
    logParser.getUniqueIPs(null, null).forEach(System.out::println);
    System.out.println("------------------------");
    logParser.getIPsForEvent(Event.SOLVE_TASK, null, null).forEach(System.out::println);
    System.out.println("------------------------");
    logParser.getIPsForUser("Amigo", null, null).forEach(System.out::println);
    System.out.println("------------------------");
    logParser.getIPsForStatus(Status.ERROR, null, null).forEach(System.out::println);
}

Такой код использовал для проверки. В папке 2 файла с расширением .log и один с расширением .txt. Мой вариант решения log читает, а txt не читает. Выдает всё, как я понимаю, согласно заданию. То же и при подстановке вместо null даты. 7 ------------------------ 120.120.120.122 146.34.15.5 192.168.100.2 12.12.12.12 127.1.1.1 127.0.0.1 146.34.15.10 ------------------------ 120.120.120.122 192.168.100.2 12.12.12.12 ------------------------ 120.120.120.122 12.12.12.12 127.0.0.1 ------------------------ 192.168.100.2 Process finished with exit code 0 Подсмотрел "корректное решение". Оно при моей проверке ничего не выдает. Видимо, я где-то не догоняю.