logo标签列表

ViewPager、PagerAdapter和Bitmap会导致内存泄漏（OutOfMemoryError）问题。

androidandroid-fragmentsmemory-leaksandroid-viewpagerandroid-imageview
6
我建立了一个Android应用程序，显示天气数据（如果您想测试问题，我可以私下提供应用程序名称）。用户可以浏览从一天到另一天以查看特定日期的天气情况。
应用程序采用片段架构（单个MainActivity与导航抽屉，调用具体的片段）。
DayPagerFragment使用ViewPager与无限数量的页面（动态片段）。每一页代表一天。
DayPagerFragment：
public class DayPagerFragment extends Fragment {

    private ViewPager mViewPager;

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container, Bundle savedInstanceState) {
        return inflater.inflate(R.layout.fragment_day, container, false);
    }

    @Override
    public void onViewCreated(View view, Bundle savedInstanceState) {
        super.onViewCreated(view, savedInstanceState);

        mViewPager = (ViewPager) view.findViewById(R.id.pager);
        mViewPager.setOffscreenPageLimit(1);
        mViewPager.setAdapter(new DayAdapter(getChildFragmentManager()));
    }

    private static class DayAdapter extends FragmentStatePagerAdapter {

        public DayAdapter(FragmentManager fm) {
            super(fm);
        }

        @Override
        public Fragment getItem(int position) {
            return DayFragment.newInstance(null);
        }

        @Override
        public int getCount() {
            // I don't know the number to put here becauseI don't have 
            // a defined number of fragments (= dynamic fragments)
            return 1; 
        }

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

    }

    public void setCurrentPagerItemPrev() {
        //mViewPager.setCurrentItem(mViewPager.getCurrentItem() - 1);
        mAdapterViewPager.getRegisteredFragment(mViewPager.getCurrentItem() - 1);
    }

    public void setCurrentPagerItemNext() {
        //mViewPager.setCurrentItem(mViewPager.getCurrentItem() + 1);
        mAdapterViewPager.getRegisteredFragment(mViewPager.getCurrentItem() + 1);
    }

}

第一项优化：使用FragmentStatePagerAdapter进行管理，因为FragmentPagerAdapter不适用于我的用途/动态片段（将整个片段存储在内存中）。

第二项优化：我使用setOffscreenPageLimit(1)设置了当前页面两侧应保留的页面数量。

DayFragment

public class DayFragment extends Fragment {

    private TextView mDay;
    private TextView mMonth;
    private Button mPrevDay;
    private Button mNextDay;
    private ImageView mCenter;
    private ImageView mLeft;
    private ImageView mRight;
    ...
    private DayRepository dayRepository;
    private Day currentDay;
    private Day prevDay;
    private Day nextDay;
    private DayUtil dayUtil;
    private DayUtil dayUtilPrev;
    private DayUtil dayUtilNext;
    private Calendar cal;
    private Calendar calPrev;
    private Calendar calNext;

    public static DayFragment newInstance(Calendar calendar) {
        DayFragment dayFragment = new DayFragment();

        Bundle args = new Bundle();
        args.putInt("year", calendar.get(Calendar.YEAR));
        args.putInt("month", calendar.get(Calendar.MONTH));
        args.putInt("day", calendar.get(Calendar.DAY_OF_MONTH));
        dayFragment.setArguments(args);

        return dayFragment;
    }

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

        mDay = (TextView) view.findViewById(R.id.textView_day);
        mMonth = (TextView) view.findViewById(R.id.textView_month);
        mCenter = (ImageView) view.findViewById(R.id.imageView_center); // Weather symbol (sun, cloud...) of D-Day
        mLeft = (ImageView) view.findViewById(R.id.imageView_left); // Weather symbol of D-1
        mRight = (ImageView) view.findViewById(R.id.imageView_right);  // Weather symbol of D-2

        //... get 6 others TextView/ImageView

        MyApplication app = (MyApplication) getActivity().getApplicationContext();

        // Get bundle args
        int day = getArguments().getInt("day");
        int month = getArguments().getInt("month");
        int year = getArguments().getInt("year");

        // Date
        this.cal = new GregorianCalendar(year, month, day);

