Introduce DTO-based user APIs, global validation handler, AjaxResult helpers, and refactor plan

docs/refactor-plan.md

@@ -0,0 +1,119 @@
+# SYTRAVEL 重构路线图（分阶段）
+
+> 目标：在不中断业务功能的前提下，把项目从“可运行”升级为“可维护、可测试、可演进”。
+
+## 0. 现状快照
+
+- 后端基于 Spring Boot 1.5.9 + Java 8，技术栈较旧。  
+- Controller/Service 层存在大量重复样板代码（分页、参数校验、日志结果组装）。  
+- 密码处理为 Base64 编码（非加密），存在明显安全风险。  
+- 前端为传统 jQuery + 多版本脚本混用，静态资源结构分散。  
+- 自动化测试非常薄弱（当前仅见基础启动测试）。
+
+## 1. 重构原则
+
+1. **先建立安全网，再动核心逻辑**：优先补测试与观测能力。  
+2. **小步快跑，保持可回滚**：每个阶段可独立上线。  
+3. **功能不变优先**：先做结构重组，再做行为变更。  
+4. **风险前置**：安全问题（密码、鉴权、输入校验）优先。
+
+## 2. 阶段计划
+
+## 阶段 A：基线与可观测（1~2 周）
+
+- 建立“重构基线”：
+  - 固定本地开发环境（JDK/Maven/MySQL 版本、初始化脚本）。
+  - 增加健康检查与关键日志字段。
+- 测试补齐（最少可用）：
+  - 为核心 Service（用户、项目、团队、产品）补充单元测试。
+  - 为 REST 接口增加 MockMvc 集成测试（覆盖增删改查主流程）。
+- 输出：
+  - 测试可执行文档。
+  - 第一版覆盖率报告（可先设 20% 目标）。
+
+## 阶段 B：后端分层与公共能力抽取（2~3 周）
+
+- 引入 DTO + 校验：
+  - 请求对象从 `JSON` 动态取值改为强类型 DTO。
+  - 使用参数校验注解替代手写判空。
+- 统一异常与返回：
+  - 引入全局异常处理（`@ControllerAdvice`）。
+  - 统一错误码、错误消息、日志追踪 ID。
+- 抽取可复用模块：
+  - 分页查询模板。
+  - 操作日志记录模板（替代每个 Service 手动拼装结果）。
+- 输出：
+  - 至少一个模块（建议 user）完成“端到端新结构示范”。
+
+## 阶段 C：安全整改（并行推进，1~2 周）
+
+- 密码体系升级：
+  - 由 Base64 改为 BCrypt/Argon2。
+  - 登录验证改为哈希比对。
+  - 增加“渐进迁移策略”（老密码首次登录后自动升级）。
+- 基础安全：
+  - 补充接口鉴权与角色控制。
+  - 统一输入过滤，避免注入/越权。
+- 输出：
+  - 安全基线清单 + 验收用例。
+
+## 阶段 D：数据层与领域模型治理（2 周）
+
+- Repository 命名与查询规范统一。
+- 领域对象瘦身：实体不承载表现层逻辑（如 `toMap`）。
+- 引入 MapStruct/手写 Mapper（Entity ↔ DTO）。
+- 关键表加索引与唯一约束复核。
+
+## 阶段 E：前端治理（2~4 周）
+
+- 静态资源整理：
+  - 清理重复/多版本 jQuery 与历史插件。
+  - 按模块归档 JS/CSS。
+- 前端接口层统一：
+  - 封装请求方法、统一错误提示。
+  - 去除页面脚本中的重复逻辑。
+- 视情况选择：
+  - 轻量化继续维护 jQuery 架构；或逐步迁移到 Vue/React（按业务价值评估）。
+
+## 阶段 F：平台升级（3~6 周，独立里程碑）
+
+- Spring Boot 1.5.x 升级到 2.7.x（再评估到 3.x）。
+- JDK 8 升级到 17 LTS。
+- 依赖升级并修复兼容性问题。
+- 完成后再启用更高版本框架特性（如新版安全配置）。
+
+## 3. 每一轮迭代的执行模板
+
+每个模块都按以下顺序推进：
+
+1. 画出现状调用链（Rest → Service → Repository）。
+2. 写回归测试，锁定当前行为。
+3. 引入 DTO + 参数校验。
+4. 抽离公共逻辑（日志、分页、异常处理）。
+5. 保持接口兼容并灰度验证。
+6. 合并后监控 1~2 天再进行下一模块。
+
+## 4. 第一批优先重构清单（建议按这个顺序）
+
+1. **用户模块（SYUserRest/SYUserService）**：风险最高（密码、权限、输入）。
+2. **登录模块（SYLoginRest/SYLoginService）**：与安全改造联动。
+3. **项目/团队/产品模块**：模板化改造，提取共性。
+4. **日志模块**：统一审计模型与查询能力。
+
+## 5. 验收标准（DoD）
+
+- 功能回归：核心接口回归通过率 100%。
+- 测试覆盖：核心业务包覆盖率达到阶段目标（20%→40%→60%）。
+- 安全：不再存在明文/可逆密码存储。
+- 可维护性：单个 Service 类平均行数明显下降，重复代码减少。
+- 文档：模块设计说明与接口说明同步更新。
+
+## 6. 我们下一步怎么做
+
+建议从 **阶段 A + 用户模块** 开始，第一迭代只做三件事：
+
+1. 为用户模块补单测与接口测试。  
+2. 把 `JSON` 入参改为 `UserCreateRequest/UserUpdateRequest`。  
+3. 落地全局异常处理和统一错误响应。  
+
+完成后你确认效果，我们再进入第二迭代（密码与鉴权改造）。

