Interoperabilità C #: il processo Excel non esce dopo l'aggiunta di un nuovo foglio di lavoro al file esistente [duplicato]

Question

  

Possibile duplicato:
   Come pulire correttamente gli oggetti di interoperabilità di Excel in C #

Ho letto molti degli altri thread qui sulla gestione dei riferimenti COM durante l'utilizzo dell'interoperabilità .Net-Excel per assicurarmi che il processo Excel esca correttamente all'uscita, e finora le tecniche hanno funzionato molto bene, ma di recente si è verificato un problema durante l'aggiunta di nuovi fogli di lavoro a un file di cartella di lavoro esistente.

Il codice seguente lascia un processo zombie Excel.

Se aggiungo un foglio di lavoro a un file di cartella di lavoro appena creato, si chiude bene. Se eseguo il codice escludendo la riga .Add(), si esce bene. (Il file esistente da cui sto leggendo è un file vuoto creato dal codice commentato)

Qualche idea?

//using Excel = Microsoft.Office.Interop.Excel;
//using System.Runtime.InteropServices;
public static void AddTest()
{
  string filename = @"C:\addtest.xls";
  object m = Type.Missing;
  Excel.Application excelapp = new Excel.Application();
  if (excelapp == null) throw new Exception("Can't start Excel");
  Excel.Workbooks wbs = excelapp.Workbooks;

  //if I create a new file and then add a worksheet,
  //it will exit normally (i.e. if you uncomment the next two lines
  //and comment out the .Open() line below):
  //Excel.Workbook wb = wbs.Add(Excel.XlWBATemplate.xlWBATWorksheet);
  //wb.SaveAs(filename, m, m, m, m, m, 
  //          Excel.XlSaveAsAccessMode.xlExclusive,
  //          m, m, m, m, m);

  //but if I open an existing file and add a worksheet,
  //it won't exit (leaves zombie excel processes)
  Excel.Workbook wb = wbs.Open(filename,
                               m, m, m, m, m, m,
                               Excel.XlPlatform.xlWindows,
                               m, m, m, m, m, m, m);

  Excel.Sheets sheets = wb.Worksheets;

  //This is the offending line:
  Excel.Worksheet wsnew = sheets.Add(m, m, m, m) as Excel.Worksheet; 

  //N.B. it doesn't help if I try specifying the parameters in Add() above

  wb.Save();
  wb.Close(m, m, m);

  //overkill to do GC so many times, but shows that doesn't fix it
  GC();
  //cleanup COM references
  //changing these all to FinalReleaseComObject doesn't help either
  while (Marshal.ReleaseComObject(wsnew) > 0) { } 
  wsnew = null;
  while (Marshal.ReleaseComObject(sheets) > 0) { }
  sheets = null;
  while (Marshal.ReleaseComObject(wb) > 0) { }
  wb = null;
  while (Marshal.ReleaseComObject(wbs) > 0) { }
  wbs = null;
  GC();
  excelapp.Quit();
  while (Marshal.ReleaseComObject(excelapp) > 0) { }
  excelapp = null;
  GC();
}

public static void GC()
{
  System.GC.Collect();
  System.GC.WaitForPendingFinalizers();
  System.GC.Collect();
  System.GC.WaitForPendingFinalizers();
}

Answer 1

Non ho il codice a portata di mano, ma ho riscontrato un problema simile. Se ricordo bene, ho finito per recuperare l'ID del processo dell'istanza di Excel e ucciderlo (dopo un periodo di attesa adeguato e quando l'altro metodo non è riuscito).

Penso di aver usato:

GetWindowThreadProcessId (tramite P / Invoke) sulla proprietà hwnd dell'oggetto excel per ottenere l'ID processo, quindi utilizzato Process.GetProcessById per ottenere un oggetto processo. Una volta fatto questo, avrei chiamato Kill sul processo.

EDIT: Devo ammetterlo, questa non è la soluzione ideale, ma se non riesci a trovare l'interfaccia canaglia che non viene rilasciata, questo lo risolverà in un vero guscio d'uovo / mazza. ;)

EDIT2: non è necessario chiamare immediatamente Close sull'oggetto processo ... È possibile provare a chiamare <=> prima di ricorrere a <=>.

Answer 2

Ho fatto una cosa simile. Creo un file Excel o ne apro uno esistente. Cancella tutti i fogli e aggiungo i miei. ecco il codice che uso per garantire la chiusura di tutti i riferimenti:

            workbook.Close(true, null, null);
            excelApp.Quit();

            if (newSheet != null)
            {
                System.Runtime.InteropServices.Marshal.ReleaseComObject(newSheet);
            }
            if (rangeSelection != null)
            {
            System.Runtime.InteropServices.Marshal.ReleaseComObject(rangeSelection);
            }
            if (sheets != null)
            {
                System.Runtime.InteropServices.Marshal.ReleaseComObject(sheets);
            }
            if (workbook != null)
            {
                System.Runtime.InteropServices.Marshal.ReleaseComObject(workbook);
            }
            if (excelApp != null)
            {
                System.Runtime.InteropServices.Marshal.ReleaseComObject(excelApp);
            }

            newSheet = null;
            rangeSelection = null;
            sheets = null;
            workbook = null;
            excelApp = null;

            GC.Collect();

