Question

I am writing an excel class and i want to release this unmanaged object automatically.
I’m using IDisposable pattern and write Dispose methods.
Example ;

class MSExcel : IDisposable
{
    ApplicationClass excel;
    bool disposed;

    public MSExcel()
    {
         disposed = false;
         excel = new ApplicationClass();
    }

    public void Dispose(bool disposing)
    {
        if (!this.disposed)
        {

            if (disposing)
            {

            }

            excel.Quit();                
            disposed = true;

        }
    }

    public void Dispose()
    {
        Dispose(true);
        GC.SuppressFinalize(this);
    }

    ~MSExcel()
    {
       Dispose(false);
    }
}

But i have classical error on exc.Quit().
“COM object that has been separated from its underlying RCW”.
Have any my mistake in code?

Answer 1

As explained in my answer to your other related question here, you should not be taking actions on reference types from within your finalizer. You enforce this by making use of the bool disposing parameter of your Disposed(bool) method, as you have done. You pass in true, when Disposed(bool) is called explicitly from the void Dispose() method, and pass in false when called from your finalizer, which you have also done.

However, you also need to protect your call to excel.Quit() so that it is not called when Disposed(bool) is called via the finalizer. That is, you should only call excel.Quit() when the bool disposing argument is true.

Therefore, the code for your Disposed(bool) method should look as follows:

public void Dispose(bool disposing)
{
    if (!this.disposed)
    {

        if (disposing)
        {
            excel.Quit();   
        }

        disposed = true;
    }
}

Once done, you could make use of your “MSExcel” class as follows:

using (MSExcel msExcel = new MSExcel)
{
   // The code calling your 'MSExcel' object goes here.
}

By doing it this way, when your code gets to the closing bracket, “}”, of your using statement block, the Dispose method on your ‘MSExcel’ class will be called automatically, ensuring that excel.Quit() is called deterministically, and not from a finalizer.

Hope this helps…

Mike