Хорош ли мой вариант сохранения состояния фрагмента в android?

Question

[Develop9999]

Добрый день, пишу проект под android и уже очень давно мучаюсь с одной проблемой, спустя время нашёл более менее нормальное решение, но не уверен хорошо ли оно.
В чём собственно суть вопроса, каждый раз когда мы создаём фрагмент и инициализируем его через
onCreateView, то мы всегда вызываем inflater.inflate(R.layout.fragment_controller, container, false);, а это операция довольно затратная и по ресурсам и по времени, часть кнопок или других визуальных объектов можно генерировать через asynktask, но в том случае когда мы не можем программно создать объекты, то приходиться использовать inflate. я узнал что можно оказывается добавлять фрагменты в fragmentManger таким образом и потом просто прятать их и показывать.

FragmentTransaction fragmentTransaction = getSupportFragmentManager().beginTransaction();
fragmentTransaction.setTransition(FragmentTransaction.TRANSIT_FRAGMENT_OPEN);
if (!fragmentsAdd[FRAGMENT_CURRENT]) {
     fragmentTransaction.add(R.id.container, fragments[FRAGMENT_CURRENT], fragmentsTag[FRAGMENT_CURRENT]);
     fragmentsAdd[FRAGMENT_CURRENT] = true;
}
for (int i = 0; i < fragments.length; i++) {
     if (i == FRAGMENT_CURRENT)
           fragmentTransaction.show(fragments[i]);
     else if (fragmentsAdd[i])
           fragmentTransaction.hide(fragments[i]);
}
fragmentTransaction.commit();

В такой реализации мы храним все фрагменты в озу, есть ли другие варианты решения или как делается по другому, к примеру загрузка страницы в вк в мобильной версии или в youtube?
Я хочу узнать другие варианты решения этой проблемы и хочу узнать насколько плох или хорош этот вариант, и при этом варианте программа съедала порядка 90 мб озу, при этом было 6-7 фрагментов.
Все переключения я обрабатываю через navigator drawer.

Answer 1

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

API есть, значит делать так можно. Но вообще-то это не очень хорошая практика, тк растёт потребление памяти. А если во вьюхах останутся ссылки на какие-нибудь большие изображения, то может и OOM случиться

Comments

[Develop9999]

А как можно решить эту проблему, чтобы не вызывать постоянно infale и onCreateView?

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

Develop9999, обычно inflate не занимает сколько-то длинное время. Если занимает, значит нужно оптимизировать. Покажи типичный xml, инфлейта которого ты хочешь избежать.

[Develop9999]

У меня inflate длится 187 милисекунд

Сам xml довольно большой, я бы сказал, что он огромный, и я использую наследование, чтобы у некоторых объектов было соотношение сторон 1к1

