JavaRush/Помощь по Java задачам/Не пойму, в чем ошибка

20 уровень

27.11.2017
5681просмотров
18комментариев

Не пойму, в чем ошибка

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

Решен

Поле инициализируется, но валидатор все равно ругается. Где ошибка? Через геттер тоже пробовал, не помогает.

И снова Singleton паттерн - синхронизация в статическом блоке.
Внутри класса OurPresident в статическом блоке создай синхронизированный блок.
Внутри синхронизированного блока инициализируй president.

Требования:

Класс OurPresident должен содержать приватное статическое поле OurPresident president.
Класс OurPresident должен содержать публичный статический метод OurPresident getOurPresident().
Класс OurPresident должен содержать приватный конструктор.
Класс OurPresident должен содержать статический блок.
Внутри статического блока класса OurPresident должен быть синхронизированный блок.
Внутри синхронизированного блока должно быть проинициализировано поле president.

package com.javarush.task.task17.task1706; public class OurPresident { private static OurPresident president; private OurPresident() { } public static OurPresident getOurPresident() { if(president == null) president = new OurPresident(); return president; } static{ synchronized(OurPresident.class){ president = new OurPresident(); } } }

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

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

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

GrigoriyBackend Developer в СберТех

18 мая 2020, 14:59

Решил с помощью такого static

static {
        synchronized (OurPresident.class){
            president = president == null? new OurPresident(): getOurPresident();
        }
    }

Evgeny StrokovEnterprise Java Developer в СберТех

17 декабря 2020, 09:57

Не вижу смысла выставлять проверку на null в статическом блоке. В любом случае объект president будет иметь null, и статический блок будет исполнен единожды при инициализации класса. Иными словами, Вы написали довольно странную и излишнюю конструкцию инициализации статического объекта.

SERGEY

Уровень 31

13 марта 2021, 12:33

то есть при запуске программы, когда мы обращаемся с одного потока к объекту, он инициализируется для всех существующих потоков программы?

Evgeny StrokovEnterprise Java Developer в СберТех

2 августа 2021, 10:47

Допустим, несколько потоков решили заполучить объект класса OurPresident. Этот класс был загружен ClassLoader, но не был инициализирован (т.е к классу не было еще никакого обращения). Потоки запросили объект с помощью вызова метода getOurPresident(). Но перед тем, как метод будет запущен, класс OurPresident должен быть инициализирован. В это время все потоки будут ожидать инициализацию данного класса. Когда класс будет инициализирован (а соответсвенно, вся инициализация будет произведена: static, non-static, вызов constructor), статическое поле president класса OurPresident будет уже инициализировано. Следовательно, можно отсюда сделать вывод, что синхронизация в static-блоке является излишней конструкцией, как и проверка поля president на null.

Андрей Щедрин

Уровень 18

18 сентября 2018, 17:37

OurPresident.class

Откуда мы взяли такой синтаксис? В уроках этого не было

+12

ВладиславJava Developer в Soft Media Group

9 ноября 2018, 07:05

https://javarush.com/groups/posts/1055-sinkhronizacija-potokov-blokirovka-obhhekta-i-blokirovka-klassa

JavaFreak

Уровень 20

28 ноября 2017, 14:36

Все, народ, разобрался. Действительно, забыл как работает статический блок, отсюда все и пошло. Всем большое спасибо за помощь))

Евгений

Уровень 24

14 декабря 2019, 20:37

как решил проблему то? ((

hidden #1380519

Уровень 21

27 ноября 2017, 23:57

Вы в статическом блоке уже проинициализировали поле президент. Поэтому публичная функция должна только вернуть его значение без проверки на null.

JavaFreak

Уровень 20

28 ноября 2017, 10:38

Пробовал, не работает.

Roman Java Developer

Expert

28 ноября 2017, 11:02

Выложи свой код метода getOurPresident() после данной рекомендации.

Pavlo PlynkoJava-разработчик в CodeGym

Expert

28 ноября 2017, 11:47

не нужна ни проверка на null, ни инициализация в getOurPresident().

JavaFreak

Уровень 20

28 ноября 2017, 12:40

ublic class OurPresident { private static OurPresident president; private OurPresident() { if(president == null) president = new OurPresident(); } public static OurPresident getOurPresident() { return president; } static{ synchronized(OurPresident.class){ president = OurPresident.getOurPresident(); } } } Переместил в конструктор, все равно ничего не работает, ошибка валидатора та же

--------

Уровень 40

Master

28 ноября 2017, 12:49

решение

president = OurPresident.getOurPresident();

С учетом содержания метода getOurPresident() эта строчка означает:

president = president;

Да еще и учитывая, что статический блок выполняется при создании объекта, а не в процессе его использования, фактически, заменяя код конструктора - в нем надо просто создать президента без лишних телодвижений)) И как раз благодаря правильному статическому блоку условие (president == null) не будет выполняться никогда, поэтому и проверять его не надо.

+10

Roman Java Developer

Expert

28 ноября 2017, 12:56

Тебе ж написали, что нужно просто убрать проверку, а не засовывать ее в конструктор )

Pavlo PlynkoJava-разработчик в CodeGym

Expert

28 ноября 2017, 13:37

Инициализацию в статическом блоке убирать не надо было.

Mark Cain

Уровень 6

19 января 2020, 05:25

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

SerhiiSoftware Developer в EPAM

28 февраля 2021, 12:21

synchronized (OurPresident.class) Здравствуйте, а можете рассказать, а что это значит?