我应该在 ViewModel 中包含 LifecycleOwner 吗？

Question

我目前需要 LifecycleOwner 才能创建观察者。

我有在 ViewModel 中创建观察者的代码，因此在我的 Fragment 中检索 ViewModel 时我附加了 LifecycleOwner。

根据谷歌的文档。

注意：ViewModel 绝不能引用视图、生命周期或任何可能包含对活动上下文的引用的类。

我是否违反了该警告？如果违反了，您建议我以何种方式移动我创建的数据返回观察者？

我只做了一个观察者，所以我想知道它是否仍然有效。因为也在谷歌的文档中它也说了。

ViewModel 对象可以包含 LifecycleObservers，例如 LiveData 对象。

主片段

private lateinit var model: MainViewModel

/**
 * Observer for our ViewModel IpAddress LiveData value.
 * @see Observer.onChanged
 * */
private val ipObserver = Observer<String> {
    textIp.text = it
    hideProgressBar()
}

override fun onCreate(savedInstanceState: Bundle?) {
    super.onCreate(savedInstanceState)
    model = ViewModelProviders.of(this).get(MainViewModel::class.java)
    model.attach(this)
}

override fun onCreateView(inflater: LayoutInflater?, container: ViewGroup?, savedInstanceState: Bundle?): View? =
        inflater?.inflate(R.layout.fragment_main, container, false)

override fun onViewCreated(view: View?, savedInstanceState: Bundle?) {
    super.onViewCreated(view, savedInstanceState)

    buttonRetrieveIp.setOnClickListener {
        showProgressBar()
        model.fetchMyIp().observe(this, ipObserver) //Here we attach our ipObserver
    }
}

override fun showProgressBar() {

    textIp.visibility = View.GONE
    progressBar.visibility = View.VISIBLE
}

override fun hideProgressBar() {

    progressBar.visibility = View.GONE
    textIp.visibility = View.VISIBLE
}

主视图模型

private var ipAddress = MutableLiveData<String>()
private lateinit var owner: LifecycleOwner

fun attach(fragment: MainFragment) {
    owner = fragment
}

/**
 * For more information regarding Fuel Request using Fuel Routing and Live Data Response.
 * @see <a href="https://github.com/kittinunf/Fuel#routing-support">Fuel Routing Support</a>
 * @see <a href="https://github.com/kittinunf/Fuel#livedata-support">Fuel LiveData Support</a>
 * */
fun fetchMyIp(): LiveData<String> {

    Fuel.request(IpAddressApi.MyIp())
            .liveDataResponse()
            .observe(owner, Observer {

                if (it?.first?.statusCode == 200) {//If you want you can add a status code checker here.

                    it.second.success {

                        ipAddress.value = Ip.toIp(String(it))?.ip
                    }
                }
            })
    return ipAddress
}

更新 1：由于 @pskink 关于使用转换的建议改进了 ViewModel。

 private lateinit var ipAddress:LiveData<String>

/**
 * Improved ViewModel since January 23, 2018 credits to <a href="https://stackoverflow.com/users/2252830/pskink">pskink</a> <a href="
 *
 * For more information regarding Fuel Request using Fuel Routing and Live Data Response.
 * @see <a href="https://github.com/kittinunf/Fuel#routing-support">Fuel Routing Support</a>
 * @see <a href="https://github.com/kittinunf/Fuel#livedata-support">Fuel LiveData Support</a>
 * */
fun fetchMyIp(): LiveData<String> {

    ipAddress = Transformations.map(Fuel.request(IpAddressApi.MyIp()).liveDataResponse(), {

        var ip:String? = ""

            it.second.success {

                ip = Ip.toIp(String(it))?.ip
            }
        ip
    })

    return ipAddress
}

原文由 Hiroga Katageri 发布，翻译遵循 CC BY-SA 4.0 许可协议

Answer 1

No. If you wish to observe changes of some LiveData inside your ViewModel you can use observeForever() which doesn’t require LifecycleOwner .

请记住在 ViewModel 的 onCleared() 事件上删除此观察者：

 val observer = new Observer() {
  override public void onChanged(Integer integer) {
    //Do something with "integer"
  }
}

…

 liveData.observeForever(observer);

…

 override fun onCleared() {
    liveData.removeObserver(observer)
    super.onCleared()
}

非常好的 observe LiveData 示例参考。

原文由 Vitaliy A 发布，翻译遵循 CC BY-SA 4.0 许可协议

Answer 2

假设：

1. Fuel 指的是你的 ViewModel
2. Fuel.request(IpAddressApi.MyIp()) 是你的方法 ViewModel
3. IpAddressApi.MyIp() 没有对你的 LifecycleOwner 的引用，

如果一切都是真的，那么你就没有违反它。只要您不传递 LifecycleOwner 对 ViewModel 的引用，您就安全了！

LifecycleOwner - 与 Activity 或 Fragment 相关，因为它拥有各种 Android 生命周期，例如 onCreate、onPause、onDestroy 等

原文由 martinomburajr 发布，翻译遵循 CC BY-SA 3.0 许可协议