Какая-та очень странная дичь, помогите. UPD: Задача перестала компилироваться на сервере.

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

Решен

Валидор пишет: Метод readKeyFromConsoleAndInitPlanet должен корректно обновлять значение переменной thePlanet в соответствии с условием задачи. Если введенная строка равна "moon", присвоить переменной thePlanet соответствующий объект. (самое забавное, что сам код работает корректно и переменная 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; import java.io.BufferedReader; import java.io.IOException; import java.io.InputStreamReader; import com.javarush.task.task14.task1421.Singleton; /* Закрепляем паттерн Singleton */ public class Solution { public static void main(String[] args) { } public static Planet thePlanet; static { readKeyFromConsoleAndInitPlanet(); } //add static block here - добавьте статический блок тут public static void readKeyFromConsoleAndInitPlanet() { // implement step #5 here - реализуйте задание №5 тут BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); String s = ""; try { s = reader.readLine(); } catch (IOException o){System.out.println("Что-то не так! \n"); o.printStackTrace();} if(s.equals(Planet.EARTH)) thePlanet = Earth.getInstance(); else if(s.equals(Planet.MOON)) thePlanet = Moon.getInstance(); else if(s.equals(Planet.SUN)) thePlanet = Sun.getInstance(); else thePlanet = null; } }

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

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

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

--------

Уровень 40

Master

17 октября 2018, 23:04

public static Moon getInstance(){
    if(instance == null){
        instance = new Moon();
        return instance;
    } else return null;
}

Как минимум, такое решение явно не верное: смысл синглтона в том, что метод как раз должен всегда возвращать один и тот же единственный объект своего класса, а не единственный раз вернуть и перестать работать

DancingShaman

Уровень 23

17 октября 2018, 23:21

Переделал под Initialization on Demand Holder. Задача до сих пор не компилируется на сервере. public class Moon implements Planet { private static class SingletonHolder{ private static final Moon instance = new Moon(); } public static Moon getInstance(){return Moon.SingletonHolder.instance;} private Moon(){}; }

--------

Уровень 40

Master

17 октября 2018, 23:45

решение

Теперь, как я понимаю, еще и исходный код перекорежил - вряд ли валидатор это поймет. Метод должен просто возвращать экземпляр, если его еще нет - сначала создать, затем вернуть, если есть - сразу вернуть. И ты уверен, что проблемы с компиляцией, а не валидацией?

DancingShaman

Уровень 23

17 октября 2018, 23:50

У меня на машине всё отлично компилируется и работает, это у них там он не компилируется :D Подумаю над твоими словами и завтра разберусь с этим, время уже позднее, спать пора.

DancingShaman

Уровень 23

18 октября 2018, 11:26

Да, чувак, ты был прав, задача решена, спасибо тебе :)

DancingShaman

Уровень 23

17 октября 2018, 22:27

До этого задача отказывалась компилироваться на сервере, но после десяти нажатий она таки скомпилировалась там, возможно, из-за обновления сегодняшнего появились какие-то баги. Например, она опять у меня не компилируется на сервере, а потратил 10 попыток, жаль, есть некое предчувствие, что с первого раза бы она решилась.