3

ネットから取得されるまで画像が回転するプログレスアニメーションで読み込まれるリストアダプタがあります。listViewを下にスクロールすると、アニメーションが前の画像を回転させているように見えます...これは非常に奇妙で望ましくない効果です。回転する必要があるのは、進行状況のアニメーションだけです。

ImageLoaderというクラスを使用して、画像を非同期的にネットフェッチします。

final int stub_id=R.drawable.progress;
public void DisplayImage(String url, Activity activity, ImageView imageView,int size)
{
    //Log.d("ImageLoader" , url);
    this.size=size;
    if(cache.containsKey(url)) {
        //If this works, then why do the OLD images still spin??
                    imageView.clearAnimation();
        //imageView.setBackgroundDrawable(null);
        //imageView.setImageDrawable(null);
        imageView.setImageBitmap(cache.get(url));
    }
    else
    {
        rotation = AnimationUtils.loadAnimation(activity, R.drawable.progress);
        rotation.setRepeatCount(Animation.INFINITE);
        imageView.startAnimation(rotation);
        queuePhoto(url, activity, imageView);
        //imageView.setImageResource(stub_id);
        //imageView.setBackgroundResource(stub_id); 
    }
    Resources r = activity.getResources();
    int dip = (int)TypedValue.applyDimension(TypedValue.COMPLEX_UNIT_DIP, size, r.getDisplayMetrics());
    LayoutParams layoutParams = imageView.getLayoutParams();
    layoutParams.height = dip;
    layoutParams.width = dip;
    imageView.setLayoutParams(layoutParams);
}

今、あなたは尋ねるかもしれません、queuePhotoは何をしますか?これは、アニメーションが開始される唯一の他の場所です。

//Used to display bitmap in the UI thread
class BitmapDisplayer implements Runnable
{
    Bitmap bitmap;
    ImageView imageView;
    public BitmapDisplayer(Bitmap b, ImageView i){bitmap=b;imageView=i;}
    public void run()
    {


        if(bitmap!=null) {
            imageView.clearAnimation();
            imageView.setImageBitmap(bitmap);
        }
        else {
            imageView.clearAnimation();
            rotation = AnimationUtils.loadAnimation(context, R.drawable.progress);
            rotation.setRepeatCount(Animation.INFINITE);
            imageView.startAnimation(rotation);
            //imageView.setImageResource(stub_id);

        }
    }
}

