1. New feature: auto create menu when web application start; 2. Optim…

…ize AppLogger: no static field, remove bean declare in applicationContext-servlet.xml
junior9919 · Oct 7, 2015 · 7b6c1fa · 7b6c1fa
1 parent a192811
commit 7b6c1fa
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 3 deletions.
diff --git a/halo-wechat-utils.jardesc b/halo-wechat-utils.jardesc
@@ -15,5 +15,6 @@
         <javaElement handleIdentifier="=WeChat/src&lt;com.halo.wechat.logger"/>
         <javaElement handleIdentifier="=WeChat/src&lt;com.halo.wechat.security"/>
         <javaElement handleIdentifier="=WeChat/src&lt;com.halo.wechat.xml.utils.converters"/>
+        <javaElement handleIdentifier="=WeChat/src&lt;com.halo.servlet.listeners"/>
     </selectedElements>
 </jardesc>
diff --git a/src/com/halo/wechat/config/applicationContext-servlet.xml b/src/com/halo/wechat/config/applicationContext-servlet.xml
@@ -12,7 +12,6 @@
 	<!-- 指定需要扫描的类包 -->
 	<context:component-scan base-package="com.*.wechat.mvc.*" />
 	<!-- 日志配置 -->
-	<bean id="appLogger" class="com.halo.wechat.logger.AppLogger" />
 	<aop:config>
 		<aop:aspect ref="appLogger">
 			<!-- 执行controller方法的切点 -->

diff --git a/src/com/halo/wechat/logger/AppLogger.java b/src/com/halo/wechat/logger/AppLogger.java
@@ -17,7 +17,8 @@
  *
  */
 public class AppLogger {
-	private static Logger logger = LogManager.getLogger(AppLogger.class.getName());
+
+	private Logger logger = LogManager.getLogger(AppLogger.class.getName());
 
 	private String[] getObjectFields(Object obj) {
 		String colNames = "";
@@ -29,7 +30,6 @@ private String[] getObjectFields(Object obj) {
 				try {
 					fieldValue = method.invoke(obj);
 				} catch (IllegalAccessException | IllegalArgumentException | InvocationTargetException e) {
-					// TODO Auto-generated catch block
 					logger.error(e.getMessage());
 				}
 				colNames += fieldName + "|";
@@ -45,6 +45,10 @@ private String[] getObjectFields(Object obj) {
 		return cols;
 	}
 
+	public Logger getLogger() {
+		return this.logger;
+	}
+
 	public void beforeMethod(JoinPoint jp) {
 		Object[] args = jp.getArgs();
 		String params = "";
@@ -144,4 +148,5 @@ public void logBeforeSignatureCompare(JoinPoint jp) {
 			logger.debug("Compare parameters: " + orginSignature + "|" + compareToSignature + "|" + echoStr);
 		}
 	}
+
 }