在构造函数参数中释放 COM 对象

Question

我正在开发一个 VSTO 插件，现在我希望对其进行优化。在我的代码中，我做了一些事情

public class ExtraOrdinaryClass
{
    public ExtraOrdinaryClass(Excel.Worksheet someGoodSheet)
    {
        tSheetName            = someGoodSheet.Name;
        tDesignSheet          = someGoodSheet;
    }
}

我刚刚知道我应该释放所有 COM 对象，但我正在寻找一种适当的方式来以适当的方式释放 someGoodSheet 对象。我怀疑如果我做下面这样的事情是有效的

public class ExtraOrdinaryClass
{
    public ExtraOrdinaryClass(Excel.Worksheet someGoodSheet)
    {
        tSheetName            = someGoodSheet.Name;
        tDesignSheet          = someGoodSheet;
        Marshal.ReleaseComObject(someGoodSheet);
        someGoodSheet = null;
    }
}

如果我有效地做到这一点，谁能帮助我并告诉我垃圾收集器何时收集参数对象？

Answer 1

我刚刚知道我应该释放所有的 COM 对象，但我正在寻找一种适当的方式来以适当的方式释放 someGoodSheet 对象

不必担心释放 COM 对象，因为您正在编写 VSTO 加载项。 VSTO 加载项是由 COM 应用程序（在本例中为 Excel）加载的in-process COM 库。 Excel 工作表所代表的 COM 对象是由 Excel 创建的，因此它具有所有权。尝试手动释放（通过Marshal.ReleaseComObject）或在您仍然拥有对它的托管引用时过度减少引用计数（如在第二个示例中），可能会使 Excel 和/或您的应用程序崩溃。

如果您正在编写一个独立的进程，例如通过 COM 启动 Excel 并摆弄一些 COM 对象，那么 是的，您需要确保 COM 对象被适当地释放。

即不要在你的构造函数中使用它：

Marshal.ReleaseComObject(someGoodSheet);

因为你实际上是在对 COM 说“我已经完成了它”，但你不是因为你仍然有一个通过 tDesignSheet 对它的托管引用。 COM/Excel 可以选择从你下面删除工作表。下次访问tDesignSheet可能会遇到异常。

---

在复制到字段的参数上调用Marshal.ReleaseComObject 有点像在调用Dispose() 之后使用Font - 两者都会导致访问已删除对象的不良情况。 （在 COM 的情况下，我假设引用计数达到 0）

---

另外，没有必要调用它（因为无论如何你都引用了同一个对象）：

someGoodSheet = null;

您的原始代码（如下所示）很好：

public class ExtraOrdinaryClass
{
    public ExtraOrdinaryClass(Excel.Worksheet someGoodSheet)
    {
        tSheetName            = someGoodSheet.Name;  // you arguably don't need this (just read tDesignSheet.Name)          
        tDesignSheet          = someGoodSheet;
    }
}

并告诉我垃圾收集器何时收集参数对象？

我想说，在这种情况下，不会收集 .NET 参数，因为现在您在 tDesignSheet 内的 ExtraOrdinaryClass 中有一个额外的引用。您要么需要将 tDesignSheet 设置为 null 并等待 GC 下一次运行，无论何时运行。

即使收集了 .NET 对象，我怀疑 .NET 也足够聪明，可以知道 COM 对象仍可能被其他本机 COM 客户端（例如 Excel 本身）使用。

因此，仅仅因为您的 .NET 现在配置的对象不再引用 COM 对象，您可能会发现 COM 对象很可能仍然处于活动状态。例如工作表仍处于打开状态。