C#　new HtmlAgilityPack.HtmlDocument();でインスタンス生成したオブジェクトの解放方法

C#でWebBrowserを使ったアプリケーションを作成しています。

WebBrowserに、Navigateで表示させたHTMLを、HtmlAgilityPack.HtmlDocumentでLoadして、いるのですが、ループの中で、HtmlAgilityPack.HtmlDocumentのnewを繰り返すと、ループ回数が多くなると、メモリ不足でエラーになります。

プログラム的には、webBrowserのDocumentCompletedで、

C#
1foreach (HtmlAgilityPack.HtmlNode data in datas)
2{
3    HtmlAgilityPack.HtmlDocument doc = new HtmlAgilityPack.HtmlDocument();
4    doc.LoadHtml(data .InnerHtml.ToString());
5
6    //読み込んだデータの処理
7
8}

と、このようにしているのですが、このときdatasの数が多いと、メモリ不足でエラーになってしまうので、なんとか毎回docを開放したいと思っておりますが、解放する方法がわからず困っております。

C#
1HtmlAgilityPack.HtmlDocument doc = new HtmlAgilityPack.HtmlDocument();

このように定義したクラスのインスタンスは、どうやって開放したらいいのでしょうか？

行動規範の内容に同意します

回答1件

ベストアンサー

HtmlDocumentにnullを代入しておいて.Netのメモリ管理に丸投げするとかで逃げられるといいのですが..

たぶんHtmlAgilityPackの中で使われているStreamReaderが開放されてなくてメモリリークを起こしてるんじゃないでしょうかね
もしそうなら根本を直さないとダメでしょう

HtmlAgilityPackはオープンソースなので適宜自分で直して使うなど工夫も要るのでしょう
クオリティがメンテナの情熱(=プロジェクトに掛けることのできる手間=注ぐことのできる無償の愛)に大きく依存してしまうというオープンソース世代の良いところであり悪いとこでもありますね
もし直せたらその成果物を還元されるとみんなに喜ばれると思いますよ
ネットサービスの大手の偉いさんも自社で使うオープンソースプロジェクトには必ず貢献してそのプロジェクトを絶やさないようにしないとプロダクトの技術的負債が大きくなる的なお話をされていたのを聞いたことがあります

あと数年前のstackoverflowにも