SQLCommandBuilderの効率

Question

msdnからこの抽出を得ました MSDN COMMAND BUILDER CLASS です。すばやく挿入、更新、またはテーブルからの削除、または非効率的なASP.NETで使用することは問題ありませんか？

public static DataSet SelectSqlRows(string connectionString, string queryString, string tableName) 
{ 
    using (SqlConnection connection = new SqlConnection(connectionString)) 
    { 


     SqlDataAdapter adapter = new SqlDataAdapter(); 
     adapter.SelectCommand = new SqlCommand(queryString, connection); 
     SqlCommandBuilder builder = new SqlCommandBuilder(adapter); 

     connection.Open(); 

     DataSet dataSet = new DataSet(); 
     adapter.Fill(dataSet, tableName); 

     //code to modify data in DataSet here 

     builder.GetUpdateCommand(); 

     //Without the SqlCommandBuilder this line would fail 
     adapter.Update(dataSet, tableName); 

     return dataSet; 
    } 
} 

Answer 1

効率性に関してコードに問題はありません。 SQL Injectionを防ぐのに役立つクエリ文字列を直接実行するのではなく、ストアドプロシージャまたはパラメータ付きクエリを使用することをお勧めします。

参考：

http://www.dotnetperls.com/sqlparameter

Answer 2

実行するSQL文と同じくらい効率的です。