Kotlin中设置MutableList数据是否存在更优实现方式？

当前方案的正确性与可优化点 #

你的现有代码逻辑上是正确的，能实现对传入MutableList的元素修改，但存在两处可优化的地方：

1. 循环中对同一个索引位置做了两次赋值，完全可以合并为一次，避免重复操作集合元素
2. 直接修改传入集合的方式带有副作用，调用方必须明确知道这个函数会改动原集合的内容

优化后的原地修改版本可以这样写：

fun setParameters(myArray: MutableList<Symbol>, instance: MutationData){ 
    myArray.forEachIndexed { index, valArr ->
        // 合并两次转换操作，仅赋值一次
        val finalValue = setValueForByReference(instance, setValues(valArr, members?.settings))
        myArray[index] = finalValue
    }
}

或者用Kotlin标准库的mapIndexedTo函数，更简洁地完成原地修改：

fun setParameters(myArray: MutableList<Symbol>, instance: MutationData){ 
    myArray.mapIndexedTo(myArray) { _, valArr ->
        setValueForByReference(instance, setValues(valArr, members?.settings))
    }
}

返回新集合的更优方案 #

如果业务场景允许，返回新集合的方式通常更推荐，这符合函数式编程的无副作用原则，代码更安全、易维护，原集合的状态不会被意外修改。

实现代码如下：

// 若只需要只读List
fun getUpdatedParameters(original: List<Symbol>, instance: MutationData): List<Symbol> {
    return original.map { valArr ->
        setValueForByReference(instance, setValues(valArr, members?.settings))
    }
}

// 若需要MutableList，可追加toMutableList()
fun getUpdatedParameters(original: List<Symbol>, instance: MutationData): MutableList<Symbol> {
    return original.map { valArr ->
        setValueForByReference(instance, setValues(valArr, members?.settings))
    }.toMutableList()
}

两种方案的适用场景 #

• 原地修改集合：适合原集合本身就是需要维护状态的可变对象，且调用方明确知晓会修改原集合的场景，优点是无需额外创建集合对象，内存开销略小。
• 返回新集合：适合大多数业务场景，尤其是不需要保留原集合修改痕迹的情况，代码更清晰，避免了副作用带来的潜在bug，是更符合Kotlin编程风格的选择。

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