From 10c218463b376967e07a9c28de565ef741a0a308 Mon Sep 17 00:00:00 2001
From: kaiyu <404897439@qq.com>
Date: Wed, 17 Mar 2021 17:26:24 +0800
Subject: [PATCH] Common模块:     修改用户不存在常量名     修改ResultMessage中使用的常量 manage模块:     账号的增删改查进行修改

---
 screen-manage/src/main/java/com/moral/api/pojo/vo/login/LoginVO.java                 |   46 --
 screen-manage/src/main/java/com/moral/api/pojo/vo/account/AccountDeleteVO.java       |   21 -
 screen-manage/src/main/java/com/moral/api/service/impl/ManageAccountServiceImpl.java |  291 +++++++++++---------
 screen-manage/src/main/java/com/moral/api/pojo/dto/account/AccountDTO.java           |    4 
 screen-manage/src/main/java/com/moral/api/pojo/dto/account/AccountInsertDTO.java     |   42 ---
 screen-manage/src/main/java/com/moral/api/service/ManageAccountService.java          |   14 
 screen-manage/src/main/java/com/moral/api/pojo/form/AccountUpdateForm.java           |    4 
 screen-manage/src/main/java/com/moral/api/pojo/vo/account/AccountVO.java             |   11 
 screen-manage/src/main/java/com/moral/api/pojo/dto/account/AccountDeleteDTO.java     |   10 
 screen-manage/src/main/java/com/moral/api/pojo/form/LogoutForm.java                  |    4 
 screen-manage/src/main/java/com/moral/api/pojo/form/AccountInsertForm.java           |    8 
 screen-manage/src/main/resources/mapper/ManageMenuMapper.xml                         |    1 
 screen-common/src/main/java/com/moral/constant/ResultMessage.java                    |   20 
 screen-manage/src/main/java/com/moral/api/pojo/form/LoginForm.java                   |    8 
 screen-common/src/main/java/com/moral/constant/ResponseCodeEnum.java                 |    2 
 screen-manage/src/main/java/com/moral/api/pojo/vo/login/AccountInfoVO.java           |   43 +-
 screen-manage/src/main/java/com/moral/api/controller/AccountController.java          |   71 ++--
 screen-manage/src/main/java/com/moral/api/pojo/dto/login/LoginDTO.java               |   37 --
 screen-manage/src/main/java/com/moral/api/pojo/vo/account/AccountQueryVO.java        |   15 
 screen-manage/src/main/java/com/moral/api/pojo/dto/account/AccountQueryDTO.java      |    4 
 screen-manage/src/main/java/com/moral/api/pojo/form/AccountDeleteForm.java           |    4 
 screen-manage/src/main/java/com/moral/api/pojo/form/AccountQueryForm.java            |    4 
 screen-manage/src/main/java/com/moral/api/pojo/vo/account/AccountInsertVO.java       |   96 ++----
 screen-manage/src/main/java/com/moral/api/pojo/dto/account/AccountUpdateDTO.java     |   19 -
 screen-manage/src/main/java/com/moral/api/pojo/vo/account/AccountUpdateVO.java       |   60 +--
 25 files changed, 345 insertions(+), 494 deletions(-)

diff --git a/screen-common/src/main/java/com/moral/constant/ResponseCodeEnum.java b/screen-common/src/main/java/com/moral/constant/ResponseCodeEnum.java
index f6c2017..1d4664d 100644
--- a/screen-common/src/main/java/com/moral/constant/ResponseCodeEnum.java
+++ b/screen-common/src/main/java/com/moral/constant/ResponseCodeEnum.java
@@ -15,7 +15,7 @@
     TOKEN_INVALID(-3,"token������"),
     TOKEN_CREATE_ERROR(-4,"token������������"),
     ACCOUNT_NOT_EXIST(-5,"���������������"),
-    ACCOUNT_INVALID(-6,"������������������"),
+    ACCOUNT_EXIST(-6,"������������������"),
     ACCOUNT_IS_DELETE(-7,"���������������������"),
     PASSWORD_INVALID(-8,"������������"),
     MOBILE_INVALID(-9,"���������������"),
