Merge pull request #116 from ballade0d/dev-dylan

feat: cache for EventService#getEvents
NJUPT-SAST · Feb 16, 2024 · a76fbb9 · a76fbb9
2 parents cc70e53 + b294b65
commit a76fbb9
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 1 deletion.
diff --git a/src/main/java/sast/evento/service/impl/DepartmentServiceImpl.java b/src/main/java/sast/evento/service/impl/DepartmentServiceImpl.java
@@ -3,6 +3,7 @@
 import com.baomidou.mybatisplus.core.conditions.query.LambdaQueryWrapper;
 import com.baomidou.mybatisplus.core.toolkit.Wrappers;
 import jakarta.annotation.Resource;
+import org.springframework.cache.annotation.CacheEvict;
 import org.springframework.stereotype.Service;
 import sast.evento.common.enums.ErrorEnum;
 import sast.evento.entitiy.Department;
@@ -56,6 +57,7 @@ public List<Department> getDepartments() {
     }
 
     @Override
+    @CacheEvict(value = "event")
     public void putDepartment(Integer departmentId, String departmentName) {
         if (departmentMapper.updateById(new Department(departmentId, departmentName)) < 1) {
             throw new LocalRunTimeException(ErrorEnum.COMMON_ERROR, "update failed");

diff --git a/src/main/java/sast/evento/service/impl/EventServiceImpl.java b/src/main/java/sast/evento/service/impl/EventServiceImpl.java
@@ -3,6 +3,8 @@
 import com.baomidou.mybatisplus.core.conditions.query.QueryWrapper;
 import com.baomidou.mybatisplus.core.conditions.update.UpdateWrapper;
 import jakarta.annotation.Resource;
+import org.springframework.cache.annotation.CacheEvict;
+import org.springframework.cache.annotation.Cacheable;
 import org.springframework.stereotype.Service;
 import org.springframework.transaction.annotation.Transactional;
 import sast.evento.common.enums.ActionState;
@@ -108,6 +110,7 @@ public List<EventModel> getNewest() {
 
     // 获取活动列表(分页）
     @Override
+    @Cacheable(value = "event", key = "#page + #size")
     public PageModel<EventModel> getEvents(Integer page, Integer size) {
         if (page == null || page < 0 || size == null || size < 0) {
             throw new LocalRunTimeException(ErrorEnum.PARAM_ERROR);
@@ -147,6 +150,7 @@ public List<EventModel> getSubscribed(String userId) {
 
     @Transactional(rollbackFor = Exception.class)
     @Override
+    @CacheEvict(value = "event")
     public Integer addEvent(EventModel eventModel, String userId) {
         Event event = new Event(eventModel);
         /* 检测必需参数是否存在 */
@@ -204,6 +208,7 @@ public Integer addEvent(EventModel eventModel, String userId) {
 
     @Transactional(rollbackFor = Exception.class)
     @Override
+    @CacheEvict(value = "event")
     public Boolean deleteEvent(Integer eventId) {
         if (eventId == null) {
             throw new LocalRunTimeException(ErrorEnum.PARAM_ERROR);
@@ -223,6 +228,7 @@ public Boolean deleteEvent(Integer eventId) {
 
     @Transactional(rollbackFor = Exception.class)
     @Override
+    @CacheEvict(value = "event")
     public Boolean updateEvent(EventModel eventModel) {
         Event event = new Event(eventModel);
 
@@ -313,6 +319,7 @@ public Boolean updateEvent(EventModel eventModel) {
     }
 
     @Override
+    @CacheEvict(value = "event")
     public void updateEvent(Event event) {
         UpdateWrapper<Event> updateWrapper = new UpdateWrapper<>();
         updateWrapper.eq("id", event.getId());
@@ -321,6 +328,7 @@ public void updateEvent(Event event) {
 
 
     @Override
+    @CacheEvict(value = "event")
     public Boolean cancelEvent(Integer eventId) {
         if (eventId == null) {
             throw new LocalRunTimeException(ErrorEnum.PARAM_ERROR);

diff --git a/src/main/java/sast/evento/service/impl/EventTypeServiceImpl.java b/src/main/java/sast/evento/service/impl/EventTypeServiceImpl.java
@@ -2,6 +2,7 @@
 
 import com.baomidou.mybatisplus.core.conditions.update.UpdateWrapper;
 import jakarta.annotation.Resource;
+import org.springframework.cache.annotation.CacheEvict;
 import org.springframework.stereotype.Service;
 import sast.evento.common.enums.ErrorEnum;
 import sast.evento.entitiy.EventType;
@@ -51,6 +52,7 @@ public List<EventType> getAllEventType() {
     }
 
     @Override
+    @CacheEvict(value = "event")
     public Boolean editEventType(EventType eventType) {
         UpdateWrapper<EventType> updateWrapper = new UpdateWrapper<>();
         updateWrapper.eq("id", eventType.getId());

diff --git a/src/main/java/sast/evento/service/impl/LocationServiceImpl.java b/src/main/java/sast/evento/service/impl/LocationServiceImpl.java
@@ -3,6 +3,7 @@
 import com.baomidou.mybatisplus.core.conditions.update.UpdateWrapper;
 import com.baomidou.mybatisplus.core.toolkit.Wrappers;
 import jakarta.annotation.Resource;
+import org.springframework.cache.annotation.CacheEvict;
 import org.springframework.stereotype.Service;
 import sast.evento.common.enums.ErrorEnum;
 import sast.evento.entitiy.Location;
@@ -61,6 +62,7 @@ public Boolean deleteLocation(Integer id) {
     }
 
     @Override
+    @CacheEvict(value = "event")
     public Boolean updateLocation(Location location) {
         if (location == null) {
             throw new LocalRunTimeException(ErrorEnum.PARAM_ERROR);

diff --git a/src/main/java/sast/evento/service/impl/PermissionServiceImpl.java b/src/main/java/sast/evento/service/impl/PermissionServiceImpl.java
@@ -2,6 +2,7 @@
 
 import com.baomidou.mybatisplus.core.toolkit.Wrappers;
 import com.baomidou.mybatisplus.extension.plugins.pagination.Page;
+import fun.feellmoose.enums.Organization;
 import jakarta.annotation.Resource;
 import org.springframework.stereotype.Service;
 import sast.evento.common.enums.ActionState;
@@ -15,7 +16,6 @@
 import sast.evento.model.treeDataNodeDTO.TreeDataNode;
 import sast.evento.service.PermissionService;
 import sast.evento.service.PermissionServiceCacheAble;
-import sast.sastlink.sdk.enums.Organization;
 
 import java.util.*;
 import java.util.stream.Collectors;