SqlConnection、SqlDataAdapter、およびSqlCommandオブジェクトを再利用する必要がありますか？

Question

次のコードのようなレイアウトで記述されたDALオブジェクトを使用しています。私はセットアップを示すために多くのコードを単純化しました。

public class UserDatabase : IDisposable 
{ 
    private SqlDataAdapter UserDbAdapter; 
    private SqlCommand UserSelectCommand; 
    private SqlCommand UserInsertCommand; 
    private SqlCommand UserUpdateCommand; 
    private SqlCommand UserDeleteCommand; 

    private System.Data.SqlClient.SqlConnection SQLConnection; 

    public UserDatabase() 
    { 
     this.SQLConnection = new System.Data.SqlClient.SqlConnection(ConnectionString); 
     this.UserDbAdapter= new SqlDataAdapter(); 
     this.UserDbAdapter.DeleteCommand = this.UserDeleteCommand; 
     this.UserDbAdapter.InsertCommand = this.UserInsertCommand; 
     this.UserDbAdapter.SelectCommand = this.UserSelectCommand; 
     this.UserDbAdapter.UpdateCommand = this.UserUpdateCommand; 
    } 

    private bool FillUsers(DataSet UserDataSet, out int numberOfRecords) 
    { 
     bool success = true; 

     numberOfRecords = 0; 
     string errorMsg = null; 

     this.UserDbAdapter.SelectCommand = this.GetUsersSelectCommand(); 

     numberOfRecords = UserDbAdapter.Fill(UserDataSet, UsersTableName); 

     return success; 
    } 

    private SqlCommand GetUserSelectCommand() 
    { 
     if (this.UserSelectCommand==null) 
      this.UserSelectCommand= new System.Data.SqlClient.SqlCommand(); 
     this.UserSelectCommand.CommandText = "dbo.Users_Select"; 
     this.UserSelectCommand.CommandType = System.Data.CommandType.StoredProcedure; 
     this.UserSelectCommand.Connection = this.SQLConnection; 
     this.UserSelectCommand.Parameters.Clear(); 
     this.UserSelectCommand.Parameters.AddRange(new System.Data.SqlClient.SqlParameter[] { 
     new System.Data.SqlClient.SqlParameter("@RETURN_VALUE", System.Data.SqlDbType.Variant, 0, System.Data.ParameterDirection.ReturnValue, false, ((byte)(0)), ((byte)(0)), "", System.Data.DataRowVersion.Current, null)}); 

     return UserSelectCommand; 
    } 

Connectionオブジェクト、SqlCommands、およびSqlDataAdapterオブジェクトを再利用するのと同じ方法で書かれている他の複数の塗りつぶしタイプの機能があります。 SqlDataAdapterは内部的にSqlConnectionのオープンとクローズを管理します。

私の質問はマルチパートです。このデザインは悪いですか？もしそうなら、なぜですか？それが悪い場合

が、それは同様に多くのローカルスコープで物事を維持するように変更する必要があり、次の

public bool FillUsers(DataSet UserDataSet) 
    { 
     using (SqlConnection conn = new SqlConnection(ConnectionString)) 
     { 
      using (SqlCommand command = GetUsersSelectCommand()) 
      { 
       using (SqlDataAdapter adapter = new SqlDataAdapter(command, conn)) 
       { 
        adapter.Fill(UserDataSet, UsersTableName); 
       } 
      } 
     } 
    } 

これは、作成処分のように思えるすべての機能のために行わなければならない、とリメイクすると、アイテムを守るよりも悪くなります。しかし、これは私がどこでもオンラインで見るセットアップのようです。

Answer 1

いいえ、問題はありません。あなたが完了したらすぐにIDisposableを実装するあなたのオブジェクトを処分する必要があります。

SqlConnectionが指定されている場合、接続を廃棄すると、基本接続がプールに返されます。あなたが思っているように、必ずしも「閉じた」わけではありません。接続プールにはその作業をさせるのが最善です。 HereはMSDNとADO.NETの接続プーリングへのリンクです。それはそれが設計されていないものを行うようにしようとしている（驚くほどこの最適化と呼ぶ人もいる）。

また、実際に測定して問題を確認してから、最適化を試みてください。 （と私は厳しい方法でこれを意味するものではなく、あなたに時間を節約するにのみ）。