OracleConnectionのDisposeについて

Nyaさんの書き込み (2009-04-15 00:25) より:
いつも参考にさせていただいております。

Visual Studio 2005 C# ASP.NET
Oracle10g

現在、イントラ内のWebアプリ開発および保守をおこなっており、DB接続にはSystem.Data.OracleClientを使用しています。
実際には、プロパティとしてSQL、Parameterを渡しDataSetを返すメソッドをもつ
ラッパークラスを作成してDBとのやりとりをおこなっています。
（メソッド内でOracleConnection,OracleCommandをusingを使用して
インスタンス生成しています）

先日までは何の疑問もなくこのラッパークラスを使用していたのですが、
このほど別会社からの納品物を修正することになり、そこに定義されていた
DBアクセス用のラッパークラスを確認したところ下記のような作りになっていました。

OracleConnection、OracleCommandはメンバ変数として定義
コンストラクタ　　　　 ：　OracleConnection、OracleCommandのインスタンス生成
DataSetを返すメソッド　：　生成したインスタンスを使用して処理
自前のCloseメソッド　　：　OracleConnectionのClose,Dispose
　　　　　　　　　　　　　 OracleCommandのDispose

この内容だと自前のCloseメソッドの呼び忘れや処理中の例外でConnectionのClose、
Disposeがおこなわれないのではないかと思い納品元に問い合わせたところ、

?連続してSQLを実行するときにいちいち接続しなくていい
?規約として必ずCloseを書けばいい

といったような回答が来ましたが、いまいち釈然としません。
せめてこのクラスにiDisposableが定義されていれば使用する側で
usingを使用してインスタンス生成すればいいような気もするのですが???。

一般的にDBアクセス用のラッパークラスを作成するとしたら、Connection等の
Disposeはどのようにおこなうものなのでしょうか？漠然とした質問で申し訳
ないのですが、どなたか「自分はこうしているよ」等のアドバイスを頂けたら
幸いです。

基本的に、ファイナライザーがオーバーロードされて、ファイナライザーで破棄されていれば大丈夫です。

連続して云々について、もう少し突っ込んで聞いてみて下さい。再接続の時間が省けるという理由であれば、「接続プールが使われるので時間はかからない」と指摘してみて下さい（事前に調べておいて下さい）。また、「閉じずに再度開こうとしたら、どうなりますか？」とも。こういったテストは、開発元でされているべきですが、問い合わせ前に軽く試してみて、望ましい結果になるか、確認しておくのもいいでしょう。

Nyaさんの書き込み (2009-04-15 22:59) より:

引用:
基本的に、ファイナライザーがオーバーロードされて、ファイナライザーで破棄されていれば大丈夫です。

とはラッパークラスに~（クラス名）を実装し、最終的にそこで必ず破棄すれば大丈夫、ということでしょうか？

ちなみに、「閉じずに再度開こうとしたら」についてはクラス内では何も意識され
ていませんでした。こちらで試したところ例外が発生していました。
テスト段階で潰せる話ではありますが、これは"ナシ"だと思いますので、含めて
指摘してみたいと思います。

どういう意味でナシでしょう？
普通にありで、むしろ適切な動作ではないかと思いますが…

あとはまあ、トランザクションの管理がどうなっているか、などが気にするべき点かなと思います。
接続を開いたままにするのはどちらかというとトランザクションの管理で意味があることの方が多いと思いますので。