从ActiveX调用时未释放WPF内存

时间:2016-04-22 05:52:31

标签: c# wpf memory-leaks activex

我有一个将1000个ComboBox填充到UI中的类,代码是

namespace ActiveXMemory
{
 /// <summary>
 /// Interaction logic for MainWindow.xaml
 /// </summary>
 public partial class MainWindow : Window
 {
    public MainWindow()
    {
        InitializeComponent();
        PopulateControls();
    }
    public void PopulateControls() {
        var newGrid = new Grid();
        for (var i = 0; i < 1000; i++)
        {
            ComboBox newcombo = new ComboBox();
            newGrid.Children.Add(newcombo);
        }
        this.Content = newGrid;
    }

    public bool OpenWindow(bool isRunning)
    {
        Dispatcher.Invoke(DispatcherPriority.Normal, new Action(() =>
        {
            try
            {
                this.ShowDialog();
            }
            catch (Exception exp)
            {
                Console.WriteLine(exp.Message);
            }
        }));
        return true;
    }

    public bool CloseWindow()
    {
        Dispatcher.Invoke(DispatcherPriority.Render, new Action(() =>
        {
            try
            {
                this.Close();
            }
            catch (Exception exp)
            {
                //Console.WriteLine(exp.Message);
            }
        }));

        return true;
    }

    private void Window_Closed(object sender, EventArgs e)
    {
        var grid = Content as Grid;

        var children = grid.Children;

        while (children.Count > 0)
        {
            var child = children[0];
            grid.Children.Remove(child);
            child = null;
        }
        grid = null;
    }

 }
}

我为类创建了一个ActiveX库,可以作为ActiveX访问,

namespace ActiveXLibrary
{

[ComVisible(true)]
[Guid("EF2EAD91-68A8-420D-B5C9-E30A6F510BDE")]
public interface IActiveXLib
{
    [DispId(1)]
    bool Initialize();
    [DispId(2)]
    bool CloseActiveX();
}

[ComVisible(true)]
[Guid("9DACD44F-0237-4F44-BCB9-0E6B729915D6"),
ClassInterface(ClassInterfaceType.None)]
[ProgId("Samp")]
public class ActiveXLib : IActiveXLib
{
    MainWindow form;
    Thread thr;
    public bool Initialize()
    {
        int i = 0;
        try
        {
            ThreadStart exeFunc = new ThreadStart(() =>
            {
                Dispatcher.CurrentDispatcher.Invoke(DispatcherPriority.Send, new Action(() =>
                {
                    start();
                }));
            });
            thr = new Thread(exeFunc);
            thr.SetApartmentState(ApartmentState.STA);
            thr.Start();

            while (form == null)
            {
                i++;
                //Console.WriteLine("form Null");
                System.Threading.Thread.Sleep(1000);
                if (i > 30)
                    break;
            }

            return true;
        }
        catch (Exception exp)
        {
            Console.WriteLine("[Initialize]" + exp.Message);
            return false;
        }
    }

    public bool CloseActiveX()
    {
        bool success = false;
        try
        {

            success = form.CloseWindow();
            form = null;
            GC.Collect(GC.MaxGeneration, GCCollectionMode.Forced);
            GC.WaitForPendingFinalizers();
            GC.Collect(GC.MaxGeneration, GCCollectionMode.Forced);
            return success;
        }
        catch (Exception exp)
        {
            Console.WriteLine("[CloseActiveX]" + exp.Message);
            return false;
        }
    }

    private void start()
    {
        try
        {
            Console.WriteLine("start() - new MainWindow()");
            form = new MainWindow();
            Console.WriteLine("OpenWindow()");
            form.OpenWindow(true);
        }
        catch (Exception exp)
        {
            MessageBox.Show(exp.Message + "\nPossible Reasons: " + exp.Source + "\n" + exp.InnerException, "Error");
        }
    }
}

}

然后我创建了一个演示项目

  ActiveXLib activeXRef;
    public MainWindow()
    {
        InitializeComponent();
    }

    private void startButton_Click(object sender, RoutedEventArgs e)
    {
        activeXRef = new ActiveXLib();
        activeXRef.Initialize();

    }

    private void stopButton_Click(object sender, RoutedEventArgs e)
    {
        activeXRef.CloseActiveX();
        GC.Collect();
    }

这里的问题是每次我点击开始和停止按钮,内存不断增加,应用程序第一次启动时创建的内存(17MB)没有被释放,即使我删除了ComboBox并设置了网格内容为null。即使GC.Collect()也没有效果,因为它只会将垃圾添加到队列中。有没有解决这个问题。

enter image description here

