Re[1]: Excel Com オブジェクトの増殖 - DOBON.NETプログラミング道掲示板

DOBON.NETプログラミング道掲示板

■35244 / 1階層)

　Excel Com オブジェクトの増殖

□投稿者/ 魔界の仮面弁士大御所(1486回)-(2022/11/27(Sun) 08:37:41)

■No35241に返信(たこさんの記事)
> Excelの処理部分はクラスにまとめ処理していますが、タスクマネージャーで見ると閉じたはずのExcelが残っています。
> いろいろやってみましたが解決できず、お知恵をお借りしたいと投稿致します。
省略されている箇所に問題があるような気もしますね。

たとえば、使用例のところで
> 　　　　ExEx1 = New ExcelEx()
> 　　　　ExEx1.WorkBook = path & "Test.xlsx"
> 　　　　ExEx1.xlsApplication.WindowState = Excel.XlWindowState.xlMaximized
> 　　　　ExEx1.Sheets(1)
> 　　　　ExEx1.WorkBook = "Close"
と書かれていますが、Sheets メソッドの実装が不明瞭ですし。

> Friend xlsApplication As Excel.Application = Nothing
これらの COM オブジェクトを非 Private として公開されていますが、
Friend で公開するにしても、ReadOnly な Property にしておくべきかと。

ただし ReadOnly だとしても、ExcelEx 外から Open や Close 等、あるいは Cells 等への
アクセスは無制限に可能となってしまいますので、COM オブジェクトを直接公開すると、
ExcelEx だけでは、参照と解放の管理をまかないきれなくなることがあります。

解放まですべてクラス側で管理するなら、委譲している Excel のオブジェクトを
直接公開するのではなく、必要な操作のみを再公開するようにしたうえで、
IDisposable パターンも追加実装しておくのが安全です。実装量は爆発的に増えますけれどね。
https://learn.microsoft.com/ja-jp/dotnet/standard/garbage-collection/implementing-dispose
https://ufcpp.net/study/csharp/rm_disposable.html

逆に、そこまでの実装コストをかけられないというのであれば、
ヘルパーライブラリとしての実装に留めるという手もあろうかと。

> Private _WorkBookIsNew As Boolean = Nothing
= False と書いた方が素直ではありませんか?

> Public Property WorkBook As String
今回の COM 解放の問題からは外れますが、こういったものはプロパティではなく、
String 引数を持ったメソッドとして実装する方が望ましいです。

クラス設計的には、プロパティは "状態" を表すものであり、
"動作" を行うことを目的としたプロパティを実装すべきではありません。
"動作" を目的とするのはメソッドの役目ですよね。

16bit 時代の Visual Basic ではプロパティで動作させていたりもしますがね…。
(例：32bit 版では非推奨になった CommonDialog.Action プロパティなど)

> Private Sub WorkBook_Open(Optional ByVal strFileName As String = Nothing)
> 　　　　'Excel アプリケーション起動
> 　　　　xlsApplication = New Excel.Application
WorkBook_Open が 2 回呼ばれると、xlsApplication 変数が
以前に編集していたインスタンスを破棄することなく、
新しい Application インスタンスを生成することになりそうですが、
何らかの再入防止策が組み込まれていますか?

インスタンス管理を考えると、Application の起動は
ExcelEx のコンストラクタで行った方が良いと思います。

> 　　　　MRComObject(xlsRange, True)
ExcelEx の他の部分の処理がどうなっているのかにもよりますが、
Range オブジェクトは複数使われることも多いので、
自分ならコレクション管理するかな…?

> 　　　　If Not xlsWorkbook Is Nothing Then xlsWorkbook.Close()
省略された部分のコードによって、この時点で Nothing になっていることがあるということですか?

提示された範囲のコードを見るだけでは、ここで Nothing になっているケースとは、外部から
　ExEx1.xlsWorkbook = Nothing
とされた場合か、あるいは WorkBook_Open がまだ呼ばれていない時ぐらいしかなさそうですが…。
(または、存在しないファイル名を指定するなどして、WorkBook_Open が失敗した場合とか)

あと、xlsWorkbook.Saved が False の時に Close を呼び出すと、
DisplayAlerts が True に戻されていた場合、保存確認のダイアログが表示されることになります。
そして保存確認でキャンセルされた場合、実際には Close されません。

もしも終了時の保存が常に不要な場合は、Close 前に Saved プロパティを True にしておくか、
もしくは Close メソッドの SaveChanges 引数に False を指定するようにします。

> Excelを閉じた時にプロセスは終了・削除されると思っているのですが、認識が違うのでしょうか…
.NET 側を解放したとしても、ReleaseComObject されねば COM 側は生き残りますが、
その逆に COM 参照が残っている状態で Excel を閉じた場合、終了せずに非表示だけで留まることがあります。

逆に、 COM 側だけが解放されたとしたら、解放済みのオブジェクトを操作するタイミングで、
HRESULT: 0x80010108 (RPC_E_DISCONNECTED) に相当する例外
たとえば「起動されたオブジェクトはクライアントから切断されました。」などが
発生する可能性があります。

