logo标签列表

碎片没有被释放出内存

androidandroid-fragmentsandroid-viewpager
20

我有一个包含View Pager的活动，其中有一个适配器FragmentStatePagerAdapter。每次进入该活动，它将占用200MB的内存，在退出该活动（调用finish()）并重新进入后，它会增加并使手机上使用的内存翻倍。

经过故障排除，似乎片段管理器没有释放片段，尽管我正在尝试删除它们，但它只是不起作用。

我尝试清空被添加的片段，以确保其不是内部问题，但问题仍然存在。

我的适配器代码如下：

   private class ChildrenPagerAdapter extends FragmentStatePagerAdapter
   {
      private List<ChildBean> childrenBean;

      public ChildrenPagerAdapter(FragmentManager fm, List<ChildBean> bean)
      {
         super(fm);
         this.childrenBean = bean;
      }

      @Override
      public int getItemPosition(Object object)
      {
         return PagerAdapter.POSITION_NONE;
      }

      @Override
      public Fragment getItem(int position)
      {

         ReportFragment reportFragment = new ReportFragment();
         reportFragment.childBean = childrenBean.get(position);
         reportFragment.position = position;
         reportFragment.mPager = mPager;
         if(position == 0)
         {
            reportFragment.mostLeft = true;
         }
         if(position == childrenNumber - 1)
         {
            reportFragment.mostRight = true;
         }

         return reportFragment;
      }

      @Override
      public int getCount()
      {
         return childrenNumber;
      }

      @Override
      public void destroyItem(ViewGroup container, int position, Object object)
      {
         // TODO Auto-generated method stub
         super.destroyItem(container, position, object);
      }
   }

我的活动代码是

    public class ReportActivity extends CustomActivity
{
   public ImageLoader imageLoader;
   private ViewPager mPager;
   private PagerAdapter mPagerAdapter;
   private int childrenNumber;
   private int currentChild;

   @Override
   protected void onDestroy()
   {
      mPager.removeAllViews();
      mPager.removeAllViewsInLayout();
      mPager.destroyDrawingCache();
      mPagerAdapter = null;
      mPager = null;
      System.gc();
      super.onDestroy();
   }

   @Override
   protected void onCreate(Bundle savedInstanceState)
   {

      super.onCreate(savedInstanceState);
      setCustomTitle(string.title_activity_reports);
      this.currentChild = getIntent().getIntExtra("itemselected", -1);

      getSupportFragmentManager().
   }

   @Override
   protected void onResume()
   {
      super.onResume();
      mPager = (ViewPager) findViewById(R.id.vpchildren);
      mPager.setOffscreenPageLimit(6);
      childrenNumber = MainActivity.bean.size();
      mPagerAdapter = new ChildrenPagerAdapter(getSupportFragmentManager(), MainActivity.bean);
      mPager.setAdapter(mPagerAdapter);
      mPager.setCurrentItem(currentChild);
   }
}

片段代码：

public class ReportFragment extends Fragment
{

   public ChildBean childBean;
   public int position;
   public ImageView img;
   public ImageLoader imageLoader;
   public DisplayImageOptions options;
   private int pee = 0;
   private int poop = 0;
   private double sleep = 0.0;
   public ViewPager mPager;
   public boolean mostLeft = false;
   public boolean mostRight = false;

   public ReportFragment()
   {

   }

   @Override
   public void onDestroyView()
   {
      super.onDestroyView();
   }

