Knowing when a View in a ListView has gone off the screen?
There is actually a simpler way to do that, you can use a listener :
mListView.setRecyclerListener(new AbsListView.RecyclerListener() {@Override public void onMovedToScrapHeap(View view) { }});
Can't believe how stupid I have been, this is a really easy one liner to solve!
Basically in my adapter if the view is being re-cycled, I simply need to null the ImageView bitmap before the view is re-used. That means the image in the list view item is blank before the next image is loaded. I no longer have the issue of the old image being there whilst the new image is being loaded in the background.
I have also made a change in the adapter that cancels any current AsynchTask bound to the view that might still be loading a previous image that is no longer needed. This was very prevalent when I scroll very fast through the list view, with often two or more image loads backing up, so the image would change several times before settling on the correct image.
Here is the new code, with the changes commented so you can see what is happening:
@Overridepublic View getView(int position, View convertView, ViewGroup parent){ ListViewHolder viewHolder; // if this is not a recycled view then create a new view for the data... if (convertView == null) { convertView = this.inflater.inflate(R.layout.target_list_view_layout, null, true); viewHolder = new ListViewHolder(); viewHolder.manufacturer = (TextView) convertView.findViewById(R.id.manufacturer); viewHolder.targetName = (TextView) convertView.findViewById(R.id.targetName); viewHolder.targetThumbnail = (ImageView) convertView.findViewById(R.id.targetThumbnail); convertView.setTag(viewHolder); } else { viewHolder = (ListViewHolder) convertView.getTag(); // Cancel the previous attempt to load an image as this is going to be superceded by the next image viewHolder.loadImageViewAsynchTask.cancel(true); // Clear down the old image so when this view is displayed, the user does not see the old image before the // new image has a chance to load in the background viewHolder.targetThumbnail.setImageBitmap(null); } TargetDescriptor targetDescriptor = this.selectedTargets.get(position); viewHolder.manufacturer.setText(targetDescriptor.manufacturer); viewHolder.targetName.setText(targetDescriptor.targetName); LoadImageViewAsynchTask loadImageViewAsynchTask = new LoadImageViewAsynchTask(viewHolder.targetThumbnail); loadImageViewAsynchTask.setTargetDescriptor(targetDescriptor); loadImageViewAsynchTask.execute(new Integer[] { 64, 64 }); // Keep a reference to the task so we can cancel it if the view is recycled next time round to prevent // un-neccessary image loads that are out of date viewHolder.loadImageViewAsynchTask = loadImageViewAsynchTask; return convertView;}
If you use a LruCache for the bitmaps and have each view in the listview keep the key to the LruCache item, you can control it better.. Then, in getView, you first try and get the bitmap from the LruCache and only download it again if it is not there.
Also, you can set an setOnScrollListener() on the ListView, and use its visibleItemCount parameter to adjust the size of the LruCache the first time the user scrolls.