异常：通过Spring aop 记录日志重复多次记录问题

异常现象

最近发现新上线的一个项目中日志记录有问题，正常每次调用应该只记录一次日志的，在日志记录表内发现有许多请求都有十来条记录，并且记录的更新时间距离写入时间有十来分钟的，也有超过一天的。

异常分析

      分析源码发现，整个记录日志的模块通过Spring aop 的方式记录Service层所有Service实现类中部分方法的请求和响应报文，在记录日志的切面类中，定义了切点对应整个service层，前置通知和后置返回通知，用ThreadLocal对应来传递前置通知和后置返回通知间传递的日志对象，最后在后置返回通知内把日志落库。

      发现在前置通知和后置返回通知内，都是先判断是否是需要记录日志的方法名，若是的话就记录，不是就不做操作；判断方法名使用的是黑名单排除法，即在名单内的就不记录日志，但看过service层中部分实现类中新增了许多方法并且是不需要记录日志的，这些方法并没有加到黑名单内，这样导致部分方法多记了日志。

      以上不是这次要查找的原因，因为这样只会多记部分方法的日志，并不会重复记录日志。再细看源码发现，在将从ThreadLocal中取出的日志对象操作入库后，并没有清除该日志对象，并且在后置返回通知中只判断了当前线程中ThreadLocal中是否有日志对象导致下一个请求使用该线程时，该日志对象还在，最后引起重复落库记录。即：

第一个请求====》前置通知，判断需要记录日志，日志对象放入ThreadLocal中====>方法a()====》后置返回通知，日志对象从ThreadLocal中取出，日志对象存在则记录日志，操作入库；不存在则不操作=====》日志对象没有销毁，继续在该线程中

第二个请求====》前置通知，判断不需要记录日志====>方法b()====》后置返回通知，日志对象从ThreadLocal中取出，日志对象存在则记录日志，操作入库；不存在则不操作=====》日志对象没有销毁，继续在该线程中

第一个请求记录日志正常，第二个请求中若方法b在黑名单内，则也会记录一条日志，这条日志实际是第一个请求的请求报文和第二个请求中的响应报文，这样就导致了重复记录日志。

解决办法

解决办法有多种，以下任选其一

1：自定义注解，在需要记录日志的方法上使用该注解

2：在后置返回通知中，每次记录入库后，使用ThreadLocal remove方法，清除该日志对象

3：后置返回通知中不依据从TheadLocal取出日志对象是否存在做为判断条件，而使用和前置通知一样的判断条件，即判断方法名是否在黑名单内，在则不记录，不在则记录