web-dev-qa-db-ja.com

Excel相互運用オブジェクトをC#で安全に破棄しますか?

私はwinformsc#visual studio2008アプリケーションに取り組んでいます。アプリはExcelファイルと通信し、これを行うためにMicrosoft.Office.Interop.Excel;を使用しています。

エラーが発生した場合でもオブジェクトが解放されていることを確認するにはどうすればよいですか?

これが私のコードです:

private void button1_Click(object sender, EventArgs e)
{
    string myBigFile="";
    OpenFileDialog openFileDialog1 = new OpenFileDialog();
    DialogResult result = openFileDialog1.ShowDialog(); // Show the dialog.
    if (result == DialogResult.OK) // Test result.
        myBigFile=openFileDialog1.FileName;

    Excel.Application xlApp;
    Excel.Workbook xlWorkBook;
    Excel.Worksheet xlWorkSheet;
    Excel.Range range;

    string str;
    int rCnt = 0;
    int cCnt = 0;

    xlApp = new Excel.ApplicationClass();
    xlWorkBook = xlApp.Workbooks.Open(myBigFile, 0, true, 5, "", "", true, Microsoft.Office.Interop.Excel.XlPlatform.xlWindows, "\t", true, false, 0, true, 1, 0);
    xlWorkSheet = (Excel.Worksheet)xlWorkBook.Worksheets.get_Item(1);

    range = xlWorkSheet.UsedRange;

    /*
    for (rCnt = 1; rCnt <= range.Rows.Count; rCnt++)
    {
        for (cCnt = 1; cCnt <= range.Columns.Count; cCnt++)
        {
            str = (string)(range.Cells[rCnt, cCnt] as Excel.Range).Value2;
            MessageBox.Show(str);
        }
    }
     */
    xlWorkSheet..EntireRow.Delete(Excel.XLDirection.xlUp)

    xlWorkBook.SaveAs(xlWorkBook.Path + @"\XMLCopy.xls",         Excel.XlFileFormat.xlXMLSpreadsheet, Type.Missing, Type.Missing,
   false, false, Excel.XlSaveAsAccessMode.xlNoChange,
   Type.Missing, Type.Missing, Type.Missing, Type.Missing, Type.Missing);

    xlWorkBook.Close(true, null, null);
    xlApp.Quit();

    releaseObject(xlWorkSheet);
    releaseObject(xlWorkBook);
    releaseObject(xlApp);
}

private void releaseObject(object obj)
{
    try
    {
        System.Runtime.InteropServices.Marshal.ReleaseComObject(obj);
        obj = null;
    }
    catch (Exception ex)
    {
        obj = null;
        MessageBox.Show("Unable to release the Object " + ex.ToString());
    }
    finally
    {
        GC.Collect();
    }
}

ブックを開いた後にエラーが発生した場合でも、オブジェクトを確実に破棄するにはどうすればよいですか。

Excel.Application xlApp;
Excel.Workbook xlWorkBook;
Excel.Worksheet xlWorkSheet;
Excel.Range range;

言い換えれば、私が実行するために次の行が必要なものに関係なく

xlWorkBook.Close(true, null, null);
xlApp.Quit();

releaseObject(xlWorkSheet);
releaseObject(xlWorkBook);
releaseObject(xlApp);

私もこれを試したので、同じ問題が発生することに注意してください

xlWorkBook.Close(false, System.Reflection.Missing.Value, System.Reflection.Missing.Value);


                xlApp.Quit();

                Marshal.ReleaseComObject(xlWorkSheet);
                Marshal.ReleaseComObject(xlWorkBook);
                Marshal.ReleaseComObject(xlApp);

                xlWorkSheet = null;
                xlWorkBook = null;
                xlApp = null;

                GC.GetTotalMemory(false);
                GC.Collect();
                GC.WaitForPendingFinalizers();
                GC.Collect();
                GC.GetTotalMemory(true);  

そして私もこれをしました:

GC.Collect()                   ;
                GC.WaitForPendingFinalizers();
                GC.Collect()                  ; 
                GC.WaitForPendingFinalizers();

                Marshal.FinalReleaseComObject(xlWorkSheet);

                xlWorkBook.Close(Type.Missing, Type.Missing, Type.Missing);
                Marshal.FinalReleaseComObject(xlWorkBook); 

                xlApp.Quit();
                Marshal.FinalReleaseComObject(xlApp); 

