zig zuКак пример:

Всё равно не очевидно. Вижу минимум 2 варианта.



Или

zig zuТак вся суть как раз в том, чтобы нигде synchronized не было.

Для записи раз в сутки очень важно чтобы методы записи были не блокирующими. Так?

zig zuМой класс Data вообще не блокирующий не для читающих ни для пишущего потока.

Я вижу. Поэтому объясняю к чем это приведет.

zig zuДля этого и локальные ссылки нужны в методах.

Не вижу связи, если вы читаете только по одному ключу. Если в get методе много операций, тогда ладно. Но вы же не напрягаете себя объяснением таких нюансов.

zig zuОбновлять может только один поток
"Я так сказал". Да? Других причин почему это так нет? Если без иронии, то вопрос простой. Кто гарантирует что это только один поток?

zig zuНесколько методов get осуществляют поиск по разным ключам.
Возможно это важно в вашем проекте, но приведенный код никак не объясняет зачем это сделано.

zig zuВот это правильное замечание, локальная ссылка в методах get может действительно не нужна?

Зависит от реализации метода, которую вы опустили.

zig zuЯ правильно понимаю, что при вызове метода get в стеке потока будет создана копия ссылки на мапу data?

Нет, не создаётся. Просто если ссылка используется только один раз, что для приведенного кода логично, то и для чего хранить эту ссылку на стеке, не ясно.

zig zuП.С. А разные методы get ищут по разным ключам.
Вариант сделать ключ аргуметом метода не рассматривается?

DEVcoachУчитывая логику этого метода - полная перезапись всей мапы - synchronized здесь не нужен совершенно.
Не нужен здесь, значит нужен где-то ещё. Потому как автор хочет тяжелые вычисления делать где-то единственным потоком.

DEVcoachОтнюдь, код, приведенный автором, - как раз таки является классическим кэшем, только вместо операции put у него bulk update.
Кешем оно не является вообще, так как код автора сводиться к одному единственному volatile полю, в итоге. Смотри.
HashMap к вопросу отношения не имеет. Значит это любой бизнес объект.
Заполнять бизнес объект в методе updateData тоже смысла нет, можно заполнить и снаружи, передавая готовое значение. Ведь метод не synchronized, значит тяжелые вычисления делаются вне его.
Точно так же getData это лишь делегат для volatileField.get();

В итоге, всё сводиться к тому что кроме объявления волатайл поля, весь остальной код это шум. Поэтому назвать поле кэшем у меня язык не поворачивается.

zig zuЯ гарантирую.

Ну, вот и я о том же.

zig zuЕсть специальный поток, который вызывает веб сервис и передает данные в полученном виде в метод updateData, там уже происходит конвертация и сохранение во внутреннею мапу.
В плане реализации это будет singleton ejb timer или спринговый шедулер.

Ню-ню.