【发布时间】:2021-02-17 09:13:02
【问题描述】:
你能告诉我我的方法是否正确吗?它有效,但我不知道它是否是正确的架构。我在某处读到,我们应该避免在负责创建片段/活动的函数上调用 viewmodel 函数,这主要是因为屏幕方向更改会调用网络请求,但我确实需要将参数从一个视图模型传递到另一个视图模型。重要的是我正在使用 Dagger Hilt 依赖注入,所以为每个视图模型创建工厂是不合理的?
假设我有项目的 RecyclerView 并且单击时我想启动带有详细信息的新片段 - 常见的事情。由于这些屏幕的逻辑很复杂,我决定将单个视图模型分成两个 - 一个用于列表片段,一个用于详细信息片段。
ItemsFragment 具有监听器并使用以下代码启动详细信息片段:
fun onItemSelected(item: Item) {
val args = Bundle().apply {
putInt(KEY_ITEM_ID, item.id)
}
findNavController().navigate(R.id.action_listFragment_to_detailsFragment, args)
}
然后在onViewCreated函数的ItemDetailsFragment类中,我收到传递的参数,将其保存在ItemDetailsViewModelitemId变量中,然后启动requestItemDetails()函数进行api调用,结果保存到LiveData,由ItemDetailsFragment
override fun onViewCreated(view: View, savedInstanceState: Bundle?) {
//...
val itemId = arguments?.getInt(KEY_ITEM_ID, -1) ?: -1
viewModel.itemId = itemId
viewModel.requestItemDetails()
//...
}
ItemDetailsViewModel
class ItemDetailsViewModel @ViewModelInject constructor(val repository: Repository) : ViewModel() {
var itemId: Int = -1
private val _item = MutableLiveData<Item>()
val item: LiveData<Item> = _item
fun requestItemDetails() {
if (itemId == -1) {
// return error state
return
}
viewModelScope.launch {
val response = repository.getItemDetails(itemId)
//...
_item.postValue(response.data)
}
}
}
【问题讨论】:
标签: android viewmodel android-livedata