Question

我有传入的参数List<Somedata>。
Somedata类包含id字段。

我的目标是从此HashMap<Somedata.id, Somedata>生成list。

接下来的方法是正确的还是有更好/更安全的方法来做到这一点？

list
    .filter { it.id != null }
    .associateTo(HashMap(), {it.id!! to it})

实际上，我无法理解，为什么我应该在!!方法中使用associateTo关键字，上面我只使用非空值过滤它。

或者也许通过?.或?.let关键字有一种很好的方式来执行此操作？

Answer 1

你可以这样做：

list.mapNotNull { e -> e.id?.let { it to e } }.toMap()

故障：

如果元素为空，则使用.let安全调用运算符调用?.将生成结果null。

因此传递给mapNotNull的lambda的类型为(Somedata) -> Pair<IdType, Somedata>。

mapNotNull会丢弃空对，而toMap会将结果List<Pair<IdType, Somedata>>转换为Map<IdType, Somedata>。

如果您想避免创建一个中间列表来保存对，您可以从一开始就将列表变为惰性Sequence：

list.asSequence().mapNotNull { e -> e.id?.let { it to e } }.toMap()

或者，因为你问：

我为什么要用!!在associateTo方法中的关键字，在上面我只用非空值过滤它。

这是因为列表仍然是List<Somedata>类型 - 这没有说明字段本身的可空性。在执行id调用时，编译器不知道associateTo字段仍然不为空。