This is the test data. I added 6th Oct at the end of list to make sure the result only pick up last consecutive date set.
List<DateTime> viewDates = new List<DateTime>();
viewDates.Add(new DateTime(10, 10, 1));
viewDates.Add(new DateTime(10, 10, 3));
viewDates.Add(new DateTime(10, 10, 3));
viewDates.Add(new DateTime(10, 10, 4));
viewDates.Add(new DateTime(10, 10, 5));
viewDates.Add(new DateTime(10, 10, 5));
viewDates.Add(new DateTime(10, 10, 5));
viewDates.Add(new DateTime(10, 10, 6));
First, we need to find out the last consecutive date:
var lastConsecutiveDate = viewDates
.Distinct()
.Reverse()
.SkipWhile(distinctDate => viewDates.Where(viewDate => (viewDate.Date == distinctDate.Date)).Count() < 2).FirstOrDefault();
We have 2 choices here, to use foreach
or LINQ
? foreach
is more readable compared to LINQ
. I will show both implementations.
This is the implementation for foreach
:
var consecutiveDates = new List<DateTime>();
foreach (var item in viewDates.Where(viewDate => viewDate <= lastConsecutiveDate).Distinct().OrderByDescending(a => a.Date))
{
//break if datediff is not 1
int count = consecutiveDates.Count;
if ((count > 0) && (consecutiveDates[count - 1].Date.AddDays(-1) != item.Date))
{
break;
}
consecutiveDates.Add(item);
}
This is the implementation for LINQ
, less readable:
var consecutiveDates = viewDates
.Where(viewDate => viewDate <= lastConsecutiveDate)
.Distinct()
.OrderByDescending(viewDate => viewDate.Date)
.Aggregate
(
new { List = new List<DateTime>() },
(result, viewDate) =>
{
int count = result.List.Count;
if ((count > 0) && (result.List[count - 1].Date.AddDays(-1) != viewDate.Date))
{
return new { result.List };
}
result.List.Add(viewDate);
return new { result.List };
},
a => a.List
);
Console.WriteLine(consecutiveDates.Count());
will print out "3".
I am expecting more refactoring on my code, but I am off for break.