違反を報告

上記関連ツリー
前の記事(元になった記事)	次の記事(この記事の返信)
←Excel Com オブジェクトの増殖 /たこ	→Re[2]: Excel Com オブジェクトの増殖 /たこ
Excel Com オブジェクトの増殖 / たこ (22/11/27(Sun) 04:14) #35241
`└` Excel Com オブジェクトの増殖 / 魔界の仮面弁士 (22/11/27(Sun) 08:37) #35244 ←Now
`└` Re[2]: Excel Com オブジェクトの増殖 / たこ (22/11/27(Sun) 10:59) #35245
`├` Re[3]: Excel Com オブジェクトの増殖 / 魔界の仮面弁士 (22/11/27(Sun) 11:36) #35247
`│└` Re[4]: Excel Com オブジェクトの増殖 / 魔界の仮面弁士 (22/11/27(Sun) 11:43) #35248
`└` Re[3]: Excel Com オブジェクトの増殖 / 魔界の仮面弁士 (22/11/27(Sun) 12:56) #35249
`└` Re[4]: Excel Com オブジェクトの増殖 / たこ (22/11/27(Sun) 16:46) #35250
`├` Re[5]: Excel Com オブジェクトの増殖 / たこ (22/11/27(Sun) 19:11) #35251
`│├` Re[6]: Excel Com オブジェクトの増殖 / 魔界の仮面弁士 (22/11/27(Sun) 22:56) #35253
`│└` Re[6]: Excel Com オブジェクトの増殖 / 魔界の仮面弁士 (22/11/28(Mon) 06:16) #35255
`└` Re[5]: Excel Com オブジェクトの増殖 / 魔界の仮面弁士 (22/11/27(Sun) 22:32) #35252
`└` Re[6]: Excel Com オブジェクトの増殖 / radian (22/11/28(Mon) 11:57) #35256
`└` Re[7]: Excel Com オブジェクトの増殖 / たこ (22/11/28(Mon) 22:38) #35257
`└` Re[8]: Excel Com オブジェクトの増殖 / たこ (22/11/28(Mon) 23:06) #35258
上記ツリーを一括表示 / 上記ツリーをトピック表示
上記の記事へ返信

入力内容にタグは利用できません。
引用部分（行頭に「>」や「■」が付いている部分）が全体の 80% 以上のコメントは投稿できません。