<?xml version="1.0" encoding="utf-8"?>
<LinearLayout
            android:layout_weight="1"
            android:layout_width="0dp"
            android:layout_height="match_parent"
            android:orientation="vertical">
            <com.ironproject.ironcontrol.CustomComponents.CustomLinearLayout
                android:background="@drawable/controller_background_button_group"
                android:layout_gravity="center"
                android:layout_margin="5dp"
                android:layout_width="match_parent"
                android:layout_height="0dp"
                android:layout_weight="1"
                android:orientation="horizontal">
                <LinearLayout
                    android:orientation="vertical"
                    android:layout_weight="1"
                    android:layout_width="0dp"
                    android:layout_height="match_parent">
                    <View
                        android:layout_width="match_parent"
                        android:layout_height="0dp"
                        android:layout_weight="1"/>
                    <com.ironproject.ironcontrol.CustomComponents.CustomButton
                        android:layout_gravity="center"
                        android:background="@drawable/controller_button_left"
                        android:id="@+id/button_left"
                        android:layout_width="match_parent"
                        android:layout_height="0dp"
                        android:layout_weight="1"/>
                    <View
                        android:layout_width="match_parent"
                        android:layout_height="0dp"
                        android:layout_weight="1"/>
                </LinearLayout>
                <LinearLayout
                    android:orientation="vertical"
                    android:layout_weight="1"
                    android:layout_width="0dp"
                    android:layout_height="match_parent">
                    <com.ironproject.ironcontrol.CustomComponents.CustomButton
                        android:layout_gravity="center"
                        android:background="@drawable/controller_button_up"
                        android:id="@+id/button_up"
                        android:layout_width="match_parent"
                        android:layout_height="0dp"
                        android:layout_weight="1"/>
                    <View
                        android:layout_width="match_parent"
                        android:layout_height="0dp"
                        android:layout_weight="1"/>
                    <com.ironproject.ironcontrol.CustomComponents.CustomButton
                        android:layout_gravity="center"
                        android:background="@drawable/controller_button_down"
                        android:id="@+id/button_down"
                        android:layout_width="match_parent"
                        android:layout_height="0dp"
                        android:layout_weight="1"/>
                </LinearLayout>
                <LinearLayout
                    android:orientation="vertical"
                    android:layout_weight="1"
                    android:layout_width="0dp"
                    android:layout_height="match_parent">
                    <View
                        android:layout_width="match_parent"
                        android:layout_height="0dp"
                        android:layout_weight="1"/>
                    <com.ironproject.ironcontrol.CustomComponents.CustomButton
                        android:layout_gravity="center"
                        android:background="@drawable/controller_button_right"
                        android:id="@+id/button_right"
                        android:layout_width="match_parent"
                        android:layout_height="0dp"
                        android:layout_weight="1"/>
                    <View
                        android:layout_width="match_parent"
                        android:layout_height="0dp"
                        android:layout_weight="1"/>
                </LinearLayout>
            </com.ironproject.ironcontrol.CustomComponents.CustomLinearLayout>

            <com.ironproject.ironcontrol.CustomComponents.DrawController
                android:layout_gravity="center"
                android:id="@+id/leftController"
                android:layout_width="match_parent"
                android:layout_height="0dp"
                android:layout_weight="1.15"/>

            <com.ironproject.ironcontrol.CustomComponents.DrawSectorController
                android:visibility="gone"
                android:layout_gravity="center"
                android:id="@+id/leftSectorController"
                android:layout_width="match_parent"
                android:layout_height="0dp"
                android:layout_weight="1.15" />
        </LinearLayout>

Скинул лишь часть кода, т.к весь не влезает, там порядка 270 строк кода, пробовал использовать gridlatoyout, но это не сильно увеличило скорость загрузки, плюс у меня 3 фрагмента в viewPagerAdapted и на 2ух других фрагментах генерируется по 96 кнопок, но эту проблему я исправил просто перенеся генерацию в asynktask
И в каждой кнопке присутствует drawable изображение

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

Develop9999, это, во-первых, стоит переписать на ConstraintLayout. Вложенные линеарлэйауты с весами это сразу нет.
Во-вторых, стоит рассмотреть возможность сделать кастомный лэйаут(ViewGroup) и в один проход там всё расположить.

96 кнопок это вообще какая-то жесть, скорее всего надо использовать RecyclerView.

UI в асинтасках не меняют, есть AsyncLayoutInflater, мб стоит использовать его.

[Develop9999]

Денис Загаевский, хорошо, тогда попробую так сделать, но у меня ещё пару вопросов, хороша ли практика использовать replace в fragmentManager, и тогда плохо ли, что при каждом переключении на фрагмент мы раз за разом опять ищем кнопки findViewById и ставим на них лисенеры? Постоянная генерация получается. В смысле так и делают в крупных проектах или нет? Или используется какой-нить паттерн? Читал примеры про mvp, mvvm, но для моей ситуации мне кажется эти паттерны не нужны.
И вы предлагаете использовать ConstraintLayout в связке с viewgroup или 2 варианта реализации - через viewgroup и через  ConstraintLayout?
И я нашёл в constraintlayout - constraintlayoutweight, можно ли его использовать или это тоже самое, что простой weight?

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

Develop9999, два разных варианта. Кури маны по measure/layout, станет понятнее.
Что значит раз за разом не очень понятно. Там такие частые переключения между экранами?

[Develop9999]

Денис Загаевский, да, довольно часто приходится переключать экраны. Про measure я знаю, за счёт measure и сделал соотношение сторон 1к1.
Так мне через replace так же делать переключение? И weight тоже использовать в constraintlayout?

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

Develop9999, да, и да

[Develop9999]

Денис Загаевский, и всё же, частое создание в onCreateView - это нормально? И по этому поводу я могу не париться, и тогда естественно не будет out of memory.

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

Develop9999, нормально, что каждый раз вьюхи пересоздаются. Если всё сделать правильно, то oom не будет.

[Develop9999]

Денис Загаевский, спасибо за помощь. И теперь я спокоен за пересоздание, а то меня всегда волновал этот вопрос, странно типо, каждый раз пересоздавать, разве это не жрёт каждый раз ресурсы процессора