Не проходит один пункт.

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

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

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

Обсуждается

Status проходит, а Event не проходит. Что за дичь если код различается только по полю?

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

Лог файл имеет следующий формат:
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:\\projects\\JavaRushTasks\\4.JavaCollections\\src\\com\\javarush\\task\\task39\\task3913\\logs")); System.out.println(logParser.getNumberOfUniqueIPs(null, null)); System.out.println(logParser.getUniqueIPs(null, new Date())); System.out.println(logParser.getIPsForUser("Eduard Petrovich Morozko",null, null)); System.out.println(logParser.getIPsForEvent(Event.WRITE_MESSAGE,null, null)); System.out.println(logParser.getIPsForStatus(Status.FAILED,null, null)); } }

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

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

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

Сергей Java Developer в Адвантум

16 сентября 2021, 13:03

Проверь конструктор:

public Log(String ip, String user, Date date, String event, Status status) {
            this.ip = ip;
            this.user = user;
            this.date = date;
            this.event = setEvent(event);
            this.num = setNum(event);
            this.status = status;
        }

Перепиши этот метод:

public List<Log> getLogs(File file) {
        List<Log> listLogs = new ArrayList<>();
        try (BufferedReader reader = new BufferedReader(new FileReader(file))) {
            while (reader.ready()) {
                String[] log = reader.readLine().split("\t");
                SimpleDateFormat format = new SimpleDateFormat("dd.M.yyyy hh:mm:ss", Locale.ENGLISH);
                Date date = format.parse(log[2]);
                listLogs.add(new Log(log[0], log[1], format.parse(log[2]), log[3], Status.valueOf(log[4])));
            }
        } catch (IOException | ParseException e) {
            e.printStackTrace();
        }

        return listLogs;
    }

Поле additionalEventParameter должно записываться отдельно. А у тебя Even и additionalEventParameter записываются в поле Event. Как это делал я:

result.setEvent(stringToEvent(incomingData[3]));
            if (result.getEvent() == Event.SOLVE_TASK || result.getEvent() == Event.DONE_TASK)
                result.setAdditionalEventParameter(stringToAdditionalEventParameter(incomingData[3]));

И вообще сядь и разберись со Stream API. Дальше эту задачу без Stream API будет трудно решить. А еще лучше изучи паттерн Команда и попробуй его применить. Удачи😉

Сергей Java Developer в Адвантум

16 сентября 2021, 13:09

Как у меня организован парсинг для Event и additionalEventParameter

private Event stringToEvent(String event) {
        if (event.contains("SOLVE_TASK"))
            return Event.SOLVE_TASK;
        if (event.contains("DONE_TASK"))
            return Event.DONE_TASK;

        switch (event) {
            case "LOGIN":
                return Event.LOGIN;
            case "DOWNLOAD_PLUGIN":
                return Event.DOWNLOAD_PLUGIN;
            case "WRITE_MESSAGE":
                return Event.WRITE_MESSAGE;
        }

        return null;
    }

    private int stringToAdditionalEventParameter(String event) {
        int additionalParameter;
        String[] eventAndParameter = event.split(" ");
        additionalParameter = Integer.parseInt(eventAndParameter[1]);
        return additionalParameter;
    }

Кисмер

Уровень 41

16 сентября 2021, 07:25

я все сравнения enum'ов делал через equals, у меня всё было ок

Евгений

Уровень 41

16 сентября 2021, 07:34

И так и так пробовал. Безполезно. Меня удивляет, что Status (тоже enum) проходит, а Event нет.