すべてのMSHTMLオブジェクトでReleaseComObjectを必ず呼び出す必要はありますか？

Question

私はWebBrowserがテキストノードのようなものにアクセスできないので、WebBrowserコントロールでMSHTMLを使用しています。ここではいくつかの記事を見てきましたが、Web上では参照するすべてのCOMオブジェクトに対してReleaseComObjectと呼ぶ必要があります。だから、私はこれを行うと言う：

var doc = myBrowser.Document.DomDocument as IHTMLDocument2;

私はdocを解放する必要がありますか？どのようにこのコードでbody：

var body = (myBrowser.Document.DomDocument as IHTMLDocument2).body;

はすぐにそれらへの参照がなくなったとして、それらを解放するRCWによって包まれ、これらのオブジェクトではありませんか？もしそうでなければ、ガベージコレクタが起動するとすぐにそれらを解放するファイナライザ（Disposeを使用するのではなく）を使って、それぞれのラッパーを作成することをお勧めします（私は手動で心配する必要はありませんそれらを処分する）？

私のアプリケーションにはメモリリークがあり、これに関連していると思います。 ANTSメモリプロファイラによれば、第2世代のメモリを使用してオブジェクトのトップリストにあるMicrosoft.CSharp.RuntimeBinder.Semantics.LocalVariableSymbolオブジェクトの束への参照を保持する関数（MSHTMLオブジェクトを使用する他の多くの機能）の1つは次のとおりです。

internal static string GetAttribute(this IHTMLDOMNode element, string name) 
{ 
    var attribute = element.IsHTMLElement() ? ((IHTMLElement)element).getAttribute(name) : null; 
    if (attribute != null) return attribute.ToString(); 
    return ""; 
} 

attributeは単なる文字列なので、何が間違っているのかわかりません。ここで

ANTSプロファイラのインスタンスの保存グラフに示されている他の機能（私はFinalReleaseComObject Sの束を追加しましたが、まだ示されている）である：私はReleaseComObjectを追加しましたが、機能はまだのようです

private void InjectFunction(IHTMLDocument2 document) 
{ 
    if (null == Document) throw new Exception("Cannot access current document's HTML or document is not an HTML."); 

    try 
    { 
     IHTMLDocument3 doc3 = document as IHTMLDocument3; 
     IHTMLElementCollection collection = doc3.getElementsByTagName("head"); 
     IHTMLDOMNode head = collection.item(0); 
     IHTMLElement scriptElement = document.createElement("script"); 
     IHTMLScriptElement script = (IHTMLScriptElement)scriptElement; 
     IHTMLDOMNode scriptNode = (IHTMLDOMNode)scriptElement; 
     script.text = CurrentFuncs; 
     head.AppendChild(scriptNode); 
     if (Document.InvokeScript(CurrentTestFuncName) == null) throw new Exception("Cannot inject Javascript code right now."); 
     Marshal.FinalReleaseComObject(scriptNode); 
     Marshal.FinalReleaseComObject(script); 
     Marshal.FinalReleaseComObject(scriptElement); 
     Marshal.FinalReleaseComObject(head); 
     Marshal.FinalReleaseComObject(collection); 
     //Marshal.FinalReleaseComObject(doc3); 
    } 
    catch (Exception ex) 
    { 
     throw ex; 
    } 
} 

何かへの参照を保持しています。ここに私の関数は、今どのように見えるかです：

private void InjectFunction(IHTMLDocument2 document) 
{ 
    if (null == Document) throw new Exception("Cannot access current document's HTML or document is not an HTML."); 

    try 
    { 
     IHTMLDocument3 doc3 = document as IHTMLDocument3; 
     IHTMLElementCollection collection = doc3.getElementsByTagName("head"); 
     IHTMLDOMNode head = collection.item(0); 
     IHTMLElement scriptElement = document.createElement("script"); 
     IHTMLScriptElement script = (IHTMLScriptElement)scriptElement; 
     IHTMLDOMNode scriptNode = (IHTMLDOMNode)scriptElement; 
     script.text = CurrentFuncs; 
     head.AppendChild(scriptNode); 
     if (Document.InvokeScript(CurrentTestFuncName) == null) throw new Exception("Cannot inject Javascript code right now."); 
     Marshal.FinalReleaseComObject(scriptNode); 
     Marshal.FinalReleaseComObject(script); 
     Marshal.FinalReleaseComObject(scriptElement); 
     Marshal.FinalReleaseComObject(head); 
     Marshal.FinalReleaseComObject(collection); 
     Marshal.ReleaseComObject(doc3); 
    } 
    catch (Exception ex) 
    { 
     MessageBox.Show("Couldn't release!"); 
     throw ex; 
    } 
} 

MessageBox.Show("Couldn't release!");ラインがヒットされることはありませんので、私はすべてが正常にリリースされていると仮定します。ここでANTSが示しものです：

私はそのサイトのコンテナの事が何であるか見当がつかない。

Answer 1

RCWはRCWがファイナライズされたときにCOMオブジェクトを解放するので、これを行うラッパーを作成する必要はありません。ファイナライズを待つことを望まないので、ReleaseComObjectに電話してください。これはDisposeパターンの根拠と同じです。だから、Dispose dは悪い考えではない可能ラッパーを作成（および実施例はそこ

var doc = myBrowser.Document.DomDocument ...;のためにありますが、あなたはまた、別の変数に.Documentを捕捉しなければならないとReleaseComObjectは、それは同様に。いつでもあなたはのプロパティを参照します別のオブジェクトを作成するCOMオブジェクト、それを解放することを確認してください。GetAttributeで

、あなたは別のインターフェイスに要素をキャストしている。COMプログラミング、that adds another referenceで。あなたはとてもあなたが解放することができvar htmlElement = (IHTMLElement) element;ような何かをする必要がありますそれだけでなく。

編集 - これは、COMオブジェクトを操作するときに使用するパターンです：

IHTMLElement element = null; 
try 
{ 
    element = <some method or property returning a COM object>; 
    // do something with element 
} 
catch (Exception ex) // although the exception type should be as specific as possible 
{ 
    // log, whatever 

    throw; // not "throw ex;" - that makes the call stack think the exception originated right here 
} 
finally 
{ 
    if (element != null) 
    { 
     Marshal.ReleaseComObject(element); 
     element = null; 
    } 
} 

これは本当にあなたが持っているすべてのCOMオブジェクト参照のために行われるべきです。

Answer 2

おそらく、この記事では、いくつかの光をもたらします：あなたのケースでは

MSDN on how COM refcounting works and some basic rules when to call AddRef and Release

、リリースはあなたの方法でCOMオブジェクトを使用している場合は、常に以来、例外をスローする前にそれらをクリーンアップからReleaseComObject