优化 AdminUserApi 的实现，和 Cloud 保持一致

YunaiV
1 parent bdf579a2
Showing 3 changed files with 19 additions and 11 deletions
yudao-module-system/yudao-module-system-api/src/main/java/cn/iocoder/yudao/module/system/api/user/AdminUserApi.java
yudao-module-system/yudao-module-system-biz/src/main/java/cn/iocoder/yudao/module/system/api/user/AdminUserApiImpl.java
yudao-module-system/yudao-module-system-biz/src/main/java/cn/iocoder/yudao/module/system/convert/user/UserConvert.java
 package cn.iocoder.yudao.module.system.api.user;
+import cn.iocoder.yudao.framework.common.util.collection.CollectionUtils;
 import cn.iocoder.yudao.module.system.api.user.dto.AdminUserRespDTO;
 import java.util.Collection;
@@ -23,6 +24,14 @@ public interface AdminUserApi {
     AdminUserRespDTO getUser(Long id);
     /**
+     * 通过用户 ID 查询用户们
+     *
+     * @param ids 用户 ID 们
+     * @return 用户对象信息
+     */
+    List<AdminUserRespDTO> getUsers(Collection<Long> ids);
+
+    /**
      * 获得指定部门的用户数组
      *
      * @param deptIds 部门数组
@@ -44,7 +53,10 @@ public interface AdminUserApi {
      * @param ids 用户编号数组
      * @return 用户 Map
      */
-    Map<Long, AdminUserRespDTO> getUserMap(Collection<Long> ids);
+    default Map<Long, AdminUserRespDTO> getUserMap(Collection<Long> ids) {
+        List<AdminUserRespDTO> users = getUsers(ids);
+        return CollectionUtils.convertMap(users, AdminUserRespDTO::getId);
+    }
     /**
      * 校验用户们是否有效。如下情况，视为无效：
@@ -9,7 +9,6 @@ import org.springframework.stereotype.Service;
 import javax.annotation.Resource;
 import java.util.Collection;
 import java.util.List;
-import java.util.Map;
 import java.util.Set;
 /**
@@ -30,6 +29,12 @@ public class AdminUserApiImpl implements AdminUserApi {
     }
     @Override
+    public List<AdminUserRespDTO> getUsers(Collection<Long> ids) {
+        List<AdminUserDO> users = userService.getUsers(ids);
+        return UserConvert.INSTANCE.convertList4(users);
+    }
+
+    @Override
     public List<AdminUserRespDTO> getUsersByDeptIds(Collection<Long> deptIds) {
         List<AdminUserDO> users = userService.getUsersByDeptIds(deptIds);
         return UserConvert.INSTANCE.convertList4(users);
@@ -42,12 +47,6 @@ public class AdminUserApiImpl implements AdminUserApi {
     }
     @Override
-    public Map<Long, AdminUserRespDTO> getUserMap(Collection<Long> ids) {
-        Map<Long, AdminUserDO> userMap = userService.getUserMap(ids);
-        return UserConvert.INSTANCE.convertMap4(userMap);
-    }
-
-    @Override
     public void validUsers(Set<Long> ids) {
         userService.validUsers(ids);
     }
@@ -14,7 +14,6 @@ import org.mapstruct.Mapper;
 import org.mapstruct.factory.Mappers;
 import java.util.List;
-import java.util.Map;
 @Mapper
 public interface UserConvert {
@@ -53,6 +52,4 @@ public interface UserConvert {
     List<AdminUserRespDTO> convertList4(List<AdminUserDO> users);
-    Map<Long, AdminUserRespDTO> convertMap4(Map<Long, AdminUserDO> map);
-
 }