Kotlin：将含可空元素的List转为无空值HashMap的写法与疑问

嘿，我来帮你理清这些问题～

一、当前实现的正确性 #

如果你的代码是先通过filterNotNull()过滤掉所有null元素，再调用associateTo构建HashMap，那这个实现本身是完全正确的——因为过滤后的集合里全是非空的Somedata实例，不会出现null风险。

至于你疑惑的!!关键字，如果已经做了过滤还在用它，那这绝对是多余的，甚至是不好的代码习惯，完全可以删掉。

二、更优、更安全的实现方式 #

Kotlin标准库提供了更简洁的工具方法，不用手动创建HashMap再一步步填充，推荐这几种方式：

1. 最简洁的标准写法 #

如果你的列表已经过滤掉null元素，直接用associateBy就能一步到位：

// 默认返回LinkedHashMap，若需要HashMap可指定
val dataMap: HashMap<Long, Somedata> = list.filterNotNull()
    .associateByTo(HashMap(), keySelector = { it.id }, valueTransform = { it })

// 如果不需要特定的HashMap类型，直接用更短的写法：
val dataMap: Map<Long, Somedata> = list.filterNotNull().associateBy { it.id }

这个方法会自动按指定的key（这里是id）将元素映射到自身，可读性和简洁性拉满。

2. 处理重复ID的安全写法 #

如果你的列表里可能存在相同ID的Somedata元素，上面的方法会自动保留最后一个出现的元素，这可能不是你想要的。这种场景下，建议先分组再校验，避免意外覆盖数据：

val dataMap = list.filterNotNull()
    .groupBy { it.id }
    .mapValues { entry ->
        if (entry.value.size > 1) {
            throw IllegalArgumentException("发现重复ID: ${entry.key}")
        }
        entry.value.first()
    }

这样能提前发现数据异常，避免隐蔽的bug。

三、关于!!、?.和?.let的使用 #

为什么过滤后还会出现!!？ #

这大概率是冗余代码——filterNotNull()会把元素类型从Somedata?自动推导为Somedata，此时元素不可能为null，完全不需要用!!强制非空断言。

用?.或?.let的替代实现 #

如果不想提前过滤，而是希望在遍历过程中跳过null元素，?.let是个很优雅的选择：

val dataMap = HashMap<Long, Somedata>()
list.forEach { item ->
    item?.let { nonNullItem ->
        dataMap[nonNullItem.id] = nonNullItem
        // 这里还可以加一些针对非空元素的额外逻辑
    }
}

或者用更紧凑的链式调用写法，和filterNotNull()效果一致，但逻辑更连贯：

val dataMap = list.mapNotNull { it }
    .associateBy { it.id }

总结一下 #

• 已经过滤空值的情况下，!!完全没必要，属于冗余代码；
• 优先用标准库的associateBy/associateByTo，简洁又安全；
• 有重复ID风险时，一定要做校验，避免数据丢失；
• ?.let适合在遍历中处理非空元素的场景，和filterNotNull()各有适用场景。

内容的提问来源于stack exchange，提问作者Goltsev Eugene