0

Images を更新しようとしていGridViewます。
アダプターを再度設定した場合にのみ、画像が更新されます。アダプターを再度設定すると、アクティビティがスクロールして一番上に戻るので、それを避けようとします。

ここにいくつかのコードがあります：

private class LoadImagesTask extends AsyncTask<Object,Void, Object> {
    private ImageAdapter imageAdapter;
    private ArrayList<Integer> idsArray;
    @Override
    protected void onPreExecute() {   
        imageAdapter = new ImageAdapter(getApplicationContext());
        Bitmap bmap = BitmapFactory.decodeResource(getResources(),
                R.drawable.locked_image);
        idsArray = dbHelper.getPicturesIdsByAlbumId(1);
        for(int i = 0;i<idsArray.size();i++){
            imageAdapter.bitmaps.add(bmap);
        }

        gridView.setAdapter(imageAdapter);
    }
    @Override
    protected Void doInBackground(Object... arg) {

        for(int i = 0;i<idsArray.size();i++){
            byte[] data = dbHelper.getPictureById(idsArray.get(i)).littlePicBytes;
            imageAdapter.bitmaps.set(i, BitmapFactory.decodeByteArray(data, 0, data.length));
            publishProgress(); //********** This line calls onProgressUpdate()  *********
        }
        return null;

    }
}

さて、このコードはビューを更新するために機能します:

@Override
protected void onProgressUpdate(Void... params) {
    gridView.setAdapter(imageAdapter);
}

しかし、これはしません:

@Override
protected void onProgressUpdate(Void... params) {
    imageAdapter.notifyDataSetChanged();
    gridView.invalidateViews();
}

ビューを更新しない理由は何ですか? ありがとうございました！

編集: ImageAdapter コード:

public class ImageAdapter extends BaseAdapter {
    private Context mContext;
    public ArrayList<Bitmap> bitmaps = new ArrayList<Bitmap>();
    public ImageAdapter(Context c) {
        mContext = c;

    }

    public int getCount() {
        return bitmaps.size();
    }

    public Object getItem(int position) {
        return position;
    }

    public long getItemId(int position) {
        return position;
    }

    // create a new ImageView for each item referenced by the Adapter
    public View getView(int position, View convertView, ViewGroup parent) {
        LayoutInflater inflater = (LayoutInflater) mContext
                .getSystemService(Context.LAYOUT_INFLATER_SERVICE);

            View gridView;

            if (convertView == null) {

                gridView = new View(mContext);

                // get layout from mobile.xml
                gridView = inflater.inflate(R.layout.image_adapter, null);

                // set image based on selected text
                ImageView imageView = (ImageView) gridView
                        .findViewById(R.id.imageView);
                imageView.setImageBitmap(bitmaps.get(position));
            }
                else{ 
                    gridView = (View) convertView;
                }
            return gridView;
    }
}
4

1 に答える 1

1

コードに 2 つの主な問題があります。

  1. doInBackground では、アダプタのデータを変更しますが、これはスレッド セーフな操作ではありません。これは、変更中にアダプタがデータから読み取る可能性があるためです。私はこの行について話している：

    imageAdapter.bitmaps.set(i, BitmapFactory.decodeByteArray(data, 0, data.length));

    代わりに、たとえば onProgressUpdate 内など、UI スレッドでのみデータを更新する必要があります。

  2. getView で、正しく実装していません。convertView が null でない場合、imageView のコンテンツを更新することはありません。それがどのようなものであるべきかは次のようなものです:

public View getView(int position, View convertView, ViewGroup parent) 
  {
  View rootView=convertView;
  ViewHolder holder;
  if (rootView != null) 
    holder=(ViewHolder) rootView.getTag();
  else
    {
    rootView = inflater.inflate(R.layout.image_adapter, null,false);
    holder = new ViewHolder();
    holder.imageView = (ImageView) rootView.findViewById(R.id.imageView);
    rootView.setTag(holder);
    }
  holder.imageView.setImageBitmap(getItem(position));
  return rootView;
  }


private Bitmap getItem(final int position)
  {
  return bitmaps.get(position); 
  }    

private static class ViewHolder
  {
  ImageView imageView;
  }
于 2013-07-13T11:11:33.507 に答える