Будет ли утечка при использовании Shared ViewModel?

Question

[Евгений]

Добрый вечер. Долго искал вопрос на просторах интернета и не нашел ответа и решил написать сюда. Допустим я использую SharedViewModel во фрагменте у которой lifeCycleOwner активити. Эта долгое время не завершает работу но в этой активити постоянно меняются фрагменты. По сути SharedViewModel используемая во фрагментах не уничтожается и получается что остается ссылка на фрагменты и они остаются в памяти. Однако LeakCanary утечки памяти не зафиксировал. Но когда я сделал тесты по очередной замене фрагментов в активити порядка 100 раз у меня образовалась область памяти которая не собиралась сборщиком мусора или мне так показалось. Вопрос в том что остаются ли фрагменты в памяти которые используют SharedViewModel до финиша активити или нет?

Comments

[Денис Загаевский]

Покажи код

[Евгений]

//Activity
const val FRAGMENT_TAG = "TEST_FRAGMENT"
class TestActivity : FragmentActivity() {

    lateinit var viewModel: TestViewModel

    override fun onCreate(savedInstanceState: Bundle?) {
        super.onCreate(savedInstanceState)
        setContentView(R.layout.activity_test)

        viewModel = ViewModelProviders.of(this).get(TestViewModel::class.java)

        supportFragmentManager.beginTransaction()
            .replace(R.id.fragmentContainer, TestFragment.newInstance(), FRAGMENT_TAG)
            .commit()
    }
}

//Fragment
class TestFragment : Fragment() {

    lateinit var viewModel: TestViewModel

    override fun onActivityCreated(savedInstanceState: Bundle?) {
        super.onActivityCreated(savedInstanceState)
        viewModel = ViewModelProviders.of(activity!!).get(TestViewModel::class.java)
    }
    companion object {
        @JvmStatic
        fun newInstance() = TestFragment()
    }
}

Вот пример как я обычно использовал SharedViewModel

Answer 1

[Денис Загаевский]

В таком варианте никакой утечки нет. Ссылка на фрагмент ниоткуда не появится во ViewModel, если ты её туда не просеттишь. Значит время жизни ViewModel не влияет на время жизни фрагмента. То, что фрагмент хранит какие-то ссылки, никак не может повлиять на его время жизни. Граф объектов - направленный.

Comments

[Евгений]

Большое спасибо за разъяснения. Могли бы вы подсказать где подробней почитать про утечки памяти?

[Денис Загаевский]

evgdev, не припомню, чтобы я что-то читал конкретно по этой теме. Утечки это просто (и в то же время сложно). Возникают, когда ссылка на объект, время жизни которого, по логике, подошло к концу, остаётся в другом объекте (или статичном поле), жизнь которого продолжается. Нужно хорошо представлять себе граф объектов, и время их жизни, и тогда проблем будет минимум. Я много видел кода новичков, которые, узнав об утечках, начинают пихать везде WeakReference. Это тоже неправильно, надо понимать, когда это необходимо.

[Евгений]

Денис Загаевский, Спасибо большое за ответ.

Answer 2

[Yar Rick]

А зачем хранить ссылки на фрагменты? Нужно предоставить управление ими менеджеру фрагментов. Он сам знает, когда выкинуть тот или иной фрагмент из памяти. В своём приложении я специально добивался того, чтобы фрагменты оставались в памяти и не пересоздавались слишком часто. Для этого в транзакции мне пришлось обойтись без replace, attach и detach, а использовать show и hide. Я не храню нигде ссылки на фрагменты и нахожу их когда нужно через менеджер фрагментов по tag. А если фрагмент не нашёлся - создаю новый инстанс.

Comments

[Евгений]

Я не говорю про хранение ссылок на фрагменты. При том же самом использовании replace я передаю туда инстанс фрагмента и я боюсь что предыдущий может висеть в памяти потомучто он содержит в себе SharedViewModel. Хотя отчасти вы помогли мне в решении моей проблемы, за это вам спасибо, но хотел бы еще получить ответ на вопрос.

[Yar Rick]

evgdev, можно не хранить ссылку на SharedViewModel во фрагменте, ведь доступ к ней всегда можно получить через ViewModelProviders. Мой опыт мне показал, что фрагменты не должны делать ничего кроме отображения пользовательского интерфейса, поэтому я их максимально облегчил, а вся логика находится во ViewModel. Правда, я также не использую и SharedViewModel.

[Евгений]

Yar Rick, с помощью ViewModelProviders вы также либо создаете viewModel либо получаете существующую.......