replace boxing Hashtable with generic Dictionary<TKey, TValue>

Author: h3xds1nz

src/Microsoft.DotNet.Wpf/src/PresentationFramework/MS/Internal/Annotations/Component/AdornerPresentationContext.cs

@@ -4,7 +4,7 @@
 
 //
 // Description:
-//      AdornerPresentationContext knows that annotation comonents are wrapped 
+//      AdornerPresentationContext knows that annotation components are wrapped 
 //      in an AnnotationAdorner and hosted in the AdornerLayer. Note, implementation-wise 
 //      a new PresentationContext is created for every annotation component. Executing 
 //      operations on a presentation context for a different annotation component 
@@ -24,7 +24,7 @@
 namespace MS.Internal.Annotations.Component
 {
     /// <summary>
-    /// AdornerPresentationContext knows that annotation comonents are wrapped in an AnnotationAdorner and hosted in the AdornerLayer.
+    /// AdornerPresentationContext knows that annotation components are wrapped in an AnnotationAdorner and hosted in the AdornerLayer.
     /// Note,  implementation-wise a new PresentationContext is created for every annotation component. Executing operations on a presentation context
     /// for a different annotation component (located in the same adorner layer) works, but is slower than using the presentation context stored in the
     /// annotation component.
@@ -108,13 +108,13 @@ internal static void SetTypeZLevel(Type type, int level)
 
             Invariant.Assert(type != null, "type is null");
 
-            if (_ZLevel.ContainsKey(type))
+            if (s_ZLevel.ContainsKey(type))
             {
-                _ZLevel[type] = level;
+                s_ZLevel[type] = level;
             }
             else
             {
-                _ZLevel.Add(type, level);
+                s_ZLevel.Add(type, level);
             }
         }
 
@@ -128,9 +128,9 @@ internal static void SetTypeZLevel(Type type, int level)
         /// <param name="max">max Z-order value for this level</param>
         internal static void SetZLevelRange(int level, int min, int max)
         {
-            if (_ZRanges[level] == null)
+            if (s_ZRanges.ContainsKey(level) == false)
             {
-                _ZRanges.Add(level, new ZRange(min, max));
+                s_ZRanges.Add(level, new ZRange(min, max));
             }
         }
 
@@ -512,12 +512,9 @@ private AnnotationAdorner GetAnnotationAdorner(IAnnotationComponent component)
         /// <returns>ZLevel</returns>
         private static int GetComponentLevel(IAnnotationComponent component)
         {
-            int level = 0;
             Type type = component.GetType();
-            if (_ZLevel.ContainsKey(type))
-                level = (int)_ZLevel[type];
 
-            return level;
+            return s_ZLevel.TryGetValue(type, out int value) ? value : 0;
         }
 
         /// <summary>
@@ -531,18 +528,21 @@ private static int GetComponentLevel(IAnnotationComponent component)
         private static int ComponentToAdorner(int zOrder, int level)
         {
             int res = zOrder;
-            ZRange range = (ZRange)_ZRanges[level];
-            if (range != null)
+
+            if (s_ZRanges.TryGetValue(level, out ZRange range))
             {
                 //adjust the Z-order (shift it with the minimal value for this range)
                 //that way the component does need to know the range for its type that is 
                 // set by the application. It always sets the z-order as it starts from 0
                 res += range.Min;
+
                 if (res < range.Min)
                     res = range.Min;
+
                 if (res > range.Max)
                     res = range.Max;
             }
+
             return res;
         }
 
@@ -559,19 +559,17 @@ private static int ComponentToAdorner(int zOrder, int level)
         /// <summary>
         /// The adornerLayer which contains the annotation component.  Basically what the presentation hides.
         /// </summary>
-        private AdornerLayer _adornerLayer;
+        private readonly AdornerLayer _adornerLayer;
 
         /// <summary>
-        /// The hashtable holds the priority level for each Component type as defined by the application
+        /// The dictionary holds the priority level for each Component type as defined by the application
         /// </summary>
-        private static Hashtable _ZLevel = new Hashtable();
+        private static readonly Dictionary<Type, int> s_ZLevel = new();
 
         /// <summary>
         /// The ZRanges for the ZLevels. 
         /// </summary>
-        private static Hashtable _ZRanges = new Hashtable();
-
-
+        private static readonly Dictionary<int, ZRange> s_ZRanges = new();
 
         #endregion Private Fields
 
@@ -581,40 +579,28 @@ private static int ComponentToAdorner(int zOrder, int level)
         /// This is to control the relationships with TextSelection which lives in the same
         /// AdornerLayer. Will be removed when more flexible Z-ordering mechanism is available
         /// </summary>
-        private class ZRange
+        private sealed class ZRange
         {
             public ZRange(int min, int max)
             {
-                //exchange values if needed
+                // Swap values if needed
                 if (min > max)
                 {
-                    int temp = min;
+                    int num = min;
                     min = max;
-                    max = temp;
+                    max = num;
                 }
-                _min = min;
-                _max = max;
-            }
 
-            public int Min
-            {
-                get
-                {
-                    return _min;
-                }
-            }
-            public int Max
-            {
-                get
-                {
-                    return _max;
-                }
+                Min = min;
+                Max = max;
             }
 
-            private int _min, _max;
+            public int Min { get; }
+            public int Max { get; }
+
         }
 
-        #endregion Internal classes
+        #endregion Private classes
     }
 }