        // Get prev/next day (for nav arrows)
        this.calPrev = (GregorianCalendar) this.cal.clone();
        this.calPrev.add(Calendar.DAY_OF_YEAR, -1);
        this.calNext = (GregorianCalendar) this.cal.clone();
        this.calNext.add(Calendar.DAY_OF_YEAR, 1);

        // Get data from database
        //...

        // Utils
        this.dayUtil = new DayUtil(currentDay, getActivity());
        this.dayUtilPrev = new DayUtil(this.prevDay, getActivity());
        this.dayUtilNext = new DayUtil(this.nextDay, getActivity());
        String dateCurrentDayName = FormatUtil.getDayName(app.getLocale()).format(this.cal.getTime());
        String dateCurrentDayNameCap = dateCurrentDayName.substring(0,1).toUpperCase() + dateCurrentDayName.substring(1);
        String dateCurrentMonthName = FormatUtil.getMonthName(month, app.getLocale());

        // Update UI
        //... lot of setText(...) using day object and utils
        mLeft.setImageResource(this.dayUtilPrev.getDrawable());
        mCenter.setImageResource(dayUtil.getDrawable());
        mRight.setImageResource(this.dayUtilNext.getDrawable());

        return view;
    }

    @Override
    public void onViewCreated(View view, Bundle savedInstanceState) {
        super.onViewCreated(view, savedInstanceState);

        // Custom fonts
        MyApplication app = (MyApplication) getActivity().getApplication();
        ViewGroup vg = (ViewGroup)getActivity().getWindow().getDecorView();
        ViewUtil.setTypeFace(app.getTrebuchet(), vg);

        // Navigation between days
        mMoonPrevDay.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {

                ((MainActivity)getActivity()).viewDay(calPrev);
            }
        });
        mMoonNextDay.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                ((MainActivity) getActivity()).viewDay(calNext);
            }
        });
    }

    // Never called!
    @Override
    public void onDestroy() {
        super.onDestroy();
        Log.w("com.example", "Fragment day destroyed");
    }

}

问题:

我的应用非常图形化，因为每个页面都显示：

  • 10个文本框（内部文本根据日期变化）
  • 4个ImageView（D-1、D-Day、D+1的天气符号）+ 另一个ImageView

当我浏览ViewPager页面时，很快就会出现OutOfMemoryError（大约在30页后）。

就像Fragment没有从内存中被释放一样。垃圾回收器并不像我期望的那样工作（我认为是因为某些内容引用了旧的片段）。

Logcat

04-06 20:01:21.683  27008-27008/com.example D/dalvikvm﹕ GC_BEFORE_OOM freed 348K, 2% free 194444K/196608K, paused 93ms, total 93ms
04-06 20:01:21.683  27008-27008/com.example E/dalvikvm-heap﹕ Out of memory on a 1790260-byte allocation.
04-06 20:01:21.693  27008-27008/com.example E/AndroidRuntime﹕ FATAL EXCEPTION: main
    Process: com.example, PID: 27008
    java.lang.OutOfMemoryError
            at android.graphics.BitmapFactory.nativeDecodeAsset(Native Method)
            at android.graphics.BitmapFactory.decodeStream(BitmapFactory.java:587)
            at android.graphics.BitmapFactory.decodeResourceStream(BitmapFactory.java:422)
            at android.graphics.drawable.Drawable.createFromResourceStream(Drawable.java:840)
            at android.content.res.Resources.loadDrawable(Resources.java:2110)
            at android.content.res.Resources.getDrawable(Resources.java:700)
            at android.widget.ImageView.resolveUri(ImageView.java:638)
            at android.widget.ImageView.setImageResource(ImageView.java:367)
            at com.example.ui.DayFragment.onCreateView(DayFragment.java:126) //...mLeft.setImageResource()
            at android.support.v4.app.Fragment.performCreateView(Fragment.java:1500)
            at android.support.v4.app.FragmentManagerImpl.moveToState(FragmentManager.java:927)
            at android.support.v4.app.FragmentManagerImpl.moveToState(FragmentManager.java:1104)
            at android.support.v4.app.BackStackRecord.run(BackStackRecord.java:682)
            at android.support.v4.app.FragmentManagerImpl.execPendingActions(FragmentManager.java:1467)
            at android.support.v4.app.FragmentManagerImpl$1.run(FragmentManager.java:440)
            at android.os.Handler.handleCallback(Handler.java:733)
            at android.os.Handler.dispatchMessage(Handler.java:95)
            at android.os.Looper.loop(Looper.java:136)
            at android.app.ActivityThread.main(ActivityThread.java:5017)
            at java.lang.reflect.Method.invokeNative(Native Method)
            at java.lang.reflect.Method.invoke(Method.java:515)
            at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:779)
            at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:595)
            at dalvik.system.NativeStart.main(Native Method)
