refactor(ReactionController): 提取获取用户ID的逻辑到独立方法
Some checks failed
test/timeline-server/pipeline/head There was a failure building this commit
Some checks failed
test/timeline-server/pipeline/head There was a failure building this commit
将重复的jwtUtils.getUserIdFromRequest调用提取为私有方法getUserIdFromRequest
This commit is contained in:
@@ -40,7 +40,7 @@ public class ReactionController {
|
||||
// 获取当前用户ID(可能为null,用于匿名访问)
|
||||
String currentUserId = null;
|
||||
try {
|
||||
currentUserId = jwtUtils.getUserIdFromRequest(request);
|
||||
currentUserId = getUserIdFromRequest(request);
|
||||
} catch (Exception e) {
|
||||
log.debug("未获取到用户ID,可能是匿名访问");
|
||||
}
|
||||
@@ -69,7 +69,7 @@ public class ReactionController {
|
||||
log.info("添加/更新反应: {} - {} - {}", entityType, entityId, reactionType);
|
||||
|
||||
try {
|
||||
String userId = jwtUtils.getUserIdFromRequest(request);
|
||||
String userId = getUserIdFromRequest(request);
|
||||
reactionService.addOrUpdateReaction(userId, entityType, entityId, reactionType);
|
||||
return ResponseEntity.success(null);
|
||||
} catch (IllegalArgumentException e) {
|
||||
@@ -93,7 +93,7 @@ public class ReactionController {
|
||||
log.info("移除反应: {} - {}", entityType, entityId);
|
||||
|
||||
try {
|
||||
String userId = jwtUtils.getUserIdFromRequest(request);
|
||||
String userId = getUserIdFromRequest(request);
|
||||
reactionService.removeReaction(userId, entityType, entityId);
|
||||
return ResponseEntity.success(null);
|
||||
} catch (IllegalArgumentException e) {
|
||||
@@ -104,4 +104,16 @@ public class ReactionController {
|
||||
return ResponseEntity.error(500, "移除反应失败");
|
||||
}
|
||||
}
|
||||
|
||||
/**
|
||||
* 从请求中获取用户ID
|
||||
*/
|
||||
private String getUserIdFromRequest(HttpServletRequest request) {
|
||||
String token = request.getHeader("Authorization");
|
||||
if (token != null && token.startsWith("Bearer ")) {
|
||||
token = token.substring(7);
|
||||
return jwtUtils.getUserIdFromToken(token);
|
||||
}
|
||||
throw new RuntimeException("Unauthorized");
|
||||
}
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user