TeamInklings/OctobotStealth
TeamInklings/OctobotStealth
Archived
1
0
Fork 0
forked from TeamInklings/Octobot
Code Pull requests Wiki Activity Actions

Use TryGetValue instead of ContainsKey + index access to avoid double lookup (#193)

Browse source 
This PR fixes an issue that is currently causing CI to fail in all pull
requests:
`Notice: "[CA1854] Prefer a 'TryGetValue' call over a Dictionary indexer
access guarded by a 'ContainsKey' check to avoid double lookup" on
/home/runner/work/Octobot/Octobot/src/Services/Update/ScheduledEventUpdateService.cs(107,4168)`
The issue is resolved by following the advice mentioned in the notice.
This commit is contained in:
Octol1ttle 2023-11-22 13:19:45 +05:00 committed by GitHub
parent c031b66eb4
commit e65c7a469d
Signed by: GitHub
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 1 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

3
src/Services/Update/ScheduledEventUpdateService.cs
View file

@ -104,14 +104,13 @@ public sealed class ScheduledEventUpdateService : BackgroundService
{ {
foreach (var @event in events) 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, data.ScheduledEvents.Add(@event.ID.Value,
new ScheduledEventData(@event.ID.Value, @event.Name, @event.ScheduledStartTime, @event.Status)); new ScheduledEventData(@event.ID.Value, @event.Name, @event.ScheduledStartTime, @event.Status));
continue; continue;
} }
var eventData = data.ScheduledEvents[@event.ID.Value];
eventData.Name = @event.Name; eventData.Name = @event.Name;
eventData.ScheduledStartTime = @event.ScheduledStartTime; eventData.ScheduledStartTime = @event.ScheduledStartTime;
if (!eventData.ScheduleOnStatusUpdated) if (!eventData.ScheduleOnStatusUpdated)