diff --git a/screen-common/src/main/java/com/moral/constant/ResultMessage.java b/screen-common/src/main/java/com/moral/constant/ResultMessage.java
index 50b9fa2..2ecfe05 100644
--- a/screen-common/src/main/java/com/moral/constant/ResultMessage.java
+++ b/screen-common/src/main/java/com/moral/constant/ResultMessage.java
@@ -36,15 +36,15 @@
 
     public static ResultMessage ok() {
         ResultMessage resultMessage = new ResultMessage();
-        resultMessage.setCode(Constants.CODE_OPERATION_SUCCESS);
-        resultMessage.setMessage(Constants.MSG_OPERATION_SUCCESS);
+        resultMessage.setCode(ResponseCodeEnum.SUCCESS.getCode());
+        resultMessage.setMessage(ResponseCodeEnum.SUCCESS.getMsg());
         resultMessage.setData(new HashMap<>());
         return resultMessage;
     }
 
     public static ResultMessage ok(String msg) {
         ResultMessage resultMessage = new ResultMessage();
-        resultMessage.setCode(Constants.CODE_OPERATION_SUCCESS);
+        resultMessage.setCode(ResponseCodeEnum.SUCCESS.getCode());
         resultMessage.setMessage(msg);
         resultMessage.setData(new HashMap<>());
         return resultMessage;
@@ -67,8 +67,8 @@
 
     public static <T> ResultMessage<T> ok(T data) {
         ResultMessage resultMessage = new ResultMessage();
-        resultMessage.setCode(Constants.CODE_OPERATION_SUCCESS);
-        resultMessage.setMessage(Constants.MSG_OPERATION_SUCCESS);
+        resultMessage.setCode(ResponseCodeEnum.SUCCESS.getCode());
+        resultMessage.setMessage(ResponseCodeEnum.SUCCESS.getMsg());
         resultMessage.setData(data == null ?new HashMap<>():data);
         return resultMessage;
     }
@@ -83,15 +83,15 @@
 
     public static ResultMessage fail() {
         ResultMessage resultMessage = new ResultMessage();
-        resultMessage.setCode(Constants.CODE_OPERATION_FAILED);
-        resultMessage.setMessage(Constants.MSG_OPERATION_FAILED);
+        resultMessage.setCode(ResponseCodeEnum.FAIL.getCode());
+        resultMessage.setMessage(ResponseCodeEnum.FAIL.getMsg());
         resultMessage.setData(new HashMap<>());
         return resultMessage;
     }
 
     public static ResultMessage fail(String msg) {
         ResultMessage resultMessage = new ResultMessage();
-        resultMessage.setCode(Constants.CODE_OPERATION_FAILED);
+        resultMessage.setCode(ResponseCodeEnum.FAIL.getCode());
         resultMessage.setMessage(msg);
         resultMessage.setData(new HashMap<>());
         return resultMessage;
@@ -107,8 +107,8 @@
 
     public static <T> ResultMessage<T> fail(T data) {
         ResultMessage resultMessage = new ResultMessage();
-        resultMessage.setCode(Constants.CODE_OPERATION_FAILED);
-        resultMessage.setMessage(Constants.MSG_OPERATION_FAILED);
+        resultMessage.setCode(ResponseCodeEnum.FAIL.getCode());
+        resultMessage.setMessage(ResponseCodeEnum.FAIL.getMsg());
         resultMessage.setData(data == null ?new HashMap<>():data);
         return resultMessage;
     }
diff --git a/screen-manage/src/main/java/com/moral/api/controller/AccountController.java b/screen-manage/src/main/java/com/moral/api/controller/AccountController.java
index 9db40fd..c624246 100644
--- a/screen-manage/src/main/java/com/moral/api/controller/AccountController.java
+++ b/screen-manage/src/main/java/com/moral/api/controller/AccountController.java
@@ -1,12 +1,11 @@
 package com.moral.api.controller;
 
-import com.google.code.kaptcha.impl.DefaultKaptcha;
 import com.moral.api.pojo.dto.account.AccountDeleteDTO;
 import com.moral.api.pojo.dto.account.AccountInsertDTO;
 import com.moral.api.pojo.dto.account.AccountQueryDTO;
 import com.moral.api.pojo.dto.account.AccountUpdateDTO;
 import com.moral.api.pojo.dto.login.LoginDTO;
-import com.moral.api.pojo.request.*;
+import com.moral.api.pojo.form.*;
 import com.moral.api.pojo.vo.account.AccountDeleteVO;
 import com.moral.api.pojo.vo.account.AccountInsertVO;
 import com.moral.api.pojo.vo.account.AccountQueryVO;
@@ -14,6 +13,7 @@
 import com.moral.api.pojo.vo.login.LoginVO;
 import com.moral.api.service.ManageAccountService;
 import com.moral.constant.Constants;
+import com.moral.constant.ResponseCodeEnum;
 import com.moral.constant.ResultMessage;
 import com.moral.util.KaptchaUtils;
 import com.moral.util.TokenUtils;
@@ -34,69 +34,64 @@
 @RestController
 @RequestMapping("/account")
 public class AccountController {
-    @Resource
-    ManageAccountService accountService;
     @Autowired
-    @Qualifier("tokenRedisTemplate")
-    RedisTemplate redisTemplate;
+    ManageAccountService accountService;
 
     @PostMapping("login")
-    public ResultMessage login(@RequestBody LoginRequest loginRequest) {
-        if (!loginRequest.valid())
+    public ResultMessage login(@RequestBody LoginForm loginForm) {
+        if (!loginForm.valid())
             return ResultMessage.fail(Constants.CODE_PARAMETER_IS_MISSING,Constants.MSG_PARAMETER_IS_MISSING);
 
-        LoginDTO loginDTO = accountService.login(loginRequest);
+        LoginDTO loginDTO = accountService.login(loginForm);
 
         LoginVO loginVO = LoginVO.convert(loginDTO);
 
-        if(loginVO.getCode().equals(loginVO.SUCCESS))
-            return ResultMessage.ok(loginVO);
-        return ResultMessage.fail(loginVO);
+        return new ResultMessage(loginDTO.getCode(),loginDTO.getMsg(),loginVO);
     }
 
 
     @PostMapping("logout")
-    public ResultMessage logout(@RequestBody LogoutRequest logoutRequest, HttpServletRequest request) {
-        if(!logoutRequest.valid())
-            return ResultMessage.fail(Constants.CODE_PARAMETER_IS_MISSING,Constants.MSG_PARAMETER_IS_MISSING);
+    public ResultMessage logout (@RequestBody LogoutForm logoutForm, HttpServletRequest request) {
+        if(!logoutForm.valid())
+            return ResultMessage.fail(ResponseCodeEnum.PARAMETERS_IS_MISSING.getCode(),
+                    ResponseCodeEnum.PARAMETERS_IS_MISSING.getMsg());
         String token = request.getHeader("token");
-        logoutRequest.setToken(token);
+        logoutForm.setToken(token);
 
-        if(accountService.logout(logoutRequest))
-            return ResultMessage.ok("������������");
-        return ResultMessage.fail("������������");
+        if(accountService.logout(logoutForm))
+            return ResultMessage.ok();
+        return ResultMessage.fail();
 
     }
 
     @PostMapping("insert")
-    public ResultMessage insert(@RequestBody AccountInsertRequest accountInsertRequest){
-        if(!accountInsertRequest.valid())
-            return ResultMessage.fail(Constants.CODE_PARAMETER_IS_MISSING,Constants.MSG_PARAMETER_IS_MISSING);
+    public ResultMessage insert(@RequestBody AccountInsertForm accountInsertForm){
+        if(!accountInsertForm.valid())
+            return ResultMessage.fail(ResponseCodeEnum.PARAMETERS_IS_MISSING.getCode(),
+                    ResponseCodeEnum.PARAMETERS_IS_MISSING.getMsg());
 
-        AccountInsertDTO accountInsertDTO = accountService.insertAccount(accountInsertRequest);
+        AccountInsertDTO accountInsertDTO = accountService.insertAccount(accountInsertForm);
 
         AccountInsertVO accountInsertVO = AccountInsertVO.convert(accountInsertDTO);
 
-        if(accountInsertVO.getCode().equals(accountInsertVO.SUCCESS))
-            return ResultMessage.ok(accountInsertVO);
-        return ResultMessage.fail(accountInsertVO);
+        return new ResultMessage(accountInsertDTO.getCode(),accountInsertDTO.getMsg(),accountInsertVO);
     }
 
     @GetMapping("query")
-    public ResultMessage query(AccountQueryRequest accountQueryRequest){
-        if(!accountQueryRequest.valid())
+    public ResultMessage query(AccountQueryForm accountQueryForm){
+        if(!accountQueryForm.valid())
             return ResultMessage.fail(Constants.CODE_PARAMETER_IS_MISSING,Constants.MSG_PARAMETER_IS_MISSING);
 
-        AccountQueryDTO accountQueryDTO = accountService.queryAccount(accountQueryRequest);
+        AccountQueryDTO accountQueryDTO = accountService.queryAccount(accountQueryForm);
 
         AccountQueryVO accountQueryVO = AccountQueryVO.convert(accountQueryDTO);
 
-        return ResultMessage.ok(accountQueryVO);
+        return new ResultMessage(accountQueryDTO.getCode(),accountQueryDTO.getMsg(),accountQueryVO);
     }
 
 
     @PostMapping("update")
-    public ResultMessage update(@RequestBody AccountUpdateRequest accountQeuryRequest){
+    public ResultMessage update(@RequestBody AccountUpdateForm accountQeuryRequest){
         if(!accountQeuryRequest.valid())
             return ResultMessage.fail(Constants.CODE_PARAMETER_IS_MISSING,Constants.MSG_PARAMETER_IS_MISSING);
 
@@ -104,23 +99,19 @@
 
         AccountUpdateVO accountUpdateVO = AccountUpdateVO.convert(accountUpdateDTO);
 
-        if(accountUpdateVO.getCode().equals(accountUpdateVO.SUCCESS))
-            return ResultMessage.ok(accountUpdateVO);
-        return ResultMessage.fail(accountUpdateVO);
+        return new ResultMessage(accountUpdateDTO.getCode(),accountUpdateDTO.getMsg(),accountUpdateVO);
     }
 
     @PostMapping("delete")
-    public ResultMessage delete(@RequestBody AccountDeleteRequest accountDeleteRequest){
-        if(!accountDeleteRequest.valid())
+    public ResultMessage delete(@RequestBody AccountDeleteForm accountDeleteForm){
+        if(!accountDeleteForm.valid())
             return ResultMessage.fail(Constants.CODE_PARAMETER_IS_MISSING,Constants.MSG_PARAMETER_IS_MISSING);
 
-        AccountDeleteDTO accountDeleteDTO = accountService.deleteAccount(accountDeleteRequest);
+        AccountDeleteDTO accountDeleteDTO = accountService.deleteAccount(accountDeleteForm);
 
         AccountDeleteVO accountDeleteVO = AccountDeleteVO.convert(accountDeleteDTO);
 
-        if(accountDeleteVO.getCode().equals(accountDeleteVO.SUCCESS))
-            return ResultMessage.ok(accountDeleteVO);
-        return ResultMessage.fail(accountDeleteVO);
+        return new ResultMessage(accountDeleteDTO.getCode(),accountDeleteDTO.getMsg(),accountDeleteVO);
     }
 
     @PostMapping("yanzhengma")
diff --git a/screen-manage/src/main/java/com/moral/api/pojo/dto/account/AccountDTO.java b/screen-manage/src/main/java/com/moral/api/pojo/dto/account/AccountDTO.java
index 2a40541..5f17674 100644
--- a/screen-manage/src/main/java/com/moral/api/pojo/dto/account/AccountDTO.java
+++ b/screen-manage/src/main/java/com/moral/api/pojo/dto/account/AccountDTO.java
@@ -20,5 +20,7 @@
 
     private List<ManageRole> roles;
 
-    private String code;
+    private Integer code;
+
+    private String msg;
 }
diff --git a/screen-manage/src/main/java/com/moral/api/pojo/dto/account/AccountDeleteDTO.java b/screen-manage/src/main/java/com/moral/api/pojo/dto/account/AccountDeleteDTO.java
index 926319c..ecab6ed 100644
--- a/screen-manage/src/main/java/com/moral/api/pojo/dto/account/AccountDeleteDTO.java
+++ b/screen-manage/src/main/java/com/moral/api/pojo/dto/account/AccountDeleteDTO.java
@@ -12,15 +12,5 @@
 @Data
 public class AccountDeleteDTO extends AccountDTO {
 
-    /*
-    * ������������
-    * */
-    public final static String SUCCESS = "0";
-
-    /*
-    * ���������������
-    * */
-    public final static String ACCOUNT_NOT_EXIST = "-1";
-
 
 }
diff --git a/screen-manage/src/main/java/com/moral/api/pojo/dto/account/AccountInsertDTO.java b/screen-manage/src/main/java/com/moral/api/pojo/dto/account/AccountInsertDTO.java
index 3d4b9d3..8f199bb 100644
--- a/screen-manage/src/main/java/com/moral/api/pojo/dto/account/AccountInsertDTO.java
+++ b/screen-manage/src/main/java/com/moral/api/pojo/dto/account/AccountInsertDTO.java
@@ -16,46 +16,4 @@
 @Data
 public class AccountInsertDTO extends AccountDTO{
 
-    /*
-     * ������������
-     * */
-    public static final String SUCCESS = "0";
-
-    /*
-     * ���������������������������
-     * */
-    public static final String ACCOUNT_LENGTH_INVALID = "-1";
-
-    /*
-     * ���������������������������
-     * */
-    public static final String PASSWORD_LENGTH_INVALID = "-2";
-
-    /*
-     * ������������������������
-     * */
-    public static final String MOBILE_INVALID = "-3";
-
-    /*
-     * ���������������������
-     * */
-    public static final String EMAIL_INVALID = "-4";
-
-    /*
-     * ���������������������
-     * */
-    public static final String ACCOUNT_EXIST = "-5";
-
-    /*
-    * ���������������������������
-    * */
-    public static final String ACCOUNT_EXIST_SPECIAL_CHAR = "-6";
-
-    /*
-    * ������������������������
-    * */
-    public static final String PASSWORD_EXIST_SPECIAL_CHAR = "-7";
-
-
-
 }
diff --git a/screen-manage/src/main/java/com/moral/api/pojo/dto/account/AccountQueryDTO.java b/screen-manage/src/main/java/com/moral/api/pojo/dto/account/AccountQueryDTO.java
index 26e2ef6..46f3888 100644
--- a/screen-manage/src/main/java/com/moral/api/pojo/dto/account/AccountQueryDTO.java
+++ b/screen-manage/src/main/java/com/moral/api/pojo/dto/account/AccountQueryDTO.java
@@ -14,6 +14,10 @@
 @Data
 public class AccountQueryDTO  {
 
+    private Integer code;
+
+    private String msg;
+
     private long pages;
 
     private long total;
diff --git a/screen-manage/src/main/java/com/moral/api/pojo/dto/account/AccountUpdateDTO.java b/screen-manage/src/main/java/com/moral/api/pojo/dto/account/AccountUpdateDTO.java
index b15c23d..b75060d 100644
--- a/screen-manage/src/main/java/com/moral/api/pojo/dto/account/AccountUpdateDTO.java
+++ b/screen-manage/src/main/java/com/moral/api/pojo/dto/account/AccountUpdateDTO.java
@@ -11,24 +11,5 @@
  **/
 @Data
 public class AccountUpdateDTO extends AccountDTO {
-    /*
-    * ������������
-    * */
-    public static final String SUCCESS = "0";
-
-    /*
-    * ���������������
-    * */
-    public static final String ACCOUNT_NOT_EXIST = "-1";
-
-    /*
-    * ������������������������
-    * */
-    public static final String MOBILE_INVALID = "-2";
-
-    /*
-    * ���������������������
-    * */
-    public static final String EMAIL_INVALID = "-3";
 
 }
diff --git a/screen-manage/src/main/java/com/moral/api/pojo/dto/login/LoginDTO.java b/screen-manage/src/main/java/com/moral/api/pojo/dto/login/LoginDTO.java
index 0701d68..6fdca56 100644
--- a/screen-manage/src/main/java/com/moral/api/pojo/dto/login/LoginDTO.java
+++ b/screen-manage/src/main/java/com/moral/api/pojo/dto/login/LoginDTO.java
@@ -12,40 +12,15 @@
 @Data
 public class LoginDTO {
 
-    /*
-    * ������������
-    * */
-    public static final String SUCCESS = "0";
-
-    /*
-    * ���������������
-    * */
-    public static final String NOT_EXIST = "-1";
-
-    /*
-    * ������������������
-    * */
-    public static final String IS_DELETE = "-2";
-
-   /*
-   * ������������
-   * */
-   public static final String PASSWORD_ERROR = "-3";
-
-   /*
-   * ������������������
-   * */
-   public static final String ROLE_EMPTY = "-4";
-
-   /*
-   * ������������������
-   * */
-   public static final String MENU_EMPTY = "-5";
-
    /*
    * ���������
    * */
-    private String code;
+    private Integer code;
+
+    /*
+    * ������������
+    * */
+    private String msg;
 
     /*
     *������token
diff --git a/screen-manage/src/main/java/com/moral/api/pojo/request/AccountDeleteRequest.java b/screen-manage/src/main/java/com/moral/api/pojo/form/AccountDeleteForm.java
similarity index 81%
rename from screen-manage/src/main/java/com/moral/api/pojo/request/AccountDeleteRequest.java
rename to screen-manage/src/main/java/com/moral/api/pojo/form/AccountDeleteForm.java
index 426717a..b70764c 100644
--- a/screen-manage/src/main/java/com/moral/api/pojo/request/AccountDeleteRequest.java
+++ b/screen-manage/src/main/java/com/moral/api/pojo/form/AccountDeleteForm.java
@@ -1,4 +1,4 @@
-package com.moral.api.pojo.request;
+package com.moral.api.pojo.form;
 
 import lombok.Data;
 
@@ -10,7 +10,7 @@
  * @Version TODO
  **/
 @Data
-public class AccountDeleteRequest {
+public class AccountDeleteForm {
 
     private Integer accountId;
 
diff --git a/screen-manage/src/main/java/com/moral/api/pojo/request/AccountInsertRequest.java b/screen-manage/src/main/java/com/moral/api/pojo/form/AccountInsertForm.java
similarity index 78%
rename from screen-manage/src/main/java/com/moral/api/pojo/request/AccountInsertRequest.java
rename to screen-manage/src/main/java/com/moral/api/pojo/form/AccountInsertForm.java
index d070492..d4f1f96 100644
--- a/screen-manage/src/main/java/com/moral/api/pojo/request/AccountInsertRequest.java
+++ b/screen-manage/src/main/java/com/moral/api/pojo/form/AccountInsertForm.java
@@ -1,6 +1,5 @@
-package com.moral.api.pojo.request;
+package com.moral.api.pojo.form;
 
-import com.moral.api.entity.ManageRole;
 import lombok.Data;
 import org.springframework.util.ObjectUtils;
 
@@ -15,7 +14,7 @@
  * @Version TODO
  **/
 @Data
-public class AccountInsertRequest implements Serializable{
+public class AccountInsertForm implements Serializable{
 
 
     private String account;
@@ -50,8 +49,7 @@
                 ObjectUtils.isEmpty(password) ||
                 ObjectUtils.isEmpty(email)    ||
                 ObjectUtils.isEmpty(mobile)   ||
-                ObjectUtils.isEmpty(userName) ||
-                ObjectUtils.isEmpty(roleIds)
+                ObjectUtils.isEmpty(userName)
             )
             return false;
         return true;
diff --git a/screen-manage/src/main/java/com/moral/api/pojo/request/AccountQueryRequest.java b/screen-manage/src/main/java/com/moral/api/pojo/form/AccountQueryForm.java
similarity index 87%
rename from screen-manage/src/main/java/com/moral/api/pojo/request/AccountQueryRequest.java
rename to screen-manage/src/main/java/com/moral/api/pojo/form/AccountQueryForm.java
index 1a8fb2f..ce49cb5 100644
--- a/screen-manage/src/main/java/com/moral/api/pojo/request/AccountQueryRequest.java
+++ b/screen-manage/src/main/java/com/moral/api/pojo/form/AccountQueryForm.java
@@ -1,4 +1,4 @@
-package com.moral.api.pojo.request;
+package com.moral.api.pojo.form;
 
 import lombok.Data;
 import org.springframework.util.ObjectUtils;
@@ -13,7 +13,7 @@
  * @Version TODO
  **/
 @Data
-public class AccountQueryRequest implements Serializable {
+public class AccountQueryForm implements Serializable {
 
     private Integer page;
 
diff --git a/screen-manage/src/main/java/com/moral/api/pojo/request/AccountUpdateRequest.java b/screen-manage/src/main/java/com/moral/api/pojo/form/AccountUpdateForm.java
similarity index 90%
rename from screen-manage/src/main/java/com/moral/api/pojo/request/AccountUpdateRequest.java
rename to screen-manage/src/main/java/com/moral/api/pojo/form/AccountUpdateForm.java
index 6745a7a..60060ff 100644
--- a/screen-manage/src/main/java/com/moral/api/pojo/request/AccountUpdateRequest.java
+++ b/screen-manage/src/main/java/com/moral/api/pojo/form/AccountUpdateForm.java
@@ -1,4 +1,4 @@
-package com.moral.api.pojo.request;
+package com.moral.api.pojo.form;
 
 import lombok.Data;
 import org.springframework.util.ObjectUtils;
@@ -13,7 +13,7 @@
  * @Version TODO
  **/
 @Data
-public class AccountUpdateRequest {
+public class AccountUpdateForm {
 
     private Integer accountId;
 
diff --git a/screen-manage/src/main/java/com/moral/api/pojo/request/LoginRequest.java b/screen-manage/src/main/java/com/moral/api/pojo/form/LoginForm.java
similarity index 64%
rename from screen-manage/src/main/java/com/moral/api/pojo/request/LoginRequest.java
rename to screen-manage/src/main/java/com/moral/api/pojo/form/LoginForm.java
index 4f60a54..7bab60f 100644
--- a/screen-manage/src/main/java/com/moral/api/pojo/request/LoginRequest.java
+++ b/screen-manage/src/main/java/com/moral/api/pojo/form/LoginForm.java
@@ -1,4 +1,4 @@
-package com.moral.api.pojo.request;
+package com.moral.api.pojo.form;
 
 import lombok.Data;
 import org.springframework.util.ObjectUtils;
@@ -13,15 +13,15 @@
  * @Version TODO
  **/
 @Data
-public class LoginRequest implements Serializable {
+public class LoginForm implements Serializable {
 
     private String account;
 
     private String password;
 
     public boolean valid(){
-        if(ObjectUtils.isEmpty(account)||ObjectUtils.isEmpty(password))
-            return false;
+            if(ObjectUtils.isEmpty(account)||ObjectUtils.isEmpty(password))
+                return false;
         return true;
     }
 }
diff --git a/screen-manage/src/main/java/com/moral/api/pojo/request/LogoutRequest.java b/screen-manage/src/main/java/com/moral/api/pojo/form/LogoutForm.java
similarity index 81%
rename from screen-manage/src/main/java/com/moral/api/pojo/request/LogoutRequest.java
rename to screen-manage/src/main/java/com/moral/api/pojo/form/LogoutForm.java
index c559c40..8847656 100644
--- a/screen-manage/src/main/java/com/moral/api/pojo/request/LogoutRequest.java
+++ b/screen-manage/src/main/java/com/moral/api/pojo/form/LogoutForm.java
@@ -1,4 +1,4 @@
-package com.moral.api.pojo.request;
+package com.moral.api.pojo.form;
 
 import lombok.Data;
 import org.springframework.util.ObjectUtils;
@@ -13,7 +13,7 @@
  * @Version TODO
  **/
 @Data
-public class LogoutRequest implements Serializable {
+public class LogoutForm implements Serializable {
 
     private String accountId;
 
diff --git a/screen-manage/src/main/java/com/moral/api/pojo/vo/account/AccountDeleteVO.java b/screen-manage/src/main/java/com/moral/api/pojo/vo/account/AccountDeleteVO.java
index f06ea46..ef0b15e 100644
--- a/screen-manage/src/main/java/com/moral/api/pojo/vo/account/AccountDeleteVO.java
+++ b/screen-manage/src/main/java/com/moral/api/pojo/vo/account/AccountDeleteVO.java
@@ -3,6 +3,7 @@
 import com.fasterxml.jackson.annotation.JsonInclude;
 import com.moral.api.entity.ManageAccount;
 import com.moral.api.pojo.dto.account.AccountDeleteDTO;
+import com.moral.constant.ResponseCodeEnum;
 import lombok.Data;
 import org.springframework.util.ObjectUtils;
 
@@ -18,24 +19,14 @@
 
 public class AccountDeleteVO extends AccountVO {
 
-    /*
-     * ������������
-     * */
-    public final static String SUCCESS = "0";
-
-    /*
-     * ���������������
-     * */
-    public final static String ACCOUNT_NOT_EXIST = "-1";
 
     public static AccountDeleteVO convert(AccountDeleteDTO dto) {
+        if (dto.getCode() != ResponseCodeEnum.SUCCESS.getCode())
+            return null;
         AccountDeleteVO vo = new AccountDeleteVO();
-        vo.setCode(dto.getCode());
-        if (!ObjectUtils.isEmpty(dto.getAccount())) {
-            ManageAccount account = dto.getAccount();
-            vo.setId(account.getId());
-            vo.setUserName(account.getUserName());
-        }
+        ManageAccount account = dto.getAccount();
+        vo.setId(account.getId());
+        vo.setUserName(account.getUserName());
         return vo;
     }
 }
diff --git a/screen-manage/src/main/java/com/moral/api/pojo/vo/account/AccountInsertVO.java b/screen-manage/src/main/java/com/moral/api/pojo/vo/account/AccountInsertVO.java
index 8cc4d31..7c4e4cf 100644
--- a/screen-manage/src/main/java/com/moral/api/pojo/vo/account/AccountInsertVO.java
+++ b/screen-manage/src/main/java/com/moral/api/pojo/vo/account/AccountInsertVO.java
@@ -3,7 +3,9 @@
 import com.fasterxml.jackson.annotation.JsonInclude;
 import com.moral.api.entity.ManageRole;
 import com.moral.api.pojo.dto.account.AccountInsertDTO;
+import com.moral.constant.ResponseCodeEnum;
 import lombok.Data;
+import org.springframework.util.ObjectUtils;
 
 import java.util.ArrayList;
 import java.util.List;
@@ -18,79 +20,37 @@
 @Data
 @JsonInclude(JsonInclude.Include.NON_EMPTY)
 public class AccountInsertVO extends AccountVO {
-    /*
-     * ������������
-     * */
-    public static final String SUCCESS = "0";
-
-    /*
-     * ���������������������������
-     * */
-    public static final String ACCOUNT_LENGTH_INVALID = "-1";
-
-    /*
-     * ���������������������������
-     * */
-    public static final String PASSWORD_LENGTH_INVALID = "-2";
-
-    /*
-     * ������������������������
-     * */
-    public static final String MOBILE_INVALID = "-3";
-
-    /*
-     * ���������������������
-     * */
-    public static final String EMAIL_INVALID = "-4";
-
-    /*
-     * ���������������������
-     * */
-    public static final String ACCOUNT_EXIST = "-5";
-
-    /*
-     * ���������������������������
-     * */
-    public static final String ACCOUNT_EXIST_SPECIAL_CHAR = "-6";
-
-    /*
-     * ������������������������
-     * */
-    public static final String PASSWORD_EXIST_SPECIAL_CHAR = "-7";
-
-
 
     /**
-    * @Description: ������������������������������������
-            * @Param: [dto]
-            * @return: com.moral.api.pojo.vo.account.AccountVO
-            * @Author: ���������
-            * @Date: 2021/3/15
-            */
-    public static AccountInsertVO convert(AccountInsertDTO dto){
+     * @Description: ������������������������������������
+     * @Param: [dto]
+     * @return: com.moral.api.pojo.vo.account.AccountVO
+     * @Author: ���������
+     * @Date: 2021/3/15
+     */
+    public static AccountInsertVO convert(AccountInsertDTO dto) {
+        if (dto.getCode() != ResponseCodeEnum.SUCCESS.getCode())
+            return null;
+
         AccountInsertVO vo = new AccountInsertVO();
-        if(dto.getCode()== AccountInsertDTO.SUCCESS){
-            Integer id = dto.getAccount().getId();
-            String userName = dto.getAccount().getUserName();
-            String email = dto.getAccount().getEmail();
-            String mobile = dto.getAccount().getMobile();
-            String wechat = dto.getAccount().getWechat();
-
-            //������������������������
-            List<ManageRole> roles = dto.getRoles();
-            List<String> roleNames = new ArrayList<>();
-            roles.forEach(role->roleNames.add(role.getName()));
-
-            vo.setId(id);
-            vo.setUserName(userName);
-            vo.setEmail(email);
-            vo.setMobile(mobile);
-            vo.setWechat(wechat);
-            vo.setRoleNames(roleNames);
+        Integer id = dto.getAccount().getId();
+        String userName = dto.getAccount().getUserName();
+        String email = dto.getAccount().getEmail();
+        String mobile = dto.getAccount().getMobile();
+        String wechat = dto.getAccount().getWechat();
+        //������������������������
+        List<ManageRole> roles = dto.getRoles();
+        List<String> roleNames = new ArrayList<>();
+        if (!ObjectUtils.isEmpty(roles)) {
+            roles.forEach(role -> roleNames.add(role.getName()));
         }
+        vo.setId(id);
+        vo.setUserName(userName);
+        vo.setEmail(email);
+        vo.setMobile(mobile);
+        vo.setWechat(wechat);
+        vo.setRoleNames(roleNames);
 
-        String code = dto.getCode();
-        vo.setCode(code);
         return vo;
     }
 
diff --git a/screen-manage/src/main/java/com/moral/api/pojo/vo/account/AccountQueryVO.java b/screen-manage/src/main/java/com/moral/api/pojo/vo/account/AccountQueryVO.java
index 16462d3..77b190b 100644
--- a/screen-manage/src/main/java/com/moral/api/pojo/vo/account/AccountQueryVO.java
+++ b/screen-manage/src/main/java/com/moral/api/pojo/vo/account/AccountQueryVO.java
@@ -3,7 +3,9 @@
 import com.fasterxml.jackson.annotation.JsonInclude;
 import com.moral.api.pojo.dto.account.AccountDTO;
 import com.moral.api.pojo.dto.account.AccountQueryDTO;
+import com.moral.constant.ResponseCodeEnum;
 import lombok.Data;
+import org.springframework.util.ObjectUtils;
 
 import java.util.ArrayList;
 import java.util.List;
@@ -29,16 +31,21 @@
 
     private List<AccountVO> accountVOs;
 
-    public static AccountQueryVO convert(AccountQueryDTO dto){
+    public static AccountQueryVO convert(AccountQueryDTO dto) {
+        if (dto.getCode() != ResponseCodeEnum.SUCCESS.getCode())
+            return null;
+
         long total = dto.getTotal();
         long current = dto.getCurrent();
         long pages = dto.getPages();
         long size = dto.getSize();
         List<AccountVO> accountVOs = new ArrayList<>();
         List<AccountDTO> accountDTOs = dto.getAccountDTOS();
-        for (AccountDTO accountDTO : accountDTOs) {
-            AccountVO vo = AccountVO.convert(accountDTO);
-            accountVOs.add(vo);
+        if (!ObjectUtils.isEmpty(accountDTOs)) {
+            for (AccountDTO accountDTO : accountDTOs) {
+                AccountVO vo = AccountVO.convert(accountDTO);
+                accountVOs.add(vo);
+            }
         }
         AccountQueryVO vo = new AccountQueryVO();
         vo.setTotal(total);
diff --git a/screen-manage/src/main/java/com/moral/api/pojo/vo/account/AccountUpdateVO.java b/screen-manage/src/main/java/com/moral/api/pojo/vo/account/AccountUpdateVO.java
index cfbf96f..a99e42d 100644
--- a/screen-manage/src/main/java/com/moral/api/pojo/vo/account/AccountUpdateVO.java
+++ b/screen-manage/src/main/java/com/moral/api/pojo/vo/account/AccountUpdateVO.java
@@ -4,7 +4,9 @@
 import com.moral.api.entity.ManageRole;
 import com.moral.api.pojo.dto.account.AccountInsertDTO;
 import com.moral.api.pojo.dto.account.AccountUpdateDTO;
+import com.moral.constant.ResponseCodeEnum;
 import lombok.Data;
+import org.springframework.util.ObjectUtils;
 
 import java.util.ArrayList;
 import java.util.List;
@@ -19,51 +21,33 @@
 @Data
 @JsonInclude(JsonInclude.Include.NON_EMPTY)
 
-public class AccountUpdateVO extends AccountVO{
-    /*
-     * ������������
-     * */
-    public static final String SUCCESS = "0";
+public class AccountUpdateVO extends AccountVO {
 
-    /*
-     * ���������������
-     * */
-    public static final String ACCOUNT_NOT_EXIST = "-1";
+    public static AccountUpdateVO convert(AccountUpdateDTO dto) {
+        if (dto.getCode() != ResponseCodeEnum.SUCCESS.getCode())
+            return null;
 
-    /*
-     * ������������������������
-     * */
-    public static final String MOBILE_INVALID = "-2";
-
-    /*
-     * ���������������������
-     * */
-    public static final String EMAIL_INVALID = "-3";
-
-    public static AccountUpdateVO convert(AccountUpdateDTO dto){
         AccountUpdateVO vo = new AccountUpdateVO();
-        if(dto.getCode()== AccountUpdateDTO.SUCCESS){
-            Integer id = dto.getAccount().getId();
-            String userName = dto.getAccount().getUserName();
-            String email = dto.getAccount().getEmail();
-            String mobile = dto.getAccount().getMobile();
-            String wechat = dto.getAccount().getWechat();
 
-            //������������������������
-            List<ManageRole> roles = dto.getRoles();
-            List<String> roleNames = new ArrayList<>();
-            roles.forEach(role->roleNames.add(role.getName()));
+        Integer id = dto.getAccount().getId();
+        String userName = dto.getAccount().getUserName();
+        String email = dto.getAccount().getEmail();
+        String mobile = dto.getAccount().getMobile();
+        String wechat = dto.getAccount().getWechat();
 
-            vo.setId(id);
-            vo.setUserName(userName);
-            vo.setEmail(email);
-            vo.setMobile(mobile);
-            vo.setWechat(wechat);
-            vo.setRoleNames(roleNames);
+        //������������������������
+        List<ManageRole> roles = dto.getRoles();
+        List<String> roleNames = new ArrayList<>();
+        if (!ObjectUtils.isEmpty(roles)) {
+            roles.forEach(role -> roleNames.add(role.getName()));
         }
+        vo.setId(id);
+        vo.setUserName(userName);
+        vo.setEmail(email);
+        vo.setMobile(mobile);
+        vo.setWechat(wechat);
+        vo.setRoleNames(roleNames);
 
-        String code = dto.getCode();
-        vo.setCode(code);
         return vo;
     }
 }
diff --git a/screen-manage/src/main/java/com/moral/api/pojo/vo/account/AccountVO.java b/screen-manage/src/main/java/com/moral/api/pojo/vo/account/AccountVO.java
index 155685e..3f6463b 100644
--- a/screen-manage/src/main/java/com/moral/api/pojo/vo/account/AccountVO.java
+++ b/screen-manage/src/main/java/com/moral/api/pojo/vo/account/AccountVO.java
@@ -5,6 +5,7 @@
 import com.moral.api.entity.ManageRole;
 import com.moral.api.pojo.dto.account.AccountDTO;
 import lombok.Data;
+import org.springframework.util.ObjectUtils;
 
 import java.util.ArrayList;
 import java.util.List;
@@ -20,8 +21,6 @@
 @JsonInclude(JsonInclude.Include.NON_EMPTY)
 public class AccountVO {
 
-    private String code;
-
     private Integer id;
 
     private String userName;
@@ -34,14 +33,16 @@
 
     private List<String> roleNames;
 
-    public static AccountVO convert(AccountDTO dto){
-        if(dto.getAccount()==null)
+    public static AccountVO convert(AccountDTO dto) {
+        if (dto.getAccount() == null)
             return null;
         AccountVO vo = new AccountVO();
         ManageAccount account = dto.getAccount();
         List<ManageRole> roles = dto.getRoles();
         List<String> roleNames = new ArrayList<>();
-        roles.forEach(role->roleNames.add(role.getName()));
+        if (!ObjectUtils.isEmpty(roles)) {
+            roles.forEach(role -> roleNames.add(role.getName()));
+        }
         vo.setId(account.getId());
         vo.setUserName(account.getUserName());
         vo.setEmail(account.getEmail());
diff --git a/screen-manage/src/main/java/com/moral/api/pojo/vo/login/AccountInfoVO.java b/screen-manage/src/main/java/com/moral/api/pojo/vo/login/AccountInfoVO.java
index 4edf55a..dbd42ca 100644
--- a/screen-manage/src/main/java/com/moral/api/pojo/vo/login/AccountInfoVO.java
+++ b/screen-manage/src/main/java/com/moral/api/pojo/vo/login/AccountInfoVO.java
@@ -6,6 +6,7 @@
 import com.moral.api.entity.ManageRole;
 import com.moral.api.pojo.dto.login.AccountInfoDTO;
 import lombok.Data;
+import org.springframework.util.ObjectUtils;
 
 import java.util.ArrayList;
 import java.util.List;
@@ -42,32 +43,36 @@
     private List<ManageMenu> menus;
 
     /**
-    * @Description: DTO������VO
-            * @Param: [dto]
-            * @return: com.moral.api.pojo.vo.login.AccountInfoVO
-            * @Author: ���������
-            * @Date: 2021/3/13
-            */
-    public static AccountInfoVO convert(AccountInfoDTO dto){
+     * @Description: DTO������VO
+     * @Param: [dto]
+     * @return: com.moral.api.pojo.vo.login.AccountInfoVO
+     * @Author: ���������
+     * @Date: 2021/3/13
+     */
+    public static AccountInfoVO convert(AccountInfoDTO dto) {
         AccountInfoVO vo = new AccountInfoVO();
         ManageAccount account = dto.getAccount();
         List<ManageMenu> menus = dto.getMenus();
         List<ManageRole> roles = dto.getRoles();
         //������roleNames
         ArrayList<String> roleNames = new ArrayList<>();
-        roles.forEach(role->roleNames.add(role.getName()));
+        if (!ObjectUtils.isEmpty(roles)) {
+            roles.forEach(role -> roleNames.add(role.getName()));
+        }
         /*������menu������������*/
-        for (ManageMenu menu : menus) {
-            menu.setCreateTime(null);
-            menu.setIsDelete(null);
-            menu.setUpdateTime(null);
-            menu.setParentId(null);
-            List<ManageMenu> children = menu.getChildren();
-            for (ManageMenu child : children) {
-                child.setCreateTime(null);
-                child.setIsDelete(null);
-                child.setUpdateTime(null);
-                child.setParentId(null);
+        if (!ObjectUtils.isEmpty(menus)) {
+            for (ManageMenu menu : menus) {
+                menu.setCreateTime(null);
+                menu.setIsDelete(null);
+                menu.setUpdateTime(null);
+                menu.setParentId(null);
+                List<ManageMenu> children = menu.getChildren();
+                for (ManageMenu child : children) {
+                    child.setCreateTime(null);
+                    child.setIsDelete(null);
+                    child.setUpdateTime(null);
+                    child.setParentId(null);
+                }
             }
         }
         vo.setAccountId(account.getId());
diff --git a/screen-manage/src/main/java/com/moral/api/pojo/vo/login/LoginVO.java b/screen-manage/src/main/java/com/moral/api/pojo/vo/login/LoginVO.java
index acc760f..52878a0 100644
--- a/screen-manage/src/main/java/com/moral/api/pojo/vo/login/LoginVO.java
+++ b/screen-manage/src/main/java/com/moral/api/pojo/vo/login/LoginVO.java
@@ -3,6 +3,7 @@
 import com.fasterxml.jackson.annotation.JsonInclude;
 import com.moral.api.pojo.dto.login.AccountInfoDTO;
 import com.moral.api.pojo.dto.login.LoginDTO;
+import com.moral.constant.ResponseCodeEnum;
 import lombok.Data;
 import org.springframework.util.ObjectUtils;
 
@@ -17,40 +18,6 @@
 @Data
 @JsonInclude(JsonInclude.Include.NON_EMPTY)
 public class LoginVO {
-    /*
-     * ������������
-     * */
-    public static final String SUCCESS = "0";
-
-    /*
-     * ���������������
-     * */
-    public static final String NOT_EXIST = "-1";
-
-    /*
-     * ������������������
-     * */
-    public static final String IS_DELETE = "-2";
-
-    /*
-     * ������������
-     * */
-    public static final String PASSWORD_ERROR = "-3";
-
-    /*
-     * ������������������
-     * */
-    public static final String ROLE_EMPTY = "-4";
-
-    /*
-     * ������������������
-     * */
-    public static final String MENU_EMPTY = "-5";
-
-    /*
-     * ���������
-     * */
-    private String code;
 
     /*
      *������token
@@ -70,16 +37,15 @@
      * @Date: 2021/3/13
      */
     public static LoginVO convert(LoginDTO dto) {
+        if (dto.getCode() != ResponseCodeEnum.SUCCESS.getCode())
+            return null;
+
         LoginVO vo = new LoginVO();
-        String code = dto.getCode();
         String token = dto.getToken();
         AccountInfoDTO accountInfoDTO = dto.getAccountInfoDTO();
-        if (!ObjectUtils.isEmpty(accountInfoDTO)) {
-            AccountInfoVO accountInfoVO = AccountInfoVO.convert(accountInfoDTO);
-            vo.setAccountInfoVO(accountInfoVO);
-        }
+        AccountInfoVO accountInfoVO = AccountInfoVO.convert(accountInfoDTO);
+        vo.setAccountInfoVO(accountInfoVO);
         vo.setToken(token);
-        vo.setCode(code);
         return vo;
     }
 }
diff --git a/screen-manage/src/main/java/com/moral/api/service/ManageAccountService.java b/screen-manage/src/main/java/com/moral/api/service/ManageAccountService.java
index 12bb024..3f1aa80 100644
--- a/screen-manage/src/main/java/com/moral/api/service/ManageAccountService.java
+++ b/screen-manage/src/main/java/com/moral/api/service/ManageAccountService.java
@@ -4,7 +4,7 @@
 import com.baomidou.mybatisplus.extension.service.IService;
 import com.moral.api.pojo.dto.account.*;
 import com.moral.api.pojo.dto.login.LoginDTO;
-import com.moral.api.pojo.request.*;
+import com.moral.api.pojo.form.*;
 
 /**
  * <p>
@@ -22,7 +22,7 @@
             * @Author: ���������
             * @Date: 2021/3/12
             */
-    LoginDTO login(LoginRequest loginRequest);
+    LoginDTO login(LoginForm loginForm);
 
    /**
    * @Description: ������
@@ -31,7 +31,7 @@
            * @Author: ���������
            * @Date: 2021/3/12
            */
-   boolean logout(LogoutRequest logoutRequest);
+   boolean logout(LogoutForm logoutForm);
 
    /**
    * @Description: ������������������
@@ -40,7 +40,7 @@
            * @Author: ���������
            * @Date: 2021/3/13
            */
-    AccountInsertDTO insertAccount(AccountInsertRequest accountInsertRequest);
+    AccountInsertDTO insertAccount(AccountInsertForm accountInsertForm);
 
     /**
     * @Description: ������������
@@ -49,7 +49,7 @@
             * @Author: ���������
             * @Date: 2021/3/15
             */
-    AccountQueryDTO queryAccount(AccountQueryRequest accountQueryRequest);
+    AccountQueryDTO queryAccount(AccountQueryForm accountQueryForm);
 
     /**
     * @Description: ������������
@@ -58,7 +58,7 @@
             * @Author: ���������
             * @Date: 2021/3/15
             */
-    AccountUpdateDTO updateAccount(AccountUpdateRequest accountUpdateRequest);
+    AccountUpdateDTO updateAccount(AccountUpdateForm accountUpdateForm);
 
     /**
     * @Description: ������������
@@ -67,5 +67,5 @@
             * @Author: ���������
             * @Date: 2021/3/16
             */
-    AccountDeleteDTO deleteAccount(AccountDeleteRequest accountDeleteRequest);
+    AccountDeleteDTO deleteAccount(AccountDeleteForm accountDeleteForm);
 }
diff --git a/screen-manage/src/main/java/com/moral/api/service/impl/ManageAccountServiceImpl.java b/screen-manage/src/main/java/com/moral/api/service/impl/ManageAccountServiceImpl.java
index 41b198b..d5f40cb 100644
--- a/screen-manage/src/main/java/com/moral/api/service/impl/ManageAccountServiceImpl.java
+++ b/screen-manage/src/main/java/com/moral/api/service/impl/ManageAccountServiceImpl.java
@@ -14,12 +14,14 @@
 import com.moral.api.pojo.dto.account.*;
 import com.moral.api.pojo.dto.login.AccountInfoDTO;
 import com.moral.api.pojo.dto.login.LoginDTO;
-import com.moral.api.pojo.request.*;
+import com.moral.api.pojo.form.*;
 import com.moral.api.service.ManageAccountService;
 import com.baomidou.mybatisplus.extension.service.impl.ServiceImpl;
+import com.moral.constant.ResponseCodeEnum;
 import com.moral.util.AESUtils;
 import com.moral.util.MD5Utils;
 import com.moral.util.TokenUtils;
+import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.beans.factory.annotation.Value;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
@@ -43,13 +45,13 @@
 
     @Value("${AES.KEY}")
     private String AESKey;
-    @Resource
+    @Autowired
     ManageAccountMapper manageAccountMapper;
-    @Resource
+    @Autowired
     ManageRoleMapper manageRoleMapper;
-    @Resource
+    @Autowired
     ManageMenuMapper manageMenuMapper;
-    @Resource
+    @Autowired
     ManageAccountRoleMapper manageAccountRoleMapper;
 
     public final static String specialCharRegEx = "[ _`~!@#$%^&*()+=|{}':;',\\[\\].<>/?~���@#���%������&*������������+|{}������������������������������������]|\n|\r|\t";
@@ -61,55 +63,53 @@
     /**
      * @Description: ������
      * @Param: [parameters]
-     * @return: java.util.Map<java.lang.String               ,               java.lang.Object>
+     * @return: java.util.Map<java.lang.String                                                                                                                                                                                                                                                               ,                                                                                                                                                                                                                                                               java.lang.Object>
      * @Author: ���������
      * @Date: 2021/3/11
      */
     @Override
-    public LoginDTO login(LoginRequest loginRequest) {
+    public LoginDTO login(LoginForm loginForm) {
         LoginDTO loginDTO = new LoginDTO();
         //������
-        String AESAccount = loginRequest.getAccount();
-        String AESPassword = loginRequest.getPassword();
+        String account = loginForm.getAccount();
+        String AESPassword = loginForm.getPassword();
         //������
-        String account = AESUtils.decrypt(AESAccount, AESKey);
         String password = AESUtils.decrypt(AESPassword, AESKey);
         //������������������
         QueryWrapper<ManageAccount> wrapper = new QueryWrapper<>();
         wrapper.eq("account", account);
         List<ManageAccount> manageAccounts = manageAccountMapper.selectList(wrapper);
         if (ObjectUtils.isEmpty(manageAccounts)) {
-            loginDTO.setCode(LoginDTO.NOT_EXIST);
+            loginDTO.setCode(ResponseCodeEnum.ACCOUNT_NOT_EXIST.getCode());
+            loginDTO.setMsg(ResponseCodeEnum.ACCOUNT_NOT_EXIST.getMsg());
             return loginDTO;
         }
         //������������������������
         ManageAccount manageAccount = null;
         for (ManageAccount value : manageAccounts) {
-            if("0".equals(value.getIsDelete()))
+            if ("0".equals(value.getIsDelete()))
                 manageAccount = value;
         }
 
         if (ObjectUtils.isEmpty(manageAccount)) {
-            loginDTO.setCode(LoginDTO.IS_DELETE);
+            loginDTO.setCode(ResponseCodeEnum.ACCOUNT_IS_DELETE.getCode());
+            loginDTO.setMsg(ResponseCodeEnum.ACCOUNT_NOT_EXIST.getMsg());
             return loginDTO;
         }
         //������������
         if (!MD5Utils.saltMD5Verify(password, manageAccount.getPassword())) {
-            loginDTO.setCode(LoginDTO.PASSWORD_ERROR);
+            loginDTO.setCode(ResponseCodeEnum.PASSWORD_INVALID.getCode());
+            loginDTO.setMsg(ResponseCodeEnum.PASSWORD_INVALID.getMsg());
             return loginDTO;
         }
 
         //������������
         List<ManageRole> roles = manageRoleMapper.getManageRoleByAccountId(manageAccount.getId());
-        if (ObjectUtils.isEmpty(roles)) {//������������������������������
-            loginDTO.setCode(LoginDTO.ROLE_EMPTY);
-            return loginDTO;
-        }
+
         //������������
-        List<ManageMenu> menus = manageMenuMapper.getParentChildrenMenusByRoles(roles);
-        if (ObjectUtils.isEmpty(menus)) {//������������������������������
-            loginDTO.setCode(LoginDTO.MENU_EMPTY);
-            return loginDTO;
+        List<ManageMenu> menus = null;
+        if (!ObjectUtils.isEmpty(roles)) {
+            menus = manageMenuMapper.getParentChildrenMenusByRoles(roles);
         }
 
         //������������������
@@ -122,7 +122,8 @@
         String token = TokenUtils.getToken(String.valueOf(manageAccount.getId()), accountInfoDTO);
 
         //������������������
-        loginDTO.setCode(LoginDTO.SUCCESS);
+        loginDTO.setCode(ResponseCodeEnum.SUCCESS.getCode());
+        loginDTO.setMsg(ResponseCodeEnum.SUCCESS.getMsg());
         loginDTO.setAccountInfoDTO(accountInfoDTO);
         loginDTO.setToken(token);
 
@@ -132,14 +133,14 @@
     /**
      * @Description: ������
      * @Param: [parameters]
-     * @return: java.util.Map<java.lang.String               ,               java.lang.Object>
+     * @return: java.util.Map<java.lang.String                                                                                                                                                                                                                                                               ,                                                                                                                                                                                                                                                               java.lang.Object>
      * @Author: ���������
      * @Date: 2021/3/11
      */
     @Override
-    public boolean logout(LogoutRequest logoutRequest) {
-        String accountId = logoutRequest.getAccountId();
-        String token = logoutRequest.getToken();
+    public boolean logout(LogoutForm logoutForm) {
+        String accountId = logoutForm.getAccountId();
+        String token = logoutForm.getToken();
         TokenUtils.destoryToken(accountId, token);
         return true;
     }
@@ -153,47 +154,52 @@
      */
     @Override
     @Transactional
-    public AccountInsertDTO insertAccount(AccountInsertRequest accountInsertRequest) {
+    public AccountInsertDTO insertAccount(AccountInsertForm accountInsertForm) {
         AccountInsertDTO accountInsertDTO = new AccountInsertDTO();
         //������
-        String AESAccount = accountInsertRequest.getAccount();
-        String AESPassword = accountInsertRequest.getPassword();
-        String account = AESUtils.decrypt(AESAccount, AESKey);
+        String account = accountInsertForm.getAccount();
+        String AESPassword = accountInsertForm.getPassword();
         String password = MD5Utils.saltMD5(AESUtils.decrypt(AESPassword, AESKey));
-        String userName = accountInsertRequest.getUserName();
-        String email = accountInsertRequest.getEmail();
-        String mobile = accountInsertRequest.getMobile();
-        String wechat = accountInsertRequest.getWechat();
-        List<String> roleIdsStr = accountInsertRequest.getRoleIds();
+        String userName = accountInsertForm.getUserName();
+        String email = accountInsertForm.getEmail();
+        String mobile = accountInsertForm.getMobile();
+        String wechat = accountInsertForm.getWechat();
+        List<String> roleIdsStr = accountInsertForm.getRoleIds();
         //������������������������������������
         /*���������������������������������������*/
         if (isSpecialChar(account)) {
-            accountInsertDTO.setCode(AccountInsertDTO.ACCOUNT_EXIST_SPECIAL_CHAR);
+            accountInsertDTO.setCode(ResponseCodeEnum.ACCOUNT_EXIST_SPECIAL_CHAR.getCode());
+            accountInsertDTO.setMsg(ResponseCodeEnum.ACCOUNT_EXIST_SPECIAL_CHAR.getMsg());
             return accountInsertDTO;
         }
         /*������������������������������������*/
         if (isSpecialChar(password)) {
-            accountInsertDTO.setCode(AccountInsertDTO.PASSWORD_EXIST_SPECIAL_CHAR);
+            accountInsertDTO.setCode(ResponseCodeEnum.PASSWORD_EXIST_SPECIAL_CHAR.getCode());
+            accountInsertDTO.setMsg(ResponseCodeEnum.PASSWORD_EXIST_SPECIAL_CHAR.getMsg());
             return accountInsertDTO;
         }
         /*���������������������������������*/
-        if (account.length() >= 20||account.length()<=6) {
-            accountInsertDTO.setCode(AccountInsertDTO.ACCOUNT_LENGTH_INVALID);
+        if (account.length() >= 20 || account.length() <= 6) {
+            accountInsertDTO.setCode(ResponseCodeEnum.ACCOUNT_LENGTH_INVALID.getCode());
+            accountInsertDTO.setMsg(ResponseCodeEnum.ACCOUNT_LENGTH_INVALID.getMsg());
             return accountInsertDTO;
         }
         /*������������������������������*/
-        if (AESUtils.decrypt(AESPassword, AESKey).length() >= 20||AESUtils.decrypt(AESPassword, AESKey).length() <= 6) {
-            accountInsertDTO.setCode(AccountInsertDTO.PASSWORD_LENGTH_INVALID);
+        if (AESUtils.decrypt(AESPassword, AESKey).length() >= 20 || AESUtils.decrypt(AESPassword, AESKey).length() <= 6) {
+            accountInsertDTO.setCode(ResponseCodeEnum.PASSWORD_LENGTH_INVALID.getCode());
+            accountInsertDTO.setMsg(ResponseCodeEnum.PASSWORD_LENGTH_INVALID.getMsg());
             return accountInsertDTO;
         }
         /*���������������������������������*/
         if (!isValidMobile(mobile)) {
-            accountInsertDTO.setCode(AccountInsertDTO.MOBILE_INVALID);
+            accountInsertDTO.setCode(ResponseCodeEnum.MOBILE_INVALID.getCode());
+            accountInsertDTO.setMsg(ResponseCodeEnum.MOBILE_INVALID.getMsg());
             return accountInsertDTO;
         }
         /*������������������������������*/
         if (!isValidEmail(email)) {
-            accountInsertDTO.setCode(AccountInsertDTO.EMAIL_INVALID);
+            accountInsertDTO.setCode(ResponseCodeEnum.EMAIL_INVALID.getCode());
+            accountInsertDTO.setMsg(ResponseCodeEnum.EMAIL_INVALID.getMsg());
             return accountInsertDTO;
         }
         /*������������������������*/
@@ -204,13 +210,10 @@
         wrapper.setEntity(exitAccount);
         List<ManageAccount> exitAccounts = manageAccountMapper.selectList(wrapper);
         if (!ObjectUtils.isEmpty(exitAccounts)) {
-            accountInsertDTO.setCode(AccountInsertDTO.ACCOUNT_EXIST);
+            accountInsertDTO.setCode(ResponseCodeEnum.ACCOUNT_EXIST.getCode());
+            accountInsertDTO.setMsg(ResponseCodeEnum.ACCOUNT_EXIST.getMsg());
             return accountInsertDTO;
         }
-
-        //String to Integer
-        List<Integer> roleIds = new ArrayList<>();
-        roleIdsStr.forEach(str -> roleIds.add(Integer.parseInt(str)));
 
         //������account
         ManageAccount manageAccount = new ManageAccount();
@@ -222,20 +225,29 @@
         manageAccount.setWechat(wechat);
         manageAccountMapper.insert(manageAccount);
         //������account_role
-        Integer accountId = manageAccount.getId();
-        roleIdsStr.forEach(
-                value -> {
-                    ManageAccountRole manageAccountRole = new ManageAccountRole();
-                    manageAccountRole.setAccountId(accountId);
-                    manageAccountRole.setRoleId(Integer.parseInt(value));
-                    manageAccountRoleMapper.insert(manageAccountRole);
-                }
-        );
+        /*String to Integer*/
+        List<ManageRole> roles = null;
+        //������������������������������������������������������
+        if (!ObjectUtils.isEmpty(roleIdsStr)) {
+            List<Integer> roleIds = new ArrayList<>();
+            roleIdsStr.forEach(str -> roleIds.add(Integer.parseInt(str)));
+
+            Integer accountId = manageAccount.getId();
+            roleIdsStr.forEach(
+                    value -> {
+                        ManageAccountRole manageAccountRole = new ManageAccountRole();
+                        manageAccountRole.setAccountId(accountId);
+                        manageAccountRole.setRoleId(Integer.parseInt(value));
+                        manageAccountRoleMapper.insert(manageAccountRole);
+                    }
+            );
+            roles = manageRoleMapper.selectBatchIds(roleIds);
+        }
         //������������������
-        List<ManageRole> roles = manageRoleMapper.selectBatchIds(roleIds);
         accountInsertDTO.setAccount(manageAccount);
         accountInsertDTO.setRoles(roles);
-        accountInsertDTO.setCode(AccountInsertDTO.SUCCESS);
+        accountInsertDTO.setCode(ResponseCodeEnum.SUCCESS.getCode());
+        accountInsertDTO.setMsg(ResponseCodeEnum.SUCCESS.getMsg());
         return accountInsertDTO;
     }
 
@@ -247,46 +259,63 @@
      * @Date: 2021/3/15
      */
     @Override
-    public AccountQueryDTO queryAccount(AccountQueryRequest accountQueryRequest) {
+    public AccountQueryDTO queryAccount(AccountQueryForm accountQueryForm) {
         AccountQueryDTO accountQueryDTO = new AccountQueryDTO();
         //������
-        Integer pageCount = accountQueryRequest.getPage();
-        Integer size = accountQueryRequest.getSize();
-        Integer accountId = accountQueryRequest.getAccountId();
-        String account = accountQueryRequest.getAccount();
-        String userName = accountQueryRequest.getUserName();
-        String email = accountQueryRequest.getEmail();
-        String mobile = accountQueryRequest.getMobile();
-        String wechat = accountQueryRequest.getWechat();
-        String isDelete = accountQueryRequest.getIsDelete();
-        //������������������
-        Map<String, Object> queryParams = new HashMap<>();
-        queryParams.put("id", accountId);
-        queryParams.put("account", account);
-        queryParams.put("user_name", userName);
-        queryParams.put("email", email);
-        queryParams.put("mobile", mobile);
-        queryParams.put("wechat", wechat);
-        if(!ObjectUtils.isEmpty(isDelete))
-            queryParams.put("is_delete", isDelete);
-        else
-            queryParams.put("is_delete","0");
-        //���������null���������
-        queryParams.values().removeIf(value -> ObjectUtils.isEmpty(value));
+        Integer pageCount = accountQueryForm.getPage();
+        Integer size = accountQueryForm.getSize();
+        Integer accountId = accountQueryForm.getAccountId();
+        String account = accountQueryForm.getAccount();
+        String userName = accountQueryForm.getUserName();
+        String email = accountQueryForm.getEmail();
+        String mobile = accountQueryForm.getMobile();
+        String wechat = accountQueryForm.getWechat();
+        String isDelete = accountQueryForm.getIsDelete();
+
         //������������
         Page<ManageAccount> page = new Page<>(pageCount, size);
         QueryWrapper<ManageAccount> wrapper = new QueryWrapper();
-        wrapper.allEq(queryParams);
+        if(!ObjectUtils.isEmpty(accountId)){
+            wrapper.eq("id",accountId);
+        }
+
+        if(!ObjectUtils.isEmpty(userName)){
+            wrapper.like("user_name",userName);
+        }
+
+        if(!ObjectUtils.isEmpty(account)){
+            wrapper.like("account",account);
+        }
+
+        if(!ObjectUtils.isEmpty(email)){
+            wrapper.like("email",email);
+        }
+
+        if(!ObjectUtils.isEmpty(mobile)){
+            wrapper.like("mobile",mobile);
+        }
+
+        if(!ObjectUtils.isEmpty(wechat)){
+            wrapper.like("wechat",wechat);
+        }
+
+        if (!ObjectUtils.isEmpty(isDelete))
+            wrapper.eq("is_delete", isDelete);
+        else
+            wrapper.eq("is_delete", "0");
+
         Page resultPage = manageAccountMapper.selectPage(page, wrapper);
         //���������������������������
         List<ManageAccount> accounts = resultPage.getRecords();
         List<AccountDTO> accountDTOS = new ArrayList<>();
-        for (ManageAccount manageAccount : accounts) {
-            AccountDTO accountDTO = new AccountDTO();
-            List<ManageRole> roles = manageRoleMapper.getManageRoleByAccountId(manageAccount.getId());
-            accountDTO.setRoles(roles);
-            accountDTO.setAccount(manageAccount);
-            accountDTOS.add(accountDTO);
+        if (!ObjectUtils.isEmpty(accounts)) {
+            for (ManageAccount manageAccount : accounts) {
+                AccountDTO accountDTO = new AccountDTO();
+                List<ManageRole> roles = manageRoleMapper.getManageRoleByAccountId(manageAccount.getId());
+                accountDTO.setRoles(roles);
+                accountDTO.setAccount(manageAccount);
+                accountDTOS.add(accountDTO);
+            }
         }
         //������������������
         accountQueryDTO.setAccountDTOS(accountDTOS);
@@ -294,28 +323,30 @@
         accountQueryDTO.setPages(resultPage.getPages());
         accountQueryDTO.setSize(resultPage.getSize());
         accountQueryDTO.setTotal(resultPage.getTotal());
+        accountQueryDTO.setCode(ResponseCodeEnum.SUCCESS.getCode());
+        accountQueryDTO.setMsg(ResponseCodeEnum.SUCCESS.getMsg());
 
         return accountQueryDTO;
     }
 
     /**
-    * @Description: ������������������
-            * @Param: [accountUpdateRequest]
-            * @return: com.moral.api.pojo.dto.account.AccountUpdateDTO
-            * @Author: ���������
-            * @Date: 2021/3/16
-            */
+     * @Description: ������������������
+     * @Param: [accountUpdateRequest]
+     * @return: com.moral.api.pojo.dto.account.AccountUpdateDTO
+     * @Author: ���������
+     * @Date: 2021/3/16
+     */
     @Override
     @Transactional
-    public AccountUpdateDTO updateAccount(AccountUpdateRequest accountUpdateRequest) {
+    public AccountUpdateDTO updateAccount(AccountUpdateForm accountUpdateForm) {
         AccountUpdateDTO accountUpdateDTO = new AccountUpdateDTO();
         //������
-        Integer accountId = accountUpdateRequest.getAccountId();
-        String email = accountUpdateRequest.getEmail();
-        String mobile = accountUpdateRequest.getMobile();
-        String wechat = accountUpdateRequest.getWechat();
-        String userName = accountUpdateRequest.getUserName();
-        List<Integer> roleIds = accountUpdateRequest.getRoleIds();
+        Integer accountId = accountUpdateForm.getAccountId();
+        String email = accountUpdateForm.getEmail();
+        String mobile = accountUpdateForm.getMobile();
+        String wechat = accountUpdateForm.getWechat();
+        String userName = accountUpdateForm.getUserName();
+        List<Integer> roleIds = accountUpdateForm.getRoleIds();
         //������������������������������
         /*������������������������������������*/
         QueryWrapper<ManageAccount> exitWrapper = new QueryWrapper<>();
@@ -324,18 +355,21 @@
         exitManageAccount.setIsDelete("0");
         exitWrapper.setEntity(exitManageAccount);
         List<ManageAccount> manageAccounts = manageAccountMapper.selectList(exitWrapper);
-        if(ObjectUtils.isEmpty(manageAccounts)){
-            accountUpdateDTO.setCode(AccountUpdateDTO.ACCOUNT_NOT_EXIST);
+        if (ObjectUtils.isEmpty(manageAccounts)) {
+            accountUpdateDTO.setCode(ResponseCodeEnum.ACCOUNT_NOT_EXIST.getCode());
+            accountUpdateDTO.setMsg(ResponseCodeEnum.ACCOUNT_NOT_EXIST.getMsg());
             return accountUpdateDTO;
         }
         /*���������������������������������*/
         if (!isValidMobile(mobile)) {
-            accountUpdateDTO.setCode(AccountUpdateDTO.MOBILE_INVALID);
+            accountUpdateDTO.setCode(ResponseCodeEnum.MOBILE_INVALID.getCode());
+            accountUpdateDTO.setMsg(ResponseCodeEnum.MOBILE_INVALID.getMsg());
             return accountUpdateDTO;
         }
         /*������������������������������*/
         if (!isValidEmail(email)) {
-            accountUpdateDTO.setCode(AccountUpdateDTO.EMAIL_INVALID);
+            accountUpdateDTO.setCode(ResponseCodeEnum.EMAIL_INVALID.getCode());
+            accountUpdateDTO.setMsg(ResponseCodeEnum.EMAIL_INVALID.getMsg());
             return accountUpdateDTO;
         }
         //������ManageAccount���
@@ -345,12 +379,12 @@
         manageAccount.setWechat(wechat);
         manageAccount.setUserName(userName);
         QueryWrapper<ManageAccount> wrapper = new QueryWrapper<>();
-        wrapper.eq("id",accountId);
-        manageAccountMapper.update(manageAccount,wrapper);
+        wrapper.eq("id", accountId);
+        manageAccountMapper.update(manageAccount, wrapper);
         //������ManageAccountRole���
         /*������������������������������*/
         QueryWrapper<ManageAccountRole> deleteWrapper = new QueryWrapper<>();
-        deleteWrapper.eq("account_id",accountId);
+        deleteWrapper.eq("account_id", accountId);
         manageAccountRoleMapper.delete(deleteWrapper);
         /*������������������*/
         for (Integer roleId : roleIds) {
@@ -362,7 +396,8 @@
         //������������������������
         List<ManageRole> manageRoles = manageRoleMapper.selectBatchIds(roleIds);
         //������������������
-        accountUpdateDTO.setCode(AccountUpdateDTO.SUCCESS);
+        accountUpdateDTO.setCode(ResponseCodeEnum.SUCCESS.getCode());
+        accountUpdateDTO.setMsg(ResponseCodeEnum.SUCCESS.getMsg());
         accountUpdateDTO.setRoles(manageRoles);
         accountUpdateDTO.setAccount(manageAccountMapper.selectById(accountId));
         return accountUpdateDTO;
@@ -370,18 +405,18 @@
 
 
     /**
-    * @Description: ������������������
-            * @Param: [accountDeleteRequest]
-            * @return: com.moral.api.pojo.dto.account.AccountDeleteDTO
-            * @Author: ���������
-            * @Date: 2021/3/16
-            */
+     * @Description: ������������������
+     * @Param: [accountDeleteRequest]
+     * @return: com.moral.api.pojo.dto.account.AccountDeleteDTO
+     * @Author: ���������
+     * @Date: 2021/3/16
+     */
     @Override
     @Transactional
-    public AccountDeleteDTO deleteAccount(AccountDeleteRequest accountDeleteRequest) {
+    public AccountDeleteDTO deleteAccount(AccountDeleteForm accountDeleteForm) {
         AccountDeleteDTO accountDeleteDTO = new AccountDeleteDTO();
         //������
-        Integer accountId = accountDeleteRequest.getAccountId();
+        Integer accountId = accountDeleteForm.getAccountId();
         //������������������
         ManageAccount manageAccount = new ManageAccount();
         manageAccount.setIsDelete("0");
@@ -389,21 +424,23 @@
         QueryWrapper<ManageAccount> queryWrapper = new QueryWrapper<>();
         queryWrapper.setEntity(manageAccount);
         ManageAccount existManageAccount = manageAccount.selectOne(queryWrapper);
-        if(ObjectUtils.isEmpty(existManageAccount)){
-            accountDeleteDTO.setCode(AccountDeleteDTO.ACCOUNT_NOT_EXIST);
+        if (ObjectUtils.isEmpty(existManageAccount)) {
+            accountDeleteDTO.setCode(ResponseCodeEnum.ACCOUNT_NOT_EXIST.getCode());
+            accountDeleteDTO.setMsg(ResponseCodeEnum.ACCOUNT_NOT_EXIST.getMsg());
             return accountDeleteDTO;
         }
         //������������������
         existManageAccount.setIsDelete("1");
         UpdateWrapper<ManageAccount> deleteAccountWrapper = new UpdateWrapper<>();
-        deleteAccountWrapper.set("is_delete","1").eq("id",manageAccount.getId());
-        manageAccountMapper.update(null,deleteAccountWrapper);
+        deleteAccountWrapper.set("is_delete", "1").eq("id", manageAccount.getId());
+        manageAccountMapper.update(null, deleteAccountWrapper);
         //������������������������������
         UpdateWrapper<ManageAccountRole> deleteManageAccountRoleWrapper = new UpdateWrapper<>();
-        deleteManageAccountRoleWrapper.set("is_delete","1").eq("account_id",manageAccount.getId());
-        manageAccountRoleMapper.update(null,deleteManageAccountRoleWrapper);
+        deleteManageAccountRoleWrapper.set("is_delete", "1").eq("account_id", manageAccount.getId());
+        manageAccountRoleMapper.update(null, deleteManageAccountRoleWrapper);
         //������������������
-        accountDeleteDTO.setCode(AccountDeleteDTO.SUCCESS);
+        accountDeleteDTO.setCode(ResponseCodeEnum.SUCCESS.getCode());
+        accountDeleteDTO.setMsg(ResponseCodeEnum.SUCCESS.getMsg());
         accountDeleteDTO.setAccount(existManageAccount);
         return accountDeleteDTO;
     }
diff --git a/screen-manage/src/main/resources/mapper/ManageMenuMapper.xml b/screen-manage/src/main/resources/mapper/ManageMenuMapper.xml
index 33dfcee..7dbef4e 100644
--- a/screen-manage/src/main/resources/mapper/ManageMenuMapper.xml
+++ b/screen-manage/src/main/resources/mapper/ManageMenuMapper.xml
@@ -69,6 +69,7 @@
     <!--���������������������������������������-->
     <select id="getParentChildrenMenusByRoles" resultMap="ParentChildrenResultMap">
         select
+        DISTINCT
         <include refid="Base_Column_List"></include>
         from
         manage_menu mm

--
Gitblit v1.8.0