Listview中的Gridview为每行重复最后一个gridview项

Question

I'm trying to make a listview with for each row: a title and grid of items. With the below code I get a list of correct titles, but the gridView on each row is filled with the items of the last row.

The logging shows that every time getView is called it gets the correct items with the correct title. I suspect there is a shared reference, or I'm writing to the wrong row somewhere.

Similar questions are all problems with the recycling of views, which I'm doing right.

public View getView(int position, View convertView, ViewGroup parent) {
    ViewHolder holder;
    if (convertView == null) {
        convertView = l_Inflater.inflate(R.layout.list_item_my_listview, null);
        holder = new ViewHolder();
        holder.txt_name = (TextView) convertView.findViewById(R.id.name);
        holder.myGridView = (GridView) convertView.findViewById(R.id.my_gridview);
        convertView.setTag(holder);
    } else {
        holder = (ViewHolder) convertView.getTag();
    }
    listItemDetails item = itemDetailsArrayList.get(position);
    holder.txt_name.setText(item.name);

    gridItemListAdapter gridArrayAdapter = new gridItemListAdapter(context, item.getGridItems());
    holder.myGridView.setAdapter(gridArrayAdapter);

    Util.Log(getClass().getName());
    Util.Log(cabinetId + " - " + cabinet.getCageList().size());
    return convertView;
}

Answer 1

Like I said in comment above, the problem occurs when you get the item's GridView on item.getGridItems() at this line:

gridItemListAdapter gridArrayAdapter = new gridItemListAdapter(context, item.getGridItems());

The problem is not the adapter which has each single instance. The last row's dup happens because - I'll try to guess your structure, I didn't know but I saw the clue -, each time you call this method, you might use the same array and so the same values.
Thus, the last update for last row change every GridItems references in each adapter instances.

When I made my test to resolve this, I avoided to get it with an extra work in getView() in my adapters by calling an object treatment. I used an object's field which was already setted. So you have to do the same as you get item's name: item.name in order to get the gridview's items as follows:

gridItemListAdapter gridArrayAdapter = new gridItemListAdapter(context, item.gridItems);

And it should be like this in your Item object:

public ArrayList<Images> gridItems; // arraylist or whatever
...
public void setGridItems(ArrayList<Images> gridItems) { // set it when you set the name
    this.gridItems = gridItems;
}

Therefore, the Item object will have new GridItems for each instance.