そして今、私のリストアダプタのgetViewの一部です:

    @Override
    public View getView(int position, View convertView, ViewGroup parent) {
        if( convertView == null ){
            vi = inflater.inflate(R.layout.trending_item, null);
            holder=new ViewHolder();
            holder.img1 = (ImageView) vi.findViewById(R.id.t_item_1);
            holder.img2 = (ImageView) vi.findViewById(R.id.t_item_2);
            holder.img3 = (ImageView) vi.findViewById(R.id.t_item_3);
            vi.setTag(holder);
        } else {
            holder=(ViewHolder)vi.getTag();
        }
        JSONObject t1= ts.get(1);
        if(t1 !=null) {
                holder.img1.setTag(t1.getString("image_small"));
                imageLoader.DisplayImage(t1.getString("image_small"), act, holder.img1,IMAGE_SIZE);
                holder.img1.setTag(TAG_T t1.getString("id"));
                holder.img1.setOnClickListener(ocl);
            } else {
                holder.img1.setImageDrawable(null);
                holder.img1.setBackgroundDrawable(null);    
                holder.img1.setTag(null);   
                holder.img1.setTag(TAG_T, null);
            }

アップデート

JBMが提案したソリューションを実装しましたが、それを機能させることができませんでした。彼はUIスレッドで実行することを推奨しています。

public class RemoteImageView extends ImageView implements RemoteLoadListener {

final int stub_id=R.drawable.progress;
int size;
private HashMap<String, Bitmap> cache=new HashMap<String, Bitmap>();
Animation rotation;
private File cacheDir;


public RemoteImageView(Context context, AttributeSet attrs, int defStyle) {
    super(context, attrs, defStyle);

}

public RemoteImageView(Context context, AttributeSet attrs) {
    super(context, attrs);
    //Make the background thead low priority. This way it will not affect the UI performance
    photoLoaderThread.setPriority(Thread.NORM_PRIORITY-1);

    //Find the dir to save cached images
    if (android.os.Environment.getExternalStorageState().equals(android.os.Environment.MEDIA_MOUNTED))
        cacheDir=new File(android.os.Environment.getExternalStorageDirectory(),context.getString(R.string.app_name));
    else
        cacheDir=context.getCacheDir();
    if(!cacheDir.exists())
        cacheDir.mkdirs();
}

public RemoteImageView(Context context) {
    super(context);

}

public void displayRemoteImage(String url, Activity activity, int size)
{
    this.myUrl = url;
    this.size=size;
    Resources r = activity.getResources();
    int dip = (int)TypedValue.applyDimension(TypedValue.COMPLEX_UNIT_DIP, size, r.getDisplayMetrics());
    LayoutParams layoutParams = this.getLayoutParams();
    layoutParams.height = dip;
    layoutParams.width = dip;
    this.setLayoutParams(layoutParams);
    if(cache.containsKey(url)) {
        this.clearAnimation();
        setImageBitmap(cache.get(url));
    }
    else
    {
        rotation = AnimationUtils.loadAnimation(activity, R.drawable.progress);
        rotation.setRepeatCount(Animation.INFINITE);
        this.startAnimation(rotation);
        queuePhoto(url, activity, this);
    }
}

@Override
public void onLoadSuccess(String url, Bitmap bmp) {
    if (url.equals(myUrl)) {
        setImageBitmap(bmp);
    } else {
        /* the arrived bitmap is stale. do nothing. */
    }
}

@Override
public void onLoadFail(String url) {
    if (url.equals(myUrl)) {
        setImageBitmap(((BitmapDrawable)getResources().getDrawable(stub_id)).getBitmap());
    } else {
        /* the failed bitmap is stale. do nothing. */
    }
}
String myUrl;
private void queuePhoto(String url, Activity activity, ImageView imageView)
{
    //This ImageView may be used for other images before. So there may be some old tasks in the queue. We need to discard them. 
    photosQueue.Clean(imageView);
    PhotoToLoad p=new PhotoToLoad(url, imageView);
    synchronized(photosQueue.photosToLoad){
        photosQueue.photosToLoad.push(p);
        photosQueue.photosToLoad.notifyAll();
    }

    //start thread if it's not started yet
    if(photoLoaderThread.getState()==Thread.State.NEW)
        photoLoaderThread.start();
}
PhotosQueue photosQueue=new PhotosQueue();

public void stopThread()
{
    photoLoaderThread.interrupt();
}

//stores list of photos to download
class PhotosQueue
{
    private Stack<PhotoToLoad> photosToLoad=new Stack<PhotoToLoad>();

    //removes all instances of this ImageView
    public void Clean(ImageView image)
    {
        for(int j=0 ;j<photosToLoad.size();){
            if(photosToLoad.get(j).imageView==image)
                photosToLoad.remove(j);
            else
                ++j;
        }
    }
}

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


class PhotosLoader extends Thread {
    public void run() {
        try {
            while(true)
            {
                //thread waits until there are any images to load in the queue
                if(photosQueue.photosToLoad.size()==0)
                    synchronized(photosQueue.photosToLoad){
                        photosQueue.photosToLoad.wait();
                    }
                if(photosQueue.photosToLoad.size()!=0)
                {
                    PhotoToLoad photoToLoad;
                    synchronized(photosQueue.photosToLoad){
                        photoToLoad=photosQueue.photosToLoad.pop();
                    }
                    Bitmap bmp=getBitmap(photoToLoad.url);
                    cache.put(photoToLoad.url, bmp);
                    Object tag=photoToLoad.imageView.getTag();
                    if(tag!=null && ((String)tag).equals(photoToLoad.url)){
                        BitmapDisplayer bd=new BitmapDisplayer(bmp, photoToLoad.imageView);
                        Activity a=(Activity)photoToLoad.imageView.getContext();
                        a.runOnUiThread(bd);
                    }
                }
                if(Thread.interrupted())
                    break;
            }
        } catch (InterruptedException e) {
            //allow thread to exit
        }
    }
}

PhotosLoader photoLoaderThread=new PhotosLoader();

//Used to display bitmap in the UI thread
class BitmapDisplayer implements Runnable
{
    Bitmap bitmap;
    ImageView imageView;
    public BitmapDisplayer(Bitmap b, ImageView i){bitmap=b;imageView=i;}
    public void run()
    {   
        if(bitmap!=null) {
            imageView.clearAnimation();
            imageView.setImageBitmap(bitmap);
        }
        else {
            imageView.clearAnimation();
            imageView.setImageResource(stub_id);
        }
    }
}
private Bitmap getBitmap(String url) 
{
    System.out.println("GET: " +url);
    if(url== null) {
        return null;
    }
    //I identify images by hashcode. Not a perfect solution, good for the demo.
    String filename=String.valueOf(url.hashCode());
    File f=new File(cacheDir, filename);

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

    //from web
    try {
        Bitmap bitmap=null;
        InputStream is=new URL(url).openStream();
        OutputStream os = new FileOutputStream(f);
        Utils.CopyStream(is, os);
        os.close();
        bitmap = decodeFile(f);
        return bitmap;
    } catch (Exception ex){
        ex.printStackTrace();
        return null;
    }
}

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

        //Find the correct scale value. It should be the power of 2.

        int width_tmp=o.outWidth, height_tmp=o.outHeight;
        int scale=1;
        while(true){
            if(width_tmp/2<size || height_tmp/2<size)
                break;
            width_tmp/=2;
            height_tmp/=2;
            scale*=2;
        }

        //decode with inSampleSize
        BitmapFactory.Options o2 = new BitmapFactory.Options();
        o2.inSampleSize=scale;
        return BitmapFactory.decodeStream(new FileInputStream(f), null, o2);
    } catch (FileNotFoundException e) {
        Log.d("ImageLoader",e.getMessage(),e);
    }
    return null;
}

}

