ベスト プラクティスに関する質問があります。BuildTourTime
DateTime と時刻を表す文字列を受け取るメソッドがあります。
internal DateTime BuildDateTime(DateTime date, string time)
{
return date.Add(TimeSpan.Parse(time));
}
これを複数の場所で使用します。現在、別のクラスで使用されています
private IEnumerable<Tour> BuildTours(Response Data)
{
//Tour tour = new Tour();
List<Tour> tours = new List<Tour>();
foreach(Events ivEvents in Data.response.events)
{
bool isCancelled = true; //Replace this with logic
tours.Add(new Tour(ivEvents.Id,Data.response.ID,Data.response.Title,BuildDateTime(ivEvents.Date,ivEvent.Start),isCancelled,ivEvents.Quantity-ivEvents.Remaining).);
}
return tours;
}
私は現在、パラメーターなしのコンストラクターを持たない Tour クラスを持っています。私が知る限り、ここにはいくつかのオプションがあります。BuildDateTime 静的を宣言します。または、Tour のパラメーターなしのコンストラクターを追加します。または、ヘルパー クラスを作成し、BuildDateTime を配置します。これは静的メソッドを使用するのに適した場所ですか? それが私が傾いているものです。
編集:なぜ反対票なのですか?投稿されたリンク(この質問を投稿する前に読んだもの)は私の質問に答えません。私の質問は、使用できるオプションを調査し、正確な質問をし、関連コードを提供したことを示しています。