Преглед изворни кода

fix(controller): 统一异常处理返回错误码

- 引入 ErrorCode 枚举类管理错误码
- 将所有控制器中的异常处理改为使用系统错误码
- 错误信息拼接具体异常描述便于调试
- 涉及血糖、血压、心率、体征和用户绑定五大模块
- 提升接口响应一致性和可维护性
mcbaiyun пре 1 месец
родитељ
комит
bf939bea3e

+ 4 - 3
src/main/java/work/baiyun/chronicdiseaseapp/controller/BloodGlucoseDataController.java

@@ -19,6 +19,7 @@ import work.baiyun.chronicdiseaseapp.model.vo.BaseQueryRequest;
 import work.baiyun.chronicdiseaseapp.model.vo.DeleteBloodGlucoseDataRequest;
 import work.baiyun.chronicdiseaseapp.service.BloodGlucoseDataService;
 import com.baomidou.mybatisplus.extension.plugins.pagination.Page;
+import work.baiyun.chronicdiseaseapp.enums.ErrorCode;
 
 @RestController
 @RequestMapping("/blood-glucose")
@@ -43,7 +44,7 @@ public class BloodGlucoseDataController {
             service.addBloodGlucoseData(req);
             return R.success(200, "ok");
         } catch (Exception e) {
-            return R.fail(500, e.getMessage());
+            return R.fail(ErrorCode.SYSTEM_ERROR.getCode(), ErrorCode.SYSTEM_ERROR.getMessage() + ": " + e.getMessage());
         }
     }
 
@@ -62,7 +63,7 @@ public class BloodGlucoseDataController {
             Page<BloodGlucoseDataResponse> page = service.listBloodGlucoseData(req);
             return R.success(200, "ok", page);
         } catch (Exception e) {
-            return R.fail(500, e.getMessage());
+            return R.fail(ErrorCode.SYSTEM_ERROR.getCode(), ErrorCode.SYSTEM_ERROR.getMessage() + ": " + e.getMessage());
         }
     }
 
@@ -81,7 +82,7 @@ public class BloodGlucoseDataController {
             service.deleteBloodGlucoseData(req.getId());
             return R.success(200, "ok");
         } catch (Exception e) {
-            return R.fail(500, e.getMessage());
+            return R.fail(ErrorCode.SYSTEM_ERROR.getCode(), ErrorCode.SYSTEM_ERROR.getMessage() + ": " + e.getMessage());
         }
     }
 }

+ 4 - 3
src/main/java/work/baiyun/chronicdiseaseapp/controller/BloodPressureDataController.java

@@ -19,6 +19,7 @@ import work.baiyun.chronicdiseaseapp.model.vo.BaseQueryRequest;
 import work.baiyun.chronicdiseaseapp.model.vo.DeleteBloodPressureDataRequest;
 import work.baiyun.chronicdiseaseapp.service.BloodPressureDataService;
 import com.baomidou.mybatisplus.extension.plugins.pagination.Page;
+import work.baiyun.chronicdiseaseapp.enums.ErrorCode;
 
 @RestController
 @RequestMapping("/blood-pressure")
@@ -43,7 +44,7 @@ public class BloodPressureDataController {
             service.addBloodPressureData(req);
             return R.success(200, "ok");
         } catch (Exception e) {
-            return R.fail(500, e.getMessage());
+            return R.fail(ErrorCode.SYSTEM_ERROR.getCode(), ErrorCode.SYSTEM_ERROR.getMessage() + ": " + e.getMessage());
         }
     }
 
@@ -62,7 +63,7 @@ public class BloodPressureDataController {
             Page<BloodPressureDataResponse> page = service.listBloodPressureData(req);
             return R.success(200, "ok", page);
         } catch (Exception e) {
-            return R.fail(500, e.getMessage());
+            return R.fail(ErrorCode.SYSTEM_ERROR.getCode(), ErrorCode.SYSTEM_ERROR.getMessage() + ": " + e.getMessage());
         }
     }
 
@@ -81,7 +82,7 @@ public class BloodPressureDataController {
             service.deleteBloodPressureData(req.getId());
             return R.success(200, "ok");
         } catch (Exception e) {
-            return R.fail(500, e.getMessage());
+            return R.fail(ErrorCode.SYSTEM_ERROR.getCode(), ErrorCode.SYSTEM_ERROR.getMessage() + ": " + e.getMessage());
         }
     }
 }

+ 4 - 3
src/main/java/work/baiyun/chronicdiseaseapp/controller/HeartRateDataController.java

@@ -19,6 +19,7 @@ import work.baiyun.chronicdiseaseapp.model.vo.BaseQueryRequest;
 import work.baiyun.chronicdiseaseapp.model.vo.DeleteHeartRateDataRequest;
 import work.baiyun.chronicdiseaseapp.service.HeartRateDataService;
 import com.baomidou.mybatisplus.extension.plugins.pagination.Page;
+import work.baiyun.chronicdiseaseapp.enums.ErrorCode;
 
 @RestController
 @RequestMapping("/heart-rate")
@@ -43,7 +44,7 @@ public class HeartRateDataController {
             service.addHeartRateData(req);
             return R.success(200, "ok");
         } catch (Exception e) {
-            return R.fail(500, e.getMessage());
+            return R.fail(ErrorCode.SYSTEM_ERROR.getCode(), ErrorCode.SYSTEM_ERROR.getMessage() + ": " + e.getMessage());
         }
     }
 
