Refactor to use TryGetValue instead of ContainsKey

Replaced redundant ContainsKey checks with TryGetValue in filter building and AddRecursive methods. This improves performance and readability by reducing dictionary lookups and streamlining key existence checks.

Author: likp

PxWeb/Code/Api2/DataSource/Cnmm/ItemSelectionResolverCnmm.cs

@@ -103,9 +103,9 @@ private static Dictionary<string, ItemSelection> RemoveUnrootedEntries(Dictionar
 
             foreach (var selection in lookupTable.Values)
             {
-                if (filter.ContainsKey(selection.Menu))
+                if (filter.TryGetValue(selection.Menu, out var selections))
                 {
-                    filter[selection.Menu].Add(selection);
+                    selections.Add(selection);
                 }
                 else
                 {
@@ -124,13 +124,11 @@ private static Dictionary<string, ItemSelection> RemoveUnrootedEntries(Dictionar
 
         private static void AddRecursive(string item, Dictionary<string, ItemSelection> result, Dictionary<string, List<ItemSelection>> lookup)
         {
-            if (!lookup.ContainsKey(item))
+            if (!lookup.TryGetValue(item, out var selections))
             {
                 return;
             }
 
-            var selections = lookup[item];
-
             if (selections != null && selections.Count > 0)
             {
                 foreach (var selection in selections)