1 уровень

03.03.2021
514просмотров
12комментариев

Перерыв

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

Обсуждается

1) Код ... как лучше? 2) Есть подозрение, что не оптимально для больших массивов данных. 3) Метод getNumberOfSuccessfulAttemptToSolveTask(int, Date, Date) должен возвращать количество успешных решений задачи с номером task за выбранный период. Нужны советы что еще попробовать: - Status.OK - снимал. в условии неоднозначно, поэтому попробовал - количества результатов от Event.SOLVE_TASK и Event.DONE_TASK складывал. - буду дописывать... P.S. задача - супер!

Реализуй интерфейс 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 }

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

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

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

Alexander Veselov

Уровень 1

4 марта 2021, 09:16

Осталось побороть вот эту проблему "get ХХХ for date = "[any_date]" and date between "[after]" and "[before]"" должен возвращать множество уникальных ХХХ, которые произвели любое действие в указанное время [any_date] в период между датами [after] и [before]. <<< - вот это не понятно. Варианты: * из any_date брать время без даты, искать между after и before (даты, без времени) - делаю, сейчас проверю. * объединить поиск: если равно any_date или между after и before - сделал так, не принято * все вышеперечисленное больное воображение, ошибка есть но не там а валидатор просто запутывает %)

Alexander Veselov

Уровень 1

3 марта 2021, 18:15

3-й пункт подобрал:

@Override
public int getNumberOfSuccessfulAttemptToSolveTask(int task, Date after, Date before) {
    return getTasks(after, before, null, Event.DONE_TASK, null, null, null).getOrDefault(task, 0);
}

Вопрос к валидатору-условию. Либо один кривит либо второе нечеткое.

Ksenia VolkovaJava Developer в DXC

Master

3 марта 2021, 18:22

А зачем тебе считать попытки для всех тасок, если нужна только одна?

Wladyslaw Java Developer

Master

3 марта 2021, 17:23

Стримы отлично ложатся на эту задачку, например. Это к вопросу о том как лучше.

Alexander Veselov

Уровень 1

3 марта 2021, 17:52

все на них основанно

private Set<String> getIPs(Date after, Date before, String user, Event event, Status status) {
        return logEntries.stream().filter(d -> d.isSuitable(after, before, user, event, status, null, null)).map(d -> d.ip).collect(Collectors.toSet());
    }

private Set<String> getUsers(Date after, Date before, String user, Event event, Status status, String ip, Integer taskId) {
        return logEntries.stream().filter(d -> d.isSuitable(after, before, user, event, status, ip, taskId)).map(d -> d.user).collect(Collectors.toSet());
    }

private Set<Event> getEvents(Date after, Date before, String user, Event event, Status status, String ip, Integer taskId) {
        return logEntries.stream().filter(d -> d.isSuitable(after, before, user, event, status, ip, taskId)).map(d -> d.event).collect(Collectors.toSet());
    }

private Set<Date> getDates(Date after, Date before, String user, Event event, Status status, String ip, Integer taskId) {
        return logEntries.stream().filter(d -> d.isSuitable(after, before, user, event, status, ip, taskId)).map(d -> d.when).collect(Collectors.toSet());
    }

Ksenia VolkovaJava Developer в DXC

Master

3 марта 2021, 18:07

getTasks тоже можно переделать на стрим

Alexander Veselov

Уровень 1

3 марта 2021, 18:16

Спасибо! Это я специально развернул для легкого дебага, было вот так:

private Map<Integer, Integer> getTasks(Date after, Date before, String user, Event event, Status status, String ip, Integer taskId) {
    HashMap<Integer, Integer> tasks = new HashMap<>();

    logEntries.stream()
            .filter(d -> d.isSuitable(after, before, user, event, status, ip, taskId))
            .filter(d -> d.taskId != null)
            .forEach(d -> tasks.put(d.taskId, tasks.getOrDefault(d.taskId, 0) + 1));
    return tasks;
}

Ksenia VolkovaJava Developer в DXC

Master

3 марта 2021, 18:19

полезный

Можно сразу из стрима мапу вернуть.

Wladyslaw Java Developer

Master

3 марта 2021, 18:55

Пиши их с новой строки каждый вызов лучше, так принято. Я вот видишь, с телефона и не заметил что у тебя стримы там.

Alexander Veselov

Уровень 1

3 марта 2021, 21:04

там сложение, показалось что нельзя. надо погуглить

Ksenia VolkovaJava Developer в DXC

Master

3 марта 2021, 21:52

Можно-можно, используй merge функцию в коллекторе )

Alexander Veselov

Уровень 1

4 марта 2021, 09:08

Ксения, спасибо за наводку! Пришлось теорию перечитать заново:

private Map<Integer, Integer> getTasks(Date after, Date before, String user, Event event, Status status,
                                       String ip, Integer taskId) {
    return logEntries.stream()
            .filter(d -> (d.isSuitable(after,
                                       before,
                                       user,
                                       event,
                                       status,
                                       ip,
                                       taskId) && d.taskId != null))
            .collect(Collectors.groupingBy(LogEntry::getTaskId, Collectors.summingInt(logEntry -> 1)));
}