Не понимаю почему код не проходит валидацию

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

Решен

Буду благодарен если кто то скажет почему мой код не проходит валидацию.

public class Solution {

    public static void main(String[] args) {
        System.out.println(getNextMonth(Month.JANUARY));
        System.out.println(getNextMonth(Month.JULY));
    }

    public static Month getNextMonth(Month month) {
        //напишите тут ваш код
        if(month.ordinal() < 11)
            month = Month.values()[month.ordinal() + 1];
        if(month.ordinal() == 11)
            month = Month.values()[0];
        return month;
    }

}

В классе Solution реализуй метод getNextMonth, который должен возвращать следующий месяц, а если передан последний месяц, возвращать первый. Используй методы values и ordinal.
Метод main не участвует в проверке.

Требования:

Реализуй метод public static Month getNextMonth(Month): он должен работать согласно условию.
В методе getNextMonth должны использоваться методы ordinal() и values().

package com.javarush.task.pro.task13.task1318; public enum Month { JANUARY, FEBRUARY, MARCH, APRIL, MAY, JUNE, JULY, AUGUST, SEPTEMBER, OCTOBER, NOVEMBER, DECEMBER }

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

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

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

Денис Enterprise Java Developer

6 июля 2023, 21:08

решение

Присвоение значения аргументу метода, а так же последовательные (не исключающие) if играют с тобой злую шутку :) Ну и лишнего хватает, зачем например нужны присвоения если можно сделать сразу возврат? Вот твой же метод, но малость причёсанный:

public static Month getNextMonth(Month month) {
    if (month.ordinal() < 11) {
        return Month.values()[month.ordinal() + 1];
    } else {
        return Month.values()[0];
    }
}

Ну и альтернативный метод решения:

public static Month getNextMonth(Month month) {
    return Arrays.stream(Month.values())
            .filter(value -> value.ordinal() == month.ordinal() + 1)
            .findFirst()
            .orElse(Month.JANUARY);
}

ForJavaRush

Уровень 15

7 июля 2023, 04:50

Спасибо, Я передаю в метод ссылку на объект который меняю в самом методе и после возвращаю... Конечно ты прав можно было сразу в return это делать... Но я так пишу скорее для наглядности... Мне так проще..) Про if это да косяк.... Но замечу что тем не менее программа полностью соответствует поставленному условию и выполняет поставленную в нем задачу.... Меня этот факт больше всего начал сбивать с толку...

Денис Enterprise Java Developer

7 июля 2023, 07:56

Так не выполняет же :) вон тебе ниже написали, что вместо декабря январь возвращает, и это именно из-за последовательных IF и присвоений. Ты проверил что ноябрь надо инкрементить, сделал его декабрём, потом проверил что декабрь последний месяц и вернул январь. И да, никак ты не меняешь этот объект, ты просто локальной переменной внутри метода присваиваешь новую ссылку, а работает метод только за счёт return'а, который эту ссылку из метода возвращает. Сделай метод void и ты удивишься) Ну а в целом, почитай про области видимости переменных и их жизненный цикл.

ForJavaRush

Уровень 15

7 июля 2023, 19:34

Спасибо всем.... Жестко тупил...

KIRAРазработчик в prosv.ru

Expert

6 июля 2023, 21:00

полезный

Должен быть декабрь

System.out.println(getNextMonth(Month.NOVEMBER));

ForJavaRush

Уровень 15

7 июля 2023, 19:34

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