code review done。

src/main/java/cn/quantgroup/xyqb/aspect/limit/AccessLimitAspect.java

@@ -26,18 +26,20 @@ public class AccessLimitAspect {
      * @throws Throwable
      */
     @Around("@annotation(cn.quantgroup.xyqb.aspect.limit.AccessLimit)")
-    public void accessLimit(ProceedingJoinPoint pjp) throws Throwable {
+    public Object accessLimit(ProceedingJoinPoint pjp) throws Throwable {
+        Object o = null;
         AccessLimit accessLimit = ((MethodSignature) pjp.getSignature()).getMethod().getAnnotation(AccessLimit.class);
         String redisKey = accessLimit.redisKey() + "_" + pjp.getArgs()[0];
         boolean lock = stringRedisTemplate.opsForValue().setIfAbsent(redisKey, "");
         if (lock) {
-            stringRedisTemplate.expire(redisKey, 1, TimeUnit.MINUTES);
             try {
-                pjp.proceed();
+                stringRedisTemplate.expire(redisKey, 1, TimeUnit.MINUTES);
+                o = pjp.proceed();
             } finally {
                 stringRedisTemplate.delete(redisKey);
             }
         }
+        return o;
     }
 
 }

src/main/java/cn/quantgroup/xyqb/service/user/impl/ModifyPhoneNoServiceImpl.java

@@ -91,6 +91,9 @@ public class ModifyPhoneNoServiceImpl implements IModifyPhoneNoService {
     @Transactional(rollbackFor = Exception.class)
     public void saveStep2(Step2Req step2Req) {
         ModifyPhoneNo modifyPhoneNo = modifyPhoneNoRepository.findOne(step2Req.getId());
+        if (modifyPhoneNo == null) {
+            throw new DataException("数据不存在。");
+        }
         modifyPhoneNo.setIdCardFaceUrl(step2Req.getIdCardFaceUrl());
         modifyPhoneNo.setIdCardRearUrl(step2Req.getIdCardRearUrl());
         modifyPhoneNo.setIdCardHoldUrl(step2Req.getIdCardHoldUrl());