半角カナは使用しないでください。文字化けの原因になります。
削除キーを覚えておくと、自分の記事の編集・削除ができます。
記事中に No*** のように書くとその記事にリンクされます(No は半角英字/*** は半角数字)。
- 使用例)
- No123 → 記事No123の記事リンクになります(指定表示)。
- No123,130,134 → 記事No123/130/134 の記事リンクになります(複数表示)。
- No123-130 → 記事No123～130 の記事リンクになります(連続表示)。

投稿前に必ずお読みください

初めて投稿される方は、必ず「書き込みのルールについて（12/4/2更新）」をお読みください。読むのが面倒、ルールを守りたくないという方は、「お気楽掲示板」へ投稿してください。
質問を投稿する場合は、「質問をする際のアドバイス」を参考にしてください。
投稿は、明記のない限り、パブリックドメインになります。
マルチポストされた投稿を報告する時は匿名で投稿し、マルチポストされている場所を併記してください。以下のリンクをクリックしていただくと、自動で下のフォームが埋まりますので、その後「送信」ボタンを押してご投稿ください。
- マルチポストの報告をする
- マルチポストの報告をするが、マルチポスト先の投稿が嫌がらせである（元の投稿者とは別人が行なっている）可能性が高い（投稿日時に開きがあり、こちらの投稿の方が早い場合は、その可能性が高いです）

Name	/
E-Mail	/
Title	/
URL	/
Comment/ 通常モード->　図表モード-> (適当に改行して下さい/半角10000文字以内) ■No35244に返信(魔界の仮面弁士さんの記事) > ■No35241に返信(たこさんの記事) >>Excelの処理部分はクラスにまとめ処理していますが、タスクマネージャーで見ると閉じたはずのExcelが残っています。 >>いろいろやってみましたが解決できず、お知恵をお借りしたいと投稿致します。 > 省略されている箇所に問題があるような気もしますね。 > > たとえば、使用例のところで >>　　　　ExEx1 = New ExcelEx() >>　　　　ExEx1.WorkBook = path & "Test.xlsx" >>　　　　ExEx1.xlsApplication.WindowState = Excel.XlWindowState.xlMaximized >>　　　　ExEx1.Sheets(1) >>　　　　ExEx1.WorkBook = "Close" > と書かれていますが、Sheets メソッドの実装が不明瞭ですし。 > > > >>Friend xlsApplication As Excel.Application = Nothing > これらの COM オブジェクトを非 Private として公開されていますが、 > Friend で公開するにしても、ReadOnly な Property にしておくべきかと。 > > ただし ReadOnly だとしても、ExcelEx 外から Open や Close 等、あるいは Cells 等への > アクセスは無制限に可能となってしまいますので、COM オブジェクトを直接公開すると、 > ExcelEx だけでは、参照と解放の管理をまかないきれなくなることがあります。 > > > 解放まですべてクラス側で管理するなら、委譲している Excel のオブジェクトを > 直接公開するのではなく、必要な操作のみを再公開するようにしたうえで、 > IDisposable パターンも追加実装しておくのが安全です。実装量は爆発的に増えますけれどね。 > https://learn.microsoft.com/ja-jp/dotnet/standard/garbage-collection/implementing-dispose > https://ufcpp.net/study/csharp/rm_disposable.html > > 逆に、そこまでの実装コストをかけられないというのであれば、 > ヘルパーライブラリとしての実装に留めるという手もあろうかと。 > > >>Private _WorkBookIsNew As Boolean = Nothing > = False と書いた方が素直ではありませんか? > > >>Public Property WorkBook As String > 今回の COM 解放の問題からは外れますが、こういったものはプロパティではなく、 > String 引数を持ったメソッドとして実装する方が望ましいです。 > > クラス設計的には、プロパティは "状態" を表すものであり、 > "動作" を行うことを目的としたプロパティを実装すべきではありません。 > "動作" を目的とするのはメソッドの役目ですよね。 > > 16bit 時代の Visual Basic ではプロパティで動作させていたりもしますがね…。 > (例：32bit 版では非推奨になった CommonDialog.Action プロパティなど) > > >>Private Sub WorkBook_Open(Optional ByVal strFileName As String = Nothing) >>　　　　'Excel アプリケーション起動 >>　　　　xlsApplication = New Excel.Application > WorkBook_Open が 2 回呼ばれると、xlsApplication 変数が > 以前に編集していたインスタンスを破棄することなく、 > 新しい Application インスタンスを生成することになりそうですが、 > 何らかの再入防止策が組み込まれていますか? > > インスタンス管理を考えると、Application の起動は > ExcelEx のコンストラクタで行った方が良いと思います。 > > >>　　　　MRComObject(xlsRange, True) > ExcelEx の他の部分の処理がどうなっているのかにもよりますが、 > Range オブジェクトは複数使われることも多いので、 > 自分ならコレクション管理するかな…? > >>　　　　If Not xlsWorkbook Is Nothing Then xlsWorkbook.Close() > 省略された部分のコードによって、この時点で Nothing になっていることがあるということですか? > > 提示された範囲のコードを見るだけでは、ここで Nothing になっているケースとは、外部から > 　ExEx1.xlsWorkbook = Nothing > とされた場合か、あるいは WorkBook_Open がまだ呼ばれていない時ぐらいしかなさそうですが…。 > (または、存在しないファイル名を指定するなどして、WorkBook_Open が失敗した場合とか) > > あと、xlsWorkbook.Saved が False の時に Close を呼び出すと、 > DisplayAlerts が True に戻されていた場合、保存確認のダイアログが表示されることになります。 > そして保存確認でキャンセルされた場合、実際には Close されません。 > > もしも終了時の保存が常に不要な場合は、Close 前に Saved プロパティを True にしておくか、 > もしくは Close メソッドの SaveChanges 引数に False を指定するようにします。 > > >>Excelを閉じた時にプロセスは終了・削除されると思っているのですが、認識が違うのでしょうか… > .NET 側を解放したとしても、ReleaseComObject されねば COM 側は生き残りますが、 > その逆に COM 参照が残っている状態で Excel を閉じた場合、終了せずに非表示だけで留まることがあります。 > > 逆に、 COM 側だけが解放されたとしたら、解放済みのオブジェクトを操作するタイミングで、 > HRESULT: 0x80010108 (RPC_E_DISCONNECTED) に相当する例外 > たとえば「起動されたオブジェクトはクライアントから切断されました。」などが > 発生する可能性があります。
File	/ アップ可能拡張子=> /.gif/.jpg/.png/.zip 1) 太字の拡張子は画像として認識されます。 2) 画像は初期状態で縮小サイズ250×250ピクセル以下で表示されます。 3) 同名ファイルがある、またはファイル名が不適切な場合、　　ファイル名が自動変更されます。 4) アップ可能ファイルサイズは1回20KB(1KB=1024Bytes)までです。 5) ファイルアップ時はプレビューは利用できません。 6) スレッド内の合計ファイルサイズ:[0/300KB] 残り:[300KB]
Icon	/ (画像を選択/サンプル一覧)
削除キー	/ (半角8文字以内)
解決済み! BOX/ 解決したらチェックしてください!（スレッドを閉じる目的ではチェックしないでください!）
クリエイティブ・コモンズ・ライセンスを Comment の末尾に挿入するには、以下のリンクをクリックしてください。（Javascript で挿入しているだけです。）表示（CC BY） \| 表示-改変禁止（CC BY-ND） \| 表示-継承（CC BY-SA） \| 表示-非営利（CC BY-NC） \| 表示-非営利-改変禁止（CC BY-NC-ND） \| 表示-非営利-継承（CC BY-NC-SA）
プレビュー/　クッキーに保存しない/