roncoo-education-common/roncoo-education-common-service/src/main/java/com/roncoo/education/common/cache/CacheRedis.java

@@ -29,7 +29,7 @@ public class CacheRedis {
     public <T> T set(String key, T t) {
         if (t != null) {
             String value = t.toString();
-            if (!(t instanceof String)) {
+            if (!(t instanceof String || t instanceof Long || t instanceof Integer)) {
                 value = JSUtil.toJsonString(t);
             }
             stringRedisTemplate.opsForValue().set(key, value, timeToLive, TimeUnit.MILLISECONDS);
@@ -43,7 +43,7 @@ public class CacheRedis {
     public <T> T set(String key, T t, int time, TimeUnit timeUnit) {
         if (t != null) {
             String value = t.toString();
-            if (!(t instanceof String)) {
+            if (!(t instanceof String || t instanceof Long || t instanceof Integer)) {
                 value = JSUtil.toJsonString(t);
             }
             stringRedisTemplate.opsForValue().set(key, value, time, timeUnit);

roncoo-education-common/roncoo-education-common-service/src/main/java/com/roncoo/education/common/service/HandlerException.java

@@ -5,11 +5,15 @@ import com.roncoo.education.common.core.base.Result;
 import com.roncoo.education.common.core.tools.JSUtil;
 import feign.FeignException;
 import org.springframework.http.HttpStatus;
+import org.springframework.util.StringUtils;
+import org.springframework.validation.FieldError;
+import org.springframework.web.bind.MethodArgumentNotValidException;
 import org.springframework.web.bind.annotation.ExceptionHandler;
 import org.springframework.web.bind.annotation.ResponseStatus;
 import org.springframework.web.bind.annotation.RestControllerAdvice;
 
 import java.util.Map;
+import java.util.StringJoiner;
 
 /**
  * @author wujing
@@ -18,27 +22,40 @@ import java.util.Map;
 public class HandlerException extends BaseController {
 
     @ExceptionHandler({FeignException.class})
-    @ResponseStatus(HttpStatus.OK)
+    @ResponseStatus(HttpStatus.INTERNAL_SERVER_ERROR)
     public Result<String> processBizException(FeignException e) {
-        String msg = e.getCause().toString().split("\n|\r\n|\r")[1];
-        log.error(msg);
-        @SuppressWarnings("unchecked")
-        Map<String, Object> m = JSUtil.parseObject(msg, Map.class);
+        String msg = e.getMessage();
+        if (!StringUtils.hasText(msg)) {
+            return Result.error("服务繁忙，请重试");
+        }
+        log.error("FeignException, {}", msg);
+        Map<String, Object> m = JSUtil.parseArray(msg.substring(msg.lastIndexOf("[{")), Map.class).get(0);
         if (null != m.get("message")) {
             return Result.error(m.get("message").toString());
         }
         return Result.error("服务繁忙，请重试");
     }
 
+    @ExceptionHandler({MethodArgumentNotValidException.class})
+    @ResponseStatus(HttpStatus.INTERNAL_SERVER_ERROR)
+    public Result<String> processException(MethodArgumentNotValidException ex) {
+        StringJoiner joiner = new StringJoiner("、");
+        for (FieldError fieldError : ex.getBindingResult().getFieldErrors()) {
+            joiner.add(fieldError.getDefaultMessage());
+        }
+        log.error("MethodArgumentNotValidException, {}", joiner);
+        return Result.error(joiner.toString());
+    }
+
     @ExceptionHandler({BaseException.class})
-    @ResponseStatus(HttpStatus.OK)
+    @ResponseStatus(HttpStatus.INTERNAL_SERVER_ERROR)
     public Result<String> processException(BaseException e) {
         log.error("BaseException", e);
         return Result.error(e.getMessage());
     }
 
     @ExceptionHandler({Exception.class})
-    @ResponseStatus(HttpStatus.OK)
+    @ResponseStatus(HttpStatus.INTERNAL_SERVER_ERROR)
     public Result<String> processException(Exception e) {
         log.error("Exception", e);
         return Result.error("服务繁忙，请重试");

roncoo-education-service/roncoo-education-service-user/src/main/java/com/roncoo/education/user/service/api/biz/ApiUsersBiz.java

@@ -178,15 +178,14 @@ public class ApiUsersBiz extends BaseBiz {
     private Boolean sendCodeCheck(String mobile) {
         String count = cacheRedis.get(Constants.RedisPre.CODE_STAT + mobile);
         if (StringUtils.hasText(count)) {
-            int countNum = Integer.valueOf(count);
+            int countNum = cacheRedis.set(Constants.RedisPre.CODE_STAT + mobile, Integer.valueOf(count).intValue() + 1, 5, TimeUnit.MINUTES);
             if (countNum < 2) {
-                cacheRedis.set(Constants.RedisPre.CODE_STAT + mobile, countNum++);
                 return Boolean.TRUE;
             }
-        } else {
-            cacheRedis.set(Constants.RedisPre.CODE_STAT + mobile, 1);
+            return Boolean.FALSE;
         }
-        return Boolean.FALSE;
+        cacheRedis.set(Constants.RedisPre.CODE_STAT + mobile, 1, 5, TimeUnit.MINUTES);
+        return Boolean.TRUE;
     }
 
     public Result<String> password(PasswordReq req) {