C# と VB.NET の質問掲示板

■No93100 (じょこびっち さん) に返信

コードを張りつけるときは図表モードにすると崩れなくていいですよ。

> 現在作っている物はフォームからエクセルを立ち上げる事ができるようになっています。
> フォームからエクセルを立ち上げてフォームを閉じると問題はないのですが
> フォームからエクセルを立ち上げずに閉じるとworkbookがnullになっていたようです。
> これを回避するにはどうすればいいでしょうか？
> 
> private void Form1_FormClosed(object sender, FormClosedEventArgs e)
> {
>   if (excelApp != null)
>   {
>     if (workBook != null)
>     {
>       workBook.Saved = true;
>     }
>     //excelApp.Quit();
>   }
>   System.Runtime.InteropServices.Marshal.ReleaseComObject(excelApp);
> }

見にくかったので、整形した形で引用させてもらいました。

上を見るとわかりますが、excelApp が null ではない場合はとみていますが、
肝心の ReleaseComObject を呼び出すところは null でも呼び出していますよね？

こんな感じにしてみるといいのでは？と思います。

private void Form1_FormClosed(object sender, FormClosedEventArgs e)
{
  if (excelApp != null)
  {
    if (workBook != null)
    {
      workBook.Saved = true;
    }
    //excelApp.Quit();
    System.Runtime.InteropServices.Marshal.ReleaseComObject(excelApp);
    excelApp = null;
  }
}

■No93103 (とっちゃん さん) に返信
> 
> コードを張りつけるときは図表モードにすると崩れなくていいですよ。

ご指摘ありがとうございます。次回から使ってみようと思います。

> こんな感じにしてみるといいのでは？と思います。
> 
> private void Form1_FormClosed(object sender, FormClosedEventArgs e)
> {
>   if (excelApp != null)
>   {
>     if (workBook != null)
>     {
>       workBook.Saved = true;
>     }
>     //excelApp.Quit();
>     System.Runtime.InteropServices.Marshal.ReleaseComObject(excelApp);
>     excelApp = null;
>   }
> }

出来ました！確かにexcelApp != nullの外でCOMの解放をしてました。

> ReleaseComObject をしているのが一つだけなので、他は書いてませんが
> workBook も COM オブジェクトだと思うので、それらの解放も必須です。
> それを怠ると、裏でエクセルが終了せずに生き残ってしまいますので、注意してください。

なるほど、勉強になります。これからコードを見直そうと思います。

とっちゃんさん、KOZさんありがとうございました。