Remove redundant containskey (#156)

* remove redundant containskey * .
JasperFx · Apr 7, 2024 · d255830 · d255830
1 parent 9716052
commit d255830
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 19 deletions.
diff --git a/src/Alba/Internal/LightweightCache.cs b/src/Alba/Internal/LightweightCache.cs
@@ -85,14 +85,7 @@ public TValue this[TKey key]
         }
         set
         {
-            if (_values.ContainsKey(key))
-            {
-                _values[key] = value;
-            }
-            else
-            {
-                _values.Add(key, value);
-            }
+            _values[key] = value;
         }
     }
 
@@ -128,15 +121,7 @@ public void Fill(TKey key, TValue value)
 
     public bool TryRetrieve(TKey key, [MaybeNullWhen(false)] out TValue value)
     {
-        value = default;
-
-        if (_values.ContainsKey(key))
-        {
-            value = _values[key];
-            return true;
-        }
-
-        return false;
+        return _values.TryGetValue(key, out value);
     }
 
     public void Each(Action<TValue> action)

diff --git a/src/IdentityServer.New/Pages/Diagnostics/ViewModel.cs b/src/IdentityServer.New/Pages/Diagnostics/ViewModel.cs
@@ -15,9 +15,8 @@ public ViewModel(AuthenticateResult result)
         {
             AuthenticateResult = result;
 
-            if (result.Properties.Items.ContainsKey("client_list"))
+            if (result.Properties.Items.TryGetValue("client_list", out var encoded))
             {
-                var encoded = result.Properties.Items["client_list"];
                 var bytes = Base64Url.Decode(encoded);
                 var value = Encoding.UTF8.GetString(bytes);