feat: 完善后端架构 - 标准化Controller层和Service层
✨ 新功能: - 创建了完整的Service层架构，包含所有业务实体的Service接口和实现类 - 新增8个标准化的Controller类，支持完整的CRUD操作 - 实现了统一的Request/Response模式和分页查询功能 - 创建了认证服务(AuthService)和令牌服务(TokenService) - 添加了Redis配置和认证拦截器 🏗️ 架构优化: - 移除Controller层所有try-catch块，使用全局异常处理机制 - 创建了专门的异常类(AuthException, TokenException, CaptchaException) - 统一了API返回格式，完善了Result类的方法 - 实现了标准的分页查询和参数校验 📦 新增文件: - 8个Controller类: Achievement, Comment, CommunityPost, Conversation, CozeApiCall, EmotionAnalysis, Reward, UserStats - 12个Service接口和对应的实现类 - 标准化的DTO类(Request/Response) - 异常处理类和拦截器 - 测试用例 🔧 重构优化: - 重写了AuthController，移除所有业务逻辑到Service层 - 优化了MessageController，使用标准的Request/Response格式 - 更新了全局异常处理器，支持多种异常类型 - 完善了WebConfig配置，添加认证拦截器 📊 代码统计: - 新增文件: 60+个 - 新增代码行数: 8000+行 - 重构代码行数: 1000+行 - 移除过时接口: 4个
This commit is contained in:
@@ -0,0 +1,233 @@
|
||||
package com.emotion.service.impl;
|
||||
|
||||
import com.baomidou.mybatisplus.core.conditions.query.LambdaQueryWrapper;
|
||||
import com.baomidou.mybatisplus.core.conditions.update.LambdaUpdateWrapper;
|
||||
import com.baomidou.mybatisplus.core.metadata.IPage;
|
||||
import com.baomidou.mybatisplus.extension.plugins.pagination.Page;
|
||||
import com.baomidou.mybatisplus.extension.service.impl.ServiceImpl;
|
||||
import com.emotion.common.BasePageRequest;
|
||||
import com.emotion.entity.GuestUser;
|
||||
import com.emotion.mapper.GuestUserMapper;
|
||||
import com.emotion.service.GuestUserService;
|
||||
import org.springframework.stereotype.Service;
|
||||
import org.springframework.util.StringUtils;
|
||||
|
||||
import java.time.LocalDateTime;
|
||||
import java.util.List;
|
||||
import java.util.UUID;
|
||||
|
||||
/**
|
||||
* 访客用户服务实现类
|
||||
*
|
||||
* @author emotion-museum
|
||||
* @date 2025-07-23
|
||||
*/
|
||||
@Service
|
||||
public class GuestUserServiceImpl extends ServiceImpl<GuestUserMapper, GuestUser> implements GuestUserService {
|
||||
|
||||
@Override
|
||||
public IPage<GuestUser> getPage(BasePageRequest request) {
|
||||
Page<GuestUser> page = new Page<>(request.getCurrent(), request.getSize());
|
||||
LambdaQueryWrapper<GuestUser> wrapper = new LambdaQueryWrapper<>();
|
||||
|
||||
if (StringUtils.hasText(request.getKeyword())) {
|
||||
wrapper.and(w -> w.like(GuestUser::getDeviceId, request.getKeyword())
|
||||
.or().like(GuestUser::getIpAddress, request.getKeyword()));
|
||||
}
|
||||
|
||||
wrapper.eq(GuestUser::getIsDeleted, 0)
|
||||
.orderByDesc(GuestUser::getCreateTime);
|
||||
|
||||
return this.page(page, wrapper);
|
||||
}
|
||||
|
||||
@Override
|
||||
public GuestUser getByDeviceId(String deviceId) {
|
||||
LambdaQueryWrapper<GuestUser> wrapper = new LambdaQueryWrapper<>();
|
||||
wrapper.eq(GuestUser::getDeviceId, deviceId)
|
||||
.eq(GuestUser::getIsDeleted, 0);
|
||||
return this.getOne(wrapper);
|
||||
}
|
||||
|
||||
@Override
|
||||
public List<GuestUser> getByIpAddress(String ipAddress) {
|
||||
LambdaQueryWrapper<GuestUser> wrapper = new LambdaQueryWrapper<>();
|
||||
wrapper.eq(GuestUser::getIpAddress, ipAddress)
|
||||
.eq(GuestUser::getIsDeleted, 0)
|
||||
.orderByDesc(GuestUser::getCreateTime);
|
||||
return this.list(wrapper);
|
||||
}
|
||||
|
||||
@Override
|
||||
public List<GuestUser> getByUserAgent(String userAgent) {
|
||||
LambdaQueryWrapper<GuestUser> wrapper = new LambdaQueryWrapper<>();
|
||||
wrapper.like(GuestUser::getUserAgent, userAgent)
|
||||
.eq(GuestUser::getIsDeleted, 0)
|
||||
.orderByDesc(GuestUser::getCreateTime);
|
||||
return this.list(wrapper);
|
||||
}
|
||||
|
||||
@Override
|
||||
public List<GuestUser> getByStatus(String status) {
|
||||
LambdaQueryWrapper<GuestUser> wrapper = new LambdaQueryWrapper<>();
|
||||
wrapper.eq(GuestUser::getStatus, status)
|
||||
.eq(GuestUser::getIsDeleted, 0)
|
||||
.orderByDesc(GuestUser::getCreateTime);
|
||||
return this.list(wrapper);
|
||||
}
|
||||
|
||||
@Override
|
||||
public List<GuestUser> getByTimeRange(LocalDateTime startTime, LocalDateTime endTime) {
|
||||
LambdaQueryWrapper<GuestUser> wrapper = new LambdaQueryWrapper<>();
|
||||
wrapper.between(GuestUser::getCreateTime, startTime, endTime)
|
||||
.eq(GuestUser::getIsDeleted, 0)
|
||||
.orderByDesc(GuestUser::getCreateTime);
|
||||
return this.list(wrapper);
|
||||
}
|
||||
|
||||
@Override
|
||||
public List<GuestUser> getByLastActiveTimeRange(LocalDateTime startTime, LocalDateTime endTime) {
|
||||
LambdaQueryWrapper<GuestUser> wrapper = new LambdaQueryWrapper<>();
|
||||
wrapper.between(GuestUser::getLastActiveTime, startTime, endTime)
|
||||
.eq(GuestUser::getIsDeleted, 0)
|
||||
.orderByDesc(GuestUser::getLastActiveTime);
|
||||
return this.list(wrapper);
|
||||
}
|
||||
|
||||
@Override
|
||||
public Long countByStatus(String status) {
|
||||
LambdaQueryWrapper<GuestUser> wrapper = new LambdaQueryWrapper<>();
|
||||
wrapper.eq(GuestUser::getStatus, status)
|
||||
.eq(GuestUser::getIsDeleted, 0);
|
||||
return this.count(wrapper);
|
||||
}
|
||||
|
||||
@Override
|
||||
public Long countByIpAddress(String ipAddress) {
|
||||
LambdaQueryWrapper<GuestUser> wrapper = new LambdaQueryWrapper<>();
|
||||
wrapper.eq(GuestUser::getIpAddress, ipAddress)
|
||||
.eq(GuestUser::getIsDeleted, 0);
|
||||
return this.count(wrapper);
|
||||
}
|
||||
|
||||
@Override
|
||||
public Long countTodayNewGuests() {
|
||||
LocalDateTime startOfDay = LocalDateTime.now().withHour(0).withMinute(0).withSecond(0);
|
||||
LambdaQueryWrapper<GuestUser> wrapper = new LambdaQueryWrapper<>();
|
||||
wrapper.ge(GuestUser::getCreateTime, startOfDay)
|
||||
.eq(GuestUser::getIsDeleted, 0);
|
||||
return this.count(wrapper);
|
||||
}
|
||||
|
||||
@Override
|
||||
public Long countActiveGuests(Integer days) {
|
||||
LocalDateTime cutoffTime = LocalDateTime.now().minusDays(days);
|
||||
LambdaQueryWrapper<GuestUser> wrapper = new LambdaQueryWrapper<>();
|
||||
wrapper.ge(GuestUser::getLastActiveTime, cutoffTime)
|
||||
.eq(GuestUser::getIsDeleted, 0);
|
||||
return this.count(wrapper);
|
||||
}
|
||||
|
||||
@Override
|
||||
public List<GuestUser> getRecentVisitors(Integer limit) {
|
||||
LambdaQueryWrapper<GuestUser> wrapper = new LambdaQueryWrapper<>();
|
||||
wrapper.eq(GuestUser::getIsDeleted, 0)
|
||||
.orderByDesc(GuestUser::getLastActiveTime)
|
||||
.last("LIMIT " + limit);
|
||||
return this.list(wrapper);
|
||||
}
|
||||
|
||||
@Override
|
||||
public List<GuestUser> getInactiveGuests(Integer days) {
|
||||
LocalDateTime cutoffTime = LocalDateTime.now().minusDays(days);
|
||||
LambdaQueryWrapper<GuestUser> wrapper = new LambdaQueryWrapper<>();
|
||||
wrapper.lt(GuestUser::getLastActiveTime, cutoffTime)
|
||||
.eq(GuestUser::getIsDeleted, 0)
|
||||
.orderByAsc(GuestUser::getLastActiveTime);
|
||||
return this.list(wrapper);
|
||||
}
|
||||
|
||||
@Override
|
||||
public List<GuestUser> getByVisitCountRange(Integer minVisits, Integer maxVisits) {
|
||||
LambdaQueryWrapper<GuestUser> wrapper = new LambdaQueryWrapper<>();
|
||||
wrapper.between(GuestUser::getVisitCount, minVisits, maxVisits)
|
||||
.eq(GuestUser::getIsDeleted, 0)
|
||||
.orderByDesc(GuestUser::getVisitCount);
|
||||
return this.list(wrapper);
|
||||
}
|
||||
|
||||
@Override
|
||||
public Double getAvgVisitCount() {
|
||||
List<GuestUser> guests = this.list(new LambdaQueryWrapper<GuestUser>()
|
||||
.eq(GuestUser::getIsDeleted, 0)
|
||||
.isNotNull(GuestUser::getVisitCount));
|
||||
return guests.stream()
|
||||
.mapToDouble(g -> g.getVisitCount() != null ? g.getVisitCount().doubleValue() : 0.0)
|
||||
.average()
|
||||
.orElse(0.0);
|
||||
}
|
||||
|
||||
@Override
|
||||
public boolean updateLastActiveTime(String id, LocalDateTime lastActiveTime) {
|
||||
LambdaUpdateWrapper<GuestUser> wrapper = new LambdaUpdateWrapper<>();
|
||||
wrapper.eq(GuestUser::getId, id)
|
||||
.set(GuestUser::getLastActiveTime, lastActiveTime)
|
||||
.set(GuestUser::getUpdateTime, LocalDateTime.now());
|
||||
return this.update(wrapper);
|
||||
}
|
||||
|
||||
@Override
|
||||
public boolean incrementVisitCount(String id) {
|
||||
LambdaUpdateWrapper<GuestUser> wrapper = new LambdaUpdateWrapper<>();
|
||||
wrapper.eq(GuestUser::getId, id)
|
||||
.setSql("visit_count = visit_count + 1")
|
||||
.set(GuestUser::getLastActiveTime, LocalDateTime.now())
|
||||
.set(GuestUser::getUpdateTime, LocalDateTime.now());
|
||||
return this.update(wrapper);
|
||||
}
|
||||
|
||||
@Override
|
||||
public boolean updateStatus(String id, String status) {
|
||||
LambdaUpdateWrapper<GuestUser> wrapper = new LambdaUpdateWrapper<>();
|
||||
wrapper.eq(GuestUser::getId, id)
|
||||
.set(GuestUser::getStatus, status)
|
||||
.set(GuestUser::getUpdateTime, LocalDateTime.now());
|
||||
return this.update(wrapper);
|
||||
}
|
||||
|
||||
@Override
|
||||
public GuestUser getOrCreateByDeviceInfo(String deviceId, String ipAddress, String userAgent) {
|
||||
GuestUser existing = getByDeviceId(deviceId);
|
||||
if (existing != null) {
|
||||
incrementVisitCount(existing.getId());
|
||||
return existing;
|
||||
}
|
||||
return createGuestUser(deviceId, ipAddress, userAgent, null);
|
||||
}
|
||||
|
||||
@Override
|
||||
public boolean cleanExpiredGuests(Integer days) {
|
||||
LocalDateTime cutoffTime = LocalDateTime.now().minusDays(days);
|
||||
LambdaUpdateWrapper<GuestUser> wrapper = new LambdaUpdateWrapper<>();
|
||||
wrapper.lt(GuestUser::getLastActiveTime, cutoffTime)
|
||||
.set(GuestUser::getIsDeleted, 1)
|
||||
.set(GuestUser::getUpdateTime, LocalDateTime.now());
|
||||
return this.update(wrapper);
|
||||
}
|
||||
|
||||
@Override
|
||||
public GuestUser createGuestUser(String deviceId, String ipAddress, String userAgent, String location) {
|
||||
GuestUser guest = GuestUser.builder()
|
||||
.id(UUID.randomUUID().toString())
|
||||
.deviceId(deviceId)
|
||||
.ipAddress(ipAddress)
|
||||
.userAgent(userAgent)
|
||||
.location(location)
|
||||
.status("active")
|
||||
.visitCount(1)
|
||||
.lastActiveTime(LocalDateTime.now())
|
||||
.build();
|
||||
this.save(guest);
|
||||
return guest;
|
||||
}
|
||||
}
|
||||
Reference in New Issue
Block a user