Kotlin中mutableList操作后始终返回空列表问题排查

你的代码有3个核心问题，是导致valuesList始终为空、运行不符合预期的原因：

• 错误使用plus方法，未修改原列表
  Kotlin中List.plus()是纯函数操作，不会修改调用者本身，只会返回一个包含原内容+新增元素的全新列表。你代码中每次调用valuesList.plus(...)生成的新列表都没有被变量接收，初始创建的空valuesList从来没有被写入过任何元素，自然始终为空。
  如果要给可变列表就地追加元素，需要先把变量声明为MutableList<Test>类型，再调用add()方法，或者使用可变列表的+=（plusAssign）操作符。
• 存在语法错误
  你贴出的代码中两次valuesList.plus(Test(...调用都缺失闭合右括号，本身无法正常通过编译。
• 链式调用逻辑不符合预期
  你在map转换的lambda中往外部列表塞元素，后续直接接forEach的写法逻辑不成立：map的返回值是每次执行lambda时最后一行表达式的结果组成的集合，你当前lambda最后一行是valuesList.plus(...)的返回值——也就是每次基于空列表生成的、仅包含1个元素的临时列表，最终forEach遍历到的是5个零散的单元素临时列表，根本不是你想要的汇总后的完整值列表。

推荐用无副作用的集合转换写法，不需要额外声明外部可变列表，逻辑更清晰：

val valuesList = (1..5).flatMap { currValue ->
    // 此处执行你的值校验逻辑，校验结果可直接在当前作用域使用
    listOf(
        Test(currValue, Instant.now()),
        Test(/* 替换为你需要的prev value */, Instant.now())
    )
}
// 此时valuesList就是包含所有元素的完整列表，可直接遍历传入目标方法
valuesList.forEach { item ->
    // 调用目标方法处理item
}

如果你习惯用可变列表收集的写法，需要注意正确调用可变列表的追加方法：

// 明确声明为可变列表类型
val valuesList = mutableListOf<Test>()
(1..5).forEach { currValue ->
    // 此处执行值校验逻辑
    valuesList.add(Test(currValue, Instant.now()))
    valuesList.add(Test(/* 替换为你需要的prev value */, Instant.now()))
}
// 收集完成后再遍历使用
valuesList.forEach { item ->
    // 调用目标方法处理item
}

注意：尽量不要在map/flatMap等转换类操作中写修改外部变量的副作用逻辑，很容易引发预期外的问题，优先用纯转换的方式生成目标集合。

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