Jim さん、別のクラス内にオープン接続メソッドとクローズ接続メソッドを持つというあなたの習慣は非常に古く、最近の .NET 開発は次のようなパターンに従っています:
using (SqlConnection conn = new SqlConnection("connection string here")) using (SqlCommand cmd = new SqlCommand("sql query", conn)) { // execute it blah blah }
プレ>ここを参照してください:SqlConnection と SqlCommand c# を閉じる または SO で何百もの質問と回答をすべて検索し、この例のような using を使用するか、同じメソッド内で try/finally を使用して接続をすぐに閉じます。1 つのメソッドを開いて on メソッドを閉じる必要はありません。その間に何かが起こった場合、エラーが発生しやすくなります。