avoid duplicate dictionary lookup in GetSession (#4550)

SimonCropp · web-flow · commit 0c1fdadfdaae · 2025-03-03T20:48:33.000-08:00
diff --git a/Source/Csla.AspNetCore/Blazor/State/SessionManager.cs b/Source/Csla.AspNetCore/Blazor/State/SessionManager.cs
@@ -26,11 +26,8 @@ public class SessionManager(ISessionIdManager sessionIdManager) : ISessionManage
     /// </summary>
     public Session GetSession()
     {
-      Session result;
       var key = _sessionIdManager.GetSessionId();
-      if (!_sessions.ContainsKey(key))
-        _sessions.TryAdd(key, []);
-      result = _sessions[key];
+      var result = _sessions.GetOrAdd(key, []);
       result.Touch();
       return result;
     }

Original file line number	Diff line number	Diff line change
`@@ -26,11 +26,8 @@ public class SessionManager(ISessionIdManager sessionIdManager) : ISessionManage`
`26`	`26`	`/// </summary>`
`27`	`27`	`public Session GetSession()`
`28`	`28`	`{`
`29`		`- Session result;`
`30`	`29`	`var key = _sessionIdManager.GetSessionId();`
`31`		`- if (!_sessions.ContainsKey(key))`
`32`		`- _sessions.TryAdd(key, []);`
`33`		`- result = _sessions[key];`
	`30`	`+ var result = _sessions.GetOrAdd(key, []);`
`34`	`31`	`result.Touch();`
`35`	`32`	`return result;`
`36`	`33`	`}`