JavaRush/Помощь по Java задачам/Два одинаковых метода, один пропускает, другой нет.

41 уровень

23.07.2021
477просмотров
17комментариев

Два одинаковых метода, один пропускает, другой нет.

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

Обсуждается

Отличие только в Event.SOLVE_TASK и Event.DONE_TASK, но первый метод пропускает а второй нет. ПАМАГИИИТЕЕЕ .... 3.5. Метод getDateWhenUserSolvedTask() должен возвращать дату, когда пользователь впервые попытался решить определенную задачу. Если такой даты в логах нет - null.

public Date getDateWhenUserSolvedTask(String user, int task, Date after, Date before) {
    List<Date> dates = new ArrayList<>();
    for (MyLog log : listFile) {
        if (isInDateRange(after, before, log)
                && log.user.equals(user)
                && log.task == ((Integer) task)
                && log.event.equals(Event.SOLVE_TASK.toString()))
            dates.add(log.date);
    }
    if(!dates.isEmpty()) {
        Collections.sort(dates);
        return dates.get(0);
    } else return null;
}

3.6. Метод getDateWhenUserDoneTask() должен возвращать дату, когда пользователь впервые решил определенную задачу. Если такой даты в логах нет - null.

public Date getDateWhenUserDoneTask(String user, int task, Date after, Date before) {
        List<Date> dates = new ArrayList<>();
        for (MyLog log : listFile) {
            if (isInDateRange(after, before, log)
                    && log.user.equals(user)
                    && log.task == ((Integer) task)
                    && log.event.equals(Event.DONE_TASK.toString()))
                dates.add(log.date);
        }
        if(!dates.isEmpty()) {
            Collections.sort(dates);
            return dates.get(0);
        } else return null;
    }

Реализуй интерфейс DateQuery у класса LogParser:
3.1. Метод getDatesForUserAndEvent() должен возвращать даты, когда определенный пользователь произвел определенное событие.
3.2. Метод getDatesWhenSomethingFailed() должен возвращать даты, когда любое событие не выполнилось (статус FAILED).
3.3. Метод getDatesWhenErrorHappened() должен возвращать даты, когда любое событие закончилось ошибкой (статус ERROR).
3.4. Метод getDateWhenUserLoggedFirstTime() должен возвращать дату, когда пользователь залогинился впервые за указанный период. Если такой даты в логах нет - null.
3.5. Метод getDateWhenUserSolvedTask() должен возвращать дату, когда пользователь впервые попытался решить определенную задачу. Если такой даты в логах нет - null.
3.6. Метод getDateWhenUserDoneTask() должен возвращать дату, когда пользователь впервые решил определенную задачу. Если такой даты в логах нет - null.
3.7. Метод getDatesWhenUserWroteMessage() должен возвращать даты, когда пользователь написал сообщение.
3.8. Метод getDatesWhenUserDownloadedPlugin() должен возвращать даты, когда пользователь скачал плагин.

Требования:

Класс LogParser должен поддерживать интерфейс DateQuery.
Метод getDatesForUserAndEvent(String, Event, Date, Date) должен возвращать множество дат, когда переданный пользователь произвел переданное событие за выбранный период.
Метод getDatesWhenSomethingFailed(Date, Date) должен возвращать множество дат, когда любое событие не выполнилось за выбранный период.
Метод getDatesWhenErrorHappened(Date, Date) должен возвращать множество дат, когда любое событие закончилось ошибкой за выбранный период.
Метод getDateWhenUserLoggedFirstTime(String, Date, Date) должен возвращать дату, когда переданный пользователь впервые залогинился за выбранный период. Если такой даты в логах нет - null.
Метод getDateWhenUserSolvedTask(String, int, Date, Date) должен возвращать дату, когда переданный пользователь впервые попытался решить задачу с номером task за выбранный период. Если такой даты в логах нет - null.
Метод getDateWhenUserDoneTask(String, int, Date, Date) должен возвращать дату, когда переданный пользователь впервые решил задачу с номером task за выбранный период. Если такой даты в логах нет - null.
Метод getDatesWhenUserWroteMessage(String, Date, Date) должен возвращать множество дат, когда переданный пользователь написал сообщение за выбранный период.
Метод getDatesWhenUserDownloadedPlugin(String, Date, Date) должен возвращать множество дат, когда переданный пользователь скачал плагин за выбранный период.

