Валик не принимает последние 4 метода. Не понимаю, что ему не нравится в этот раз?

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

Решен

В комментах уже обсуждали некорректность условий и валика. Что понял - исправил вместо Solved - Done. Но с последними 4мя методами проблема осталась.

Реализуй интерфейс EventQuery у класса LogParser:
4.1. Метод getNumberOfAllEvents() должен возвращать количество событий за указанный период.
4.2. Метод getAllEvents() должен возвращать все события за указанный период.
4.3. Метод getEventsForIP() должен возвращать события, которые происходили с указанного IP.
4.4. Метод getEventsForUser() должен возвращать события, которые инициировал
определенный пользователь.
4.5. Метод getFailedEvents() должен возвращать события, которые не выполнились.
4.6. Метод getErrorEvents() должен возвращать события, которые завершились ошибкой.
4.7. Метод getNumberOfAttemptToSolveTask() должен возвращать количество попыток
решить определенную задачу.
4.8. Метод getNumberOfSuccessfulAttemptToSolveTask() должен возвращать количество
успешных решений определенной задачи.
4.9. Метод getAllSolvedTasksAndTheirNumber() должен возвращать мапу (номер_задачи :
количество_попыток_решить_ее).
4.10. Метод getAllDoneTasksAndTheirNumber() должен возвращать мапу (номер_задачи :
сколько_раз_ее_решили).

Требования:

Класс LogParser должен поддерживать интерфейс EventQuery.
Метод getNumberOfAllEvents(Date, Date) должен возвращать количество уникальных событий за выбранный период.
Метод getAllEvents(Date, Date) должен возвращать множество уникальных событий за выбранный период.
Метод getEventsForIP(String, Date, Date) должен возвращать множество уникальных событий, которые происходили с переданного IP адреса за выбранный период.
Метод getEventsForUser(String, Date, Date) должен возвращать множество уникальных событий, которые произвел переданный пользователь за выбранный период.
Метод getFailedEvents(Date, Date) должен возвращать множество уникальных событий, у которых статус выполнения FAILED за выбранный период.
Метод getErrorEvents(Date, Date) должен возвращать множество уникальных событий, у которых статус выполнения ERROR за выбранный период.
Метод getNumberOfAttemptToSolveTask(int, Date, Date) должен возвращать количество попыток решить задачу с номером task за выбранный период.
Метод getNumberOfSuccessfulAttemptToSolveTask(int, Date, Date) должен возвращать количество успешных решений задачи с номером task за выбранный период.
Метод getAllSolvedTasksAndTheirNumber(Date, Date) должен возвращать мапу (номер_задачи : количество_попыток_решить_ее) за выбранный период.
Метод getAllDoneTasksAndTheirNumber(Date, Date) должен возвращать мапу (номер_задачи : сколько_раз_ее_решили) за выбранный период.

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

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

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

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

Ksenia VolkovaJava Developer в DXC

Master

7 ноября 2021, 20:15

полезный

@Override
public int getNumberOfAttemptToSolveTask(int task, Date after, Date before) {
    HashSet<Event> events = new HashSet<>();
    readFiles();
    for (LogRecord logRecord : logRecords) {
        if (logRecord.getEvent().equals(Event.SOLVE_TASK) && logRecord.getTaskNumber() == task && filteredByDate(logRecord, after, before)) {
            events.add(logRecord.getEvent());
        }
    }
    return events.size();
}

У тебя в сет events по определению могут попасть только ивенты SOLVE_TASK. То есть events.size() может быть либо 1, либо 0.

Андрей

Уровень 41

7 ноября 2021, 20:29

Верно. Спасибо за наводку. Заменил HashSet<Event> на HashSet<LogRecord> и на ArrayList<LogRecord> иииии.... не помогло :(

Ksenia VolkovaJava Developer в DXC

Master

7 ноября 2021, 20:56

Покажи, что получилось. И кстати, это не очень хорошая идея - каждый раз дергать readFiles() и читать файлы заново.

Андрей

Уровень 41

7 ноября 2021, 20:59

@Override
public int getNumberOfAttemptToSolveTask(int task, Date after, Date before) {
    HashSet<LogRecord> logRecordsFiltered = new HashSet<>();
    readFiles();
    for (LogRecord logRecord : logRecords) {
        if (logRecord.getEvent().equals(Event.SOLVE_TASK) && logRecord.getTaskNumber() == task && filteredByDate(logRecord, after, before)) {
            logRecordsFiltered.add(logRecord);
        }
    }
    return logRecordsFiltered.size();
}

Андрей

Уровень 41

7 ноября 2021, 21:10

Насчёт каждый раз читать: изначально я читал только 1 раз в конструкторе парсера. Но уже тогда в обсуждениях возникали справедливые мысли, что если файлы изменятся между разными вызовами методов. Они же действительно могут меняться. Тогда получается, их действительно нужно читать каждый раз в момент вызова метода. Или "дочитывать", то есть читать те файлы, которые не были прочитаны раньше или изменились с прошлого чтения.

Ksenia VolkovaJava Developer в DXC

Master

7 ноября 2021, 21:10

Попробуй поменять на logRecord.getEvent() == Event.SOLVE_TASK Для енума использовать equals не имеет смысла.

Ksenia VolkovaJava Developer в DXC

Master

7 ноября 2021, 21:14

решение

Тогда уж лучше предусмотреть способ отслеживать изменения и читать, если они есть. А если у тебя сотни логов в папке лежат, а ты их на каждый чих заново все перечитываешь? В рамках этой задачи не предполагается, что логи будут меняться в процессе.

Андрей

Уровень 41

7 ноября 2021, 22:17

Увы, не помогло.

Андрей

Уровень 41

7 ноября 2021, 23:28

Странно, но перенос readFiles() из каждого метода в конструктор парсера действительно помог.