From fff1bde158803dfde60e9505b4f9d3293f0989ab Mon Sep 17 00:00:00 2001
From: Octol1ttle <l1ttleofficial@outlook.com>
Date: Wed, 22 Nov 2023 06:19:06 +0500
Subject: [PATCH] fix: use TryGetValue instead of ContainsKey + index access to
 avoid double lookup

---
 src/Services/Update/ScheduledEventUpdateService.cs | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

diff --git a/src/Services/Update/ScheduledEventUpdateService.cs b/src/Services/Update/ScheduledEventUpdateService.cs
index f1d3524..8872ddc 100644
--- a/src/Services/Update/ScheduledEventUpdateService.cs
+++ b/src/Services/Update/ScheduledEventUpdateService.cs
@@ -104,14 +104,13 @@ public sealed class ScheduledEventUpdateService : BackgroundService
     {
         foreach (var @event in events)
         {
-            if (!data.ScheduledEvents.ContainsKey(@event.ID.Value))
+            if (!data.ScheduledEvents.TryGetValue(@event.ID.Value, out var eventData))
             {
                 data.ScheduledEvents.Add(@event.ID.Value,
                     new ScheduledEventData(@event.ID.Value, @event.Name, @event.ScheduledStartTime, @event.Status));
                 continue;
             }
 
-            var eventData = data.ScheduledEvents[@event.ID.Value];
             eventData.Name = @event.Name;
             eventData.ScheduledStartTime = @event.ScheduledStartTime;
             if (!eventData.ScheduleOnStatusUpdated)