From 3a9e0ea8a7c9334b397b0b79f126c8fe07ff6ab8 Mon Sep 17 00:00:00 2001 From: Juergen Hoeller Date: Sat, 22 Jul 2023 00:31:09 +0200 Subject: [PATCH] Polishing --- .../cache/caffeine/CaffeineCache.java | 15 ++++++++------- 1 file changed, 8 insertions(+), 7 deletions(-) diff --git a/spring-context-support/src/main/java/org/springframework/cache/caffeine/CaffeineCache.java b/spring-context-support/src/main/java/org/springframework/cache/caffeine/CaffeineCache.java index c47497f85e..c768347cf2 100644 --- a/spring-context-support/src/main/java/org/springframework/cache/caffeine/CaffeineCache.java +++ b/spring-context-support/src/main/java/org/springframework/cache/caffeine/CaffeineCache.java @@ -59,8 +59,8 @@ public class CaffeineCache extends AbstractValueAdaptingCache { * given internal {@link com.github.benmanes.caffeine.cache.Cache} to use. * @param name the name of the cache * @param cache the backing Caffeine Cache instance - * @param allowNullValues whether to accept and convert {@code null} - * values for this cache + * @param allowNullValues whether to accept and convert {@code null} values + * for this cache */ public CaffeineCache(String name, com.github.benmanes.caffeine.cache.Cache cache, boolean allowNullValues) { @@ -86,7 +86,7 @@ public class CaffeineCache extends AbstractValueAdaptingCache { @SuppressWarnings("unchecked") @Override @Nullable - public T get(Object key, final Callable valueLoader) { + public T get(Object key, Callable valueLoader) { return (T) fromStoreValue(this.cache.get(key, new LoadFunction(valueLoader))); } @@ -106,7 +106,7 @@ public class CaffeineCache extends AbstractValueAdaptingCache { @Override @Nullable - public ValueWrapper putIfAbsent(Object key, @Nullable final Object value) { + public ValueWrapper putIfAbsent(Object key, @Nullable Object value) { PutIfAbsentFunction callable = new PutIfAbsentFunction(value); Object result = this.cache.get(key, callable); return (callable.called ? null : toValueWrapper(result)); @@ -140,7 +140,7 @@ public class CaffeineCache extends AbstractValueAdaptingCache { @Nullable private final Object value; - private boolean called; + boolean called; public PutIfAbsentFunction(@Nullable Object value) { this.value = value; @@ -159,16 +159,17 @@ public class CaffeineCache extends AbstractValueAdaptingCache { private final Callable valueLoader; public LoadFunction(Callable valueLoader) { + Assert.notNull(valueLoader, "Callable must not be null"); this.valueLoader = valueLoader; } @Override - public Object apply(Object o) { + public Object apply(Object key) { try { return toStoreValue(this.valueLoader.call()); } catch (Exception ex) { - throw new ValueRetrievalException(o, this.valueLoader, ex); + throw new ValueRetrievalException(key, this.valueLoader, ex); } } }