imageloader mixes up when using viewholder pattern about lazylist

[thest1]

Probably you just do something wrong. I always use viewholder - it should not be a problem. Probably you can show us your code?

[letz]

    @Override
    public View getView(int position, View convertView, ViewGroup parent) {

        ViewHolder holder;
        if(convertView == null){
            holder = new ViewHolder();
            LayoutInflater inflater = (LayoutInflater) context.getSystemService(Context.LAYOUT_INFLATER_SERVICE);
            convertView = inflater.inflate(R.layout.live_feed_row, parent, false);

            holder.title = (TextView) convertView.findViewById(R.id.live_feed_title);
            holder.timeAgo = (TextView) convertView.findViewById(R.id.live_feed_time_ago);
            holder.avatar = (AvatarView) convertView.findViewById(R.id.live_feed_avatar);
            convertView.setTag(holder);
        }else
            holder = (ViewHolder) convertView.getTag();

        final NewsActivity activity = getItem(position);
        final String activityType = activity.getActivityType();
        holder.title.setText(Html.fromHtml(activity.getFormattedTitle()));
        holder.timeAgo.setText(Utils.timeAgo(context, activity.getActiveOn()));

        // Display image
        if (activity.hasRelatedUser() && activity.getRelatedUser().getId() > 0) {
            User user = activity.getRelatedUser();
            imageLoader.displayImage(user.getAvatarUrl(), holder.avatar);
        } else {
            holder.avatar.setImageDrawable(context.getResources().getDrawable(R.drawable.mobitto_avatar));
        }

        return convertView;

I make an instance of imageloader on the adapter construtor

[thest1]

You can't just call holder.avatar.setImageDrawable because there may be a thread downloading image right now into the same ImageView.

It could be something like that:

// Display image
if (...) {
    ...
    imageLoader.displayImage(...);
} else {
    imageLoader.reset(holder.avatar);
    holder.avatar.setImageDrawable(...);
}

And reset method inside ImageLoader would be:

public void reset(ImageView imageView){
   imageViews.put(imageView, ""); 
}

[letz]

still doesn't work.

if (...) {
...
imageLoader.display(...); // this downloads from web a image
} else {
 imageLoader.reset(holder.avatar); // <--- didn't solve
holder.avatar.setImageDrawable(...); // this get's a default image in app resources
}

[thest1]

You can also change stub_id in ImageLoader. If you want default image you call display() with empty url. And ImageLoader should should display stub but don't start the download if url if empty.

[letz]

i think it worked! thanks!