src/main/java/com/sy/travel/common/AjaxResult.java

@@ -20,6 +20,14 @@ public AjaxResult(int code, String msg, T data) {
 		this.data = data;
 	}
 
+	public static <T> AjaxResult<T> success(T data) {
+		return new AjaxResult<T>(200, ApiMessages.SUCCESS, data);
+	}
+
+	public static <T> AjaxResult<T> failed(int code, T data) {
+		return new AjaxResult<T>(code, ApiMessages.FAILED, data);
+	}
+
 	public int getCode() {
 		return code;
 	}

src/main/java/com/sy/travel/common/ApiMessages.java

@@ -0,0 +1,6 @@
+package com.sy.travel.common;
+
+public interface ApiMessages {
+	String SUCCESS = "success";
+	String FAILED = "failed";
+}

src/main/java/com/sy/travel/common/Commons.java

@@ -104,7 +104,15 @@ public class Commons {
 	public final static String USER_DELETE = "删除用户";
 	public final static String USER_DELETE_FAILED = "删除用户失败";
 	public final static String USER_UPDATE = "修改用户";
+	@Deprecated
 	public final static String USER_UPDATE_NOT_FOUNT ="该用户不存在";
+	public final static String USER_UPDATE_NOT_FOUND ="该用户不存在";
+	@Deprecated
 	public final static String USER_UPDTE_PWD = "修改超级管理员密码";
+	public final static String USER_UPDATE_PWD = "修改超级管理员密码";
 	public final static String USER_UPDATE_PWD_NOT_PERMISSION = "只有超级管理员才有修改密码的权限";
+	public final static String USER_NOT_LOGIN = "用户未登录";
+	public final static String USER_ADMIN_OLD_PASSWORD_EMPTY = "超级管理员原密码不能为空";
+	public final static String USER_ADMIN_OLD_PASSWORD_ERROR = "超级管理员原密码输入错误";
+	public final static String USER_ADMIN_NEW_PASSWORD_EMPTY = "超级管理员新密码不能为空";
 }

src/main/java/com/sy/travel/common/ErrorCodes.java

@@ -0,0 +1,6 @@
+package com.sy.travel.common;
+
+public interface ErrorCodes {
+	String VALIDATION_ERROR = "VALIDATION_ERROR";
+	String INTERNAL_ERROR = "INTERNAL_ERROR";
+}

src/main/java/com/sy/travel/dto/user/UserAdminPwdUpdateRequest.java

@@ -0,0 +1,36 @@
+package com.sy.travel.dto.user;
+
+import javax.validation.constraints.NotBlank;
+
+public class UserAdminPwdUpdateRequest {
+	@NotBlank(message = "operator不能为空")
+	private String operator;
+	@NotBlank(message = "password不能为空")
+	private String password;
+	@NotBlank(message = "newPwd不能为空")
+	private String newPwd;
+
+	public String getOperator() {
+		return operator;
+	}
+
+	public void setOperator(String operator) {
+		this.operator = operator;
+	}
+
+	public String getPassword() {
+		return password;
+	}
+
+	public void setPassword(String password) {
+		this.password = password;
+	}
+
+	public String getNewPwd() {
+		return newPwd;
+	}
+
+	public void setNewPwd(String newPwd) {
+		this.newPwd = newPwd;
+	}
+}