@@ -62,7 +63,7 @@ public class HeartRateDataController {
             Page<HeartRateDataResponse> page = service.listHeartRateData(req);
             return R.success(200, "ok", page);
         } catch (Exception e) {
-            return R.fail(500, e.getMessage());
+            return R.fail(ErrorCode.SYSTEM_ERROR.getCode(), ErrorCode.SYSTEM_ERROR.getMessage() + ": " + e.getMessage());
         }
     }
 
@@ -81,7 +82,7 @@ public class HeartRateDataController {
             service.deleteHeartRateData(req.getId());
             return R.success(200, "ok");
         } catch (Exception e) {
-            return R.fail(500, e.getMessage());
+            return R.fail(ErrorCode.SYSTEM_ERROR.getCode(), ErrorCode.SYSTEM_ERROR.getMessage() + ": " + e.getMessage());
         }
     }
 }

+ 4 - 3
src/main/java/work/baiyun/chronicdiseaseapp/controller/PhysicalDataController.java

@@ -19,6 +19,7 @@ import work.baiyun.chronicdiseaseapp.model.vo.BaseQueryRequest;
 import work.baiyun.chronicdiseaseapp.model.vo.DeletePhysicalDataRequest;
 import work.baiyun.chronicdiseaseapp.service.PhysicalDataService;
 import com.baomidou.mybatisplus.extension.plugins.pagination.Page;
+import work.baiyun.chronicdiseaseapp.enums.ErrorCode;
 
 @RestController
 @RequestMapping("/physical")
@@ -43,7 +44,7 @@ public class PhysicalDataController {
             physicalDataService.addPhysicalData(req);
             return R.success(200, "ok");
         } catch (Exception e) {
-            return R.fail(500, e.getMessage());
+            return R.fail(ErrorCode.SYSTEM_ERROR.getCode(), ErrorCode.SYSTEM_ERROR.getMessage() + ": " + e.getMessage());
         }
     }
 
@@ -62,7 +63,7 @@ public class PhysicalDataController {
             Page<PhysicalDataResponse> page = physicalDataService.listPhysicalData(req);
             return R.success(200, "ok", page);
         } catch (Exception e) {
-            return R.fail(500, e.getMessage());
+            return R.fail(ErrorCode.SYSTEM_ERROR.getCode(), ErrorCode.SYSTEM_ERROR.getMessage() + ": " + e.getMessage());
         }
     }
 
@@ -81,7 +82,7 @@ public class PhysicalDataController {
             physicalDataService.deletePhysicalData(req.getId());
             return R.success(200, "ok");
         } catch (Exception e) {
-            return R.fail(500, e.getMessage());
+            return R.fail(ErrorCode.SYSTEM_ERROR.getCode(), ErrorCode.SYSTEM_ERROR.getMessage() + ": " + e.getMessage());
         }
     }
 }

+ 5 - 4
src/main/java/work/baiyun/chronicdiseaseapp/controller/UserBindingController.java

@@ -21,6 +21,7 @@ import work.baiyun.chronicdiseaseapp.model.vo.DeleteUserBindingRequest;
 import work.baiyun.chronicdiseaseapp.model.vo.CheckUserBindingResponse;
 import work.baiyun.chronicdiseaseapp.model.vo.CheckUserBindingRequest;
 import work.baiyun.chronicdiseaseapp.model.vo.UserBindingPageResult;
+import work.baiyun.chronicdiseaseapp.enums.ErrorCode;
 import work.baiyun.chronicdiseaseapp.service.UserBindingService;
 
 @RestController
@@ -46,7 +47,7 @@ public class UserBindingController {
             userBindingService.createUserBinding(req);
             return R.success(200, "ok");
         } catch (Exception e) {
-            return R.fail(500, e.getMessage());
+            return R.fail(ErrorCode.SYSTEM_ERROR.getCode(), ErrorCode.SYSTEM_ERROR.getMessage() + ": " + e.getMessage());
         }
     }
 
@@ -65,7 +66,7 @@ public class UserBindingController {
             userBindingService.deleteUserBinding(req.getPatientUserId(), req.getBoundUserId());
             return R.success(200, "ok");
         } catch (Exception e) {
-            return R.fail(500, e.getMessage());
+            return R.fail(ErrorCode.SYSTEM_ERROR.getCode(), ErrorCode.SYSTEM_ERROR.getMessage() + ": " + e.getMessage());
         }
     }
 
@@ -84,7 +85,7 @@ public class UserBindingController {
             Page<UserBindingResponse> page = userBindingService.listBindingsByPatient(patientUserId, bindingType, req);
             return R.success(200, "ok", page);
         } catch (Exception e) {
-            return R.fail(500, e.getMessage());
+            return R.fail(ErrorCode.SYSTEM_ERROR.getCode(), ErrorCode.SYSTEM_ERROR.getMessage() + ": " + e.getMessage());
         }
     }
 
@@ -103,7 +104,7 @@ public class UserBindingController {
             Page<UserBindingResponse> page = userBindingService.listBindingsByBoundUser(boundUserId, bindingType, req);
             return R.success(200, "ok", page);
         } catch (Exception e) {
-            return R.fail(500, e.getMessage());
+            return R.fail(ErrorCode.SYSTEM_ERROR.getCode(), ErrorCode.SYSTEM_ERROR.getMessage() + ": " + e.getMessage());
         }
     }