04-06 20:01:21.773  27008-27008/com.example I/dalvikvm-heap﹕ Clamp target GC heap from 197.480MB to 192.000MB
04-06 20:01:21.773  27008-27008/com.example D/dalvikvm﹕ GC_FOR_ALLOC freed 565K, 2% free 193932K/196608K, paused 73ms, total 73ms

我有一个内存泄漏问题，但不知道出在哪里和为什么。我尝试使用Eclipse MAT (Memory Analyser)，但是还是找不到问题所在。
你能帮我吗？
编辑： 为了加载字体，我使用了以下代码： DayFragment.java 
// Custom fonts
MyApplication app = (MyApplication) getActivity().getApplication();
ViewGroup vg = (ViewGroup)getActivity().getWindow().getDecorView();
ViewUtil.setTypeFace(app.getTrebuchet(), vg);

MyApplication.java

public Typeface getTrebuchet() {
    if (trebuchet == null){
        trebuchet = Typeface.createFromAsset(getAssets(), Consts.PATH_TYPEFACE_TREBUCHET);
    }
    return trebuchet;
}

我的DDMS显示了内存泄漏：

enter image description here

编辑2：重要提示！

我的应用程序中使用了导航抽屉，由唯一的活动MainActivity处理。导航抽屉使用碎片（而不是活动）。

这就是为什么DayPagerFragment扩展自Fragment（而不是FragmentActivity或Activity）。

为了在日期之间进行滑动，用户必须触摸两个按钮（prev / next）。我在DayFragment上使用setOnClickListener来处理这些按钮（请参见我的更新代码）。

问题是我调用了((MainActivity)getActivity()).viewDay(calPrev);

MainActivity

public class MainActivity extends ActionBarActivity implements NavigationDrawerFragment.NavigationDrawerCallbacks {

    private NavigationDrawerFragment mNavigationDrawerFragment;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        ...
        // Set up the navigation drawer
        mNavigationDrawerFragment.setUp(R.id.navigation_drawer, (DrawerLayout) findViewById(R.id.drawer_layout));
    }
    ...
    public void viewDay(Calendar calendar) {
        DayFragment dayFragment = DayFragment.newInstance(calendar); // The problem is here I think !
        FragmentManager fragmentManager = getSupportFragmentManager();
        fragmentManager.beginTransaction()
                .setCustomAnimations(android.R.anim.fade_in, android.R.anim.fade_out)
                .replace(R.id.container, dayFragment)
                .addToBackStack(null)
                .commit();
    }

}

所以...我认为因为我每次实例化一个新的fragment，ViewPager就不能发挥它的作用！而且MainActivity保留了对每个Fragment的引用：这就是为什么垃圾回收器不释放内存的原因。

现在：

  1. 我不知道我的理论是否正确
  2. 如何更正？如何从setOnClickListener调用setCurrentPagerItemPrev和setCurrentPagerItemNext方法（请查看我在DayPagerFragment中的更新代码）？

NB: 我使用mAdapterViewPager.getRegisteredFragment()代替mViewPager.setCurrentItem，因为我的DayAdapter扩展自SmartFragmentStatePagerAdapter，但它们是相同的。

