Первый элемент в списке неправильный Изображение

Question

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

Эта ошибка появляется только тогда, когда у моего первого элемента нет действительного URL-адреса, и это происходит только для первого элемента. После прокрутки вниз загружается правильное изображение.

Основная идея заключается в том, что я помещаю временное изображение, которое будет заменено изображением, загруженным из JSON. Если ни один URL (или недействительный) не должен отображаться по умолчанию, в зависимости от типа отображаемого мной животного (если собака отображает изображение собаки, если это кошка, будет отображаться изображение кошки и т. Д.).

Вот код в классе ImageLoader я использую:

package ro.nextlogic.petsplus.utils; 

import java.io.File; 
import java.io.FileInputStream; 
import java.io.FileNotFoundException; 
import java.io.FileOutputStream; 
import java.io.IOException; 
import java.io.InputStream; 
import java.io.OutputStream; 
import java.net.HttpURLConnection; 
import java.net.URL; 
import java.util.Collections; 
import java.util.Map; 
import java.util.WeakHashMap; 
import java.util.concurrent.ExecutorService; 
import java.util.concurrent.Executors; 

import ro.nextlogic.petsplus.R; 
import android.app.Activity; 
import android.content.Context; 
import android.graphics.Bitmap; 
import android.graphics.BitmapFactory; 
import android.util.Log; 
import android.widget.ImageView; 

public class ImageLoader { 

    static MemoryCache memoryCache=new MemoryCache(); 
    static FileCache fileCache; 
    private Map<ImageView, String> imageViews=Collections.synchronizedMap(new WeakHashMap<ImageView, String>()); 
    ExecutorService executorService;  
    /** 
    * The maximum number of threads used when loading images. 
    */ 
    private static final int MAX_THREADS = 5; 
    private Context context; 

    private volatile static ImageLoader instance; 

    /** Returns singleton class instance */ 
    public static ImageLoader getInstance(Context context) { 
     if (instance == null) { 
      synchronized (ImageLoader.class) { 
       if (instance == null) { 
        instance = new ImageLoader(context); 
       } 
      } 
     } 
     return instance; 
    } 

    private ImageLoader(Context context) { 
     this.context = context; 
     fileCache=new FileCache(context); 

     executorService = Executors.newFixedThreadPool(MAX_THREADS); 
    } 

    final int stub_id = R.drawable.default_other; 
    public void displayImage(String url, ImageView imageView, final int REQUIRED_SIZE) { 
     if (url == null) { 
      return; 
     } 
     Log.i("BITMAP", "imageView = " + imageView + "\nurl = " + url); 
     imageViews.put(imageView, url); 
     Bitmap bitmap=memoryCache.get(url); 
     if (bitmap!=null && !bitmap.isRecycled()) { 
      imageView.setImageBitmap(bitmap); 
     } else { 
      queuePhoto(url, imageView, REQUIRED_SIZE); 
      imageView.setImageResource(stub_id); 
     } 
    } 

    private void queuePhoto(String url, ImageView imageView, final int REQUIRED_SIZE) { 
     PhotoToLoad p=new PhotoToLoad(url, imageView, REQUIRED_SIZE); 
     executorService.submit(new PhotosLoader(p)); 
    } 

    public static Bitmap getBitmap(final String url, final int REQUIRED_SIZE) { 
     File f = fileCache.getFile(url); 

     //from SD cache 
     Bitmap b = decodeFile(f, REQUIRED_SIZE); 
     if(b != null) 
      return b; 

     //from web 
     try { 
      Bitmap bitmap=null; 
      URL imageUrl = new URL(url); 
      HttpURLConnection conn = (HttpURLConnection)imageUrl.openConnection(); 
      conn.setConnectTimeout(30000); 
      conn.setReadTimeout(30000); 
      conn.setInstanceFollowRedirects(true); 
      InputStream is=conn.getInputStream(); 
      OutputStream os = new FileOutputStream(f); 
      Utils.CopyStream(is, os); 
      os.close(); 
      bitmap = decodeFile(f, REQUIRED_SIZE); 
      return bitmap; 
     } catch (Throwable ex){ 
      ex.printStackTrace(); 
      if(ex instanceof OutOfMemoryError) 
       memoryCache.clear(); 
      return null; 
     } 
    } 