   @Override
   public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState)
   {
      ViewGroup rootView = (ViewGroup) inflater.inflate(R.layout.report_fragment, container, false);

      if(mostLeft)
      {
         rootView.findViewById(id.btnleft).setVisibility(View.GONE);
      }
      if(mostRight)
      {
         rootView.findViewById(id.btnright).setVisibility(View.GONE);
      }

      rootView.findViewById(id.btnleft).setOnClickListener(new OnClickListener()
      {

         @Override
         public void onClick(View v)
         {
            mPager.setCurrentItem(mPager.getCurrentItem() - 1);

         }
      });

      rootView.findViewById(id.btnright).setOnClickListener(new OnClickListener()
      {

         @Override
         public void onClick(View v)
         {
            mPager.setCurrentItem(mPager.getCurrentItem() + 1);

         }
      });

      SimpleDateFormat sdf = new SimpleDateFormat("dd-MM-yyyy", Locale.ENGLISH);
      Date dobchild = new Date();

      ((TextView) rootView.findViewById(id.tvday)).setText(sdf.format(dobchild));

      ImageView childimg = (ImageView) rootView.findViewById(id.img_child);
      ((TextView) rootView.findViewById(id.tvchildname)).setText(childBean.childname);
      ((TextView) rootView.findViewById(id.tvclassname)).setText(((CustomApplication) getActivity().getApplication()).preferenceAccess.getCurrentClassName());

      Date dob = null;
      String age = "";
      try
      {
         dob = sdf.parse(childBean.childdob);
         age = GeneralUtils.getAge(dob.getTime(), getString(string.tv_day), getString(string.tv_month), getString(string.tv_year));
      }
      catch(ParseException e)
      {
         // TODO:
      }
      ((CustomTextView) rootView.findViewById(id.tvchildage)).setText(age);

      DisplayImageOptions options =
         new DisplayImageOptions.Builder().showImageForEmptyUri(drawable.noimage).showImageOnFail(drawable.noimage).showStubImage(drawable.noimage).cacheInMemory()
            .imageScaleType(ImageScaleType.NONE).build();

      imageLoader = ImageLoader.getInstance();
      imageLoader.displayImage(childBean.childphoto, childimg, options);
      final TextView tvpee = (TextView) rootView.findViewById(id.tvpeetime);
      final TextView tvpoop = (TextView) rootView.findViewById(id.tvpootimes);
      final TextView tvsleep = (TextView) rootView.findViewById(id.tvsleeptime);

      rootView.findViewById(id.btnaddpee).setOnClickListener(new OnClickListener()
      {
         @Override
         public void onClick(View v)
         {
            pee = pee + 1;
            if(pee > 9)
            {
               Toast.makeText(getActivity(), getString(string.tvareyousurepee), Toast.LENGTH_LONG).show();
            }
            tvpee.setText(String.format(getString(string.tvtimes), pee));
         }
      });

      rootView.findViewById(id.btnminuspee).setOnClickListener(new OnClickListener()
      {
         @Override
         public void onClick(View v)
         {
            if(pee > 0)
            {
               pee = pee - 1;
               tvpee.setText(String.format(getString(string.tvtimes), pee));
            }
         }
      });

      rootView.findViewById(id.btnpluspoo).setOnClickListener(new OnClickListener()
      {
         @Override
         public void onClick(View v)
         {
            poop = poop + 1;
            if(poop > 9)
            {
               Toast.makeText(getActivity(), getString(string.tvareyousurepoop), Toast.LENGTH_LONG).show();
            }
            tvpoop.setText(String.format(getString(string.tvtimes), poop));
         }
      });

      rootView.findViewById(id.btnminuspoo).setOnClickListener(new OnClickListener()
      {
         @Override
         public void onClick(View v)
         {
            if(poop > 0)
            {
               poop = poop - 1;
               tvpoop.setText(String.format(getString(string.tvtimes), poop));
            }
         }
      });

      rootView.findViewById(id.btnaddsleep).setOnClickListener(new OnClickListener()
      {
         @Override
         public void onClick(View v)
         {
            sleep = sleep + 0.25;
            tvsleep.setText(String.format(getString(string.tvhours), sleep));
         }
      });

      rootView.findViewById(id.btnminussleep).setOnClickListener(new OnClickListener()
      {
         @Override
         public void onClick(View v)
         {
            if(sleep > 0)
            {
               sleep = sleep - 0.25;
               tvsleep.setText(String.format(getString(string.tvhours), sleep));
            }
         }
      });

      rootView.findViewById(id.btnsave).setOnClickListener(new OnClickListener()
      {
         @Override
         public void onClick(View v)
         {
            Toast.makeText(getActivity(), "Report Saved.", Toast.LENGTH_LONG).show();
            getActivity().finish();
         }
      });

      return rootView;
   }
}

请提供建议... 谢谢

