まとめて返信させて下さい。
http://gushwell.ldblog.jp/archives/2009-11.html
上記リンク先では、sealedクラスによって、私が言ってきたような形で実装していますので、
今回のような質問に限って言えば、ファイナライザとsuppressfinalizeは両方とも書かないようなコーディングにしようと思います。
一応月曜日にコードを載せると書きましたので、一部載せますが、一応解決としたいと思います。
ありがとうございました。
渋木宏明さん
回答ありがとうございます。
＞心配なら、省略することなんか考えないで、Disposable パターンに忠実に実装すればいいんじゃないでしょうか。
おっしゃるとおりです。最悪の場合そうしようと思ったのですが、やはり一般的なコーディングを身につけることが大事だと思いましたで、しぶとく質問させていただきました（笑）
IDisposableを実装する理由によっては、必要な場合があるのですね。了解です。
私自身のレベルがあがったらその辺の質問が出ることがあるかもしれません。
これからもなにとぞよろしくお願いします。
-toyo-さん
回答ありがとうございます。
わたしもちょこっとだけDB接続について調べてみましたが、おっしゃるとおり、長くつないでおくよりは細かい接続を何度もやったほうがいいみたいですね。（たぶん）
その辺はちょっと見直そうかなと思います。
K.Takaokaさん
いつも回答ありがとうございます。
sealedしない場合には、IDisposableパターンにのっとり、ファイナライザもsuppressfinalizeも両方とも記述するようにしようと思います。
Assertという新たな手法が登場しましたが、後日調べたいと思います。
とりあえず今回の問題は解決としたいと思います。
最後に自分のコードを載せます。
using System;
using System.Collections.Generic;
using System.Text;
using System.Data;
using System.Data.SqlClient;
namespace AAA
{
public class AAASample:IDisposable
{
private bool _disposed;
public DataTable _DTable;
private string _ConStr;
private string _SelStr;
private string _InsStr;
private SqlConnection _SqlConn;
private SqlDataAdapter _SqlDa;
private SqlCommandBuilder _CmdBldr;
public AAASample
{
//初期化
}
/**********************/
/*****いろいろなメソッド****/
/**********************/
#region IDisposable メンバ
//コメントアウト
/*~AAASample()
{
Dispose(false);
}*/
//GC.SuppressFinalize()はコメントアウト
public void Dispose()
{
Dispose(true);
// Take yourself off the Finalization queue
// to prevent finalization code for this object
// from executing a second time.
//GC.SuppressFinalize(this);
}
//引数無しDispose()に書けばよいのだが、以下も作成
private void Dispose(bool disposing)
{
if (!this._disposed)
{
if (disposing)
{
if (_DTable != null)
{
_DTable.Dispose();
_DTable = null;
}
if (_CmdBldr != null)
{
_CmdBldr.Dispose();
_CmdBldr = null;
}
if (_SqlDa != null)
{
_SqlDa.Dispose();
_SqlDa = null;
}
if (_SqlConn != null)
{
_SqlConn.Dispose();
_SqlConn = null;
}
}
}
_disposed = true;
}
#endregion
}
}
いろいろ突っ込んでいただいてもいいですし、スルーしていただいても構いません。
とりあえず期限が近いためこれで行きたいと思います。
