diff --git a/continew-module-system/src/main/java/top/continew/admin/system/service/impl/DictItemServiceImpl.java b/continew-module-system/src/main/java/top/continew/admin/system/service/impl/DictItemServiceImpl.java index 4aebb310..461b98e4 100644 --- a/continew-module-system/src/main/java/top/continew/admin/system/service/impl/DictItemServiceImpl.java +++ b/continew-module-system/src/main/java/top/continew/admin/system/service/impl/DictItemServiceImpl.java @@ -16,6 +16,7 @@ package top.continew.admin.system.service.impl; +import cn.hutool.core.collection.CollUtil; import cn.hutool.core.util.ClassUtil; import cn.hutool.core.util.StrUtil; import com.alicp.jetcache.anno.Cached; @@ -77,6 +78,9 @@ public class DictItemServiceImpl extends BaseServiceImpl dictIds) { + if (CollUtil.isEmpty(dictIds)) { + return; + } baseMapper.lambdaUpdate().in(DictItemDO::getDictId, dictIds).remove(); RedisUtils.deleteByPattern(CacheConstants.DICT_KEY_PREFIX + StringConstants.ASTERISK); } diff --git a/continew-module-system/src/main/java/top/continew/admin/system/service/impl/FileServiceImpl.java b/continew-module-system/src/main/java/top/continew/admin/system/service/impl/FileServiceImpl.java index 08548a4a..42419d25 100644 --- a/continew-module-system/src/main/java/top/continew/admin/system/service/impl/FileServiceImpl.java +++ b/continew-module-system/src/main/java/top/continew/admin/system/service/impl/FileServiceImpl.java @@ -125,6 +125,9 @@ public class FileServiceImpl extends BaseServiceImpl storageIds) { + if (CollUtil.isEmpty(storageIds)) { + return 0L; + } return baseMapper.lambdaQuery().in(FileDO::getStorageId, storageIds).count(); } diff --git a/continew-module-system/src/main/java/top/continew/admin/system/service/impl/MessageUserServiceImpl.java b/continew-module-system/src/main/java/top/continew/admin/system/service/impl/MessageUserServiceImpl.java index 134baa21..05801de7 100644 --- a/continew-module-system/src/main/java/top/continew/admin/system/service/impl/MessageUserServiceImpl.java +++ b/continew-module-system/src/main/java/top/continew/admin/system/service/impl/MessageUserServiceImpl.java @@ -80,6 +80,9 @@ public class MessageUserServiceImpl implements MessageUserService { @Override public void readMessage(List ids) { + if (CollUtil.isEmpty(ids)) { + return; + } baseMapper.lambdaUpdate() .set(MessageUserDO::getIsRead, true) .set(MessageUserDO::getReadTime, LocalDateTime.now()) @@ -90,6 +93,9 @@ public class MessageUserServiceImpl implements MessageUserService { @Override public void deleteByMessageIds(List messageIds) { + if (CollUtil.isEmpty(messageIds)) { + return; + } baseMapper.lambdaUpdate().in(MessageUserDO::getMessageId, messageIds).remove(); } } \ No newline at end of file diff --git a/continew-module-system/src/main/java/top/continew/admin/system/service/impl/RoleDeptServiceImpl.java b/continew-module-system/src/main/java/top/continew/admin/system/service/impl/RoleDeptServiceImpl.java index 5d94fada..9e158e17 100644 --- a/continew-module-system/src/main/java/top/continew/admin/system/service/impl/RoleDeptServiceImpl.java +++ b/continew-module-system/src/main/java/top/continew/admin/system/service/impl/RoleDeptServiceImpl.java @@ -62,12 +62,18 @@ public class RoleDeptServiceImpl implements RoleDeptService { @Override @Transactional(rollbackFor = Exception.class) public void deleteByRoleIds(List roleIds) { + if (CollUtil.isEmpty(roleIds)) { + return; + } baseMapper.lambdaUpdate().in(RoleDeptDO::getRoleId, roleIds).remove(); } @Override @Transactional(rollbackFor = Exception.class) public void deleteByDeptIds(List deptIds) { + if (CollUtil.isEmpty(deptIds)) { + return; + } baseMapper.lambdaUpdate().in(RoleDeptDO::getDeptId, deptIds).remove(); } diff --git a/continew-module-system/src/main/java/top/continew/admin/system/service/impl/RoleMenuServiceImpl.java b/continew-module-system/src/main/java/top/continew/admin/system/service/impl/RoleMenuServiceImpl.java index f5f3ff0b..0154ac89 100644 --- a/continew-module-system/src/main/java/top/continew/admin/system/service/impl/RoleMenuServiceImpl.java +++ b/continew-module-system/src/main/java/top/continew/admin/system/service/impl/RoleMenuServiceImpl.java @@ -64,6 +64,9 @@ public class RoleMenuServiceImpl implements RoleMenuService { @Override @Transactional(rollbackFor = Exception.class) public void deleteByRoleIds(List roleIds) { + if (CollUtil.isEmpty(roleIds)) { + return; + } baseMapper.lambdaUpdate().in(RoleMenuDO::getRoleId, roleIds).remove(); } diff --git a/continew-module-system/src/main/java/top/continew/admin/system/service/impl/RoleServiceImpl.java b/continew-module-system/src/main/java/top/continew/admin/system/service/impl/RoleServiceImpl.java index dad7e75f..11142df4 100644 --- a/continew-module-system/src/main/java/top/continew/admin/system/service/impl/RoleServiceImpl.java +++ b/continew-module-system/src/main/java/top/continew/admin/system/service/impl/RoleServiceImpl.java @@ -158,6 +158,9 @@ public class RoleServiceImpl extends BaseServiceImpl listNameByIds(List ids) { + if (CollUtil.isEmpty(ids)) { + return Collections.emptyList(); + } List roleList = baseMapper.lambdaQuery().select(RoleDO::getName).in(RoleDO::getId, ids).list(); return roleList.stream().map(RoleDO::getName).toList(); } @@ -165,6 +168,9 @@ public class RoleServiceImpl extends BaseServiceImpl listCodeByUserId(Long userId) { List roleIdList = userRoleService.listRoleIdByUserId(userId); + if (CollUtil.isEmpty(roleIdList)) { + return Collections.emptySet(); + } List roleList = baseMapper.lambdaQuery().select(RoleDO::getCode).in(RoleDO::getId, roleIdList).list(); return roleList.stream().map(RoleDO::getCode).collect(Collectors.toSet()); } @@ -172,6 +178,9 @@ public class RoleServiceImpl extends BaseServiceImpl listByUserId(Long userId) { List roleIdList = userRoleService.listRoleIdByUserId(userId); + if (CollUtil.isEmpty(roleIdList)) { + return Collections.emptySet(); + } List roleList = baseMapper.lambdaQuery().in(RoleDO::getId, roleIdList).list(); return new HashSet<>(BeanUtil.copyToList(roleList, RoleContext.class)); } diff --git a/continew-module-system/src/main/java/top/continew/admin/system/service/impl/UserPasswordHistoryServiceImpl.java b/continew-module-system/src/main/java/top/continew/admin/system/service/impl/UserPasswordHistoryServiceImpl.java index 3d276843..e7d91136 100644 --- a/continew-module-system/src/main/java/top/continew/admin/system/service/impl/UserPasswordHistoryServiceImpl.java +++ b/continew-module-system/src/main/java/top/continew/admin/system/service/impl/UserPasswordHistoryServiceImpl.java @@ -54,6 +54,9 @@ public class UserPasswordHistoryServiceImpl implements UserPasswordHistoryServic @Override public void deleteByUserIds(List userIds) { + if (CollUtil.isEmpty(userIds)) { + return; + } baseMapper.lambdaUpdate().in(UserPasswordHistoryDO::getUserId, userIds).remove(); } diff --git a/continew-module-system/src/main/java/top/continew/admin/system/service/impl/UserRoleServiceImpl.java b/continew-module-system/src/main/java/top/continew/admin/system/service/impl/UserRoleServiceImpl.java index 9f43513f..c9a4e623 100644 --- a/continew-module-system/src/main/java/top/continew/admin/system/service/impl/UserRoleServiceImpl.java +++ b/continew-module-system/src/main/java/top/continew/admin/system/service/impl/UserRoleServiceImpl.java @@ -91,6 +91,9 @@ public class UserRoleServiceImpl implements UserRoleService { @Override public void deleteByUserIds(List userIds) { + if (CollUtil.isEmpty(userIds)) { + return; + } baseMapper.lambdaUpdate().in(UserRoleDO::getUserId, userIds).remove(); } @@ -124,6 +127,9 @@ public class UserRoleServiceImpl implements UserRoleService { @Override public boolean isRoleIdExists(List roleIds) { + if (CollUtil.isEmpty(roleIds)) { + return false; + } return baseMapper.lambdaQuery().in(UserRoleDO::getRoleId, roleIds).exists(); } } diff --git a/continew-module-system/src/main/java/top/continew/admin/system/service/impl/UserServiceImpl.java b/continew-module-system/src/main/java/top/continew/admin/system/service/impl/UserServiceImpl.java index 0c3d0eb9..13289052 100644 --- a/continew-module-system/src/main/java/top/continew/admin/system/service/impl/UserServiceImpl.java +++ b/continew-module-system/src/main/java/top/continew/admin/system/service/impl/UserServiceImpl.java @@ -463,6 +463,9 @@ public class UserServiceImpl extends BaseServiceImpl deptIds) { + if (CollUtil.isEmpty(deptIds)) { + return 0L; + } return baseMapper.lambdaQuery().in(UserDO::getDeptId, deptIds).count(); }