1
你能发布ReportFragment代码吗？ - Henrique
1
你的应用真的需要200MB吗？FragmentStatePagerAdapter被用于提高效率，以保持内存中最少量的片段。然而，在你的应用中，你使用了mPager.setOffscreenPageLimit(6);，基本上在内存中保留了最多13个片段，这有点违背了原来的设计。 - user
@Luksprog 是的，我达到了600 MB，然后活动就关闭了，我加入了这个来使应用程序内部平滑滚动...奇怪的是，每次我回滚到已经滚动过的片段时，它都会占用更多的内存...我已经花费了20多个小时在这上面...但没有运气...感谢您的帮助...顺便说一句，我将把setoofscreenpagelimit设置为2。感谢您注意到这一点。 - N Jay
使用 traceview 来查看发生了什么。 - user
1
如果堆大小的最大值远小于600MB，那么你的应用程序如何使用了这么多RAM？你使用JNI吗？如果是这样，当你不再需要它时，你必须自己释放其内存。另外，我认为你应该尝试一个完全新的项目，仅演示问题，而不是向我们展示整个代码（因为其中大部分可能与问题无关）。尽量编写最小化的代码以显示问题。 - android developer
显示剩余6条评论
4个回答
22

ViewPager本身有一个方法setOffscreenPageLimit，允许您指定适配器保留的页面数量。因此，远离当前页面的片段将被销毁。

首先，看着你的代码，我没有看到你在片段的onDestroy()中执行任何内存释放措施。片段本身被销毁和垃圾回收并不意味着您分配的所有资源也被删除了。

例如，我的主要关注点是：

imageLoader = ImageLoader.getInstance();
imageLoader.displayImage(childBean.childphoto, childimg, options);

从我看到的情况来看，似乎有一个静态实例的ImageLoader，每次出现新的片段时都会被触发，但我看不到哪个死亡的片段会要求ImageLoader卸载其内容。这对我来说看起来很可疑。

如果我是你，我会在活动重新启动后，应用程序多占用了200MB内存（如你所说）的那一刻，转储HPROF文件，并通过MAT（内存分析工具）分析引用。显然，你遇到了内存泄漏问题，我非常怀疑问题不在于片段本身没有被销毁。

如果你不知道如何分析内存堆，这里有一个好的video。它帮助我识别和消除我的应用程序中的内存泄漏问题，而且我已经看了无数遍了。

3
不要在Fragment中存储对ViewPager或ImageView的“强”引用。这将创建一个循环引用，会使所有内容保留在内存中。相反，如果您必须在Activity外部保留对ViewPager或任何其他引用其上下文的元素的引用，请尝试使用WeakReference，例如:
private WeakReference<ViewPager> mPagerRef; 
... 
mPagerRef = new WeakReference<ViewPager>(mPager);
...
final ViewPager pager = mPagerRef.get();

if (pager != null) {
    pager.setCurrentItem(...);
}

遵循这种模式来处理存储对Activity或Application上下文的引用的对象（提示：任何ViewGroup、ImageView、Activity等）可以防止出现“保留周期”形式的“内存泄漏”。
你能提供一个这种情况更好的例子吗？是针对每个片段中的每个视图吗？ - Jonas Borggren
2
1
在使用Eclipse中的内存分析器工具后，我发现占用内存的是我的片段的实际布局，具体来说是相对布局。原因是我创建了一个自定义字体为typeface的CustomTextView。
 Typeface face=Typeface.createFromAsset(context.getAssets(), "Helvetica_Neue.ttf"); 
 this.setTypeface(face);

为了解决内存泄漏问题，我只需按照这里找到的答案进行以下操作：
public class FontCache {

    private static Hashtable<String, Typeface> fontCache = new Hashtable<String, Typeface>();

    public static Typeface get(String name, Context context) {
        Typeface tf = fontCache.get(name);
        if(tf == null) {
            try {
                tf = Typeface.createFromAsset(context.getAssets(), name);
            }
            catch (Exception e) {
                return null;
            }
            fontCache.put(name, tf);
        }
        return tf;
    }
}
1
你认为不接受实际上给了你正确方向和信息的答案是恰当的吗？ - EvilDuck
网页内容由stack overflow 提供, 点击上面的
可以查看英文原文，
原文链接