Why is my custom SimpleCursorAdapter returning different reults when ListView is scrolling

Go To StackoverFlow.com

0

I'm trying to implement a custom SimpleCursorAdapter in order to switch out layouts in a ListView but I'm getting very random results while scrolling.

My issue here is that when I scroll up and down, the ListView seemingly by random, mix up the layouts. For instance, a row can have the listview_item_row layout at first but when scrolling in and out of the screen it can be replaced by listview_item_reply_row and back again. I can't say I've really understood how newView works. I have successfully been able to use bindView to determine if I'm to hide an image in the layout or not but new View is veiled in darkness to me about its implementation and why the list scrolling behaves the way it does.

My goal is to have a list with x amount of items. Depending on if the item is a reply or a new message I want to load a specific layout on that row. Depending on if the row has an image or not I want to show/hide the imageview in the row layout.

What I have omitted in the code are the imports and row layouts. I'm trying to implement this by using Fragments and SimpleCursorAdapter in the v4 support package. The row layouts for the ListFragment are visibly different but contain the same widgets.

The ListView layout:

<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
android:layout_width="fill_parent"
android:layout_height="fill_parent"
android:orientation="vertical" >

<TextView
    android:id="@+id/text_feed_header_random"
    android:layout_width="fill_parent"
    android:layout_height="wrap_content"
    android:gravity="center_vertical|center_horizontal"
    android:padding="4dp"
    android:text="Allmänt"
    android:textAppearance="?android:attr/textAppearanceMedium"
    android:textColor="#FFF" />

<!--
The frame layout is here since we will be showing either
the empty view or the list view.

-->

<FrameLayout
    android:layout_width="fill_parent"
    android:layout_height="fill_parent"
    android:layout_below="@id/text_feed_header_random"
    android:layout_above="@+id/footer" >

    <!--
         Here is the list. Since we are using a ListActivity, we
         have to call it "@android:id/list" so ListActivity will
         find it

    -->

    <ListView
        android:id="@android:id/list"
        android:layout_width="fill_parent"
        android:layout_height="fill_parent"
        android:drawSelectorOnTop="false" />

    <!-- Here is the view to show if the list is emtpy -->

    <TextView
        android:id="@android:id/empty"
        android:layout_width="fill_parent"
        android:layout_height="fill_parent"
        android:text="No items."
        android:textAppearance="?android:attr/textAppearanceMedium" />
</FrameLayout>

<LinearLayout
    android:id="@+id/footer"
    style="@android:style/ButtonBar"
    android:layout_width="fill_parent"
    android:layout_height="wrap_content"
    android:layout_alignParentBottom="true"
    android:orientation="horizontal" >

    <Button
        android:id="@+id/button_random_post"
        android:layout_width="0dp"
        android:layout_height="wrap_content"
        android:layout_weight="1"
        android:text="Gör ett inlägg!" />

    <Button
        android:id="@+id/button_random_refresh"
        android:layout_width="0dp"
        android:layout_height="wrap_content"
        android:layout_weight="1"
        android:text="Refresh list!" />
</LinearLayout>
</RelativeLayout>

A condensed Fragment using the layout above:

public class RandomFragment extends ListFragment implements LOG {
private DatabaseHelper mDbHelper;
private KarenfeedCursorAdapter mAdapter;
private Cursor mCursor;

@Override
public void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    Log.d(TAG, "RANDOMFRAGMENT START!");

    mDbHelper = new DatabaseHelper(getActivity());
    mDbHelper.open();
    mDbHelper.setTable(Posts.TABLE_RANDOM_POSTS);

    //TODO: Replace SimpleCursorAdapter with a FragmentList instead...
    mCursor = mDbHelper.getAllPostsSortedCursor();
    String[] columns = {    Posts.COLUMN_ID,        Posts.COLUMN_CREATED,   Posts.COLUMN_USER,  Posts.COLUMN_COMMENT };
    int[] to = {            R.id.imageItemPhoto,    R.id.textItemDate,      R.id.textItemUser,  R.id.textItemComment };
    int flags = 0;
    mAdapter = new FeedCursorAdapter(getActivity(), R.layout.listview_item_row, mCursor, columns, to, flags);
    this.setListAdapter(mAdapter);
    initFeedList(); // This call in the end executes mCursor = mDbHelper.getAllPostsSorted(); mAdapter.changeCursor(mCursor); mAdapter.notifyDataSetChanged();
}
}

