は、私は私のコードでは、この非常に多くのことを書くように見えるのDataReader

Question

から返されるだけ単一の行を強制しますか？

Answer 1

私は知りませんが、このコードは、拡張メソッドに委任することができます。

public static R Single<R>(this DataReader reader, Func<DataReader,R> selector) { 
    R result = default(R); 
    if (reader.Read()) 
     result = selector(reader); 
    if (reader.Read()) 
     throw new DataException("multiple rows returned from query"); 
    return result; 
} 

はそのように使用する：

using (var reader = cmd.ExecuteReader()) 
{ 
    User u = reader.Single(r => new User((int)r["UserId"], r["UserName"].ToString())) 
} 

は、コードの重複からあなたを保存します。

Answer 2

SQLを使用してデータを取得する場合は、データリーダーを使用する必要があるたびにその種のコードを削除すると便利です。

SELECT TOP ([Number of rows you want to be selected]) 
FROM [Table Name] 
WHERE [Condition] 

EX：

SELECT TOP (1) 
FROM tblUsers 
WHERE Username = 'Allan Chua' 

別のチップを使用ストアドプロシージャ、SQLクエリと、不要なコーディングの繰り返しを最小限に抑えることができ、それらを使用します。

Answer 3

これは、あなたの目標が何であるかに応じて役立つかもしれません。適切な例外をスローするために複数の行が返されたことを検出する必要がある場合、これは役に立ちません。

結果が1つのみ返されるようにするには、このメソッドを使用してパフォーマンスバンプを取得する可能性があります。私が理解しているところでは、データプロバイダはこれを使用して、単一の行結果を予期してクエリを最適化することができます。

いずれの場合でも、SqlCommand.ExecuteReaderを使用してデータリーダーを作成し、CommandBehavior列挙（特にCommandBehavior.SingleRow）から引数を渡します。 ExecuteReaderはこれを受け入れるためにオーバーロードされます。

CommandBehavior enum

SqlCommand.ExecuteReader overload

だからあなたのコードは次のようになります。

using (var reader = cmd.ExecuteReader(CommandBehavior.SingleRow)) 
{ 
    if (reader.Read()) 
    { 
     result = new User((int)reader["UserId"], reader["UserName"].ToString()); 
    } 
}