Блог

Моя проблема связана с тем, что пользователь прокручивает ListView. Я огляделся и увидел многочисленные примеры «listview lazy image», также посмотрел видео Google IO, в котором говорится о «хорошей практике» для выполнения этой работы. Но моя проблема сохраняется, когда пользователь перемещается вверх и вниз по ListView.

Что происходит, так это то, что при прокрутке списка изображения, которые были загружены для каждого элемента, перетасовываются, и аватар каждого элемента, переходящий к следующему элементу, заканчивается. Я не знаю, ясно ли я выражаюсь, но я покажу с изображением.

При запуске элементы, у которых нет изображения, остаются со стандартным изображением.

Изображение 1: http://boxandroid.com/app/weguide/itsok.png Перед тем, как пользователь прокручивает ListView: http://boxandroid.com/app/weguide/nook.png

Обратите внимание, что изображения были перетасованы среди других элементов.

в моем адаптере:

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

    ViewHolder viewHolder = new ViewHolder();
    if(convertView == null){
        convertView = _inflate.inflate(R.layout.layout_list, null);
        viewHolder.text = (TextView) convertView.findViewById(R.id.title);
        viewHolder.owner = (TextView) convertView.findViewById(R.id.owner);
        viewHolder.image = (ImageView) convertView.findViewById(R.id.thumb);
        convertView.setTag(viewHolder);
    }else{
        viewHolder = (ViewHolder) convertView.getTag();
    }

    HashMap<String, String> item = (HashMap<String, String>) getItem(position);

    viewHolder.text.setText( item.get("poiName").toString() );
    viewHolder.owner.setText( item.get("owner").toString() );

    ImageView imageView = viewHolder.image;
    imageView.setTag(item.get("thumbs"));

    if(!item.get("thumbs").equals("null")){
        Drawable cacheImage = loader.loadDrawable(item.get("thumbs"), new ImageManage.ImageCallback() {
            public void imageLoaded(Drawable imageDrawable, String imageUrl) {
                ImageView imageViewByTag = (ImageView) _listView.findViewWithTag(imageUrl);
                if(imageViewByTag != null)
                    imageViewByTag.setBackgroundDrawable(imageDrawable);
            }
        });
        imageView.setImageDrawable(cacheImage);
        notifyDataSetChanged();
    }

    return convertView;
}
  

За основу взят пример из блога Android.

http://android-developers.blogspot.com/2010/07/multithreading-for-performance.html

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

    ViewHolder viewHolder = new ViewHolder();
    if(convertView == null){
        convertView = _inflate.inflate(R.layout.layout_list, null);
        viewHolder.text = (TextView) convertView.findViewById(R.id.title);
        viewHolder.owner = (TextView) convertView.findViewById(R.id.owner);
        viewHolder.image = (ImageView) convertView.findViewById(R.id.thumb);
        convertView.setTag(viewHolder);
    }else{
        viewHolder = (ViewHolder) convertView.getTag();
    }

    HashMap<String, String> item = (HashMap<String, String>) getItem(position);

    viewHolder.text.setText( item.get("poiName").toString() );
    viewHolder.owner.setText( item.get("owner").toString() );
    viewHolder.image.setTag(item.get("thumbs"));
    imageDowload.download(item.get("thumbs"), viewHolder.image);


    return convertView;
}
  

теперь работает, спасибо.

Ваша проблема в том, что вы сопоставляете notifyDataSetChange() внутри метода getView

 if(!item.get("thumbs").equals("null")){
    Drawable cacheImage = loader.loadDrawable(item.get("thumbs"), new ImageManage.ImageCallback() {
        public void imageLoaded(Drawable imageDrawable, String imageUrl) {
            ImageView imageViewByTag = (ImageView) _listView.findViewWithTag(imageUrl);
            if(imageViewByTag != null)
                imageViewByTag.setBackgroundDrawable(imageDrawable);
        }
    });
    imageView.setImageDrawable(cacheImage);
    notifyDataSetChanged();
}
  

приведенный выше код должен быть выполнен вне метода getView.