2013-09-11 55 views
0

我的ApplicationContext需要如下Spring AOP的控制器执行两次

<?xml version="1.0" encoding="UTF-8"?> 
<beans xmlns=".."> 
    <mvc:annotation-driven/> 
    <task:annotation-driven/> 
    <mvc:resources mapping="/resources/**" location="/resources/"/> 
    <aop:aspectj-autoproxy /> 
    <context:component-scan base-package="com.abc"> 
     <context:include-filter type="aspectj" expression="com.abc.aspects.LogControllerAspect"/> 
    </context:component-scan> 
    <context:annotation-config /> 
</beans> 

有2个看点Java类,LogControllerAspect(用于记录所有的呼叫到Spring控制器)和LogDAOAspect(用于记录所有来电DB)。

@Aspect 
@Service 
public class LogDAOAspect { 
    private Logger logger = LoggerFactory.getLogger(this.getClass()); 

    @Around("execution(* com.*.*DAOImpl.*(..))") 
    public Object logAround(ProceedingJoinPoint joinPoint) throws Throwable { 
     String methodId = joinPoint.getTarget().getClass().getSimpleName()+" : "+joinPoint.getSignature().getName() + " : " + ((joinPoint.getArgs()==null||joinPoint.getArgs().length<1)?"":(Arrays.toString(joinPoint.getArgs()))); 
     Object returnVal = null; 
     StopWatch sw = new StopWatch(methodId); 
     try { 
      sw.start(); 
      returnVal= joinPoint.proceed(joinPoint.getArgs()); 
      sw.stop(); 
     } catch (Throwable e) { 
      logger.error(methodId+"\n"+e); 
      throw e; 
     } 
     logger.debug(methodId + ":" +sw.getTotalTimeMillis()); 
     return returnVal; 
    } 
} 


@Aspect 
public class LogControllerAspect { 
    private Logger logger = LoggerFactory.getLogger(this.getClass()); 

    @Around("execution(* com.*.*Controller.*(..))") 
    public Object logAround(ProceedingJoinPoint joinPoint) throws Throwable { 
     String methodId = joinPoint.getTarget().getClass().getSimpleName()+" : "+joinPoint.getSignature().getName() + " : " + ((joinPoint.getArgs()==null||joinPoint.getArgs().length<1)?"":(Arrays.toString(joinPoint.getArgs()))); 
     Object returnVal = null; 
     StopWatch sw = new StopWatch(methodId); 
     try { 
      sw.start(); 
      returnVal= joinPoint.proceed(joinPoint.getArgs()); 
      sw.stop(); 
     } catch (Throwable e) { 
      logger.error(methodId+"\n"+e); 
      throw e; 
     } 
     logger.debug(methodId + ":" +sw.getTotalTimeMillis()); 
     return returnVal; 
    } 
} 

LogDAOAspect是好的,但LogControllerAspect正在记录两次(logAround方法正在执行两次)时,我要求的一些页。我可以理解,该方面代表了两次,但不知道如何避免这种情况。帮助赞赏。

+0

你确定控制器方法运行一次吗?也许有两个方法调用。 – erencan

+0

该方法只被调用一次(我设置了一个断点,它只打了一次) –

+0

你应该检查COntroller软件包'* com。*。* Controller。*(..) – erencan

回答

0

这是一个愚蠢的错误。这甚至不是一个春天相关的问题! 我已经安装log4j如下!!! ??? !!!

<appender name="AppAppender" class="org.apache.log4j.DailyRollingFileAppender"> 
    <param name="DatePattern" value=".yyyy-MM-dd.HH"/> 
    <param name="File" value="logs/logfile.log"/> 
    <param name="Append" value="true"/> 
    <layout class="org.apache.log4j.PatternLayout"> 
     <param name="ConversionPattern" value="%-5p : %d{ISO8601} : %m%n"/> 
    </layout> 
</appender> 

<logger name="com.abc.aspects.LogControllerAspect"> 
    <priority value="debug"></priority> 
    <appender-ref ref="AppAppender"/> 
</logger> 

<root> 
    <priority value="error" /> 
    <appender-ref ref="AppAppender" /> 
</root> 

谢谢@erencan。你的问题确实帮助我仔细观察了该方法内真正发生的事情。

改变是在下面,它工作正常。应该log4j到这个问题的标签!

<logger name="com.abc.aspects.LogControllerAspect" additivity="false"> 
    <priority value="debug"></priority> 
    <appender-ref ref="AppAppender"/> 
</logger>