From 9a1155e660f88b53e4b6529c4354726ac313165f Mon Sep 17 00:00:00 2001 From: liuzhigang Date: Fri, 15 Sep 2023 14:20:57 +0800 Subject: [PATCH] refactor: modify non-standard naming --- .../spring/context/DubboSpringInitializer.java | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/dubbo-config/dubbo-config-spring/src/main/java/org/apache/dubbo/config/spring/context/DubboSpringInitializer.java b/dubbo-config/dubbo-config-spring/src/main/java/org/apache/dubbo/config/spring/context/DubboSpringInitializer.java index 7f79c7223a0..8dc1a97930b 100644 --- a/dubbo-config/dubbo-config-spring/src/main/java/org/apache/dubbo/config/spring/context/DubboSpringInitializer.java +++ b/dubbo-config/dubbo-config-spring/src/main/java/org/apache/dubbo/config/spring/context/DubboSpringInitializer.java @@ -43,7 +43,7 @@ public class DubboSpringInitializer { private static final Logger logger = LoggerFactory.getLogger(DubboSpringInitializer.class); - private static final Map contextMap = new ConcurrentHashMap<>(); + private static final Map REGISTRY_CONTEXT_MAP = new ConcurrentHashMap<>(); public DubboSpringInitializer() { } @@ -54,7 +54,7 @@ public static void initialize(BeanDefinitionRegistry registry) { DubboSpringInitContext context = new DubboSpringInitContext(); // Spring ApplicationContext may not ready at this moment (e.g. load from xml), so use registry as key - if (contextMap.putIfAbsent(registry, context) != null) { + if (REGISTRY_CONTEXT_MAP.putIfAbsent(registry, context) != null) { return; } @@ -66,18 +66,18 @@ public static void initialize(BeanDefinitionRegistry registry) { } public static boolean remove(BeanDefinitionRegistry registry) { - return contextMap.remove(registry) != null; + return REGISTRY_CONTEXT_MAP.remove(registry) != null; } public static boolean remove(ApplicationContext springContext) { AutowireCapableBeanFactory autowireCapableBeanFactory = springContext.getAutowireCapableBeanFactory(); - for (Map.Entry entry : contextMap.entrySet()) { + for (Map.Entry entry : REGISTRY_CONTEXT_MAP.entrySet()) { DubboSpringInitContext initContext = entry.getValue(); if (initContext.getApplicationContext() == springContext || initContext.getBeanFactory() == autowireCapableBeanFactory || initContext.getRegistry() == autowireCapableBeanFactory ) { - DubboSpringInitContext context = contextMap.remove(entry.getKey()); + DubboSpringInitContext context = REGISTRY_CONTEXT_MAP.remove(entry.getKey()); logger.info("Unbind " + safeGetModelDesc(context.getModuleModel()) + " from spring container: " + ObjectUtils.identityToString(entry.getKey())); return true; @@ -87,11 +87,11 @@ public static boolean remove(ApplicationContext springContext) { } static Map getContextMap() { - return contextMap; + return REGISTRY_CONTEXT_MAP; } static DubboSpringInitContext findBySpringContext(ApplicationContext applicationContext) { - for (DubboSpringInitContext initContext : contextMap.values()) { + for (DubboSpringInitContext initContext : REGISTRY_CONTEXT_MAP.values()) { if (initContext.getApplicationContext() == applicationContext) { return initContext; } @@ -184,7 +184,7 @@ private static void registerSingleton(ConfigurableListableBeanFactory beanFactor } private static DubboSpringInitContext findContextForApplication(ApplicationModel applicationModel) { - for (DubboSpringInitContext initializationContext : contextMap.values()) { + for (DubboSpringInitContext initializationContext : REGISTRY_CONTEXT_MAP.values()) { if (initializationContext.getApplicationModel() == applicationModel) { return initializationContext; }