Что не так? Уже все облазил. "Если введенная строка равна "moon", присвоить переменной thePlanet соответствующий объект." Остальное верно

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

Закрепляем паттерн Singleton

Java Core, 5 уровень, 12 лекция

Решен

Если введенная строка равна "moon", присвоить переменной 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(){ // #3 if(instance == null){ //если объект еще не создан instance = new Sun(); //создать новый объект } return instance; // вернуть ранее созданный объект } }

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

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

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

Владимир Созанский

Уровень 24

22 июля 2019, 13:46

ошибка вот в чем String a; если сравнивать == то сравниваются ссылки на объекты нужно использовать метод equals и есть 2 варианта "vaska".equals(key) и key.equals("vaska") или в Вашем случае Planet.MOON.equals(a) или a.equals(Planet.MOON) Так вот первый вариант предпочтителен, потому, что первый объект точно не null

Владислав

Уровень 23

22 июля 2019, 15:00

if (Planet.SUN.equals(a) || Planet.EARTH.equals(a) || Planet.MOON.equals(a)){
             if (Planet.MOON.equals(a)) thePlanet = Moon.getInstance();
             if (Planet.SUN.equals(a)) thePlanet = Sun.getInstance();
             if (Planet.EARTH.equals(a)) thePlanet = Earth.getInstance();

Поменял вот так, все равно та же ошибка

Haumi Java Developer

22 июля 2019, 15:45

a.equalsIgnoreCase(Planet.SUN) и так далее if (a == Planet.SUN || a == Planet.EARTH || a == Planet.MOON) - вообще лишняя строчка вам достаточно условий то что ниже... просто проверяете какое значение хранится в переменной a. Если оно равно какой либо константе, то и и вызывайте нужный метод.

Владимир Созанский

Уровень 24

22 июля 2019, 20:10

решение

Тогда уж пишите if (a == null) thePlanet = null; else if (Planet.MOON.equals(a)) thePlanet = Moon.getInstance(); else if (Planet.SUN.equals(a)) thePlanet = Sun.getInstance(); else if (Planet.EARTH.equals(a)) thePlanet = Earth.getInstance(); else thePlanet = null;

Владислав

Уровень 23

23 июля 2019, 08:58

Да, спасибо, компилятор принял Ваш вариант, но я все ещё не могу понять, почему именно так писать было правильно..

Владимир Созанский

Уровень 24

23 июля 2019, 09:05

Ваша "a" это ссылка на объект. Ссылка может быть пустой, это случай a == null А может иметь значение. И тут правило: Сравнение для String делать с помощью метода equals()