Issue When Fast Scroll A Listview

I am working on a small project where I create a listview bound to an ArrayAdapter. In the getView() function of ArrayAdapter, I do a loading of images from web urls on thread, and

Solution 1:

I have same issue when scroll quickly it alternate the vales of some item to others, just like background color of some items if changes randomly. I solved this issue by searching a lot and find exact solution is just adding these two methods in your adapter if you are using ViewHolder in your adapter

@OverridepublicintgetViewTypeCount() {
        return getCount();
    }

    @OverridepublicintgetItemViewType(int position) {
        return position;
    }

Solution 2:

Assuming that you are not caching the downloaded image.. lets see the following code:

    if (!(item.imageHref).equalsIgnoreCase("null")) {
        mDrawableManager.fetchDrawableOnThread(item.imageHref, viewHolder.image);
    } else {
        viewHolder.image.setImageDrawable(null);
    }

Now if the image view is getting reused then it would already have the old image for the assigned list item. So until the thread download the image from the network it would display the old image and when the thread download the image for the current item it would be replaced with the new image. Try to change it to:

    if (!(item.imageHref).equalsIgnoreCase("null")) {
        viewHolder.image.setImageDrawable(SOME_DEFULAT_IMAGE);
        mDrawableManager.fetchDrawableOnThread(item.imageHref, viewHolder.image);
    } else {
        viewHolder.image.setImageDrawable(null);
    }

Or you can use something link smart image view that supports HTTP URI and also caches the images. Check out following link for smart image view:

https://github.com/loopj/android-smart-image-view http://loopj.com/android-smart-image-view/

Solution 3:

Add ImageLoader class from below link in your project. link

just call DisplayImage() methode of Image loader class as below in getView()

ImageLoaderimageLoader=newImageLoader();


 yourImageView.setTag(URL_FOR_Your_Image);

 imageLoader.DisplayImage(URL_FOR_Your_Image,ACTIVITY.this, yourImageView);

Your images will load in background as you want without wait.

Solution 4:

I think you should declare your downloader method fetchDrawableOnThread() as "synchronized" . Because a lot of threads are working simultaneously and any thread which started later, can end earlier. So there are chances of images being misplaced.

It happened to me for a long time. Finally "synchronized" helped me do it cleanly. I hope it helps you too.

Solution 5:

I give it a try with synchronization again. Either synchronize the fetchDrawableOnThread(), or synchronize the global hashmap data within fetchDrawableOnThread(). First i thought the issue is gone, but when i tried more times later, i found the issue is still there.

Then i thought about the synchronization. fetchDrawableOnThread() is called from getView(), and getview() itself does not have a concurrency issue. Even if as Yogesh said, what happened INSIDE getView() is thread-based, and return early or late, it can not affect the correctness of getView(), i.e. the list item's display, only the sooner or later.

What i did(synchronization) inside fetchDrawableOnThread() i think it's still correct, 'cause i used a hashmap to cache images downloaded from remote url, and this hashmap is read/write upon in a multi-thread situation, so it should be locked. But i do not think it's the rootcause of the UI misplace, if hashmap is messed up, the image misplacement will be permanent.

Then i looked further on convertView reuse mechanism based on Praful's explanation. He explained clearly what happened when image always comes from remote and no cache locally, but my situation is i waited my list to display fully, i.e. all images download complete and cached complete, before i do the fast scroll. So in my experiment, the images are read from cache.

Then when inspecting my code, i saw one minor difference in the use of convertView as in getView() method, a lot of the example usages are like this:

public View getView(int position, View convertView, ViewGroup parent) {  // case 1Viewv= convertView;
    ....  // modify vreturn v;
}

However the example i happened to copy from use:

public View getView(int position, View convertView, ViewGroup parent){  // case 2
    ....  // modify convertViewreturn convertView;
}

I thought it makes no difference at first, 'cause according to what android says, 'ListView sends the Adapter an old view that it's not used any more in the convertView param.', so why not use 'convertView' para directly?

But i guess i was wrong. I changed my getView() code to case 1. Boom! everything works. No funny business ever no matter how fast i scroll the list.

Quite strange, is convertView only old, or is it old & in-use? If the later, we should only get a copy and then modify..... ??