Безопасное удаление объектов взаимодействия Excel в C #? - PullRequest
14 голосов
/ 01 апреля 2012

Я работаю над приложением winforms c # visual studio 2008.приложение обращается к файлам 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); 

на данный момент я не думаю, что можно закрыть Excel от Visual Studio 2008. Возможно, это ошибка или что-то в этом роде, но я попробовал 20 лучших сайтов на этом и получаютот же результат: Excel по какой-то причине открывает два экземпляра, а когда я выполняю сборку мусора и т. д. (или нет), он закрывает только ОДИН экземпляр.

, когда я пытаюсь открыть файл, он говорит, что естьошибка или она повреждена.

когда я иду кпопросите менеджера и убейте процесс Excel, файл откроется без проблем.]

Есть ли способ закрыть Excel с Visual Studio 2008?если да, можете ли вы дать мне руководство или решение для этого

Ответы [ 2 ]

20 голосов
/ 01 апреля 2012

Сначала я представлю измененный 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, чтобы упростить эту задачу.

4 голосов
/ 02 апреля 2012

Убедитесь, что в вашем коде есть две проблемы:

  • Что при закрытии программы Excel остается как работающий процесс
  • Что при открытии файла Excel ваша программа создает ошибка в Excel, говорящая о том, что файл поврежден или что-то подобное

Я скопировал обработчик щелчков button1 и метод pst releaseObject в отредактированном вопросе в чистое приложение Winform VS2008, C # 3.5 и внес несколько небольших изменений, чтобы устранить обе проблемы, перечисленные выше.

Чтобы Excel не выгружался из памяти, вызовите releaseObject для созданного вами объекта range. Сделайте это до вызова releaseObject(xlWorkSheet); Помните, что все эти ссылки делают программирование COM Interop таким интересным.

Чтобы исправить проблему с поврежденным файлом Excel, обновите вызов метода WorkBook.SaveAs, чтобы заменить второй параметр (Excel.XlFileFormat.xlXMLSpreadsheet) на Type.Missing. Метод SaveAs по умолчанию будет обрабатывать это правильно.

Я уверен, что код, который вы разместили в своем вопросе, упрощен, чтобы помочь устранить проблемы, с которыми вы столкнулись. Вы должны использовать try..finally блок pst демонстрирует.

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...