我无法找到解决我的问题的方法。没有人能帮助我或者有什么想法吗？ - Guicara
尝试使用类似 http://square.github.io/picasso/ 的库来维护图像内存处理。 - Pavandroid
你最终解决了问题并可以分享你是如何做到的吗？ - akohout
很遗憾，我已经移除了ViewPager并手动实现了手势控制来替代ViewPager的行为。虽然不完美，但是... - Guicara
7个回答
4
您需要检查两个方面：图像和字体。
图像使用大量内存，因此您需要做一些工作来快速释放它们。我使用以下代码来帮助清理我的内存，它会删除引用，从而有助于更快地清理。
同时，在短时间内加载许多高质量图像可能会导致错误，因为Android在增加堆空间方面有点慢。请参见我的答案：Android Understanding Heap Sizes。您可能需要在清理图像和字体后才设置android:largeHeap="true"在您的清单文件中。
使用DDMS堆视图检查内存是否保持水平，这意味着您正在清理已滚动的页面。这需要在IDE上安装Android工具插件。
public abstract class SimplePagerAdapter extends PagerAdapter {
// ...
@Override
public void destroyItem(ViewGroup container, int position, Object object) {
    container.removeView((View) object);
    unbindDrawables((View) object);
    object = null;
}
protected void unbindDrawables(View view) {
    if (view.getBackground() != null) {
        view.getBackground().setCallback(null);
    }
    if (view instanceof ViewGroup) {
        for (int i = 0; i < ((ViewGroup) view).getChildCount(); i++) {
            unbindDrawables(((ViewGroup) view).getChildAt(i));
        }
        ((ViewGroup) view).removeAllViews();
    }
}
}

接下来，请检查您获取字体的方式。确保缓存所有对 Typeface.createFromAsset(...) 的调用，就像下面的示例一样：

public Typeface getFont(String font, Context context) {
    Typeface typeface = fontMap.get(font);
    if (typeface == null) {
        typeface = Typeface.createFromAsset(context.getResources().getAssets(), "fonts/" + font);
        fontMap.put(font, typeface);
    }
    return typeface;
}

编辑：问题更新后的额外信息

将您的适配器更改为使用对象列表：

在 getCount() 中返回该列表的大小，

添加对象到列表时调用：

notifyDataSetChanged();
mPagerContainer.invalidate();

当你接近当前列表的末尾时，请请求更多项并将它们添加到列表中，再次调用上面的代码。

我已经实现了类似的东西，这是一个带有大量高分辨率图片的无限滚动器，但我没有使用片段，所以我对它们从列表中删除的方式不太清楚。

我可以看到您没有覆盖isViewFromObject，也许您想在您的页面适配器中添加以下方法：

@Override
public boolean isViewFromObject(View view, Object object) {
    View _view = (View) object;
    return view == _view;
}
我已经编辑了我的第一个问题，向您展示如何加载TypeFace。我认为TypeFace的加载不是问题所在。我尝试使用您的代码（PagerAdapter），但内存泄漏问题仍然存在。请查看附加的DDMS控制台屏幕截图（字节数组> 176 Mb！）。 - Guicara
@Guicara，我更新了我的回答，我认为你需要改变填充列表的方式。此外，我可能在说胡话，但如果上述方法都不起作用，请重写*saveState()*函数并返回null，然后进行测试。 - tbraun
请查看我的更新问题（编辑2）。谢谢！如果我错了，我会测试您的更新答案。 - Guicara
我认为这行代码 "object = null;" 没有任何意义，因为你只是删除了传递给方法的引用的副本。 - Leo DroidCoder
@Leo，方法作用域的结尾应该负责清理它。在Java中将对象设置为null是没有意义的。 - tbraun
1
我已经编辑了我的第一个问题，向您展示我如何加载TypeFace。我不认为TypeFace的加载是问题所在。 - Guicara
1

我看了一下资源，首先想到的是没有人真正调用destroyItem()。你可能应该自己去做。

0

很抱歉重新翻出了一篇旧帖子，但我想贡献我的方法。

在我的应用程序中，我通过使用以下方式解决了它：

tiv.setImageBitmap(BitmapFactory.decodeFile(outFile.getPath()));

替代

tiv.setImageDrawable(Drawable.createFromPath(outFile.getAbsolutePath()));

