Не могу понять в чём проблема?

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

Обсуждается

Проходит всё кроме - Метод readKeyFromConsoleAndInitPlanet() должен корректно обновлять значение переменной thePlanet в соответствии с условием задачи.

Для решения этой задачи тебе нужно:

Найти пример паттерна Singleton с ленивой реализацией (lazy initialization).
По образу и подобию в отдельных файлах создать в отдельных файлах три класса синглтона — Sun, Moon, Earth.
Реализовать интерфейс Planet в классах Sun, Moon, Earth.
В статическом блоке класса Solution вызвать метод readKeyFromConsoleAndInitPlanet().
Реализовать функционал метода readKeyFromConsoleAndInitPlanet():
- для этого считай с консоли один параметр типа String.
- если он равен одной из констант интерфейса Planet, создай соответствующий объект и присвой его Planet thePlanet, иначе — обнули Planet thePlanet.

Требования:

Класс Sun не должен позволять создавать объекты своего типа за пределами класса.
Класс Sun должен содержать приватное статическое поле instance типа Sun.
В классе Sun должен быть реализован публичный статический метод getInstance(), возвращающий значение поля instance.
Метод getInstance() в классе Sun должен всегда возвращать один и тот же объект.
Поле instance класса Sun должно быть инициализировано только после первого обращения к методу getInstance().
Класс Moon не должен позволять создавать объекты своего типа за пределами класса.
Класс Moon должен содержать приватное статическое поле instance типа Moon.
В классе Moon должен быть реализован публичный статический метод getInstance(), возвращающий значение поля instance.
Метод getInstance() в классе Moon должен всегда возвращать один и тот же объект.
Поле instance класса Moon должно быть инициализировано только после первого обращения к методу getInstance().
Класс Earth не должен позволять создавать объекты своего типа за пределами класса.
Класс Earth должен содержать приватное статическое поле instance типа Earth.
В классе Earth должен быть реализован публичный статический метод getInstance(), возвращающий значение поля instance.
Метод getInstance в классе Earth должен всегда возвращать один и тот же объект.
Поле instance класса Earth должно быть инициализировано только после первого обращения к методу getInstance().
Метод readKeyFromConsoleAndInitPlanet() должен быть вызван в статическом блоке класса Solution.
Метод readKeyFromConsoleAndInitPlanet() должен считывать одну строку с клавиатуры.
Метод readKeyFromConsoleAndInitPlanet() должен корректно обновлять значение переменной thePlanet в соответствии с условием задачи.
Классы Sun, Moon и Earth должны быть созданы в отдельных файлах.

package com.javarush.task.task15.task1522; public class Sun implements Planet{ private static Sun instance; private Sun(){ } public static Sun getInstance(){ if (instance == null){ instance = new Sun(); } return instance; } }

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

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

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

Денис Enterprise Java Developer

25 сентября 2023, 09:06

Во первых, условие не требует ничего выводить в консоль. Во вторых, у тебя есть статическая переменная thePlanet в классе, ты же работаешь с локальной, соответственно твой метод ничего не делает и не соответствует требованию. Здесь более чем напрашивается Switch к слову.

public static void readKeyFromConsoleAndInitPlanet() {
    Scanner scanner = new Scanner(System.in);

    switch (scanner.nextLine()) {
        case Planet.EARTH:
            thePlanet = Earth.getInstance();
            break;
        case Planet.MOON:
            thePlanet = Moon.getInstance();
            break;
        case Planet.SUN:
            thePlanet = Sun.getInstance();
            break;
        default:
            thePlanet = null;
    }
}

Semyon Baryshnikov QA Automation Engineer

25 сентября 2023, 09:49

Спасибо, сейчас исправлю

Kirill

Уровень 22

3 января 2024, 17:26

Привет! Влетаю в дмалог) не понимаю, почему у нас синтексис Planet.SUN. Что это вызов такой?

Денис Enterprise Java Developer

3 января 2024, 19:05

Обычное обращение к статическому полю класса.

Long_byte

Уровень 57

19 февраля 2024, 05:46

это статические поля интерфейса Planet так как поля статические мы обращаемся к ним через имя интерфейса поправьте меня если ошибаюсь