JavaRush/Помощь по Java задачам/Что не так?

41 уровень

19.12.2017
989просмотров
16комментариев

Что не так?

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

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

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

Решен

сабж

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

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

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

АлександрFull Stack Developer в VTB

19 декабря 2017, 14:55

Прошел задачу. Потихоньку развиваю телепатию. в getNumberOfAttemptToSolveTask и getNumberOfSuccessfulAttemptToSolveTask меняем порядок условий на

record.event == Event.SOLVE_TASK && record.taskNumber == task

А в getAllSolvedTasksAndTheirNumber и getAllDoneTasksAndTheirNumber просто забыл проверку на null для дат. Спасибо мне за очередное решение!

Roman Java Developer

Expert

19 декабря 2017, 15:53

Выложи код этих двух методов, пожалуйста.

АлександрFull Stack Developer в VTB

19 декабря 2017, 15:58

@Override
    public int getNumberOfAttemptToSolveTask(int task, Date after, Date before)
    {
        ...
            if (isBetween(record.date,after,before) &&  record.event == Event.SOLVE_TASK && record.taskNumber == task ) count++;
        ...
    }
//Метод getNumberOfSuccessfulAttemptToSolveTask() должен возвращать количество
//успешных решений определенной задачи.

    @Override
    public int getNumberOfSuccessfulAttemptToSolveTask(int task, Date after, Date before)
    {
        ...
            if (isBetween(record.date,after,before) && record.event == Event.DONE_TASK  && record.taskNumber == task ) count++;
        ...
    }

Roman Java Developer

Expert

19 декабря 2017, 14:34

Запустил твой метод getNumberOfAttemptToSolveTask(int, Date, Date) на тестовом файле:

public static void main(String[] args) {
        LogParser logParser = new LogParser(Paths.get("...\\JavaRushTasks\\4.JavaCollections\\src\\com\\javarush\\task\\task39\\task3913\\logs")); // тут реальный путь к папке с тестовым файлом example.log
        String dateAfter = "01.01.2020";
        String dateBefore = "01.01.2029";
        DateFormat format = new SimpleDateFormat("dd.mm.yyyy");
        Date after = new Date();
        Date before = new Date();
        try {
            after = format.parse(dateAfter);
            before = format.parse(dateBefore);
        } catch (Exception e) {
            e.printStackTrace();
        }
        System.out.println(logParser.getNumberOfAttemptToSolveTask(18, after, before));
}

и получил:

Exception in thread "main" java.lang.NullPointerException

АлександрFull Stack Developer в VTB

19 декабря 2017, 14:46

Это уже прошло валидацию.

Не проходят getAllSolvedTasksAndTheirNumber(Date, Date)

getAllDoneTasksAndTheirNumber(Date, Date)

АлександрFull Stack Developer в VTB

19 декабря 2017, 14:53

значит там нет файла или данные не по формату.

Roman Java Developer

Expert

19 декабря 2017, 14:57

Тестирую на коде (все остальное как в предыдущий раз):

System.out.println(logParser.getAllSolvedTasksAndTheirNumber(after, before)); // after - "01.01.2020"; before - "01.01.2029";
System.out.println(logParser.getAllSolvedTasksAndTheirNumber(null, null));

У тебя вывод одинаковый - неправильно для случая с заданными датами:

{1=1, 18=3}
{1=1, 18=3}

АлександрFull Stack Developer в VTB

19 декабря 2017, 14:58

это все не важно, валидатор принял)

АлександрFull Stack Developer в VTB

19 декабря 2017, 14:59

Спасибо за помощь!

Roman Java Developer

Expert

19 декабря 2017, 15:02

Не за что. Валидатором был принят неправильный вариант?

АлександрFull Stack Developer в VTB

19 декабря 2017, 15:04

Нет, я уже нашел ошибку. В getAllSolvedTasksAndTheirNumber и getAllDoneTasksAndTheirNumber просто забыл проверку на null для дат.

АлександрFull Stack Developer в VTB

19 декабря 2017, 14:30

Но пока еще 2 последние функции не пропускает, что делать?

АлександрFull Stack Developer в VTB

19 декабря 2017, 14:30

Спасибо мне за решение. Оказалось, что это глючное УГ, под названием валидатор ругался на порядок условий. Поменял на

if (isBetween(record.date,after,before) &&  record.event == Event.SOLVE_TASK && record.taskNumber == task ) count++;

и пропустило.

Roman Java Developer

Expert

19 декабря 2017, 16:32

Валидатор тут не при чем. Порядок условий играет важную роль! При использовании &&, если одно из первых условий не выполняется, то до следующих даже не дойдет. В этом твоя ошибка. Если ты сначала проверяешь номер задачи, то у некоторых record он может быть null и ты получишь исключение (если протестируешь свой код), а если сначала проверить event, то у тех record, у которых record.event == Event.SOLVE_TASK всегда есть номер задачи. Либо, если хочешь именно такой порядок, то перед проверкой record.taskNumber == task проверяй record.taskNumber != null.

АлександрFull Stack Developer в VTB

20 декабря 2017, 07:05

Это уже косяк архитектуры. Не должен я думать, получу NPE или нет в if

-2

АлександрFull Stack Developer в VTB

19 декабря 2017, 13:56

Утомляет угадайка. Ментор пишет Метод getNumberOfAttemptToSolveTask(int, Date, Date) класса LogParser должен правильно возвращать количество попыток решить задачу с номером task за период с null по null. С нулами работает корректно. Что не так?