C＃リファクタリングこの乱雑なコード！

Question

基本的に、曜日のグリッドを持つタイムシートページ（ASP.NET MVC）のフラットビューモデルを作成しています。 WorkTimeプロパティは、データベースからの既存のWorkTimeであるか、既存のWorkTimeが存在しない場合はnullでなければなりません。

1週間しか表示されません（土曜日から金曜日）。私はそこに個々のプロパティを持っているので、私のビューの構文はもっと簡単になるはずです。

+------+-----+-----+-----+-----+-----+-----+-----+ 
| Desc | Sat | Sun | Mon | Tue | Wed | Thu | Fri | 
+------+-----+-----+-----+-----+-----+-----+-----+ 
|______|_____|_____|_____|_____|_____|_____|_____| 
|______|_____|_____|_____|_____|_____|_____|_____| 
|______|_____|_____|_____|_____|_____|_____|_____| 
|______|_____|_____|_____|_____|_____|_____|_____| 

から来て、データベースに永続化されている日付のすべての時間がない要素（すべて真夜中の時間）を持っていないと土曜日から金曜日までのDateTimeプロパティはすべて、すでに設定されています。

私は、このヘルパーメソッドと

public WorkTime SaturdayWorkTime { get; private set; } 
public WorkTime SundayWorkTime { get; private set; } 
public WorkTime MondayWorkTime { get; private set; } 
public WorkTime TuesdayWorkTime { get; private set; } 
public WorkTime WednesdayWorkTime { get; private set; } 
public WorkTime ThursdayWorkTime { get; private set; } 
public WorkTime FridayWorkTime { get; private set; } 

現在の繰り返し...

public DateTime Saturday { get; private set; } 
public DateTime Sunday { get; private set; } 
public DateTime Monday { get; private set; } 
public DateTime Tuesday { get; private set; } 
public DateTime Wednesday { get; private set; } 
public DateTime Thursday { get; private set; } 
public DateTime Friday { get; private set; } 

_workTimes = _workTimeRepository.GetByWorkAssignmentID(WorkAssignment.ID, Saturday, Friday); 
SaturdayWorkTime = GetWorkTimeForDay(DayOfWeek.Saturday); 
SundayWorkTime = GetWorkTimeForDay(DayOfWeek.Sunday); 
MondayWorkTime = GetWorkTimeForDay(DayOfWeek.Monday); 
TuesdayWorkTime = GetWorkTimeForDay(DayOfWeek.Tuesday); 
WednesdayWorkTime = GetWorkTimeForDay(DayOfWeek.Wednesday); 
ThursdayWorkTime = GetWorkTimeForDay(DayOfWeek.Thursday); 
FridayWorkTime = GetWorkTimeForDay(DayOfWeek.Friday); 

を設定していますプロパティ...

private WorkTime GetWorkTimeForDay(DayOfWeek dow) 
{ 
    return _workTimes.FirstOrDefault(x => x.Date.DayOfWeek == dow); 
} 

Answer 1

は小さなスタートだ：代わりに個々の変数を持つことの

private WorkTime GetWorkTimeForDay(DayOfWeek dw) 
{ 
    return workTimes.FirstOrDefault(x => x.Date.DayOfWeek == dw); 
} 

Answer 2

あなたはのDayOfWeek列挙型を使用することができます。ここで

DateTime now = DateTime.Now; 
var dayOfWeek = now.DayOfWeek; 

link

Answer 3

、あなたはWorkTimeへのDayOfWeekからマップ、および日時へのDayOfWeekから別のマップを持っているだろうか？申し訳ありませんが、私のC＃が錆びているので、これはおそらく、かなり右ではないですが、あなたはそれがより次のようになり、ということがあった場合：

for (DayOfWeek day : allDays) 
    workTimes(day) = repository.FirstOrDefault(dateTimes(day)); 

Answer 4

なぜないだけではなく、1つのWorkTimeプロパティを持って、そして作ること辞書を使用し、辞書の代わりに辞書を職場のリポジトリから返します。

Answer 5

これらの作業時間を保存する辞書を作成しないのはなぜですか？ _workTimeRepository.GetByWorkAssignmentIDを使用して、この辞書を移植

private Dictionary<DayOfWeek, WorkTime> _workTimes; 

、非常に単純である必要があります。