如何修复Jetpack Compose中的"Unsupported concurrent change during composition"错误？

错误信息 #

Recomposer.applyAndCheck
java.lang.IllegalStateException - Unsupported concurrent change during composition. A state object was modified by composition as well as being modified outside composition.

问题本质与viewModelScope的作用 #

这个错误属于Compose状态并发冲突：同一个状态对象同时被「Compose重组流程内」和「重组流程外」的代码修改，或是多个非主线程协程同时修改状态导致的。

viewModelScope不是强制要求，但它是推荐的状态更新上下文：

• 它默认绑定Main调度器，确保状态更新在主线程执行（Compose状态必须在主线程修改）；
• 生命周期与ViewModel绑定，避免内存泄漏；
• 但并非唯一选项——比如LaunchedEffect内的协程（同样默认主线程）更新状态也是合法的，核心要求是：状态更新必须在主线程，且不能在重组过程中直接修改。

代码片段分析 #

事件处理代码

原代码：

is HomeEvent.OnFreeCouponDeclined -> {
    homeState.update {
        copy(isDeclined = false)
    }
}

如果这段代码是在ViewModel的事件处理逻辑中（比如接收UI发送的事件），直接调用update是没问题的——因为ViewModel的事件触发通常在主线程。但如果这段代码是在非主线程协程或**@Composable函数内部（重组过程中）**执行，就会触发错误。

修改后的代码：

is HomeEvent.OnFreeCouponDeclined -> {
    viewModelScope.launch {
        homeState.update {
            copy(isDeclined = false)
        }
    }
}

属于冗余写法——viewModelScope默认就是主线程调度，除非当前上下文被切换到了其他线程，否则没必要额外套launch。

库存监听代码

这段代码存在明显问题：

private fun observeInventory() {
    viewModelScope.launch {
        (Dispatchers.Default) {
            getInventoryInteractor.invoke().collectLatest {
                (Dispatchers.Main) {
                    homeState.value = homeState.value.copy(categories = it.map {
                        it.copy(subCategories = emptyList())
                    })
                }
            }
        }
    }
}

1. 语法错误：(Dispatchers.Default)和(Dispatchers.Main)写法无效，正确应该用withContext(Dispatchers.XXX)切换上下文；
2. 线程逻辑冗余：数据转换（map）可以放在Default线程执行，避免占用主线程，但当前写法会导致线程切换混乱；
3. 潜在并发风险：如果流在多线程发射数据，或同时有其他地方修改homeState，可能引发冲突。

优化后的规范写法：

private fun observeInventory() {
    viewModelScope.launch {
        getInventoryInteractor.invoke()
            .map { inventoryList ->
                // 数据处理放在Default线程
                inventoryList.map { it.copy(subCategories = emptyList()) }
            }
            .flowOn(Dispatchers.Default)
            .collectLatest { processedCategories ->
                // 主线程更新状态
                homeState.value = homeState.value.copy(categories = processedCategories)
            }
    }
}

自动轮播代码

这段代码存在冗余和潜在风险：

LaunchedEffect("${state.currentPage}_${sliderHoldTimestamp}_${sliderItems.size}") {
    delay(AUTO_SLIDER_DELAY)
    coroutineScope.launch {
        if (sliderItems.isNotEmpty() && state.pageCount != 0) {
            var newPosition = state.currentPage + 1
            if (newPosition > sliderItems.size - 1) newPosition = 0
            state.animateScrollToPage(newPosition.mod(state.pageCount))
        }
    }
}

1. LaunchedEffect Key不规范：字符串拼接可能导致不必要的协程重启（比如Long类型转字符串的精度问题），建议直接传多个参数作为Key：LaunchedEffect(state.currentPage, sliderHoldTimestamp, sliderItems.size)；
2. 冗余协程嵌套：LaunchedEffect本身就是协程上下文，不需要额外调用coroutineScope.launch，否则可能导致多个协程同时修改Pager状态；
3. 逻辑可简化：用mod直接计算新位置，避免冗余判断。

优化后代码：

LaunchedEffect(state.currentPage, sliderHoldTimestamp, sliderItems.size) {
    delay(AUTO_SLIDER_DELAY)
    if (sliderItems.isNotEmpty() && state.pageCount != 0) {
        val newPosition = (state.currentPage + 1).mod(sliderItems.size)
        state.animateScrollToPage(newPosition)
    }
}

通用排查建议 #

• 强制主线程更新：在状态更新前可加检查：check(Looper.getMainLooper().isCurrentThread)，快速定位非主线程更新的问题；
• 禁止重组中直接改状态：@Composable函数内只能读取状态，修改必须通过LaunchedEffect等副作用API或ViewModel处理；
• 统一状态更新上下文：优先用viewModelScope或LaunchedEffect协程，避免零散使用GlobalScope等无绑定生命周期的协程；
• 原子更新状态：修改复杂状态时，优先用MutableState.update（原子操作）替代直接赋值value = ...，避免并发赋值冲突；
• 流处理规范：网络/数据库流尽量用flowOn指定数据处理线程，最终在主线程collect并更新状态。

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