我也试过像link那样刷新内存,但即便如此,内存仍然存在。(因为我不确定当前进程的句柄是来自demo项目还是来自ActiveXMemory)

修改

我加入了

 this.Dispatcher.Invoke(DispatcherPriority.Render, GCDelegate);

到Window_Closed事件

现在内存被清除

enter image description here

但我现在面临的一个问题是,如果我试图立即关闭(调用CloseActiveX()),那么就不会发生这种情况。(即)

    private void startButton_Click(object sender, RoutedEventArgs e)
    {
        activeXRef = new ActiveXLib();
        activeXRef.Initialize();
        activeXRef.CloseActiveX();
        activeXRef = null;
        GC.Collect();

    }

上面的代码仍然锁定了内存,我不明白其中的区别,这只是另一个事件,有没有人对此有任何想法?

2 个答案:

答案 0 :(得分:3)

我认为虽然我无法告诉你的MainWindow.Open/CloseWindow()方法是什么样的,但我还是有了这个问题。使用线程肯定是问题的一部分,你必须做一个非直观的事情来防止泄漏具有线程亲和力的内部WPF管道对象。

必须关闭线程的调度程序。当UI线程终止时,这个通常只在WPF应用程序中发生一次。同样非常重要的是调度员进行调度,WPF在很大程度上取决于它以确保&#34;弱事件&#34;实际上很弱。

MainWindow的一个不泄漏的示例实现:

public class MainWindow : Window {
    public void OpenWindow(bool noidea) {
        this.ShowDialog();

    }
    public bool CloseWindow() {
        this.Dispatcher.Invoke(() => {
            this.Dispatcher.InvokeShutdown();   // <== Important!
        });
        return true;
    }
}

ShowDialog()确保调度程序执行调度,而InvokeShutdown()确保发生清理。通过使用10毫秒DispatcherTimer而不是按钮进行测试,因此发生率非常高。我可以删除GC.Collect()调用和内存使用情况稳定。

答案 1 :(得分:1)

目前我没有对ActiveX部分发表评论(会仔细检查并通知你),但我可以提出与WPF代码相关的建议。

这应该使组合物品一次性

public class DispCombo : ComboBox, IDisposable
{
    public void Dispose()
    {
        GC.SuppressFinalize(this);
    }
}
public class DispGrid : Grid, IDisposable
{
    public void Dispose()
    {
        GC.SuppressFinalize(this);
    }
}

您将填充一次性控件

public void PopulateControls()
{
    var newGrid = new DispGrid();
    for (var i = 0; i < 1000; i++)
    {
        DispCombo newcombo = new DispCombo();
        newGrid.Children.Add(newcombo);
    }
    this.Content = newGrid;
}

因此结束阶段将成为(处置而不是设置为null)

private void Window_Closed(object sender, EventArgs e)
{
    var grid = Content as DispGrid;

    var children = grid.Children;

    while (children.Count > 0)
    {
        var child = children[0] as DispCombo;
        grid.Children.Remove(child);
        child.Dispose();
    }
    grid.Dispose();

}

那就是说,现在你可以将WPF视图包装在using子句

public class MainLibrary
    {
        public bool OpenWindow() //found no ref to bool isRunning
        {

            using (var mw = new MainWindow())
            {
                mw.ShowDialog();
                mw.Close();
            }

            return true;
        }

        public bool CloseWindow()
        {
            GC.Collect(GC.MaxGeneration, GCCollectionMode.Forced);
            GC.WaitForPendingFinalizers();
            GC.Collect(GC.MaxGeneration, GCCollectionMode.Forced);
            return true;
        }
    }

关闭垃圾收集。 这些是我在这些更改后的内存快照,在开始时采用(1),然后在ShowDialog中的主OpenWindow之后(2)CloseWindow

enter image description here

修改

最后,如果你需要启动一个Task,调用Dispatcher,这里就是函数

public bool CloseWindow()
        {
            winClose = () =>
            {
                mw.Dispatcher.Invoke(() =>
                   {
                       mw.Close();
                       mw.Dispose();
                   });
            };
            cleanProc = () =>
            {
                mw.Dispatcher.Invoke(() =>
                {
                    GC.Collect(GC.MaxGeneration, GCCollectionMode.Forced);
                    GC.WaitForPendingFinalizers();
                    GC.Collect(GC.MaxGeneration, GCCollectionMode.Forced);
                });
            };
            Task.Run(() =>
           {
               winClose.Invoke();
           }).ContinueWith(x =>

               {
                   cleanProc.Invoke();
               });

            return true;
        }

public bool OpenWindow() 
        {
            mw = new MainWindow();
            mw.Show();
            return true;
        }