ExpandableListView in Fragment Issue
Asked Answered
A

2

9

I am trying to implement an Expandable listView in a fragment. There are no errors coming and when I try to log the output from both the List<String> and HashMap<String, List<String>>, I get the actual data logged.

The issue I am getting is when I show the actual Expandable Listview in the fragment. Only the first item of the list is being shown and I cannot expand it (as per screenshot below):

.screenshot

This is the code I am using:

Fragment class

public static class LineupFragment extends Fragment {

    ExpandableListAdapter listAdapter;
    ExpandableListView expListView;
    List<String> listDataHeader;
    HashMap<String, List<String>> listDataChild;

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {

        View rootView = inflater.inflate(R.layout.fragment_lineup, null);

        expListView = (ExpandableListView) rootView.findViewById(R.id.expListView);

        prepareListData();

        listAdapter = new ExpandableListAdapter(getActivity(), listDataHeader, listDataChild);
        expListView.setAdapter(listAdapter);

        Log.i("groups", listDataHeader.toString());
        Log.i("details", listDataChild.toString());

        // Listview Group click listener
        expListView.setOnGroupClickListener(new OnGroupClickListener() {

            @Override
            public boolean onGroupClick(ExpandableListView parent, View v,int groupPosition, long id) {
                // Toast.makeText(getApplicationContext(),
                // "Group Clicked " + listDataHeader.get(groupPosition),
                // Toast.LENGTH_SHORT).show();
                return false;
            }
        });

        // Listview Group expanded listener
        expListView.setOnGroupExpandListener(new OnGroupExpandListener() {

            @Override
            public void onGroupExpand(int groupPosition) {
                Toast.makeText(getActivity().getApplicationContext(),listDataHeader.get(groupPosition) + " Expanded",Toast.LENGTH_SHORT).show();
            }
        });

        // Listview Group collasped listener
        expListView.setOnGroupCollapseListener(new OnGroupCollapseListener() {

            @Override
            public void onGroupCollapse(int groupPosition) {
                Toast.makeText(getActivity().getApplicationContext(),listDataHeader.get(groupPosition) + " Collapsed",Toast.LENGTH_SHORT).show();
            }
        });

        // Listview on child click listener
        expListView.setOnChildClickListener(new OnChildClickListener() {

            @Override
            public boolean onChildClick(ExpandableListView parent, View v,
                    int groupPosition, int childPosition, long id) {
                Toast.makeText(getActivity().getApplicationContext(),listDataHeader.get(groupPosition) + " : " + listDataChild.get(listDataHeader.get(groupPosition)).get(childPosition), Toast.LENGTH_SHORT).show();
                return false;
            }
        });   
    return rootView;
    }

    private void prepareListData() {
        listDataHeader = new ArrayList<String>();
        listDataChild = new HashMap<String, List<String>>();

        // Adding child data
        listDataHeader.add("Top 250");
        listDataHeader.add("Now Showing");
        listDataHeader.add("Coming Soon..");

        // Adding child data
        List<String> top250 = new ArrayList<String>();
        top250.add("The Shawshank Redemption");
        top250.add("The Godfather");
        top250.add("The Godfather: Part II");
        top250.add("Pulp Fiction");
        top250.add("The Good, the Bad and the Ugly");
        top250.add("The Dark Knight");
        top250.add("12 Angry Men");

        List<String> nowShowing = new ArrayList<String>();
        nowShowing.add("The Conjuring Despicable Me TurboGrown Ups 2 Red 2 the Wolverine The Conjuring Despicable Me TurboGrown Ups 2 Red 2 the Wolverine");

        List<String> comingSoon = new ArrayList<String>();
        comingSoon.add("2 Guns");
        comingSoon.add("The Smurfs 2");
        comingSoon.add("The Spectacular Now");
        comingSoon.add("The Canyons");
        comingSoon.add("Europa Report");

        listDataChild.put(listDataHeader.get(0), top250); // Header, Child data
        listDataChild.put(listDataHeader.get(1), nowShowing);
        listDataChild.put(listDataHeader.get(2), comingSoon);

    }

**Adapter class**

    public class ExpandableListAdapter extends BaseExpandableListAdapter {

                private Activity _context;
                private List<String> _listDataHeader; // header titles
                // child data in format of header title, child title
                private HashMap<String, List<String>> _listDataChild;