package com.javarush.task.task39.task3913; import java.nio.file.Paths; import java.text.DateFormat; import java.text.ParseException; import java.text.SimpleDateFormat; import java.util.Date; public class Solution { public static void main(String[] args) throws ParseException { LogParser logParser = new LogParser(Paths.get("C:\\JAVA\\JavaRushTasks\\4.JavaCollections\\src\\com\\javarush\\task\\task39\\task3913\\logs")); DateFormat format = new SimpleDateFormat("d.M.y H:m:s"); Date d1 = format.parse("21.10.2021 19:45:25"); System.out.println(logParser.getNumberOfUniqueIPs(null, d1)); // System.out.println(logParser.getNumberOfUniqueIPs(null, null)); // System.out.println(logParser.getUniqueIPs(null, null)); // System.out.println(logParser.getIPsForUser("Amigo" ,null, null)); // System.out.println(logParser.getIPsForEvent(Event.LOGIN,null, null)); // System.out.println(logParser.getIPsForStatus(Status.OK,null, null)); System.out.println(logParser.getDateWhenUserDoneTask("Vasya Pupkin",15, null,d1)); } }

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

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

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

Ksenia Volkova Java Developer в DXC

Master

23 июля 2021, 07:27

Из того, что сразу бросается в глаза - почему ты Integer сравниваешь через == ?

-2

Дмитрий

Уровень 41

23 июля 2021, 07:42

после приведения не исправил, заменил на equals, ошибка осталась

Ksenia Volkova Java Developer в DXC

Master

23 июля 2021, 08:23

А если log.task = null ?

-2

Дмитрий

Уровень 41

23 июля 2021, 08:37

по условию: Для событий SOLVE_TASK и DONE_TASK существует дополнительный параметр, который указывается через пробел, это номер задачи.

Дмитрий

Уровень 41

23 июля 2021, 09:00

конкретно для этого метода добавил

&& log.task != null

Ksenia Volkova Java Developer в DXC

Master

23 июля 2021, 09:01

&& log.task == ((Integer) task)
&& log.event.equals(Event.DONE_TASK.toString()))

У тебя порядок проверки таков, что сначала проверяется log.task, и только потом event. А значит log.task будет проверяться для всех ивентов. Покажи актуальный код метода целиком.

-2

Дмитрий

Уровень 41

23 июля 2021, 09:05

исправил, ошибка осталась (и почему в предыдущем методе валидатор это пропустил?)

public Date getDateWhenUserDoneTask(String user, int task, Date after, Date before) {
       List<Date> dates = new ArrayList<>();
       for (MyLog log : listFile) {
           if (isInDateRange(after, before, log)
                   && log.user.equals(user)
                   && log.event.equals(Event.DONE_TASK.toString())
                   && log.task != null
                   && log.task.intValue() == task
                  )
               dates.add(log.date);
       }
       if(!dates.isEmpty()) {
           Collections.sort(dates);
           return dates.get(0);
       } else return null;

Ksenia Volkova Java Developer в DXC

Master

23 июля 2021, 09:47

Пропустил - потому что тесты валидатора несовершенны. Если для конкретного юзера в указанный период не будет других ивентов, то ошибка не вылезет и тест пройдет. Статус, думаю, тоже имеет значение.

-2

Дмитрий

Уровень 41

23 июля 2021, 10:28

Ксения, спасибо за терпение и понимание, сам был на саппорте, знаю как чайники иногда задалбывают. Но после внесения всех изменений, проблема осталась.

for (MyLog log : listFile) {
           if (log.user.equals(user)
                   && isInDateRange(after, before, log)
                   && log.event.equals(Event.DONE_TASK.toString())
                   && log.task != null
                   && log.task == task
                   && log.status.equals(Status.OK.toString())
                  )
               dates.add(log.date);
       }

уже изменил даже последовательность проверки. Можете дать подсказку типа: диапазон дат не верен или проверка пользователя выполнена некоректно?

Ksenia Volkova Java Developer в DXC

Master

23 июля 2021, 12:40

Я бы попробовала все же включить крайние даты в диапазон.

-2

Дмитрий

Уровень 41

23 июля 2021, 14:12

Я включил, изменил это

if (after == null || log.date.after(after)) {
            if ((before == null || log.date.before(before))) {
                dateVerification = true;
            }
        }

на это:

if (after == null || log.date.getTime() >= after.getTime()) {
           if ((before == null || log.date.getTime() <= before.getTime())) {
               dateVerification = true;
           }
       }

не работает

Павел Безумный учёный

Expert

23 июля 2021, 15:39

Выполните проверки в таком порядке:

if (log.user.equals(user)
        && isInDateRange(after, before, log)
        && log.event.equals(Event.DONE_TASK.toString())
        && log.task == task
)

В проверке на null нет необходимости, поскольку, как уже сказала Ксения, событие DONE_TASK гарантирует наличие конкретного значения в поле log.task. Проверка статуса OK не требуется по условию.

Ksenia Volkova Java Developer в DXC

Master

26 июля 2021, 10:00

))) У меня нет никакого доступа к тестам валидатора - если что. Если честно, я бы вообще отрефакторила весь этот код. Почему, например, у тебя в MyLog ивенты и статусы - строки? гораздо удобнее использовать соответсвующие типы. Для всех методов get...() удобно было бы использовать стримы (почему-то в logToList() ты их используешь, а в других местах - нет).