    //decodes image and scales it to reduce memory consumption 
    private static Bitmap decodeFile(final File f, final int REQUIRED_SIZE) { 
     try { 
      // Decode image size 
      BitmapFactory.Options o = new BitmapFactory.Options(); 
      o.inJustDecodeBounds = true; 
      FileInputStream stream1=new FileInputStream(f); 
      BitmapFactory.decodeStream(stream1,null,o); 
      stream1.close(); 

      // The new size we want to scale to 
//   final int REQUIRED_SIZE=70; // 70 is best for Thumbnail 
      // Get the width and height of the image 
      int width_tmp=o.outWidth, height_tmp=o.outHeight; 
      // Find the correct scale value. It should be the power of 2. 
      int scale=1; 
      while(true){ 
       if(width_tmp/2<REQUIRED_SIZE || height_tmp/2<REQUIRED_SIZE) 
        break; 
       width_tmp/=2; 
       height_tmp/=2; 
       scale*=2; 
      } 
      // Decode with inSampleSize 
      BitmapFactory.Options o2 = new BitmapFactory.Options(); 
      o2.inSampleSize=scale; 
      FileInputStream stream2=new FileInputStream(f); 
      Bitmap bitmap = BitmapFactory.decodeStream(stream2, null, o2); 
      stream2.close(); 
      return bitmap; 
     } catch (FileNotFoundException e1) { 
//   Log.e("IMAGELOADER", "FileNotFoundException: ", e1); 
     } catch (IOException e2) { 
      Log.e("IMAGELOADER", "IOException: ", e2); 
     } 
     return null; 
    } 

    //Task for the queue 
    private class PhotoToLoad { 
     public final String url; 
     public final ImageView imageView; 
     public final int REQUIRED_SIZE; 
     public PhotoToLoad(final String u, final ImageView i, final int rq){ 
      url=u; 
      imageView=i; 
      REQUIRED_SIZE = rq; 
     } 
    } 

    class PhotosLoader implements Runnable { 
     PhotoToLoad photoToLoad; 
     PhotosLoader(PhotoToLoad photoToLoad) { 
      this.photoToLoad=photoToLoad; 
     } 

     @Override 
     public void run() { 
      try{ 
       if(imageViewReused(photoToLoad)) 
        return; 
       Bitmap bmp = getBitmap(photoToLoad.url, photoToLoad.REQUIRED_SIZE); 
       memoryCache.put(photoToLoad.url, bmp); 
       if(imageViewReused(photoToLoad)) 
        return; 
       BitmapDisplayer bd=new BitmapDisplayer(bmp, photoToLoad); 
       Activity a=(Activity)photoToLoad.imageView.getContext(); 
       a.runOnUiThread(bd); 
      }catch(Throwable th){ 
       th.printStackTrace(); 
      } 
     } 
    } 

    boolean imageViewReused(PhotoToLoad photoToLoad) { 
     String tag=imageViews.get(photoToLoad.imageView); 
     if(tag==null || !tag.equals(photoToLoad.url)) 
      return true; 
     return false; 
    } 

    //Used to display bitmap in the UI thread 
    class BitmapDisplayer implements Runnable { 
     Bitmap bitmap; 
     PhotoToLoad photoToLoad; 
     public BitmapDisplayer(Bitmap b, PhotoToLoad p) { 
      bitmap = b; 
      photoToLoad = p; 
     } 
     public void run() { 
      if(imageViewReused(photoToLoad)) 
       return; 
      if(bitmap != null) 
       Utils.imageViewAnimatedChange(context, photoToLoad.imageView, bitmap); 
//    photoToLoad.imageView.setImageBitmap(bitmap); 
     } 
    } 

    public void clearCache() { 
     memoryCache.clear(); 
     fileCache.clear(); 
    } 
} 

и это, как я это называю:

