Просмотр исходного кода

refactor(security): 替换当前用户ID获取方式

- 使用SecurityUtils工具类获取当前用户ID
- 删除各服务实现类中的getCurrentUserId方法
- 统一用户ID获取逻辑,提高代码复用性
mcbaiyun 2 месяцев назад
Родитель
Сommit
c3a204e8ab

+ 3 - 6
src/main/java/work/baiyun/chronicdiseaseapp/service/impl/BloodGlucoseDataServiceImpl.java

@@ -23,7 +23,7 @@ public class BloodGlucoseDataServiceImpl implements BloodGlucoseDataService {
 
     @Override
     public void addBloodGlucoseData(AddBloodGlucoseDataRequest request) {
-        Long userId = getCurrentUserId();
+        Long userId = work.baiyun.chronicdiseaseapp.util.SecurityUtils.getCurrentUserId();
         BloodGlucoseData record = new BloodGlucoseData();
         BeanUtils.copyProperties(request, record);
         record.setUserId(userId);
@@ -32,7 +32,7 @@ public class BloodGlucoseDataServiceImpl implements BloodGlucoseDataService {
 
     @Override
     public Page<BloodGlucoseDataResponse> listBloodGlucoseData(BaseQueryRequest request) {
-        Long userId = getCurrentUserId();
+        Long userId = work.baiyun.chronicdiseaseapp.util.SecurityUtils.getCurrentUserId();
         Page<BloodGlucoseData> page = new Page<>(request.getPageNum(), request.getPageSize());
         LambdaQueryWrapper<BloodGlucoseData> wrapper = new LambdaQueryWrapper<>();
         wrapper.eq(BloodGlucoseData::getUserId, userId)
@@ -56,8 +56,5 @@ public class BloodGlucoseDataServiceImpl implements BloodGlucoseDataService {
         return responsePage;
     }
 
-    private Long getCurrentUserId() {
-        // TODO: replace with actual security context lookup
-        return 1L;
-    }
+    
 }

+ 3 - 6
src/main/java/work/baiyun/chronicdiseaseapp/service/impl/BloodPressureDataServiceImpl.java

@@ -23,7 +23,7 @@ public class BloodPressureDataServiceImpl implements BloodPressureDataService {
 
     @Override
     public void addBloodPressureData(AddBloodPressureDataRequest request) {
-        Long userId = getCurrentUserId();
+        Long userId = work.baiyun.chronicdiseaseapp.util.SecurityUtils.getCurrentUserId();
         BloodPressureData record = new BloodPressureData();
         BeanUtils.copyProperties(request, record);
         record.setUserId(userId);
@@ -32,7 +32,7 @@ public class BloodPressureDataServiceImpl implements BloodPressureDataService {
 
     @Override
     public Page<BloodPressureDataResponse> listBloodPressureData(BaseQueryRequest request) {
-        Long userId = getCurrentUserId();
+        Long userId = work.baiyun.chronicdiseaseapp.util.SecurityUtils.getCurrentUserId();
         Page<BloodPressureData> page = new Page<>(request.getPageNum(), request.getPageSize());
         LambdaQueryWrapper<BloodPressureData> wrapper = new LambdaQueryWrapper<>();
         wrapper.eq(BloodPressureData::getUserId, userId)
@@ -56,8 +56,5 @@ public class BloodPressureDataServiceImpl implements BloodPressureDataService {
         return responsePage;
     }
 
-    private Long getCurrentUserId() {
-        // TODO: replace with actual security context lookup
-        return 1L;
-    }
+    
 }

+ 3 - 6
src/main/java/work/baiyun/chronicdiseaseapp/service/impl/HeartRateDataServiceImpl.java

@@ -23,7 +23,7 @@ public class HeartRateDataServiceImpl implements HeartRateDataService {
 
     @Override
     public void addHeartRateData(AddHeartRateDataRequest request) {
-        Long userId = getCurrentUserId();
+        Long userId = work.baiyun.chronicdiseaseapp.util.SecurityUtils.getCurrentUserId();
         HeartRateData record = new HeartRateData();
         BeanUtils.copyProperties(request, record);
         record.setUserId(userId);
@@ -32,7 +32,7 @@ public class HeartRateDataServiceImpl implements HeartRateDataService {
 
     @Override
     public Page<HeartRateDataResponse> listHeartRateData(BaseQueryRequest request) {
-        Long userId = getCurrentUserId();
+        Long userId = work.baiyun.chronicdiseaseapp.util.SecurityUtils.getCurrentUserId();
         Page<HeartRateData> page = new Page<>(request.getPageNum(), request.getPageSize());
         LambdaQueryWrapper<HeartRateData> wrapper = new LambdaQueryWrapper<>();
         wrapper.eq(HeartRateData::getUserId, userId)
@@ -56,8 +56,5 @@ public class HeartRateDataServiceImpl implements HeartRateDataService {
         return responsePage;
     }
 
-    private Long getCurrentUserId() {
-        // TODO: replace with actual security context lookup
-        return 1L;
-    }
+    
 }

+ 3 - 6
src/main/java/work/baiyun/chronicdiseaseapp/service/impl/PhysicalDataServiceImpl.java

@@ -25,7 +25,7 @@ public class PhysicalDataServiceImpl implements PhysicalDataService {
 
     @Override
     public void addPhysicalData(AddPhysicalDataRequest request) {
-        Long userId = getCurrentUserId();
+        Long userId = work.baiyun.chronicdiseaseapp.util.SecurityUtils.getCurrentUserId();
         PhysicalData physicalData = new PhysicalData();
         BeanUtils.copyProperties(request, physicalData);
         physicalData.setUserId(userId);
@@ -34,7 +34,7 @@ public class PhysicalDataServiceImpl implements PhysicalDataService {
 
     @Override
     public Page<PhysicalDataResponse> listPhysicalData(BaseQueryRequest request) {
-        Long userId = getCurrentUserId();
+        Long userId = work.baiyun.chronicdiseaseapp.util.SecurityUtils.getCurrentUserId();
         Page<PhysicalData> page = new Page<>(request.getPageNum(), request.getPageSize());
         LambdaQueryWrapper<PhysicalData> wrapper = new LambdaQueryWrapper<>();
         wrapper.eq(PhysicalData::getUserId, userId)
@@ -70,8 +70,5 @@ public class PhysicalDataServiceImpl implements PhysicalDataService {
         return weight.divide(heightInMeters.pow(2), 2, RoundingMode.HALF_UP);
     }
 
-    private Long getCurrentUserId() {
-        // TODO: replace with actual security context lookup
-        return 1L;
-    }
+    
 }