                public ExpandableListAdapter(Activity context, List<String> listDataHeader,
                        HashMap<String, List<String>> listChildData) {
                    this._context = context;
                    this._listDataHeader = listDataHeader;
                    this._listDataChild = listChildData;
                }

                @Override
                public Object getChild(int groupPosition, int childPosititon) {
                    return this._listDataChild.get(this._listDataHeader.get(groupPosition))
                            .get(childPosititon);
                }

                @Override
                public long getChildId(int groupPosition, int childPosition) {
                    return childPosition;
                }

                @Override
                public View getChildView(int groupPosition, final int childPosition,
                        boolean isLastChild, View convertView, ViewGroup parent) {

                    final String childText = (String) getChild(groupPosition, childPosition);

                    if (convertView == null) {
                        LayoutInflater infalInflater = (LayoutInflater) this._context
                                .getSystemService(Context.LAYOUT_INFLATER_SERVICE);
                        convertView = infalInflater.inflate(R.layout.list_item, null);
                    }

                    TextView txtListChild = (TextView) convertView.findViewById(R.id.lblListItem);

                    txtListChild.setText(childText);
                    return convertView;
                }

                @Override
                public int getChildrenCount(int groupPosition) {
                    return this._listDataChild.get(this._listDataHeader.get(groupPosition))
                            .size();
                }

                @Override
                public Object getGroup(int groupPosition) {
                    return this._listDataHeader.get(groupPosition);
                }

                @Override
                public int getGroupCount() {
                    return this._listDataHeader.size();
                }

                @Override
                public long getGroupId(int groupPosition) {
                    return groupPosition;
                }

                @Override
                public View getGroupView(int groupPosition, boolean isExpanded,
                        View convertView, ViewGroup parent) {
                    String headerTitle = (String) getGroup(groupPosition);
                    if (convertView == null) {
                        LayoutInflater infalInflater = (LayoutInflater) this._context
                                .getSystemService(Context.LAYOUT_INFLATER_SERVICE);
                        convertView = infalInflater.inflate(R.layout.list_group, null);
                    }

                    TextView lblListHeader = (TextView) convertView.findViewById(R.id.lblListHeader);
                    lblListHeader.setTypeface(null, Typeface.BOLD);
                    lblListHeader.setText(headerTitle);

                    return convertView;
                }

                @Override
                public boolean hasStableIds() {
                    return false;
                }

                @Override
                public boolean isChildSelectable(int groupPosition, int childPosition) {
                    return true;
                }
            }
        }

Edit: This is the code for the fragment_lineup.xml file:

<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
    android:id="@+id/LinearLayout2"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    android:background="@drawable/bg_black"
    android:orientation="vertical" >

    <ExpandableListView
        android:id="@+id/expListView"
        android:layout_width="match_parent"
        android:layout_height="310dp"
        android:layout_weight="0.14" >
    </ExpandableListView>

</LinearLayout>

Then I have two other lavout files for a custom layout.

One for the **list_group.xml**:

    <LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="fill_parent"
    android:layout_height="wrap_content"
    android:background="@drawable/bg_black"
    android:orientation="horizontal" >

    <LinearLayout
        android:layout_width="52dp"
        android:layout_height="52dp"
        android:orientation="vertical" >

        <ImageView
            android:id="@+id/photo"
            android:layout_width="match_parent"
            android:layout_height="match_parent"
            android:contentDescription="@string/app_name"
            android:fitsSystemWindows="true" />

    </LinearLayout>

    <LinearLayout
        android:layout_width="match_parent"
        android:layout_height="52dp"
        android:orientation="vertical" >

        <TextView
            android:id="@+id/lblListHeader"
            android:layout_width="match_parent"
            android:layout_height="match_parent"
            android:gravity="center_vertical"
            android:paddingLeft="6dip"
            android:paddingTop="6dip"
            android:textColor="#CC0000"
            android:textSize="20dp"
            android:textStyle="bold"
            tools:ignore="SpUsage" />

    </LinearLayout>

</LinearLayout>

And the other for the **list_item.xml**:

   <LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
    android:layout_width="match_parent"
    android:layout_height="120dp"
    android:orientation="vertical" >

    <TextView
        android:id="@+id/lblListItem"
        android:layout_width="fill_parent"
        android:layout_height="wrap_content"
        android:background="@color/white"
        android:paddingBottom="5dp"
        android:paddingLeft="?android:attr/expandableListPreferredChildPaddingLeft"
        android:paddingTop="5dp"
        android:textSize="13sp" />