Ho provato questo con molte opzioni diverse e non ho ancora fallito con me.

Answer 3

ecco il mio codice completo per eliminare Excel creato con la libreria di interoperabilità di Office12 .Net: Godere, -Alan.

using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Runtime.InteropServices;
using System.Diagnostics;
using Microsoft.Office.Interop.Excel;

class Program
{

    /// <summary> 
    /// Win32 API import for getting the process Id. 
    /// The out param is the param we are after. I have no idea what the return value is. 
    /// </summary> 
    [DllImport("user32.dll")]
    private static extern IntPtr GetWindowThreadProcessId(IntPtr hWnd, out IntPtr ProcessId); 

    static void Main(string[] args)
    {
        var app = new Application();
        IntPtr hwnd = new IntPtr(app.Hwnd);
        IntPtr processId;
        IntPtr foo = GetWindowThreadProcessId(hwnd, out processId);
        Process proc = Process.GetProcessById(processId.ToInt32());
        proc.Kill(); // set breakpoint here and watch the Windows Task Manager kill this exact EXCEL.EXE
        app.Quit(); // should give you a "Sorry, I can't find this Excel session since you killed it" Exception.
    }
}

Answer 4

Questo funziona molto bene per me, senza eccezioni.

Public Class ExcelHlpr

    Declare Function EndTask Lib "user32.dll" (ByVal hWnd As IntPtr, ByVal ShutDown As Boolean, ByVal Force As Boolean) As Integer

    Dim cXlApp As Microsoft.Office.Interop.Excel.Application

    Public Function GetExcel() As Microsoft.Office.Interop.Excel.Application
        cXlApp = New Microsoft.Office.Interop.Excel.Application
        Return cXlApp
    End Function

    Public Function EndExcel() As Integer
        Dim xlHwnd As New IntPtr(cXlApp.Hwnd)
        Return EndTask(xlHwnd, False, True)
    End Function

End Class

Answer 5

Non molto costruttivo, lo so, ma ho testato il codice esattamente come mostrato sopra e il mio processo Excel termina come previsto, il mio C: \ addtest.xls è seduto con 8 nuovi fogli e nessun processo Excel è in esecuzione.
versione interop essere la causa mi chiedo? Ho provato con 11 & Amp; 12.

Answer 6

Sto usando VB.NET 3.5 SP1 e il seguente codice STILL lascia EXCEL.EXE aperto:

        xlWorkbook.Close(SaveChanges:=False)
        xlApplication.Quit()

        System.Runtime.InteropServices.Marshal.ReleaseComObject(xlRange)
        System.Runtime.InteropServices.Marshal.ReleaseComObject(xlWorksheet)
        System.Runtime.InteropServices.Marshal.ReleaseComObject(xlSheets)
        System.Runtime.InteropServices.Marshal.ReleaseComObject(xlWorkbook)
        System.Runtime.InteropServices.Marshal.ReleaseComObject(xlApplication)

        xlRange = Nothing
        xlWorksheet = Nothing
        xlSheets = Nothing
        xlWorkbook = Nothing
        xlApplication = Nothing

        GC.GetTotalMemory(False)
        GC.Collect()
        GC.WaitForPendingFinalizers()

        GC.Collect()
        GC.WaitForPendingFinalizers()
        GC.Collect()
        GC.GetTotalMemory(True)

Answer 7

Andrew, ecco il codice che ho trovato che funziona. Ho pensato di postarlo qui per gli altri che si imbattono:

namespace WindowHandler
{
using System;
using System.Text;
using System.Collections;
using System.Runtime.InteropServices;

/// <summary>
/// Window class for handling window stuff.
/// This is really a hack and taken from Code Project and mutilated to this small thing.
/// </summary>
public class Window
{
    /// <summary>
    /// Win32 API import for getting the process Id.
    /// The out param is the param we are after. I have no idea what the return value is.
    /// </summary>
    [DllImport("user32.dll")]
    private static extern IntPtr GetWindowThreadProcessId(IntPtr hWnd, out IntPtr ProcessId);

    /// <summary>
    /// Gets a Window's process Id.
    /// </summary>
    /// <param name="hWnd">Handle Id.</param>
    /// <returns>ID of the process.</returns>
    public static IntPtr GetWindowThreadProcessId(IntPtr hWnd)
    {
        IntPtr processId;
        IntPtr returnResult = GetWindowThreadProcessId(hWnd, out processId);

        return processId;
    }
}
}