CollectionView导致内存泄露?

本文将创建一个示例项目,运行后探查内存,发现本应被垃圾回收的UI控件没有被回收。进一步发现是CollectionView导致控件不能被回收。最后,通过查看.NET Framework源代码,发现其实不是内存泄露,虚惊一场。

发现问题

创建一个用户控件GroupControl,有AddGroup(object header, object[] items)方法。这个方法就是创建一个GroupBox,设置Header和GroupBox里面的ListBox.ItemsSource。

GroupControl.xaml

<ContentControl x:Class="Gqqnbig.TranscendentKill.GroupControl"
             xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
             xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
             xmlns:mc="http://schemas.openxmlformats.org/markup-compatibility/2006" 
             xmlns:d="http://schemas.microsoft.com/expression/blend/2008" 
             mc:Ignorable="d" d:DesignHeight="300" d:DesignWidth="300">
    <ItemsControl Name="selectionGroupPanel" x:FieldModifier="private" HorizontalAlignment="Left" VerticalAlignment="Top"/>
</ContentControl>

GroupControl.xaml.cs

    public partial class GroupControl
    {
        public GroupControl()
        {
            InitializeComponent();
        }

        public event SelectionChangedEventHandler SelectionChanged;

        public void AddGroup(object header, object[] items)
        {
            GroupBox groupBox = new GroupBox();
            groupBox.Header = header;
            ListBox listBox = new ListBox();
            listBox.ItemsSource = items;
            listBox.SelectionChanged += listBox_SelectionChanged;
            groupBox.Content = listBox;

            selectionGroupPanel.Items.Add(groupBox);
        }

        void listBox_SelectionChanged(object sender, SelectionChangedEventArgs e)
        {
            if (SelectionChanged != null)
                SelectionChanged(this, e);
        }
    }

然后主窗口使用这个GroupControl,在窗口加载的时候往GroupControl里填数据,当用户选择GroupControl里任意一项的时候,卸载这个GroupControl。

MainWindow.xaml

<Window x:Class="Gqqnbig.TranscendentKill.UI.MainWindow"
        xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation"
        xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
        Width="800" x:ClassModifier="internal" Loaded="Window_Loaded_1">

</Window>

MainWindow.xaml.cs

    internal partial class MainWindow : Window
    {
        public MainWindow()
        {
            InitializeComponent();
        }


        private void Window_Loaded_1(object sender, RoutedEventArgs e)
        {
            Tuple<string, object[]>[] cps = new Tuple<string, object[]>[2];
            cps[0] = new Tuple<string, object[]>("时间", new[] { (object)DateTime.Now.ToShortTimeString() });
            cps[1] = new Tuple<string, object[]>("日期", new[] { (object)DateTime.Now.ToShortDateString() });


            GroupControl win = new GroupControl();

            for (int i = 0; i < cps.Length; i++)
            {
                ContentPresenter[] items = new ContentPresenter[cps[i].Item2.Length];

                for (int j = 0; j < cps[i].Item2.Length; j++)
                    items[j] = new ContentPresenter { Content = cps[i].Item2[j] };

                win.AddGroup(cps[i].Item1, items);
            }

            win.SelectionChanged += win_SelectionChanged;
            Content = win;
        }

        void win_SelectionChanged(object sender, SelectionChangedEventArgs e)
        {
            GroupControl win = (GroupControl)this.Content;
            win.SelectionChanged -= win_SelectionChanged;

            Content = null;


            GC.Collect();
        }
    }


当卸载了GroupControl之后,尽管也调用了GC,我用.NET Memory Profiler查看,发现它还是存在。

CollectionView导致内存泄露?_第1张图片

图1


CollectionView导致内存泄露?_第2张图片

图2

图2表示GroupBox._contextStorage保存了我的GroupControl;ListBox._parent保存了前面的GroupBox; ItemsPresenter保存了前面的ListBox;以此类推。因为有对GroupControl的引用链存在,所以它无法被垃圾回收。

不彻底的解决方案

从引用链可以发现,ContentPresenter引用了父元素ListBoxItem,所以尝试从ContentPresenter入手。不生成ContentPresenter,直接用原始的集合。

把MainWindow.cs的

            for (int i = 0; i < cps.Length; i++)
            {
                ContentPresenter[] items = new ContentPresenter[cps[i].Item2.Length];

                for (int j = 0; j < cps[i].Item2.Length; j++)
                    items[j] = new ContentPresenter { Content = cps[i].Item2[j] };

                win.AddGroup(cps[i].Item1, items);
            }
改为

            for (int i = 0; i < cps.Length; i++)
            {
                //ContentPresenter[] items = new ContentPresenter[cps[i].Item2.Length];

                //for (int j = 0; j < cps[i].Item2.Length; j++)
                //    items[j] = new ContentPresenter { Content = cps[i].Item2[j] };

                win.AddGroup(cps[i].Item1, cps[i].Item2);
            }
。这样探查内存,发现GroupControl消失了。问题疑似解决。


但这样的解决方案留给人几点疑惑:

  1. 控件之间的相互引用不应阻碍垃圾回收,只要它们没有被外部的长生命周期的实例引用。
  2. 这个解决方案似乎不能得出什么一般性的原则来避免疑似由ContentPresenter引起的内存泄露。众所周知,WPF里大量使用ContentPresenter,难道都会泄露?


仔细查看内存探查的结果(图3),会发现ListCollectionView(也存在于图2中的第7行)并没有被垃圾回收。

CollectionView导致内存泄露?_第3张图片

图3

所以,需要一个更彻底的解决方案。

寻找原因/彻底的解决方案

图3说明ListCollectionView跟外部做了什么交互,导致自己被引用上了,所以一连串都不能被回收。

我在VS里输入ListCollectionView,然后按F12转到定义。我装了Resharper,做过查看.net源代码的配置,所以就可以转到ListCollectionView的源代码。可是不知为什么,ListCollectionView的代码是空的。于是我转到CollectionView。

然后查找哪里使用了CollectionView。Reshaper 7.0疑似跟以前相比改进过了,可以查找.net类库里使用的类,于是我找到了ViewTable.Purge(),而且ViewTable也正好在引用链里面。

CollectionView导致内存泄露?_第4张图片

图4

查找ViewTable.Purge()的调用方,可以找到ViewManager.Purge()。继续查找,定位到 DataBindEngine.GetViewRecord(object collection, CollectionViewSource key, Type collectionViewType, bool createView, Func<object, object> GetSourceItem):ViewRecord。所以差不多明白了,每当创建新的CollectionView的时候,就会调用Purge,就会删除未使用的旧的CollectionView。

于是尝试解决办法,在MainWindow.cs的GC.Collect()之后,加上

            ListBox listBox = new ListBox();
            int[] n = { 1, 2, 3, 4 };
            listBox.ItemsSource = n;
            Content = listBox;

再探查内存,发现只有一个ListCollectionView,值为1234;而旧的“时间”“日期”的CollectionView已经被销毁了。

经过此番研究,结论是移除对ItemsControl的引用后,ItemsControl的CollectionView不会销毁,ItemsControl本身可能也不会销毁。如果再创建一个新的ItemsControl,并填充数据,旧的CollectionView和ItemsControl就会被销毁了。




你可能感兴趣的:(.net,内存,memory,profiler)