Android使用片段作为照片库,怎么样?

时间:2012-12-20 08:35:50

标签: android gallery fragment

我正在尝试实施一个画廊。

由于图库窗口小部件已被弃用...要实现它并具有分页滚动效果,我使用片段(android兼容性v4)来执行此操作。

一切看起来都不错,直到向后滚动...我的APP因为以下原因而强制关闭:

java.lang.IllegalStateException: The specified child already has a parent. You must call removeView() on the child's parent first.

当向后滚动显示上一张照片时,看起来它会重新获得ImageView。

我不知道如何解决这个问题...

这是我的xml&代码

[XML]

<android.support.v4.view.ViewPager android:id="@+id/pager" android:layout_below="@id/head" android:layout_width="wrap_content" android:layout_height="350dp" android:layout_marginTop="35dp" />

[代码]

public class PhotoView extends FragmentActivity
{
  private Button btnBack=null;
  private ClickHandler click=null;
  private ViewPager pager=null;
  private ArrayList<ImageView> imgList=new ArrayList<ImageView>();

  @Override
  public void onCreate(Bundle savedInstanceState)
  {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.photo_show);
    initUI();
    handler.sendEmptyMessage(0);
  }

  private void initUI()
  {
    btnBack=(Button)findViewById(R.id.btnBack);
    btnBack.setOnClickListener(click);
    pager=(ViewPager)findViewById(R.id.pager);
  }

  private Handler handler=new Handler()
  {
    public void handleMessage(Message msg)
    {
      loadPhotos();
    }
  };

  private void loadPhotos()
  {
    for(int i=0; i<Constants.PHOTO_LIST.size(); i++)
    {
      ImageView img=new ImageView(this);
      img.setLayoutParams(new LayoutParams(LayoutParams.WRAP_CONTENT, LayoutParams.WRAP_CONTENT));
      imgList.add(img);
      new ImageDL(Constants.PHOTO_LIST.get(i).getFileId(), Params.IMAGE_SIZE_PARAM_ORIGINAL).execute(img); // async D/L photo
    }
    pager.setAdapter(new PhotoAdapter(getSupportFragmentManager()));
  }

  @Override
  public void onBackPressed()
  {
    super.onBackPressed();
    btnBack.performClick();
    return;
  }

  @Override
  protected void onStop()
  {
    super.onStop();
    for(int i=0; i<imgList.size(); i++) //Do recycle when activity stopped, but I'm not yet sure this code is right...
    {
      Drawable drawable=imgList.get(i).getDrawable();
      if(drawable instanceof BitmapDrawable)
      {
        BitmapDrawable bitmapDrawable=(BitmapDrawable) drawable;
        Bitmap bitmap=bitmapDrawable.getBitmap();
        bitmap.recycle();
      }
    }
    imgList.clear();
    imgList=null;
  }

  private class ClickHandler implements OnClickListener
  {
    public void onClick(View v)
    {
      if(v==btnBack)
      {
        finish();
        overridePendingTransition(R.anim.lr1, R.anim.lr2);
        System.gc();
      }
    }
  }

  private class PhotoAdapter extends FragmentPagerAdapter
  {
    public PhotoAdapter(FragmentManager mgr)
    {super(mgr);}

    @Override
    public int getCount()
    {
      return(imgList.size());
    }

    @Override
    public Fragment getItem(int position)
    {
      return(new PhotoFragment(position));
    }
  }

  private class PhotoFragment extends Fragment 
  {
    private int index=-1;

    private PhotoFragment(int index)
    {this.index=index;}

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) 
    {
      return imgList.get(index);
    }
  }
}

这是我第一次在代码中使用Fragment API,我不确定我所做的代码逻辑是否正确......

有任何想法可以帮助我解决这个问题,或者在Android 2.3~4.x中实现图库(带分页滚动)的另一种更好的方法吗?

1 个答案:

答案 0 :(得分:0)

你不应该使用imgList作为缓存。

onCreateView()的实现问题是:  。在第0页,ViewPager将imageView添加到第0页,并将imageView添加到其布局(父视图)(我认为)。 。当您向后滚动到第0页时,ViewPager将再次调用onCreateView(),因此在尝试将已添加的imageView重新添加到其布局时会出现错误:

java.lang.IllegalStateException:指定的子节点已经有父节点。

所以你不应该在loadPhotos()方法中使用imgList;而是在onCreateView()中初始化imageView。

修改后的代码:

 private void loadPhotos()
  {

    pager.setAdapter(new PhotoAdapter(getSupportFragmentManager()));
  }


public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) 
    {
      ImageView img=new ImageView(this);
      img.setLayoutParams(new LayoutParams(LayoutParams.WRAP_CONTENT, LayoutParams.WRAP_CONTENT));

      new ImageDL(Constants.PHOTO_LIST.get(i).getFileId(),  Params.IMAGE_SIZE_PARAM_ORIGINAL).execute(img); // async D/L photo


      return img;
}