Usa viewLifecycleOwner come LifecycleOwner

Ho un frammento:

class MyFragment : BaseFragment() {

   // my StudentsViewModel instance
   lateinit var viewModel: StudentsViewModel

   override fun onCreateView(...){
        ...
   }

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

       viewModel = ViewModelProviders.of(this).get(StudentsViewModel::class.java)
       updateStudentList()
   }

   fun updateStudentList() {
        // Compiler error on 'this': Use viewLifecycleOwner as the LifecycleOwner
        viewModel.students.observe(this, Observer {
            //TODO: populate recycler view
        })
    }
}

Nel mio frammento, ho un'istanza di StudentsViewModel che viene avviata in onViewCreated(...).

In, StudentsViewModel, studentsè una LiveData:

class StudentsViewModel : ViewModel() {
    val students = liveData(Dispatchers.IO) {
          ...
    }
}

Tornando a MyFragment, nella funzione updateStudentList()ricevo un errore del compilatore che si lamenta del thisparametro passato a .observe(this, Observer{...})quelloUse viewLifecycleOwner as the LifecycleOwner

Perché visualizzo questo errore? Come sbarazzarsene?

Perché visualizzo questo errore?

Lint consiglia di utilizzare il ciclo di vita delle viste del frammento ( viewLifecycleOwner) anziché il ciclo di vita del frammento stesso ( this). Ian Lake e Jeremy Woods di Google esaminano la differenza come parte di questa presentazione del Summit degli sviluppatori Android , e Ibrahim Yilmaz copre le differenze in questo post Medium In breve:

• viewLifecycleOwnerè legato a quando il frammento ha (e perde) la sua UI ( onCreateView(), onDestroyView())

• thisè legato al ciclo di vita complessivo del frammento ( onCreate(), onDestroy()), che può essere sostanzialmente più lungo

Come sbarazzarsene?

Sostituire:

viewModel.students.observe(this, Observer {
        //TODO: populate recycler view
    })

con:

viewModel.students.observe(viewLifecycleOwner, Observer {
        //TODO: populate recycler view
    })

Nel tuo codice attuale, se onDestroyView()viene chiamato, ma onDestroy()non lo è, continuerai a osservare LiveData, forse l'arresto anomalo quando provi a popolare un inesistente RecyclerView. Usando viewLifecycleOwner, eviti quel rischio.

Invece di thisusare viewLifecycleOwnerper osservareLiveData

viewModel.students.observe(viewLifecycleOwner, Observer {
    //TODO: populate recycler view
})