List.contains踩坑

背景

表结构

存在字段 产品经理id ，存储所有的产品经理id，例如 31834,41437,44710

复制代码

`pm_user_ids` varchar(50)  DEFAULT NULL COMMENT '产品经理id',

代码

java 复制代码

private boolean isPM(Long userId) {
    List<String> pmUserIds= projectResponsiblePersonMapper.pmAllUserIds();
    if (pmUserIds.contains(String.valueOf(userId))) {
        log.debug("用户ID为：{}，是产品经理", userId);
        return true;
    } else {
        log.debug("用户ID为：{}，不是产品经理", userId);
        return false;
    }
}

sql 复制代码

<select id="pmAllUserIds" resultType="java.lang.String">
    select
        pm_user_ids
    from inja_project_responsible_person
</select>

问题

userId=44710，判断时，被判断为不是产品经理

原因

查询返回的是pm_user_ids字段的值，而不是用户ID列表。

从数据库表结构可以看出，pm_user_ids是一个包含多个用户ID的字符串（用逗号分隔），而不是单个用户ID。

java 复制代码

List<String> pmUserIds= projectResponsiblePersonMapper.pmAllUserIds();

这个方法获取到的是一个List，其中每个元素都是一个pm_user_ids字符串（如"45,46,47"），而不是单个用户ID的列表（如["45", "46", "47"]）。

所以当检查pmUserIds.contains("47")时，实际上是在检查List中是否有元素恰好等于"47"，而不是检查是否有元素包含"47"。

解决

修改pmAllUserIds查询，将所有pm_user_ids字段中的用户ID提取出来并去重。

xml 复制代码

<select id="pmAllUserIds" resultType="java.lang.String">
    SELECT DISTINCT SUBSTRING_INDEX(SUBSTRING_INDEX(pm_user_ids, ',', numbers.n), ',', -1) AS user_id
    FROM inja_project_responsible_person
    CROSS JOIN (
        SELECT 1 n UNION SELECT 2 UNION SELECT 3 UNION SELECT 4 UNION SELECT 5
        UNION SELECT 6 UNION SELECT 7 UNION SELECT 8 UNION SELECT 9 UNION SELECT 10
    ) numbers
    WHERE CHAR_LENGTH(pm_user_ids) - CHAR_LENGTH(REPLACE(pm_user_ids, ',', '')) >= numbers.n - 1
    AND pm_user_ids IS NOT NULL
    AND pm_user_ids != ''
</select>

或者修改java代码

java 复制代码

// ... existing code ...
    private boolean isPM(Long userId) {
        List<String> pmUserIds = projectResponsiblePersonMapper.pmAllUserIds();
        String userIdStr = String.valueOf(userId);
        
        // 遍历所有pm_user_ids字符串，检查是否包含当前用户ID
        for (String pmIds : pmUserIds) {
            if (pmIds != null && !pmIds.isEmpty()) {
                // 按逗号分割并检查是否包含userId
                String[] ids = pmIds.split(",");
                for (String id : ids) {
                    if (id.trim().equals(userIdStr)) {
                        log.debug("用户ID为：{}，是产品经理", userId);
                        return true;
                    }
                }
            }
        }
        
        log.debug("用户ID为：{}，不是产品经理", userId);
        return false;
    }
// ... existing code ...