私はC＃の

Question

でコンストラクタを組み合わせることができます私は、次のコードを持っている：

public AccountService(ModelStateDictionary modelStateDictionary, string dataSourceID) 
    { 
     this._modelState = modelStateDictionary; 
     this._accountRepository = StorageHelper.GetTable<Account>(dataSourceID); 
     this._productRepository = StorageHelper.GetTable<Product>(dataSourceID); 
    } 

    public AccountService(string dataSourceID) 
    { 
     this._accountRepository = StorageHelper.GetTable<Account>(dataSourceID); 
     this._productRepository = StorageHelper.GetTable<Product>(dataSourceID); 
    } 

を私はそれぞれがStorageHelper呼び出しを行う必要はありませんコンストラクタを簡素化することができますいくつかの方法はありますか？

また、これを指定する必要があります。 ？

Answer 1

public AccountService(ModelStateDictionary modelStateDictionary, string dataSourceID) 
    : this(dataSourceID) 
{ 
    this._modelState = modelStateDictionary; 

} 

これは、最初に他のコンストラクタを呼び出します。 base(...を使用して、ベースコンストラクタを呼び出すこともできます。

thisを意味します。別の方法に

1）抽象共通初期化ロジックと各コンストラクタからそれを呼び出す：

Answer 2

はい、あなたは選択肢のカップルを持っています。あなたは（_modelStateがそれの後に初期化する_accountRepositoryを必要とする場合IE）の項目が初期化される順序を制御するために必要な場合は、この方法が必要になります

public AccountService(ModelStateDictionary modelStateDictionary, string dataSourceID) 
{ 
    this._modelState = modelStateDictionary; 
    Initialize(dataSourceID); 
} 

public AccountService(string dataSourceID) 
{ 
    Initialize(dataSourceID); 
} 

private void Initialize(string dataSourceID) 
{ 
    this._accountRepository = StorageHelper.GetTable<Account>(dataSourceID); 
    this._productRepository = StorageHelper.GetTable<Product>(dataSourceID); 
} 

2）カスケードコンストラクタを最後にthisを追加することによって：

public AccountService(ModelStateDictionary modelStateDictionary, string dataSourceID) : this(dataSourceID) 
{ 
    this._modelState = modelStateDictionary; 
} 

public AccountService(string dataSourceID) 
{ 
    this._accountRepository = StorageHelper.GetTable<Account>(dataSourceID); 
    this._productRepository = StorageHelper.GetTable<Product>(dataSourceID); 
}