4

2 に答える 2

3

ここでの問題はImageViewDisplayImage関数との間の競合です。リストが上下にスクロールすると、行が再利用されるため、リモート イメージが到着したときに、そのターゲット ビューが既に別の行に属していることがよくあります。これを行う唯一の信頼できる方法はRemoteImageView extends ImageView、画像のフェッチを内部で処理する独自のクラスを作成することです。その後、RemoteImageView は適切な同期を行うことができます。

queuePhoto次のように、メソッドRemoteLoadListenerに ImageView の代わりにa を使用させます。

public interface RemoteLoadListener {
    void onLoadFail(String url);
    void onLoadSuccess(String url, Bitmap bmp);
}

次に、RemoteImageViewaを作成RemoteLoadListenerし、すべてを内部で行います。

public class RemoteImageView extends ImageView implements RemoteLoadListener {

    final int static stub_id=R.drawable.progress;
    public void displayImage(String url, Activity activity, int size)
    {
        myUrl = url;

        if(cache.containsKey(url)) {
            ...
            setImageBitmap(cache.get(url));
        }
        else
        {
            ...
            imageView.startAnimation(rotation);
            queuePhoto(url, activity, this);
        }
    }

    @Override
    public void onLoadSuccess(String url, Bitmap bmp) {
        if (url.equals(myUrl)) {
            setImageBitmap(bmp);
        } else {
            /* the arrived bitmap is stale. do nothing. */
        }
    }

    @Override
    public void onLoadFail(String url) {
        if (url.equals(myUrl)) {
            setImageBitmap(placeholder);
        } else {
            /* the failed bitmap is stale. do nothing. */
        }
    }
}

したがって、あなたのgetViewメソッドでは、これを行うだけです:

holder.img1.displayImage(t1.getString("image_small"), act, IMAGE_SIZE);

アップデート