現時点では、Visual Studio 2008からExcelを閉じることはできないと思います。バグか何かである必要がありますが、これで上位20のWebサイトを試し、同じ結果が得られました。Excelは何らかの理由で2つのインスタンスを開いています。ガベージコレクションなどを実行すると(または実行しないと)、1つのインスタンスだけが閉じられます。

ファイルを開こうとすると、エラーがあるか破損していると表示されます。

タスクマネージャーに移動してExcelプロセスを強制終了すると、ファイルは問題なく開きます。]

visual Studio 2008でExcelを閉じる方法はありますか?もしそうなら、これに対するガイダンスまたは解決策を私に提供していただけませんか

最初に変更されたreleaseObjectを提示し、次にそれを使用するためのパターンを提供します。

using Marshal = System.Runtime.InteropServices.Marshal;
private void releaseObject(ref object obj) // note ref!
{
    // Do not catch an exception from this.
    // You may want to remove these guards depending on
    // what you think the semantics should be.
    if (obj != null && Marshal.IsComObject(obj)) {
        Marshal.ReleaseComObject(obj);
    }
    // Since passed "by ref" this assingment will be useful
    // (It was not useful in the original, and neither was the
    //  GC.Collect.)
    obj = null;
}

ここで、使用するパターン:

private void button1_Click(object sender, EventArgs e)
{
    // Declare. Assign a value to avoid a compiler error.
    Excel.Application xlApp = null;
    Excel.Workbook xlWorkBook = null;
    Excel.Worksheet xlWorkSheet = null;

    try {
        // Initialize
        xlApp = new Excel.ApplicationClass();
        xlWorkBook = xlApp.Workbooks.Open(myBigFile, 0, true, 5, "", "", true, Microsoft.Office.Interop.Excel.XlPlatform.xlWindows, "\t", true, false, 0, true, 1, 0);
        // If the cast fails this like could "leak" a COM RCW
        // Since this "should never happen" I wouldn't worry about it.
        xlWorkSheet = (Excel.Worksheet)xlWorkBook.Worksheets.get_Item(1);
        ...
    } finally {
        // Release all COM RCWs.
        // The "releaseObject" will just "do nothing" if null is passed,
        // so no need to check to find out which need to be released.
        // The "finally" is run in all cases, even if there was an exception
        // in the "try". 
        // Note: passing "by ref" so afterwords "xlWorkSheet" will
        // evaluate to null. See "releaseObject".
        releaseObject(ref xlWorkSheet);
        releaseObject(ref xlWorkBook);
        // The Quit is done in the finally because we always
        // want to quit. It is no different than releasing RCWs.
        if (xlApp != null) {
            xlApp.Quit();
        }
        releaseObject(ref xlApp);    
    }
}

この単純なアプローチは、ほとんどの状況で拡張/ネストできます。このタスクを簡単にするために、IDisposableを実装するカスタムラッパークラスを使用します。

20
user166390

コードに2つの問題が発生していることを確認します。

  • プログラムを閉じると、Excelは実行中のプロセスとして残ります
  • プログラムが作成するExcelファイルを開くと、ファイルが破損しているなどのエラーがExcelに表示されます。

編集した質問のbutton1クリックハンドラーとpstのreleaseObjectメソッドをクリーンなVS2008、C#3.5 Winformアプリケーションにコピーし、上記の両方の問題を解消するためにいくつかの小さな変更を加えました。

Excelがメモリからアンロードされない問題を修正するには、作成したreleaseObjectオブジェクトでrangeを呼び出します。 releaseObject(xlWorkSheet);を呼び出す前にこれを行ってください。これらすべての参照を覚えておくことが、COM相互運用プログラミングを非常に楽しいものにします。

破損したExcelファイルの問題を修正するには、WorkBook.SaveAsメソッド呼び出しを更新して、2番目のパラメーター(Excel.XlFileFormat.xlXMLSpreadsheet)をType.Missingに置き換えます。 SaveAsメソッドは、デフォルトでこれを正しく処理します。

質問に投稿したコードは、発生している問題のデバッグに役立つように簡略化されていると思います。 pstが示すtry..finallyブロックを使用する必要があります。

4
Jay Riggs