feat: use icollection instead

Author: jcs090218

Assets/JCSUnity/Scripts/Util/JCS_Util.cs

@@ -135,55 +135,26 @@ public static bool WithInAcceptRange(float range, float acceptRange, float curre
         /// <returns></returns>
         public static bool WithInRange(float minRange, float maxRange, float currentVal)
         {
-            if (currentVal >= minRange && currentVal <= maxRange)
-                return true;
-            return false;
-        }
-
-        /// <summary>
-        /// With in array range. (Array)
-        /// </summary>
-        /// <returns></returns>
-        public static bool WithInRange<T>(int index, T[] arr)
-        {
-            return index >= 0 && index < arr.Length;
+            return (currentVal >= minRange && currentVal <= maxRange);
         }
 
         /// <summary>
-        /// With in array range. (List)
+        /// With in collection range.
         /// </summary>
         /// <returns></returns>
-        public static bool WithInRange<T>(int index, List<T> arr)
+        public static bool WithInRange<T>(int index, ICollection<T> arr)
         {
             return index >= 0 && index < arr.Count;
         }
 
         /// <summary>
-        /// Loop in an array. (Array)
-        /// </summary>
-        /// <typeparam name="T"> Type. </typeparam>
-        /// <param name="index"> Index </param>
-        /// <param name="arr"> Array. </param>
-        /// <returns> index that looped. </returns>
-        public static int LoopIn<T>(int index, T[] arr)
-        {
-            // loop through the array, if at the tail of the array set it to head.
-            if (index < 0)
-                index = arr.Length - 1;
-            // loop through the array, if at head of the array we set it to the tail.
-            else if (index >= arr.Length)
-                index = 0;
-            return index;
-        }
-
-        /// <summary>
-        /// Loop in an array. (List)
+        /// Loop in a collection.
         /// </summary>
         /// <typeparam name="T"> Type. </typeparam>
         /// <param name="index"> Index </param>
         /// <param name="arr"> List. </param>
         /// <returns> index that looped. </returns>
-        public static int LoopIn<T>(int index, List<T> arr)
+        public static int LoopIn<T>(int index, ICollection<T> arr)
         {
             // loop through the array, if at the tail of the array set it to head.
             if (index < 0)