feat(): refactor

Author: Rawven

apollo-client/src/main/java/com/ctrip/framework/apollo/internals/ConfigManager.java

@@ -38,4 +38,6 @@ public interface ConfigManager {
    * @return the config file instance for the namespace
    */
   ConfigFile getConfigFile(String namespace, ConfigFileFormat configFileFormat);
+  
+  
 }

apollo-client/src/main/java/com/ctrip/framework/apollo/internals/ConfigMonitorInitializer.java

@@ -24,6 +24,7 @@
 import com.ctrip.framework.apollo.monitor.internal.exporter.ApolloClientMetricsExporter;
 import com.ctrip.framework.apollo.monitor.internal.jmx.ApolloClientJmxMBeanRegister;
 import com.ctrip.framework.apollo.monitor.internal.ApolloClientMonitorContext;
+import com.ctrip.framework.apollo.monitor.internal.listener.ApolloClientMonitorEventListener;
 import com.ctrip.framework.apollo.monitor.internal.listener.impl.DefaultApolloClientBootstrapArgsApi;
 import com.ctrip.framework.apollo.monitor.internal.listener.impl.DefaultApolloClientExceptionApi;
 import com.ctrip.framework.apollo.monitor.internal.listener.impl.DefaultApolloClientNamespaceApi;
@@ -37,6 +38,7 @@
 import com.ctrip.framework.apollo.tracer.spi.MessageProducer;
 import com.ctrip.framework.apollo.util.ConfigUtil;
 import com.ctrip.framework.foundation.internals.ServiceBootstrap;
+import com.google.common.collect.Lists;
 import java.util.List;
 
 /**
@@ -64,39 +66,45 @@ private static void doInit() {
     initializeMetricsEventListener();
     initializeMetricsExporter();
     initializeJmxMonitoring();
-    hasInitialized = true;
   }
 
 
   private static void initializeJmxMonitoring() {
     if (m_configUtil.isClientMonitorJmxEnabled()) {
-      monitorContext.getCollectors().forEach(metricsCollector ->
+      monitorContext.getApolloClientMonitorEventListeners().forEach(metricsListener ->
           ApolloClientJmxMBeanRegister.register(
-              MBEAN_NAME + metricsCollector.getName(), metricsCollector)
+              MBEAN_NAME + metricsListener.getName(), metricsListener)
       );
     }
   }
 
   private static void initializeMetricsEventListener() {
-    DefaultConfigManager configManager = (DefaultConfigManager) ApolloInjector.getInstance(
+    ConfigManager configManager = ApolloInjector.getInstance(
         ConfigManager.class);
-    monitorContext.setApolloClientBootstrapArgsMonitorApi(new DefaultApolloClientBootstrapArgsApi(
-        m_configUtil));
-    monitorContext.setApolloClientExceptionMonitorApi(new DefaultApolloClientExceptionApi());
-    monitorContext.setApolloClientNamespaceMonitorApi(new DefaultApolloClientNamespaceApi(
-        configManager.m_configs, configManager.m_configFiles));
-    monitorContext.setApolloClientThreadPoolMonitorApi(new DefaultApolloClientThreadPoolApi(
+    DefaultApolloClientBootstrapArgsApi defaultApolloClientBootstrapArgsApi = new DefaultApolloClientBootstrapArgsApi(
+        m_configUtil);
+    DefaultApolloClientExceptionApi defaultApolloClientExceptionApi = new DefaultApolloClientExceptionApi();
+    DefaultApolloClientNamespaceApi defaultApolloClientNamespaceApi = new DefaultApolloClientNamespaceApi(
+        configManager);
+    DefaultApolloClientThreadPoolApi defaultApolloClientThreadPoolApi = new DefaultApolloClientThreadPoolApi(
         RemoteConfigRepository.m_executorService,
         AbstractConfig.m_executorService, AbstractConfigFile.m_executorService,
-        AbstractApolloClientMetricsExporter.m_executorService));
+        AbstractApolloClientMetricsExporter.m_executorService);
+    
+    monitorContext.setApolloClientBootstrapArgsMonitorApi(defaultApolloClientBootstrapArgsApi);
+    monitorContext.setApolloClientExceptionMonitorApi(defaultApolloClientExceptionApi);
+    monitorContext.setApolloClientNamespaceMonitorApi(defaultApolloClientNamespaceApi);
+    monitorContext.setApolloClientThreadPoolMonitorApi(defaultApolloClientThreadPoolApi);
+    monitorContext.setApolloClientMonitorEventListeners(Lists.newArrayList(defaultApolloClientBootstrapArgsApi,
+        defaultApolloClientNamespaceApi,defaultApolloClientThreadPoolApi,defaultApolloClientExceptionApi));
   }
 
   private static void initializeMetricsExporter(
   ) {
     ApolloClientMetricsExporterFactory exporterFactory = ApolloInjector.getInstance(
         ApolloClientMetricsExporterFactory.class);
     ApolloClientMetricsExporter metricsReporter = exporterFactory.getMetricsReporter(
-        monitorContext.getCollectors());
+        monitorContext.getApolloClientMonitorEventListeners());
     if(metricsReporter != null) {
       monitorContext.setApolloClientMetricsExporter(metricsReporter);
     }

apollo-client/src/main/java/com/ctrip/framework/apollo/monitor/api/ApolloClientNamespaceMonitorApi.java

@@ -35,11 +35,6 @@ public interface ApolloClientNamespaceMonitorApi {
    */
   Integer getNamespacePropertySize(String namespace);
 