@Override 
public View getView(int position, View convertView, ViewGroup parent) { 
    View rowView = convertView; 
    final ViewHolder holder; 

    if (convertView == null) { 
     rowView = inflator.inflate(R.layout.shelter_animal_rowlayout, parent, false); 
     holder = new ViewHolder(); 
     holder.animalImg = (ImageView) rowView.findViewById(R.id.shelter_animal_image); 
     holder.animalName = (TextView) rowView.findViewById(R.id.shelter_animal_name); 
     holder.animalDescription = (TextView) rowView.findViewById(R.id.shelter_animal_description); 
     rowView.setTag(holder);    
    } else { 
     holder = ((ViewHolder) rowView.getTag()); 
    } 

    AnimalItem animalItem = filteredModelItemsArray.get(position); 
    if (animalItem != null) { 
     // Display the animal name, set "Unknown" if not available 
     if (!TextUtils.isEmpty(animalItem.name) &&     // Not empty 
       !animalItem.name.contains("Unknown")) {  // Not Unknown 
      holder.animalName.setText(animalItem.name); 
     } else { 
      holder.animalName.setText(R.string.shelter_animal_name); 
     } 

     // Display the animal description, set "Unknown" if not available 
     if (!TextUtils.isEmpty(animalItem.description) &&     // Not empty 
       !animalItem.description.contains("Unknown")) { // Not Unknown 
      holder.animalDescription.setText(Html.fromHtml(animalItem.description));  
     } else { 
      holder.animalDescription.setText(R.string.shelter_animal_description); 
     } 

     // Display the animal image 
     if (animalItem.photo != null) { 
      imageLoader.displayImage(animalItem.photo, holder.animalImg, 70); 
     } else if (animalItem.animal.contains("Dog")) { 
      holder.animalImg.setImageResource(R.drawable.default_dog); 
     } else if (animalItem.animal.contains("Cat")) { 
      holder.animalImg.setImageResource(R.drawable.default_cat); 
     } else { 
      holder.animalImg.setImageResource(android.R.drawable.ic_menu_help); 
     } 
    } else { 
     Toast.makeText(context, "NO animals retrieved from server!", Toast.LENGTH_LONG).show(); 
    } 

return rowView; 

}

animalItem.photo является URL из JSON animalItem.animal является типом животного, также полученным от JSON

Следует отметить, что текст отображается нормально ... только изображение неправильное и только для первого элемента (когда фотография недоступна).

Если кто-то может указать мне в правильном направлении или сказать, что я делаю неправильно, было бы очень признательно.

EDIT: Я думаю, что исправили проблему, не используя WeakHashMap больше и сохранение в карте хэш-код для каждого ImageView.Так это то, что я изменил:

private Map<ImageView, String> imageViews=Collections.synchronizedMap(new WeakHashMap<ImageView, String>()); 

в

private Map<Integer, String> imageViews=Collections.synchronizedMap(new HashMap<Integer, String>()); 

и для сохранения и получения значений:

imageViews.put(imageView, url); 

в

imageViews.put(imageView.hashCode(), url); 

и

imageViews.get(photoToLoad.imageView); 

в

imageViews.get(photoToLoad.imageView.hashCode()); 

РЕШЕНИЕ: Я установил проблему без влияния на производительность, добавив метод, который отменяет загрузку изображения:

Это метод (объявленная in ImageLoader):

public void cancelDisplayTaskFor(ImageView imageView) { 
    imageViews.remove(imageView); 
} 

И вызовет этот метод в обычай ArrayAdapter где я установить изображение:

  // Display the animal image 
     if (animalItem.photo != null) { 
      imageLoader.displayImage(animalItem.photo, holder.animalImg, 70); 
     } else if (animalItem.animal.contains("Dog")) { 
      imageLoader.cancelDisplayTaskFor(holder.animalImg); 
      holder.animalImg.setImageResource(R.drawable.default_dog); 
     } else if (animalItem.animal.contains("Cat")) { 
      imageLoader.cancelDisplayTaskFor(holder.animalImg); 
      holder.animalImg.setImageResource(R.drawable.default_cat); 
     } else { 
      imageLoader.cancelDisplayTaskFor(holder.animalImg); 
      holder.animalImg.setImageResource(android.R.drawable.ic_menu_help); 
     } 

Надеется, что это поможет кому-то еще :)

Answer 1

, но вы должны заменить список адаптера или массив перед оповещением (забыли упомянуть), то notifyDataChanged()

Answer 2

// change this code inside your imageloader 


     public void run() { 
     if(imageViewReused(photoToLoad)) 
      return; 
     if(bitmap != null) 
     { 
       Utils.imageViewAnimatedChange(context, photoToLoad.imageView, bitmap); 

     } 
     else 
     { 
       imageView.setImageResource(stub_id);// so if bitmap is null it will set this default image 
     } 
    }