最初のアダプターの画像が正しくない

Question

何らかの理由で、最初の画像が正しく表示され、別のユーザーの画像で上書きされてしまいます。任意のアイデア：

public View getView(int position, View convertView, ViewGroup parent) { 
     View vi=convertView; 
     ViewHolder holder; 
     if(convertView == null){ 
       vi = inflater.inflate(R.layout.feed_item, null); 
       holder=new ViewHolder(); 
       holder.userImage = (ImageView) vi.findViewById(R.id.feed_userimage); 
       vi.setTag(holder); 
     } else { 
       holder=(ViewHolder)vi.getTag(); 
     } 
     if(user.has("image") && 
      user.getString("image") != null && 
      !user.getString("image").equals("null")) { 
       holder.userImage.setTag(user.getString("image")); 
       imageLoader.DisplayImage(user.getString("image"), act, holder.userImage,USER_IMAGE_SIZE); 
     } else { 
       holder.userImage.setImageDrawable(null); 
     } 

Answer 1

あなたがconvertViewを使用しているので、それがhappenningさ

if(user.has("image") && 
    user.getString("image") != null && 
    !user.getString("image").equals("null")) { 
    holder.userImage.setTag(user.getString("image")); 
    imageLoader.DisplayImage(user.getString("image"), act, holder.userImage,USER_IMAGE_SIZE); 
} else { 
    holder.userImage.setTag(null);//add this line 
    holder.userImage.setImageDrawable(null); 
} 

Answer 2

を修正し、これを試してみてください。 getViewメソッドに渡されたconvertViewは、前のgetView呼び出し（これはもはや表示されないので、もはや必要ではありません）で作成したビュー・オブジェクトです（スクロールすることにより）。

getViewが呼び出されるたびに、新しいビューが作成され、実際に以前に作成されたビューを使用していると仮定しています。 convertViewは最適化として渡されるため、少数しか表示されていない場合には多くのビューを作成する必要はありません。 したがって、setTagコールは、実際に以前に作成されたビューのtagを上書きします。

tagsを使用せずにコードの書き換えを再考する必要があります。または、convertViewがnullではない場合にのみ膨張させるのではなく、常にビューを膨張させることができます。しかし、私は長いリストのためにこのアプローチを推奨していませんが、それはメモリ内で不必要なビューが多すぎることを意味します。