From 1070cb00f8f19a11549cf9802a4c62fd316d1d74 Mon Sep 17 00:00:00 2001 From: Jeffery Thompson Date: Thu, 12 Jul 2018 15:48:45 -0400 Subject: [PATCH] fix: use TryFirst instead of comparison to default(DateTime) --- ICD.Common.Utils/Extensions/DateTimeExtensions.cs | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/ICD.Common.Utils/Extensions/DateTimeExtensions.cs b/ICD.Common.Utils/Extensions/DateTimeExtensions.cs index 8a95078..63ca40f 100644 --- a/ICD.Common.Utils/Extensions/DateTimeExtensions.cs +++ b/ICD.Common.Utils/Extensions/DateTimeExtensions.cs @@ -42,9 +42,9 @@ namespace ICD.Common.Utils.Extensions if (times.Length == 0) return null; - DateTime[] orderedTimes = times.OrderBy(dt => dt).ToArray(); - var time = orderedTimes.FirstOrDefault(dt => target < dt); - return time == default(DateTime) ? (DateTime?) null : time; + DateTime earliestTime; + bool success = times.OrderBy(dt => dt).TryFirst(dt => target < dt, out earliestTime); + return success ? earliestTime : (DateTime?) null; } /// @@ -59,9 +59,9 @@ namespace ICD.Common.Utils.Extensions if (times.Length == 0) return null; - DateTime[] orderedTimes = times.OrderByDescending(dt => dt).ToArray(); - var time = orderedTimes.FirstOrDefault(dt => target > dt); - return time == default(DateTime) ? (DateTime?) null : time; + DateTime latestTime; + bool success = times.OrderByDescending(dt => dt).TryFirst(dt => target > dt, out latestTime); + return success ? latestTime : (DateTime?) null; } } }