src/main/java/com/sy/travel/dto/user/UserCreateRequest.java

@@ -0,0 +1,57 @@
+package com.sy.travel.dto.user;
+
+import javax.validation.constraints.NotBlank;
+import javax.validation.constraints.Pattern;
+
+public class UserCreateRequest {
+	@NotBlank(message = "operator不能为空")
+	private String operator;
+	@NotBlank(message = "username不能为空")
+	private String username;
+	@NotBlank(message = "password不能为空")
+	private String password;
+	@NotBlank(message = "permission不能为空")
+	@Pattern(regexp = "^[0-5]$", message = "permission取值范围为0-5")
+	private String permission;
+	private String remark;
+
+	public String getOperator() {
+		return operator;
+	}
+
+	public void setOperator(String operator) {
+		this.operator = operator;
+	}
+
+	public String getUsername() {
+		return username;
+	}
+
+	public void setUsername(String username) {
+		this.username = username;
+	}
+
+	public String getPassword() {
+		return password;
+	}
+
+	public void setPassword(String password) {
+		this.password = password;
+	}
+
+	public String getPermission() {
+		return permission;
+	}
+
+	public void setPermission(String permission) {
+		this.permission = permission;
+	}
+
+	public String getRemark() {
+		return remark;
+	}
+
+	public void setRemark(String remark) {
+		this.remark = remark;
+	}
+}

src/main/java/com/sy/travel/dto/user/UserUpdateRequest.java

@@ -0,0 +1,56 @@
+package com.sy.travel.dto.user;
+
+import javax.validation.constraints.NotBlank;
+import javax.validation.constraints.NotNull;
+import javax.validation.constraints.Pattern;
+
+public class UserUpdateRequest {
+	@NotBlank(message = "operator不能为空")
+	private String operator;
+	@NotNull(message = "id不能为空")
+	private Integer id;
+	private String password;
+	@Pattern(regexp = "^[0-5]$", message = "permission取值范围为0-5")
+	private String permission;
+	private String remark;
+
+	public String getOperator() {
+		return operator;
+	}
+
+	public void setOperator(String operator) {
+		this.operator = operator;
+	}
+
+	public Integer getId() {
+		return id;
+	}
+
+	public void setId(Integer id) {
+		this.id = id;
+	}
+
+	public String getPassword() {
+		return password;
+	}
+
+	public void setPassword(String password) {
+		this.password = password;
+	}
+
+	public String getPermission() {
+		return permission;
+	}
+
+	public void setPermission(String permission) {
+		this.permission = permission;
+	}
+
+	public String getRemark() {
+		return remark;
+	}
+
+	public void setRemark(String remark) {
+		this.remark = remark;
+	}
+}

src/main/java/com/sy/travel/handler/GlobalExceptionHandler.java

@@ -0,0 +1,33 @@
+package com.sy.travel.handler;
+
+import java.util.stream.Collectors;
+
+import org.springframework.http.HttpStatus;
+import org.springframework.web.bind.MethodArgumentNotValidException;
+import org.springframework.web.bind.annotation.ControllerAdvice;
+import org.springframework.web.bind.annotation.ExceptionHandler;
+import org.springframework.web.bind.annotation.ResponseBody;
+import org.springframework.web.bind.annotation.ResponseStatus;
+
+import com.sy.travel.common.AjaxResult;
+import com.sy.travel.common.ErrorCodes;
+
+@ControllerAdvice
+public class GlobalExceptionHandler {
+
+	@ExceptionHandler(MethodArgumentNotValidException.class)
+	@ResponseStatus(HttpStatus.BAD_REQUEST)
+	@ResponseBody
+	public AjaxResult<String> handleValidation(MethodArgumentNotValidException ex) {
+		String message = ex.getBindingResult().getFieldErrors().stream().map(fieldError -> fieldError.getDefaultMessage())
+				.collect(Collectors.joining(";"));
+		return new AjaxResult<String>(400, ErrorCodes.VALIDATION_ERROR, message);
+	}
+
+	@ExceptionHandler(Exception.class)
+	@ResponseStatus(HttpStatus.INTERNAL_SERVER_ERROR)
+	@ResponseBody
+	public AjaxResult<String> handleUnexpected(Exception ex) {
+		return new AjaxResult<String>(500, ErrorCodes.INTERNAL_ERROR, ex.getMessage());
+	}
+}