</LinearLayout>

Kindly note that I need to use a fragment because this class is part of a larger app which I am developing, and uses tabs. The whole code works fine in an activity.

I am suspecting that my issue is in the adapter class.

Agribusiness answered 6/6, 2014 at 14:11 Comment(8)
Can you post the layout of the fragment? The whole code works fine in an activity. - what exactly does this mean? Which part works?Amain
All of the code above works perfectly in an activity but does not in a fragmentAgribusiness
As I said, post the layout of the fragment, at a quick look there doesn't seem to be anything wrong with the adapter class.Amain
Did you tried scrolling the element visible ?. is it scrollable ?Glister
no it is not scrollable. Only that element is being shown. It should show 4 elements in all.Agribusiness
Can you show us the code for the activity which adds this fragment?Nummary
This won't solve your issue but your inflate line should look like this: View rootView = inflater.inflate(R.layout.fragment_lineup, container, false);Nummary
Thanks @Jay Soyer. I managed to solve the issue myself and I have shared the code too in my answer.Agribusiness
A
22

I have finally found out how this can be made. I am pasting the code below for anyone who has been encountering the same problem. Feel free to use:

public static class LineupFragment extends Fragment {

View rootView;
ExpandableListView lv;
private String[] groups;
private String[][] children;


public LineupFragment() {

}

public void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);

    groups = new String[] { "Test Header 1", "Test Header 2", "Test Header 3", "Test Header 4" };

    children = new String [][] {
        { "s simply dummy text of the printing and typesetting industry. Lorem Ipsum has been the industry's standard dummy text ever since the 1500s, when an unknown printer took a galley of type and scrambled it to make a type specimen book. It has survived not only five centuries, but also the leap into electronic typesetting, remaining essentially unchanged. It was popularised in the 1960s with the release of Letraset sheets containing Lorem Ipsum passages, and more recently with desktop publishing software like Aldus PageMaker including versions of Lorem Ipsum." },
        { "Contrary to popular belief, Lorem Ipsum is not simply random text. It has roots in a piece of classical Latin literature from 45 BC, making it over 2000 years old. Richard McClintock, a Latin professor at Hampden-Sydney College in Virginia, looked up one of the more obscure Latin words, consectetur, from a Lorem Ipsum passage, and going through the cites of the word in classical literature, discovered the undoubtable source. Lorem Ipsum comes from sections 1.10.32 and 1.10.33 of comes from a line in section 1.10.32." },
        { "It is a long established fact that a reader will be distracted by the readable content of a page when looking at its layout. The point of using Lorem Ipsum is that it has a more-or-less normal distribution of letters, as opposed to using 'Content here, content here', making it look like readable English. Many desktop publishing packages and web page editors now use Lorem Ipsum as their default model text, and a search for 'lorem ipsum' will uncover many web sites still in their infancy. Various versions have evolved over the years, sometimes by accident, sometimes on purpose (injected humour and the like)." },
        { "There are many variations of passages of Lorem Ipsum available, but the majority have suffered alteration in some form, by injected humour, or randomised words which don't look even slightly believable. If you are going to use a passage of Lorem Ipsum, you need to be sure there isn't anything embarrassing hidden in the middle of text. All the Lorem Ipsum generators on the Internet tend to repeat predefined chunks as necessary, making this the first true generator on the Internet. It uses a dictionary of over 200 Latin words, combined with a handful of model sentence structures, to generate Lorem Ipsum which looks reasonable. The generated Lorem Ipsum is therefore always free from repetition, injected humour, or non-characteristic words etc." }
    };
}

@Override
public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {
    rootView = inflater.inflate(R.layout.fragment_lineup, container, false);  

return rootView;
}

@Override
public void onViewCreated(View view, Bundle savedInstanceState) {
    super.onViewCreated(view, savedInstanceState);

    lv = (ExpandableListView) view.findViewById(R.id.expListView);
    lv.setAdapter(new ExpandableListAdapter(groups, children));
    lv.setGroupIndicator(null);

}

public class ExpandableListAdapter extends BaseExpandableListAdapter {

    private final LayoutInflater inf;
    private String[] groups;
    private String[][] children;