Дмитрий

Уровень 41

26 июля 2021, 10:58

Когда начинаешь решать задачу, ты не видишь всЁ условие целиком. - в MyLog ивенты и статусы - потому что в первых задачах они практически не использовались, и такая реализация была логична. - в процессе решения и реализации новых условий, я делаю рефакторинг, это можно заметить сравнив каждую последующую реализацию. - так как здесь объяснили стримы, так я их и использую, на сколько понял и знаю, списывать не хочу, но валидатор не оставляет выбора - стримы, это спорный вопрос. Реализация через стрим уменьшает количество кода, но увеличивает нагрузку на систему. В условиях задачи не указано использовать стримы.

Ksenia Volkova Java Developer в DXC

Master

26 июля 2021, 11:05

Енумы Event и Status вроде бы были даны изначально - и можно было понять, что это неспроста. Использование енумов вместо множества отдельных идентичных строк куда эффективнее. Когда ты делаешь сравнение типа log.event.equals(event.toString()) , ты для каждого такого сравнения создаешь лишний объект String - и тут тебя почему-то увеличение нагрузки на систему не беспокоит. А сокращение количества кода может облегчить поиск ошибок.

Дмитрий

Уровень 41

26 июля 2021, 11:32

И на том спасибо. Иду дальше, проблема не решена. Замена местами аргументов в операторе if - это не проблема в коде, это проблема в валидаторе.

Ksenia Volkova Java Developer в DXC

Master

26 июля 2021, 11:45

Причем тут замена местами аргументов? Не в месте же дело. И валидатору это как раз все равно. Просто это делает твой код менее эффективным. Валидатор - это тест, он такое пропустит. А вот живой код-ревьюер - вряд ли. Несовершенство валидатора как раз в том, что он нередко пропускает то, что не должен бы. В коротких задачах это просто менее заметно, а в "многоходовых" пропущенное на ранних шагах может вызывать проблемы в дальнейшем. Поэтому там труднее искать ошибку - она может быть совсем не в том месте, на которое ругается валидатор.