Skip to content

Revamp the UiState handling from the ViewModel to adapt multiple complex UI state logic #576

@luongvo

Description

@luongvo

Why

According to the problem in the recent projects to handle multiple complex UI state logic, such as skeleton loading with partial loading or separate/multiple error handling between data flows, we had revamped the UiState handling from the ViewModel to adapt and cover these cases. It's time to bring that new approach to replace the existing deprecated approach injectLoading from the base template.

/**
* To show loading manually, should call `hideLoading` after
*/
protected fun showLoading() {
if (loadingCount == 0) {
_isLoading.value = true
}
loadingCount++
}
/**
* To hide loading manually, should be called after `showLoading`
*/
protected fun hideLoading() {
loadingCount--
if (loadingCount == 0) {
_isLoading.value = false
}
}
protected fun launch(context: CoroutineContext = EmptyCoroutineContext, job: suspend () -> Unit) =
viewModelScope.launch(context) {
job.invoke()
}
protected fun <T> Flow<T>.injectLoading(): Flow<T> = this
.onStart { showLoading() }
.onCompletion { hideLoading() }

What we will do:

  • Define the base UiState for data flow execution states.
  • Define the base uiState StateFlow<HashMap<Int, UiState>> in BaseViewModel to handle separate UiState for each data flow. This helps avoid defining many states for all data flows in a ViewModel.
  • Sync all UiState with isLoading StateFlow to handle the "global loading state".

Who Benefits?

Developers

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

Relationships

None yet

Development

No branches or pull requests

Issue actions