C＃コンストラクタでより簡潔な構文ですか？

Question

私は次のようにコンストラクタに何かを持っている：

using Microsoft.Data.Extensions; 

public class Complaint 
{ 
    public int Id {get; set;} 
    public int Transcript {get; set;} 
    //... etc. ... Lots more properties 

    public Complaint(int id) 
    { 
    var command = dataContext.CreateStoreCommand(
        "dbo.stp_Complaint_Get", 
        CommandType.StoredProcedure, 
        new SqlParameter("Id", id)); 

    var complaint = command.Materialize(x => 
         new Complaint 
         { 
          Id = x.Field<int>("Id"), 
          Transcript = x.Field<string>("Transcript"); 
          //... etc. ... Lots more fields from db 

         } 

    this.Id = complaint.Id; 
    this.Transcript = complaint.Transcript; 
    //... etc. ... Lots more properties to set 

    } 

} 

は私が1つのステップの代わりに、2で最後の部分を実行できるようになるC＃での構文はありますか？つまり、概念的に次のようなものです。

this = command.Materialize(x => 
        new Complaint 
        { 
         Id = x.Field<int>("Id"), 
         Transcript = x.Field<string>("Transcript"); 
        } 

Answer 1

本質的にはありません。あなたは、クラス内に不満オブジェクトを格納し、コンストラクタからすべてを設定するのではなく、すべてのプロパティをポイントすることができます。

例えば

public class Complaint 
{ 
    private readonly {type of complaint} m_Complaint;   

    public int Id 
    { 
     get { return m_Complaint.Id; } 
    } 

    // ...etc 
} 

あなたはm_Complaintのセッター持っていない場合は、より複雑に得ることができる - NULL可能なバッキングフィールド維持し、チェックあなたが

Answer 2

まあm_Complaintプロパティにアクセスする前に、あなたができること代わりにコンストラクタの静的メソッドを使用します。

public static Complaint FromId(int id) 
{ 
    var command = dataContext.CreateStoreCommand(
       "dbo.stp_Complaint_Get", 
       CommandType.StoredProcedure, 
       new SqlParameter("Id", id)); 

    return command.Materialize(x => 
        new Complaint 
        { 
         Id = x.Field<int>("Id"), 
         Transcript = x.Field<string>("Transcript"); 
         //... etc. ... Lots more fields from db 

        }); 
} 

Answer 3

を私はあなたがこのような何かを試すことが信じている：

var currentInstance = this; 
var complaint = command.Materialize(x => 
         new Complaint 
         { 
          Id = currentInstance.Id = x.Field("Id"), 
          Transcript = currentInstance.Transcript = x.Field("Transcript"); 
          //... etc. ... Lots more fields from db 

         }); 

私はクロージャーでこれをキャプチャすることはできないと思うので、そのcurrentInstanceトリックを使用すると助けになるはずですが、それは冗長になることもあります。 さらに、そのようなコードは、現在のソリューションと比較してわかりにくいものです。私はあなたのコードはそれほど素晴らしいと信じています。

Answer 4

これらのプロパティをすべて2回設定しないようにしようとしているのは、構造を変更すると2箇所のプロパティを更新する必要があるからです。あるインスタンスから別のインスタンスにプロパティをコピーするためにリフレクションを使用することもできます。

var complaint = command.Materialize(x => 
         new Complaint 
         { 
          Id = x.Field<int>("Id"), 
          Transcript = x.Field<string>("Transcript"); 
          //... etc. ... Lots more fields from db 

         } 

foreach (var prop in typeof(Complaint).GetProperties()) 
    ... 

Answer 5

クレームオブジェクトをメンバー変数にして、get/setプロパティが基本となるクレームのプロパティにアクセスしますか？