将图像设置到我的ImageView中。我的应用程序现在定期清理堆内存至25-30MB左右。使用之前的方法，我的堆内存无限增长。看起来Bitmapfactory可以更好地处理它的内存使用。

谢谢。我会在我的应用的下一个更新中尝试您的解决方案。 - Guicara
0
尝试使用https://github.com/nostra13/Android-Universal-Image-Loader来填充您的ImageView，而不是使用默认方法。这个库会处理内存，缓存图像等等。你的问题可能出在

mLeft.setImageResource(this.dayUtilPrev.getDrawable());
mCenter.setImageResource(dayUtil.getDrawable());
mRight.setImageResource(this.dayUtilNext.getDrawable());

这些函数在每次调用新片段时被调用，使用了许多位图解码函数，例如：

at android.graphics.BitmapFactory.nativeDecodeAsset(Native Method) at android.graphics.BitmapFactory.decodeStream(BitmapFactory.java:587) at android.graphics.BitmapFactory.decodeResourceStream(BitmapFactory.java:422) at android.graphics.drawable.Drawable.createFromResourceStream(Drawable.java:840) at android.content.res.Resources.loadDrawable(Resources.java:2110)

如果没有正确处理，这些函数将很快填满您的内存。

我感到困惑，因为在UIL项目页面上有这样的注释：注意：只有在确实需要时才使用drawable://！始终考虑本地加载drawable的方式 - ImageView.setImageResource(...)，而不是使用ImageLoader。 参考：github.com/nostra13/Android-Universal-Image-Loader。我会尝试使用这个库并回来这里。 - Guicara
当库从内存缓存中加载图像（因此图像已经被缓存=从ViewPager查看的页面），堆大小是稳定的。但是，当应用程序加载新图像（ViewPager的新页面）时，堆大小会不断增长！我不明白为什么垃圾收集器不释放内存... - Guicara
0

您已将意图过滤器MAIN添加到启动页活动中。它结束后，接下来执行所有繁重工作的线程(MainActivity)具有默认优先级(0)。 Android活动优先级可以从0到1000。除此之外，我还在您的清单文件中添加了android:largeHeap。

另外，您可以拥有两个带有意图过滤器MAIN的活动。 请按照以下方式更新您的清单文件：

<?xml version="1.0" encoding="utf-8"?>
<manifest xmlns:android="http://schemas.android.com/apk/res/android"
   package="com.example" >

    <uses-sdk
       android:minSdkVersion="11"
       android:targetSdkVersion="19" />

    <uses-permission ...>

    <supports-screens
       android:smallScreens="false"
       android:normalScreens="true"
       android:largeScreens="true"
       android:xlargeScreens="true"
       android:anyDensity="true" />

    <application
       android:name="com.example.MyApplication"
       android:largeHeap="true"
       android:allowBackup="true"
       android:icon="@drawable/ic_launcher"
       android:label="@string/app_name">

        <!-- ACTIVITIES -->
        <activity
           android:name="com.example.ui.SplashActivity"
           android:label="@string/app_name"
           android:screenOrientation="portrait" >
            <intent-filter>
                <action android:name="android.intent.action.MAIN" />
                <category android:name="android.intent.category.LAUNCHER" />
            </intent-filter>
        </activity>
        <activity
           android:name="com.example.ui.MainActivity"
           android:screenOrientation="portrait"
           android:label="@string/app_name" 
           android:priority:"900">
            <intent-filter>
                <action android:name="android.intent.action.MAIN" />
                <category android:name="android.intent.category.DEFAULT" />
            </intent-filter>
        </activity>

        <!-- ROBOSPICE SERVICES -->
        <service
           android:name="com.example.network.CalendarSpiceService"
           android:exported="false" />

    </application>

</manifest>
1
谢谢。我已经更新了我的AndroidManifest文件，但是我没有看到明显的变化（但是代码会因此而变得更好）。 - Guicara
0
2
我已经实现了SmartFragmentStatePagerAdapter适配器，但问题仍然存在。因此我开始思考问题可能出在我的Day fragment上：一些东西一定阻止了Garbage Collector完成它的工作。 - Guicara
网页内容由stack overflow 提供, 点击上面的
可以查看英文原文，
原文链接