0

速度が低下しないように、各ビューの imageView を非同期に更新できるカスタム リストビュー アダプターを作成しようとしています。

これまでのところ、イメージビューのビットマップを格納するベクトルを使用しており、アダプター コンストラクターですべてのビットマップを取得するスレッドを呼び出しています。getView() メソッドで、vector[position] が null でないかどうかをチェックして、imageView ビットマップを実際に変更します。

コードは次のとおりです。

public class SongAdapter extends ArrayAdapter<Song> {
private int resource;
private LayoutInflater inflater;
private ArrayList<Song> items;
private Context cntx;
private final Utilities util = new Utilities(); //una per tutte le canzoni
private Bitmap[] covers; //la utilizzo per fare un cashing delle cover

public SongAdapter(Context cntx, int resId, ArrayList<Song> objects)
{
    super(cntx, resId, objects);
    this.cntx = cntx;
    items = objects;
    resource = resId;
    inflater = LayoutInflater.from(cntx);
    covers = new Bitmap[objects.size()];
    asyncLoadImagesCover(); //lacio un thread per caricare le immagini
}


//carico le immagini nella listview in background
public void asyncLoadImagesCover()
{
    new Thread(new Runnable() {

        @Override
        public void run() {
            for(int i = 0; i < items.size(); i++)
                covers[i] = util.getAlbumArtFromSong(items.get(i).getPath(), cntx);

            //notifyDataSetChanged();
        }
    }).start();
}


//cerco di mantenere in memoria l'oggetto in modo da evitare il ricaricamento
private static class ViewHolder
{
    TextView titolo;
    TextView artista;
    ImageView cover;
}

//////////////////////////////////////////////////////////////////////////////////////

//Funzione che riscrive gli elementi della listview
@Override
public View getView(int position, View v, ViewGroup parent)
{
    //recupero l'oggetto canzone
    Song song = getItem(position);
    ViewHolder holder;

    if(v == null)
    {
        //creo la view partendo dal layout
        v = inflater.inflate(resource, parent, false);
        holder = new ViewHolder();
        holder.titolo = (TextView)v.findViewById(R.id.songTitle);
        holder.artista = (TextView)v.findViewById(R.id.artistText);
        holder.cover = (ImageView)v.findViewById(R.id.albumImage);
        v.setTag(holder);
    }
    else
        holder = (ViewHolder) v.getTag();

    //setto i valori a partire dall'holder, in questo modo non devo fare findviewbyid ogni volta
    holder.titolo.setText(song.getTitle());
    holder.artista.setText(song.getArtist());
    if(covers[position] != null)
        holder.cover.setImageBitmap(covers[position]);

    return v;
}
}

とにかく、これはうまく機能していません。50% の確率でアクティビティがクラッシュしますが、それは新しいスレッドが原因だと思います。Web で検索したところ、さまざまな解決策が見つかりましたが、まずコードがクラッシュする理由を理解したいと思います。また、getView で v と位置を渡してリストビュー GUI を更新するスレッドを作成するにはどうすればよいですか?

////////////////////////////////////////////// 編集：

興味のある人のために、これは私がやったことです: UI を更新するための 1 つのハンドラーと、関数によって画像を取得するスレッドをスタックするための ExecutorService を使用する 2 つの別個のオブジェクトを持つクラスを作成しました。

public class CoverLoader  {

Utilities util = new Utilities();
Context cntx;
ExecutorService executorService;
Handler handler = new Handler(); //si occupa di aggiornare la UI

public CoverLoader(Context cntx)
{
    this.cntx = cntx;
    executorService = Executors.newFixedThreadPool(5);
}

public void DisplayImage(ImageView iv, String path)
{
    executorService.submit(new ImageLoader(iv,path));
}


//Separo la parte del caricamento dalla parte di aggiornamento della gui
//questa classe si occupa solo di recuperare la bmp
public class ImageLoader implements Runnable
{

    ImageView iv;
    String path;

    public ImageLoader(ImageView iv, String path)
    {
        this.iv = iv;
        this.path = path;
    }

    @Override
    public void run()
    {
        Bitmap bmp = util.getAlbumArtFromSong(path, iv.getContext());
        handler.post(new BitmapDisplayer(iv,bmp));
    }
}

//Si occupa di aggiornare la UI, richiamata dall'handler a termine del processo di caricamento

class BitmapDisplayer implements Runnable
{
    Bitmap bitmap;
    ImageView iv;

    public BitmapDisplayer(ImageView iv, Bitmap bitmap)
    {
        this.iv = iv;
        this.bitmap = bitmap;
    }

    @Override
    public void run()
    {
        iv.setImageBitmap(bitmap);
    }
}

}

getView() で:

@Override
public View getView(int position, View v, ViewGroup parent)
{
    Song song = getItem(position);
    ViewHolder holder;

    if(v == null)
    {
        //creo la view partendo dal layout
        v = inflater.inflate(resource, parent, false);
        holder = new ViewHolder();
        holder.titolo = (TextView)v.findViewById(R.id.songTitle);
        holder.artista = (TextView)v.findViewById(R.id.artistText);
        holder.cover = (ImageView)v.findViewById(R.id.albumImage);
        v.setTag(holder);
    }
    else
        holder = (ViewHolder) v.getTag();

    //setto i valori a partire dall'holder, in questo modo non devo fare findviewbyid ogni volta
    holder.titolo.setText(song.getTitle());
    holder.artista.setText(song.getArtist());
    coverLoader.DisplayImage(holder.cover, song.getPath()); //Richiamo uno stack di thread

    //if(covers[position] != null)
        //holder.cover.setImageBitmap(covers[position]);

    return v;
}

これまでのところ、私はすでに持っている画像を保存する方法（おそらく配列？）を見つけるだけです

4

1 に答える 1

0

Activity コンテキストにバインドされたオブジェクト内で Threads を実行するべきではありません。これは短い生き物 (メモリ リーク) であるためです。

また、すべての画像をダウンロードするのではなく、getView コードで要求されたこれらの画像だけをダウンロードする必要があります (パフォーマンス)。

また、複数のスレッド (同期) によってアクセスされる変数を避けるようにしてください。

于 2013-04-20T20:03:20.780 に答える