diff --git a/src/main/java/com/qs/serve/modules/bir/service/impl/BirActivityCenterGoodsServiceImpl.java b/src/main/java/com/qs/serve/modules/bir/service/impl/BirActivityCenterGoodsServiceImpl.java
index 78ef61b1..d4217169 100644
--- a/src/main/java/com/qs/serve/modules/bir/service/impl/BirActivityCenterGoodsServiceImpl.java
+++ b/src/main/java/com/qs/serve/modules/bir/service/impl/BirActivityCenterGoodsServiceImpl.java
@@ -4,6 +4,7 @@ import com.baomidou.mybatisplus.core.conditions.query.LambdaQueryWrapper;
 import com.baomidou.mybatisplus.core.conditions.query.QueryWrapper;
 import com.baomidou.mybatisplus.extension.service.impl.ServiceImpl;
 import com.qs.serve.common.model.dto.DateSplitDTO;
+import com.qs.serve.common.util.CollectionUtil;
 import com.qs.serve.common.util.DateSplitUtil;
 import com.qs.serve.common.util.PageUtil;
 import com.qs.serve.modules.bir.entity.BirBaseActivity;
@@ -203,23 +204,25 @@ public class BirActivityCenterGoodsServiceImpl extends ServiceImpl<BirActivityCe
     }
 
     private void buildRemoveRecode(List<Long> activityIds) {
-        LambdaQueryWrapper<BirActivityCenterGoods> birBaseActivityLqw = new LambdaQueryWrapper<>();
-        birBaseActivityLqw.in(BirActivityCenterGoods::getActivityId, activityIds);
-
-        List<BirActivityCenterGoods> removeBirList = this.list(birBaseActivityLqw);
-        if(removeBirList.size()>0){
-            List<BirRemoveId> removeIds = removeBirList.stream().map(a-> {
-                BirRemoveId birRemoveId = new BirRemoveId();
-                birRemoveId.setDeleteTime(LocalDateTime.now());
-                birRemoveId.setId(a.getId());
-                return birRemoveId;
-            }).collect(Collectors.toList());
-
-            removeIds.forEach(a->{
-                birRemoveIdMapper.insert(a);
-            });
+        if(CollectionUtil.isNotEmpty(activityIds)) {
+            LambdaQueryWrapper<BirActivityCenterGoods> birBaseActivityLqw = new LambdaQueryWrapper<>();
+            birBaseActivityLqw.in(BirActivityCenterGoods::getActivityId, activityIds);
+
+            List<BirActivityCenterGoods> removeBirList = this.list(birBaseActivityLqw);
+            if (removeBirList.size() > 0) {
+                List<BirRemoveId> removeIds = removeBirList.stream().map(a -> {
+                    BirRemoveId birRemoveId = new BirRemoveId();
+                    birRemoveId.setDeleteTime(LocalDateTime.now());
+                    birRemoveId.setId(a.getId());
+                    return birRemoveId;
+                }).collect(Collectors.toList());
+
+                removeIds.forEach(a -> {
+                    birRemoveIdMapper.insert(a);
+                });
+            }
+            this.remove(birBaseActivityLqw);
         }
-        this.remove(birBaseActivityLqw);
     }
 
     /**