最初にシステムの複雑さを減らしてみてください。写真のキューとキャッシュを削除し、Web から画像をダウンロードすることに置き換えました (この部分はあなたのコードに基づいています)。このコードは実行していません。入力しただけです。しかし、それはあなたに明確な考えを与えるはずです。

public class RemoteImageView extends ImageView implements RemoteLoadListener {

    public RemoteImageView(Context context, AttributeSet attrs) {
        super(context, attrs);
    }

    public RemoteImageView(Context context) {
        super(context);

    }

    public void displayRemoteImage(String url, Activity activity, int size)
    {
        this.myUrl = url;
        this.size=size;
        Resources r = activity.getResources();
        int dip = (int)TypedValue.applyDimension(TypedValue.COMPLEX_UNIT_DIP, size, r.getDisplayMetrics());
        LayoutParams layoutParams = this.getLayoutParams();
        layoutParams.height = dip;
        layoutParams.width = dip;
        this.setLayoutParams(layoutParams);

        {
            rotation = AnimationUtils.loadAnimation(activity, R.drawable.progress);
            rotation.setRepeatCount(Animation.INFINITE);
            this.startAnimation(rotation);
            queuePhoto(url, activity, this);
        }
    }

    @Override
    public void onLoadSuccess(String url, Bitmap bmp) {
        if (url.equals(myUrl)) {
            setImageBitmap(bmp);
        } else {
            /* the arrived bitmap is stale. do nothing. */
        }
    }

    @Override
    public void onLoadFail(String url) {
        if (url.equals(myUrl)) {
            setImageBitmap(((BitmapDrawable)getResources().getDrawable(stub_id)).getBitmap());
        } else {
            /* the failed bitmap is stale. do nothing. */
        }
    }
    String myUrl;

    private void queuePhoto(final String url, final RemoteLoadListener listener)
    {
        new AsyncTask<Void, Void, Bitmap>() {
            @Override
            protected Bitmap doInBackground(Void... params) {
                //from web
                Bitmap bitmap = null;
                InputStream is = null;
                OutputStream os = null;
                try {
                    is = new URL(url).openStream();
                    os = new FileOutputStream(f);
                    Utils.CopyStream(is, os);
                    bitmap = decodeFile(f);
                } catch (Exception ex){
                    bitmap = null;
                } finally {
                    if (is != null) try { is.close(); } catch (IOException e) { /* ignore */ };
                    if (os != null) try { os.close(); } catch (IOException e) { /* ignore */ };
                }
                return bitmap;
            }

            @Override
            protected void onPostExecute(Bitmap result) {
                if (result != null) {
                    listener.onLoadSuccess(url, result);
                } else {
                    listener.onLoadFail(url);
                }
            };
        }.execute();
    }
}

PS:try - finallyストリームを操作するときはブロックに注意してください。

于 2011-07-18T17:48:00.503 に答える
0

私も同じ問題を抱えていました。私はそれに1日以上費やし、ついに簡単な解決策を見つけました。次のように、クラス extends BaseAdapter の getView() 関数の「if else」条件を削除するだけです...

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

    vi = inflater.inflate(R.layout.trending_item, null);
    holder=new ViewHolder();
    holder.img1 = (ImageView) vi.findViewById(R.id.t_item_1);
    holder.img2 = (ImageView) vi.findViewById(R.id.t_item_2);
    holder.img3 = (ImageView) vi.findViewById(R.id.t_item_3);
    JSONObject t1= ts.get(1);
    if(t1 !=null) {
            holder.img1.setTag(t1.getString("image_small"));
            imageLoader.DisplayImage(t1.getString("image_small"), act, holder.img1,IMAGE_SIZE);
            holder.img1.setTag(TAG_T t1.getString("id"));
            holder.img1.setOnClickListener(ocl);
        } else {
            holder.img1.setImageDrawable(null);
            holder.img1.setBackgroundDrawable(null);    
            holder.img1.setTag(null);   
            holder.img1.setTag(TAG_T, null);
        }
于 2014-07-17T04:32:09.143 に答える