Warum gibt meine benutzerdefinierte SimpleCursorAdapter verschiedene Ergebnisse zurück, wenn ListView scrollt

Question

Ich versuche, einen benutzerdefinierten SimpleCursorAdapter zu implementieren, um Layouts in einer ListView zu wechseln, aber ich bekomme sehr zufällige Ergebnisse beim Scrollen.

Mein Problem hier ist, dass, wenn ich scrollen rauf und runter, die ListView scheinbar zufällig, mischen Sie die Layouts. Zum Beispiel kann eine Zeile das Layout "listview_item_row" haben, aber beim Ein- und Ausblenden kann sie durch listview_item_reply_row und wieder zurück ersetzt werden. Ich kann nicht sagen, dass ich wirklich verstanden habe, wie newView funktioniert. Ich bin erfolgreich in der Lage gewesen, bindView zu verwenden, um festzustellen, ob ich ein Bild im Layout verstecken soll oder nicht, aber die neue Ansicht ist für mich dunkel über die Implementierung verschleiert und warum das Scrollen der Liste sich so verhält.

Mein Ziel ist es, eine Liste mit x Menge von Elementen zu haben. Abhängig davon, ob das Element eine Antwort oder eine neue Nachricht ist, möchte ich ein bestimmtes Layout für diese Zeile laden. Abhängig davon, ob die Zeile ein Bild enthält oder nicht, möchte ich die Bildansicht im Zeilenlayout ein-/ausblenden.

Was ich im Code weggelassen habe, sind die Importe und Zeilenlayouts. Ich versuche dies zu implementieren, indem ich Fragments und SimpleCursorAdapter im v4 Support Package verwende. Die Zeilenlayouts für das ListFragment unterscheiden sich sichtbar, enthalten jedoch dieselben Widgets.

Das 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> 

Ein verkürzter Fragment mit dem Layout oben:

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(); 
} 
} 

Die SimpleCursorAdapter, dass der ListFragment einer Verbindung herstellt:

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); 
    } 
} 
} 

Answer 1

nach der Lektüre Ihrer Frage und der Code ich denke, dass Sie wissen sollten, dass, wann immer Sie einen Listenansicht nach oben oder unten scrollen i t kommuniziert mit dem Adapter, um die neuen Elemente zu füllen, die in den Fokus kamen, und für den Fall einer anderen Zeile für verschiedene Daten sollten Sie eine Arraylist machen, die den Namen oder die ID der Elemente enthält, deren Hintergründe unterschiedlich sind und dann dort anfangen Hintergrund je nach Bedarf bei jedem Aufruf des Adapters.