Получить предыдущий рабочий день

Question

мне нужно, чтобы получить предыдущий рабочий день:

public DateTime GetPreviousWorkDay(DateTime date) 
{ 
    DateTime prevDay = date.AddDays(-1); 
    bool isHoliday = IsHoliday(prevDay); 
    while (isHoliday || prevDay.DayOfWeek == DayOfWeek.Saturday || prevDay.DayOfWeek == DayOfWeek.Sunday) 
    { 
     prevDay = prevDay.AddDays(-1); 
     isHoliday = IsHoliday(prevDay); 
    } 
    return prevDay; 
} 

Является ли этот метод безопасен? Может ли это быть бесконечным циклом? Есть ли более простой способ?

Answer 1

Чтобы сделать логику более ясной, я извлек метод IsWeekend и удалил ненужные временные переменные. Простой do..while цикл будет выполнять работу:

public DateTime PreviousWorkDay(DateTime date) 
{ 
    do 
    {   
     date = date.AddDays(-1);   
    } 
    while(IsHoliday(date) || IsWeekend(date)) 

    return date; 
} 

private bool IsWeekend(DateTime date) 
{ 
    return date.DayOfWeek == DayOfWeek.Saturday || 
      date.DayOfWeek == DayOfWeek.Sunday; 
}