-  /**
-   * get ConfigFile namespaces
-   */
-  List<String> getConfigFileNamespaces();
-
   /**
    * get not found namespaces
    */

apollo-client/src/main/java/com/ctrip/framework/apollo/monitor/internal/ApolloClientMonitorConstant.java

@@ -24,12 +24,6 @@
  */
 public class ApolloClientMonitorConstant {
 
-  /**
-   * util
-   */
-  public static final DateTimeFormatter DATE_FORMATTER = DateTimeFormatter.ofPattern(
-      "yyyy-MM-dd HH:mm:ss").withZone(ZoneId.systemDefault());
-
   /**
    * common
    */
@@ -61,7 +55,7 @@ public class ApolloClientMonitorConstant {
   public static final String APOLLO_CLIENT_NAMESPACE_FIRST_LOAD_SPEND = "Apollo.Client.NamespaceFirstLoadSpendTime";
 
   /**
-   * collector tag
+   * listener tag
    */
   public static final String TAG_ERROR = "ErrorMonitor";
   public static final String TAG_NAMESPACE = "NamespaceMonitor";

apollo-client/src/main/java/com/ctrip/framework/apollo/monitor/internal/ApolloClientMonitorContext.java

@@ -40,7 +40,20 @@ public class ApolloClientMonitorContext {
   private ApolloClientBootstrapArgsMonitorApi apolloClientBootstrapArgsMonitorApi = new NullClientBootstrapArgsMonitorApi();
   private ApolloClientThreadPoolMonitorApi apolloClientThreadPoolMonitorApi = new NullClientThreadPoolMonitorApi();
   private ApolloClientMetricsExporter apolloClientMetricsExporter = new NullApolloClientMetricsExporter();
-
+  private List<ApolloClientMonitorEventListener> listener = Lists.newArrayList();
+  
+  public void addApolloClientMonitorEventListener(ApolloClientMonitorEventListener listener){
+    this.listener.add(listener);
+  }
+  
+  public void setApolloClientMonitorEventListeners(List<ApolloClientMonitorEventListener> listeners){
+    listener = listeners;
+  }
+  
+  public List<ApolloClientMonitorEventListener> getApolloClientMonitorEventListeners(){
+    return listener;
+  }
+  
   public void setApolloClientExceptionMonitorApi(
       ApolloClientExceptionMonitorApi apolloClientExceptionMonitorApi) {
     this.apolloClientExceptionMonitorApi = apolloClientExceptionMonitorApi;
@@ -66,14 +79,6 @@ public void setApolloClientMetricsExporter(
     this.apolloClientMetricsExporter = apolloClientMetricsExporter;
   }
 
-  public List<ApolloClientMonitorEventListener> getCollectors() {
-    return Lists.newArrayList(
-        (ApolloClientMonitorEventListener) apolloClientBootstrapArgsMonitorApi,
-        (ApolloClientMonitorEventListener) apolloClientThreadPoolMonitorApi,
-        (ApolloClientMonitorEventListener) apolloClientExceptionMonitorApi,
-        (ApolloClientMonitorEventListener) apolloClientNamespaceMonitorApi);
-  }
-
   public ApolloClientExceptionMonitorApi getExceptionApi() {
     return apolloClientExceptionMonitorApi;
   }

apollo-client/src/main/java/com/ctrip/framework/apollo/monitor/internal/event/ApolloClientMonitorEventFactory.java

@@ -24,7 +24,7 @@
  */
 public class ApolloClientMonitorEventFactory {
 
-  public static volatile ApolloClientMonitorEventFactory INSTANCE;
+  private static volatile ApolloClientMonitorEventFactory INSTANCE;
 
   private ApolloClientMonitorEventFactory() {
   }

apollo-client/src/main/java/com/ctrip/framework/apollo/monitor/internal/event/ApolloClientMonitorEventPublisher.java

@@ -26,23 +26,23 @@
  */
 public class ApolloClientMonitorEventPublisher {
 
-  private static ApolloClientMonitorContext COLLECTOR_MANAGER = ApolloInjector.getInstance(
+  private static ApolloClientMonitorContext MONITOR_CONTEXT = ApolloInjector.getInstance(
       ApolloClientMonitorContext.class);
   private static ConfigUtil m_configUtil = ApolloInjector.getInstance(ConfigUtil.class);
 
   public static void publish(ApolloClientMonitorEvent event) {
     if (m_configUtil.isClientMonitorEnabled()) {
-      for (ApolloClientMonitorEventListener collector : COLLECTOR_MANAGER.getCollectors()) {
-        if (collector.isSupported(event)) {
-          collector.collect(event);
+      for (ApolloClientMonitorEventListener listener : MONITOR_CONTEXT.getApolloClientMonitorEventListeners()) {
+        if (listener.isSupported(event)) {
+          listener.collect(event);
           return;
         }
       }
     }
   }
 
   protected static void reset() {
-    COLLECTOR_MANAGER = ApolloInjector.getInstance(
+    MONITOR_CONTEXT = ApolloInjector.getInstance(
         ApolloClientMonitorContext.class);
     m_configUtil = ApolloInjector.getInstance(ConfigUtil.class);
 

apollo-client/src/main/java/com/ctrip/framework/apollo/monitor/internal/exporter/AbstractApolloClientMetricsExporter.java

@@ -44,14 +44,14 @@ public abstract class AbstractApolloClientMetricsExporter implements ApolloClien
         ApolloThreadFactory.create(ApolloClientMetricsExporter.class.getName(), true));
   }
 
-  protected List<ApolloClientMonitorEventListener> collectors;
+  protected List<ApolloClientMonitorEventListener> listeners;
 
   @Override
-  public void init(List<ApolloClientMonitorEventListener> collectors, long collectPeriod) {
-    log.info("Initializing metrics exporter with {} collectors and collect period of {} seconds.",
-        collectors.size(), collectPeriod);
+  public void init(List<ApolloClientMonitorEventListener> listeners, long collectPeriod) {
+    log.info("Initializing metrics exporter with {} listeners and collect period of {} seconds.",
+        listeners.size(), collectPeriod);
     doInit();
-    this.collectors = collectors;
+    this.listeners = listeners;
     initScheduleMetricsCollectSync(collectPeriod);
     log.info("Metrics collection scheduled with a period of {} seconds.", collectPeriod);
   }
@@ -73,10 +73,10 @@ private void initScheduleMetricsCollectSync(long collectPeriod) {
 
   protected void updateMetricsData() {
     log.debug("Start to update metrics data job");
-    collectors.forEach(collector -> {
-      if (collector.isMetricsSampleUpdated()) {
-        log.debug("Collector {} has updated samples.", collector.getName());
-        collector.export().forEach(this::registerSample);
+    listeners.forEach(listeners -> {
+      if (listeners.isMetricsSampleUpdated()) {
+        log.debug("Listener {} has updated samples.", listeners.getName());
+        listeners.export().forEach(this::registerSample);
       }
     });
   }

apollo-client/src/main/java/com/ctrip/framework/apollo/monitor/internal/exporter/ApolloClientMetricsExporter.java

@@ -29,7 +29,7 @@ public interface ApolloClientMetricsExporter extends Ordered {
   /**
    * init method
    */
-  void init(List<ApolloClientMonitorEventListener> collectors, long collectPeriod);
+  void init(List<ApolloClientMonitorEventListener> listeners, long collectPeriod);
 
   /**
    * Used to access custom monitoring systems

apollo-client/src/main/java/com/ctrip/framework/apollo/monitor/internal/exporter/ApolloClientMetricsExporterFactory.java

@@ -24,5 +24,5 @@
  */
 public interface ApolloClientMetricsExporterFactory {
 
-  ApolloClientMetricsExporter getMetricsReporter(List<ApolloClientMonitorEventListener> collectors);
+  ApolloClientMetricsExporter getMetricsReporter(List<ApolloClientMonitorEventListener> listeners);
 }