Excel COM对象未被释放

时间:2015-09-14 19:16:55

标签: c# marshalling excel-interop

我在加载用户的设置文件后尝试使用干净的excel中断。我遇到了一些我无法弄清楚的东西,除了它与词典有关。如果我注释掉一个(或两个)Dictionary填充括号,则设置文件会加载然后释放,但如果两者都在运行,则excel应用程序将不会释放。我是否也通过从他们那里获取数据来优化词典?

我确信还有其他方法可以创建全球词典,但这是我目前唯一的知情方式,但我愿意学习是否有更好的词汇。

“字典填充代码”是带有i&的for循环。 Y:

for (int i = 0; i < lastRow - 1; i++)
    {
        string key = settingsSheet.Range["B" + (i + 2)].Value; 
        string value = settingsSheet.Range["A" + (i + 2)].Value; 
        DictionaryLoad.DIC.Add(key, value);
    }

这是完整的代码:

public Form1()
    {
        InitializeComponent();
        txtFileNamePreface.Enabled = false;
        string fileName = "F:\\Shared\\Projects\\State Assoc Clients\\Data Management\\Download Site\\KeyStats Download Statistics\\Naming Conventions.xls";
        LoadProductName(fileName);
    }

    public static class DictionaryLoad
    {
        public static IDictionary<string, string> DIC;
        public static IDictionary<string, string> DIC2;
        static DictionaryLoad()
        {
            DIC = new Dictionary<string, string>();
            DIC2 = new Dictionary<string, string>();
        }
    }

    private void LoadProductName(string fileName)
    {
        //starting up and defining the Excel references
        Excel.Application excelApp = new Excel.Application(); //excel open here
        Excel.Workbook settingsBook = null;
        Excel.Worksheet settingsSheet = null;
        excelApp.Visible = false;
        excelApp.DisplayAlerts = false;

        settingsBook = excelApp.Workbooks.Open(fileName);
        settingsSheet = settingsBook.Sheets["NamingConventions"]; 

        int lastRow = findFirstBlankRow(settingsSheet, "A1") - 1;
        fillComboBox(cbProductType, lastRow, settingsSheet, "A"); 
        fillComboBox(cbYear, lastRow, settingsSheet, "D");

        int lastRow2 = findFirstBlankRow(settingsSheet, "E1");
        fillComboBox(cbRule, lastRow2, settingsSheet, "E");

        for (int i = 0; i < lastRow - 1; i++)
        {
            string key = settingsSheet.Range["B" + (i + 2)].Value; 
            string value = settingsSheet.Range["A" + (i + 2)].Value; 
            DictionaryLoad.DIC.Add(key, value);
        }

        cbProductName.Items.Clear();
        foreach (KeyValuePair<string, string> entry in DictionaryLoad.DIC)
        {
            if (entry.Value == cbProductType.Text)
            { cbProductName.Items.Add(entry.Key); }
        }
        try { cbProductName.SelectedIndex = 0; }
        catch { }

        for (int j = 0; j < lastRow - 1; j++)
        {
            string key = settingsSheet.Range["B" + (j + 2)].Value; 
            string value = settingsSheet.Range["C" + (j + 2)].Value;
            DictionaryLoad.DIC2.Add(key, value);
        }

        cbRule.SelectedIndex = 0;
        cbYear.Text = DateTime.Now.Year.ToString();
        cbQuarter.SelectedIndex = 0;
        cbMonth.Text = DateTime.Now.ToString("MMMM");
        cbProductType.SelectedIndex = 0;
        string workBookName = excelApp.ActiveWorkbook.FullName;
        txtOutputFolder.Text = Path.GetDirectoryName(workBookName);

        settingsBook.Close();
        excelApp.Quit();
        appCleanup(excelApp);
        appCleanup(settingsBook);
        appCleanup(settingsSheet);
        garbageCleanup();
        Application.Exit();
    }
   public void appCleanup(object application1, object application2 = null, object application3 = null)
    {
        Marshal.ReleaseComObject(application1);
        application1 = null;
    }

    public void garbageCleanup()
    {
        GC.Collect();
        GC.WaitForPendingFinalizers();
    }

2 个答案:

答案 0 :(得分:2)

  1. 您永远不需要在此上下文中调用@IBOutlet var okOutlet: UIButton! @IBAction func okAction(sender: UIButton){} 。运行时完全能够跟踪COM对象,并在不再引用它们时释放它们。调用Marshal.ReleaseComObject是一种令人困惑的反模式,甚至有些微软文档错误地暗示了这种模式。

    在您的情况下,这意味着应删除Marshal.ReleaseComObject中的调用,但您可能仍需要设置appCleanup以清除引用 - 您的代码不应该如何使用此变量。< / p>

  2. 您应该两次调用垃圾收集器例程 - 您可能会遇到引用形成一个循环的情况,并且第一个GC调用将打破循环,但COM对象可能只在第二次调用时正确释放。

  3. 最后,您必须在调试版本中小心使用这种代码。方法中的引用被人为地保持活动直到方法结束,以便它们仍然可以在调试器中访问。这意味着通过在该方法中调用GC,不会清除本地application1 = null变量。 要避免此问题,您可以遵循以下模式:

    excelApp

答案 1 :(得分:1)

我想也许你应该使用string.Copy,因为否则你的字典会持有对链接到Excel对象的字符串对象的引用。这将创建字符串对象的新实例:

for (int i = 0; i < lastRow - 1; i++)
{
    string key = string.Copy(settingsSheet.Range["B" + (i + 2)].Value); 
    string value = string.Copy(settingsSheet.Range["A" + (i + 2)].Value); 
    DictionaryLoad.DIC.Add(key, value);
}