The SimpleCursorAdapter that the ListFragment connects to:

public class FeedCursorAdapter extends SimpleCursorAdapter implements LOG {
private Context mContext;
private int mLayout;

public FeedCursorAdapter(Context context, int layout, Cursor c, String[] from, int[] to, int flags) {
    super(context, layout, c, from, to, flags);
    // TODO Auto-generated constructor stub
    mContext = context;
    mLayout = layout;
}

@Override
public View newView(Context context, Cursor cursor, ViewGroup parent) {
    LayoutInflater inflater = LayoutInflater.from(context);
    View view;
    int id = cursor.getInt(cursor.getColumnIndex(Posts.COLUMN_ID));
    int parentId = cursor.getInt(cursor.getColumnIndex(Posts.COLUMN_PARENT_ID));
    Log.d(TAG, "id: " +id+ " parentId: " +parentId);
    int hasImage = cursor.getInt(cursor.getColumnIndex(Posts.COLUMN_IMAGE));
    if(id == parentId) {
        view = inflater.inflate(R.layout.listview_item_row, parent, false);
    } else {
        view = inflater.inflate(R.layout.listview_item_reply_row, parent, false);
    }
    return view;
}

@Override
public void bindView(View view, Context context, Cursor cursor) {
    Log.d(TAG, "bindView()");
    int id = cursor.getInt(cursor.getColumnIndex(Posts.COLUMN_ID));
    int parentId = cursor.getInt(cursor.getColumnIndex(Posts.COLUMN_PARENT_ID));
    int hasImage = cursor.getInt(cursor.getColumnIndex(Posts.COLUMN_IMAGE));
    String date = cursor.getString(cursor.getColumnIndex(Posts.COLUMN_CREATED));
    String user = cursor.getString(cursor.getColumnIndex(Posts.COLUMN_USER));
    String comment = cursor.getString(cursor.getColumnIndex(Posts.COLUMN_COMMENT));
    TextView dateView = (TextView) view.findViewById(R.id.textItemDate);
    TextView userView = (TextView) view.findViewById(R.id.textItemUser);
    TextView commentView = (TextView) view.findViewById(R.id.textItemComment);
    ImageView imageView = (ImageView) view.findViewById(R.id.imageItemPhoto);

    dateView.setText(date);
    userView.setText(user);
    commentView.setText(comment);
    if(hasImage == 0) {
        imageView.setVisibility(ImageView.GONE);
    } else {
        String bitmapPath = Environment.getExternalStorageDirectory().getPath() + "/feed/" + id + "_thumb.jpg";
        Bitmap bitmap = BitmapFactory.decodeFile(bitmapPath);
        BitmapDrawable bitmapDrawable = new BitmapDrawable(bitmap);
        imageView.setImageDrawable(bitmapDrawable);
        imageView.setVisibility(ImageView.VISIBLE);
    }
}
}
2012-04-04 08:09
by shellström


0

after reading your question and the code i think you should know that whenever you scroll a listview up or down it communicates with the adapter to populate the new items that came in to focus and for the case of different row for different data you should make a arraylist that will contain the name or id of the items whose backgrounds are different and then you start making there background according to your need on every call to adapter.

2012-04-04 08:48
by varun bhardwaj
The scrolling worked very well with how I constructed this application before; by using ArrayAdapter and ArrayList to populate the ListView. There is however differences in how that implementation worked and how this SimpleCursorAdapter, Cursor and ListFragment works and I can't really pinpoint what's going wrong or why the behaviour is so erratic. Could you try and elaborate? I use a cursor to avoid the use of an ArrayList, which I at least assume shouldn't be needed - shellström 2012-04-04 13:50
I think I've gotten a somewhat clearer picture of how a CursorAdapter works in this case and you certainly did have a point, I was just to new to this in order to understand what you explained. Anywho, I'll accept this as an answer to my question - shellström 2012-05-28 13:36
What i mean to say was when ever u scroll up or down it will communicate again with adapter and may be st that time the cursor is not at the right position compared to the item in the list so i think that is why it is showing different results on scroll. Anyways i ll try this code today after office and let you know what i got - varun bhardwaj 2012-05-31 04:13
Ads