Подскажите пожалуйста где ошибка?

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

Обсуждается

Не принимает по 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; import java.nio.file.Paths; import java.util.Date; public class Solution { public static void main(String[] args) { Date date = new Date(); LogParser logParser = new LogParser(Paths.get("C:/JavaRushTasks/4.JavaCollections/src/com/javarush/task/task39/task3913/logs/example.log")); System.out.println(logParser.getNumberOfAttemptToSolveTask(18,null,null)); //C:\JavaRushTasks\4.JavaCollections\src\com\javarush\task\task39\task3913\logs\example.log } }

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

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

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

Pavlo Plynko Java-разработчик в CodeGym

Expert

18 июня 2019, 09:47

logDir - директория с логами (логов может быть несколько,
все они должны иметь расширение log).

не вяжется с:

BufferedReader bufferedReader = new BufferedReader(new FileReader(new File(String.valueOf(path))));

Леонид Шевченко

Уровень 41

14 июня 2019, 13:15

Попробуйте везде заменить log.getEvent() == Event.SOLVE_TASK на equals()

Max Hayday

Уровень 36

16 июня 2019, 09:33

Пробовал, не пропускает (

Леонид Шевченко

Уровень 41

18 июня 2019, 10:23

немного отличная логика:

@Override
    public int getNumberOfAttemptToSolveTask(int task, Date after, Date before) {
        int count = 0;
        for (MyLog log : getLogsByDate(after, before))
            if (log.taskNumber == task && log.event.equals(Event.SOLVE_TASK))
                count ++;
        return count;
    }

    @Override
    public int getNumberOfSuccessfulAttemptToSolveTask(int task, Date after, Date before) {
        int count = 0;
        for (MyLog log : getLogsByDate(after, before))
            if (log.taskNumber == task && log.event.equals(Event.DONE_TASK))
                count ++;
        return count;
    }

    @Override
    public Map<Integer, Integer> getAllSolvedTasksAndTheirNumber(Date after, Date before) {
        Map<Integer, Integer> map = new HashMap<>();
        for (MyLog log : getLogsByDate(after, before)){
            if (log.taskNumber != 0 && log.event.equals(Event.SOLVE_TASK)) {
                checkMap(map, log);
            }
        }
        return map;
    }

    @Override
    public Map<Integer, Integer> getAllDoneTasksAndTheirNumber(Date after, Date before) {
        Map<Integer, Integer> map = new HashMap<>();
        for (MyLog log : getLogsByDate(after, before)) {
            if (log.taskNumber != 0) {
                if (log.event.equals(Event.DONE_TASK)) {
                    checkMap(map, log);
                }
            }
        }
        return map;
    }

Max Hayday

Уровень 36

23 июня 2019, 19:30

как это понимать?

Леонид Шевченко

Уровень 41

25 июня 2019, 11:22

К примеру, в методе getNumberOfSuccessfulAttemptToSolveTaskвы проверяете на Status.OK. Зачем? в условии этого не требуют Почему там же проверка Event.SOLVE_TASK?