Tukukkk

My app is several checkboxes areas. On some event data from all checkboxes will be saved/sended/etc.

All worked, but this Activity loading about 7 seconds because my code is bad. So i need help in refactoring and optimize it.

How it looks in app:

img1

img2

So, i have 10(!) forms with ~ 100 checkboxes summary.
To draw it, i'm using 10 recyclers with 10 adaptes. Every adapter initialize with his own list and have own layout manager.

How to make this code good?

I can't pass all code, it's too large, so i give it apart.

To drow one recycler i use it(variable names changed):

//toolbar and questform

...

<RelativeLayout android:id="@+id/rv_wrapper1"

                android:layout_width="match_parent"

                android:layout_height="wrap_content"

                android:padding="@dimen/padding_default">



                <TextView

                    android:layout_width="match_parent"

                    android:layout_height="wrap_content"

                    android:layout_alignParentStart="true"

                    android:layout_alignParentLeft="true"

                    android:text="@string/title1"

                    android:textSize="@dimen/text_size_big" />



                <ImageView

                    android:id="@+id/img_arrow_1"

                    android:layout_width="@dimen/image_size_small"

                    android:layout_height="@dimen/image_size_small"

                    android:layout_alignParentRight="true"

                    android:layout_marginEnd="@dimen/margin_default"

                    android:layout_marginRight="@dimen/margin_default"

                    android:src="@drawable/ic_expand" />





                <com.github.chuross.library.ExpandableLayout

                    android:id="@+id/rv_expaner1"

                    android:layout_width="match_parent"

                    android:layout_height="wrap_content"

                    android:layout_below="@id/img_arrow_1"

                    app:exl_duration="1000">



                    <android.support.v7.widget.RecyclerView

                        android:id="@+id/rv_1"

                        android:layout_width="match_parent"

                        android:layout_height="wrap_content"

                        android:layout_marginTop="@dimen/padding_small">

                    </android.support.v7.widget.RecyclerView>

                </com.github.chuross.library.ExpandableLayout>

            </RelativeLayout>

            ///another recycles

and i duplicate it 9 times on xml.

Activity

RecyclerView rv1 = findViewById(R.id.rv1);

RecyclerView rv2 = findViewById(R.id.rv2);

//another 8 RV.

...



rv1Adapter = new RVAdapter(this, initializList1(this));

rv2Adapter = new RVAdapter(this, initializList2(this));

//another 8 adapters

...



rv1.setAdapter(rv1Adapter);

rv2.setAdapter(rv2Adapter);

...

//another 8 setters



RecyclerView.LayoutManager layoutManagerForRv1 = new GridLayoutManager(this, 3);

RecyclerView.LayoutManager layoutManagerForRv2 = new GridLayoutManager(this, 3);

...

//another 8 managers





RelativeLayout rv1Expander = findViewById(R.id.rv_expander1);

RelativeLayout rv2Expander = findViewById(R.id.rv_expander2);

...

//another 8 RL



rv1Expander.setOnClickListener(new ExpanderLayoutHelper(R.id.rv_expander1));

rv2Expander.setOnClickListener(new ExpanderLayoutHelper(R.id.rv_expander2));

 ...

//another 8 listeners

ExpanderLayoutHelper - using for expand/collaps animation of RV.

public class ExpanderLayoutHelper implements View.OnClickListener {

        RelativeLayout expandWrapper;

        ImageView arrow;

        ExpandableLayout expandableLayout;



        ExpanderLayoutHelper(int relativeLayoutId) {

            this.expandWrapper = findViewById(relativeLayoutId);

            this.arrow = (ImageView) expandWrapper.getChildAt(1);

            this.expandableLayout = (ExpandableLayout) expandWrapper.getChildAt(2);

        }



        @Override

        public void onClick(View v) {

            if (expandableLayout.isCollapsed()) {

                expandableLayout.expand();

                expandWrapper.getChildAt(1).animate().rotation(90f).setDuration(1000);

            } else {

                expandableLayout.collapse();

                arrow.animate().rotation(0).setDuration(1000);

            }

        }

    }

Another points will be here if i remember something important.

1) initializeList() is just 1-20 list.add("something") operation. For every form i have uniq list, so i have 10 initialize methods now.