-3

ベスト プラクティスに関する質問があります。BuildTourTimeDateTime と時刻を表す文字列を受け取るメソッドがあります。

 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 を配置します。これは静的メソッドを使用するのに適した場所ですか? それが私が傾いているものです。

編集:なぜ反対票なのですか?投稿されたリンク(この質問を投稿する前に読んだもの)は私の質問に答えません。私の質問は、使用できるオプションを調査し、正確な質問をし、関連コードを提供したことを示しています。

4

2 に答える 2