banner省份都删除报错

liquidnet-bus-client/liquidnet-client-admin/liquidnet-client-admin-zhengzai/src/main/java/com/liquidnet/client/admin/zhengzai/kylin/service/impl/KylinBannersServiceImpl.java

@@ -32,6 +32,7 @@ import org.springframework.data.mongodb.core.convert.MongoConverter;
 import org.springframework.data.mongodb.core.query.Criteria;
 import org.springframework.data.mongodb.core.query.Query;
 import org.springframework.stereotype.Service;
+import org.springframework.util.CollectionUtils;
 import org.springframework.util.StringUtils;
 
 import java.time.LocalDateTime;
@@ -94,8 +95,8 @@ public class KylinBannersServiceImpl extends ServiceImpl<KylinBannersMapper, Kyl
                 kylinBannersRelationsMapper.insert(kylinBannersRelations);
             }
             List<String> provincesList = bannersParam.getProvincesList();
-            List<String> provincesNameList = bannersParam.getProvincesNameList();
-            if (!provincesList.isEmpty()) {
+            if (!CollectionUtils.isEmpty(provincesList)) {
+                List<String> provincesNameList = bannersParam.getProvincesNameList();
                 for (int i = 0; i < provincesList.size(); i++) {
                     kylinBannersRelations.setFieldId(provincesList.get(i));
                     kylinBannersRelations.setFieldIdType(2);
@@ -167,12 +168,13 @@ public class KylinBannersServiceImpl extends ServiceImpl<KylinBannersMapper, Kyl
                 kylinBannersRelationsMapper.insert(kylinBannersRelations);
             }
             List<String> provincesList = bannersParam.getProvincesList();
-            List<String> provincesNameList = bannersParam.getProvincesNameList();
-            if (!provincesList.isEmpty()) {
+            if (CollectionUtils.isEmpty(provincesList)) {
                 // 入数据库 先删除之前的
                 kylinBannersRelationsMapper.delete(
                         new UpdateWrapper<KylinBannersRelations>().eq("banners_id", bannersId).eq("field_id_type", 2)
                 );
+            } else {
+                List<String> provincesNameList = bannersParam.getProvincesNameList();
                 if (!provincesList.isEmpty()) {
                     for (int i = 0; i < provincesList.size(); i++) {
                         kylinBannersRelations.setFieldId(provincesList.get(i));