    public ExpandableListAdapter(String[] groups, String[][] children) {
        this.groups = groups;
        this.children = children;
        inf = LayoutInflater.from(getActivity());
    }

    @Override
    public int getGroupCount() {
        return groups.length;
    }

    @Override
    public int getChildrenCount(int groupPosition) {
        return children[groupPosition].length;
    }

    @Override
    public Object getGroup(int groupPosition) {
        return groups[groupPosition];
    }

    @Override
    public Object getChild(int groupPosition, int childPosition) {
        return children[groupPosition][childPosition];
    }

    @Override
    public long getGroupId(int groupPosition) {
        return groupPosition;
    }

    @Override
    public long getChildId(int groupPosition, int childPosition) {
        return childPosition;
    }

    @Override
    public boolean hasStableIds() {
        return true;
    }

    @Override
    public View getChildView(int groupPosition, final int childPosition, boolean isLastChild, View convertView, ViewGroup parent) {

         ViewHolder holder;
            if (convertView == null) {
                convertView = inf.inflate(R.layout.list_item, parent, false);
                holder = new ViewHolder();

                holder.text = (TextView) convertView.findViewById(R.id.lblListItem);
                convertView.setTag(holder);
            } else {
                holder = (ViewHolder) convertView.getTag();
            }

            holder.text.setText(getChild(groupPosition, childPosition).toString());

            return convertView;
    }

    @Override
    public View getGroupView(int groupPosition, boolean isExpanded, View convertView, ViewGroup parent) {
        ViewHolder holder;

        if (convertView == null) {
            convertView = inf.inflate(R.layout.list_group, parent, false);

            holder = new ViewHolder();
            holder.text = (TextView) convertView.findViewById(R.id.lblListHeader);
            convertView.setTag(holder);
        } else {
            holder = (ViewHolder) convertView.getTag();
        }

        holder.text.setText(getGroup(groupPosition).toString());

        return convertView;
    }

    @Override
    public boolean isChildSelectable(int groupPosition, int childPosition) {
        return true;
    }

    private class ViewHolder {
        TextView text;
    }
}

And this is how the new layout now looks:

Working Expandable Listview

Feel free to use this in your code and hope I have helped you somehow :)

Agribusiness answered 11/6, 2014 at 9:44 Comment(12)
I had same problem.... you seem to have solve it but there is no any explanation in your answer about what causes it and how you solve it... can you explain your answer....Welltimed
If I remember well, basically what i was doing is setting the textview in the xml as match_parent in the textview. Therefore the first element took a whole screen.Agribusiness
but why you didnt use hashmap... hashmap is more efficient and simpler than multidimensional array..Welltimed
because I had discarded that option and used a multi-dimensional array instead. In my opinion they are both of the same difficulty to implementAgribusiness
can you post your code in a question please and send me the link here?Agribusiness
can you please upvote my answer if it has helped you?Agribusiness
@IsharaAmarasekera Can you please upvote if this has helped you? ThanksAgribusiness
Nice post, thank you, but I get a java.lang.NullPointerException: Attempt to invoke virtual method 'void android.widget.ExpandableListView.setAdapter(android.widget.ExpandableListAdapter)' on a null object reference (I just copied your code) Can you help please?Astrology
I think your problem is that you haven't created the xml files. From the error you mentioned, it seems that you haven't created the widget in the layout file. Take a look at the following links which contain the layout xml files you need: dropbox.com/s/vaf36sytbg03dvm/fragment_lineup.xml?dl=0, dropbox.com/s/90bc52f7mpabpc5/list_item.xml?dl=0 and dropbox.com/s/pi76alb4o2k7lvo/list_group.xml?dl=0Agribusiness
Well, I found the solution, I created the xml file and renamed the expList - but not in the Fragment file :D thanks for your helpAstrology
Do you have an idea how to make this dynamic?Astrology
Dynamic in the sense of getting data from a database?Agribusiness
W
4

Dev.mi's solution works fine. But I was stuck in similar problem bcoz of some stupid mistake. I was making bit sophisticated design and had used scrollView as root parent for ExpandableListView.

ExpandableListView with wrapcontent inside linear layout is enough to make it scrollable.

Adding Scrollable View outside caused Expandable ListView to shrink to show only one header.

Dont place ExpandableListView inside Scroll View

Welltimed answered 11/8, 2015 at 8:40 Comment(0)

© 2022 - 2024 — McMap. All rights reserved.