refactor(extensions): 重命名集合扩展方法并优化 RemoveIf

- Random 重命名为 RandomElement，避免与 System.Random 命名冲突
- Upset 重命名为 Shuffle，使用更直观的命名
- RemoveIf 使用 RemoveAll 替代循环查找，提升性能

Author: AlianBlank

GameFrameX.Foundation.Extensions/CollectionExtensions.cs

@@ -158,15 +158,15 @@ public static int RemoveIf<TKey, TValue>(this Dictionary<TKey, TValue> self, Fun
     #region List<T>
 
     /// <summary>
-    /// 从列表中随机获取一个对象。
+    /// 从列表中随机获取一个元素。
     /// </summary>
     /// <typeparam name="T">列表元素的类型。</typeparam>
     /// <param name="list">要随机的列表，不能为 null 且不能为空。</param>
     /// <returns>随机选择的列表元素。</returns>
     /// <exception cref="ArgumentNullException">当 <paramref name="list"/> 为 null 时抛出</exception>
     /// <exception cref="ArgumentException">当 <paramref name="list"/> 为空时抛出</exception>
     /// <remarks>使用System.Random.Shared来生成随机数，确保线程安全。</remarks>
-    public static T Random<T>(this List<T> list)
+    public static T RandomElement<T>(this List<T> list)
     {
         ArgumentNullException.ThrowIfNull(list, nameof(list));
         if (list.Count == 0)
@@ -188,7 +188,7 @@ public static T Random<T>(this List<T> list)
     /// <returns>从列表中随机选择的元素。</returns>
     /// <exception cref="ArgumentNullException">当 <paramref name="list"/> 或 <paramref name="random"/> 为 null 时抛出</exception>
     /// <exception cref="ArgumentException">当 <paramref name="list"/> 为空时抛出</exception>
-    public static T Random<T>(this List<T> list, Random random)
+    public static T RandomElement<T>(this List<T> list, Random random)
     {
         ArgumentNullException.ThrowIfNull(list, nameof(list));
         ArgumentNullException.ThrowIfNull(random, nameof(random));
@@ -202,7 +202,7 @@ public static T Random<T>(this List<T> list, Random random)
     }
 
     /// <summary>
-    /// 打乱列表中的元素顺序。
+    /// 打乱列表中的元素顺序（洗牌）。
     /// </summary>
     /// <typeparam name="T">列表元素的类型。</typeparam>
     /// <param name="list">要打乱顺序的列表，不能为 null。</param>
@@ -211,7 +211,7 @@ public static T Random<T>(this List<T> list, Random random)
     /// 使用Fisher-Yates洗牌算法实现列表随机排序。
     /// 该方法会直接修改原列表的顺序。
     /// </remarks>
-    public static void Upset<T>(this List<T> list)
+    public static void Shuffle<T>(this List<T> list)
     {
         ArgumentNullException.ThrowIfNull(list, nameof(list));
 
@@ -224,27 +224,21 @@ public static void Upset<T>(this List<T> list)
     }
 
     /// <summary>
-    /// 从列表中移除满足条件的元素。
+    /// 从列表中移除满足条件的所有元素。
     /// </summary>
     /// <typeparam name="T">列表元素的类型。</typeparam>
     /// <param name="list">要操作的列表，不能为 null。</param>
     /// <param name="condition">用于判断元素是否满足移除条件的委托，返回true表示需要移除，不能为 null。</param>
     /// <exception cref="ArgumentNullException">当 <paramref name="list"/> 或 <paramref name="condition"/> 为 null 时抛出</exception>
     /// <remarks>
-    /// 此方法会重复查找并移除所有满足条件的元素，直到列表中不再存在满足条件的元素为止。
-    /// 注意：如果列表中有多个元素满足条件，会从前往后逐个移除。
+    /// 此方法使用 List{T}.RemoveAll 方法一次性移除所有满足条件的元素。
     /// </remarks>
     public static void RemoveIf<T>(this List<T> list, Predicate<T> condition)
     {
         ArgumentNullException.ThrowIfNull(list, nameof(list));
         ArgumentNullException.ThrowIfNull(condition, nameof(condition));
 
-        var idx = list.FindIndex(condition);
-        while (idx >= 0)
-        {
-            list.RemoveAt(idx);
-            idx = list.FindIndex(condition);
-        }
+        list.RemoveAll(condition);
     }
 
     #endregion

GameFrameX.Foundation.Tests/Extensions/CollectionExtensionsTests.cs

@@ -279,64 +279,64 @@ public void RemoveIf_Dictionary_ValidParameters_ShouldRemoveMatchingItems()
 
     #endregion
 
-    #region Random Tests
+    #region RandomElement Tests
 
     [Fact]
-    public void Random_NullList_ShouldThrowArgumentNullException()
+    public void RandomElement_NullList_ShouldThrowArgumentNullException()
     {
         // Arrange
         List<string> list = null;
 
         // Act & Assert
-        Assert.Throws<ArgumentNullException>(() => list.Random());
+        Assert.Throws<ArgumentNullException>(() => list.RandomElement());
     }
 
     [Fact]
-    public void Random_EmptyList_ShouldThrowArgumentException()
+    public void RandomElement_EmptyList_ShouldThrowArgumentException()
     {
         // Arrange
         var list = new List<string>();
 
         // Act & Assert
-        Assert.Throws<ArgumentException>(() => list.Random());
+        Assert.Throws<ArgumentException>(() => list.RandomElement());
     }
 
     [Fact]
-    public void Random_ValidList_ShouldReturnItem()
+    public void RandomElement_ValidList_ShouldReturnItem()
     {
         // Arrange
         var list = new List<string> { "item1", "item2", "item3" };
 
         // Act
-        var result = list.Random();
+        var result = list.RandomElement();
 
         // Assert
         Assert.Contains(result, list);
     }
 
     #endregion
 
-    #region Upset Tests
+    #region Shuffle Tests
 
     [Fact]
-    public void Upset_NullList_ShouldThrowArgumentNullException()
+    public void Shuffle_NullList_ShouldThrowArgumentNullException()
     {
         // Arrange
         List<string> list = null;
 
         // Act & Assert
-        Assert.Throws<ArgumentNullException>(() => list.Upset());
+        Assert.Throws<ArgumentNullException>(() => list.Shuffle());
     }
 
     [Fact]
-    public void Upset_ValidList_ShouldNotThrow()
+    public void Shuffle_ValidList_ShouldNotThrow()
     {
         // Arrange
         var list = new List<string> { "item1", "item2", "item3" };
         var originalItems = new List<string>(list);
 
         // Act
-        list.Upset();
+        list.Shuffle();
 
         // Assert
         Assert.Equal(originalItems.Count, list.Count);
@@ -347,13 +347,13 @@ public void Upset_ValidList_ShouldNotThrow()
     }
 
     [Fact]
-    public void Upset_EmptyList_ShouldNotThrow()
+    public void Shuffle_EmptyList_ShouldNotThrow()
     {
         // Arrange
         var list = new List<string>();
 
         // Act & Assert
-        list.Upset(); // Should not throw
+        list.Shuffle(); // Should not throw
         Assert.Empty(list);
     }
 

README.md

@@ -74,7 +74,7 @@ string decrypted = AesHelper.Decrypt(encrypted, "your-key");
 
 // 扩展方法使用
 var list = new List<int> { 1, 2, 3, 4, 5 };
-var randomItem = list.Random(); // 随机获取元素
+var randomItem = list.RandomElement(); // 随机获取元素
 var isNullOrEmpty = myString.IsNullOrEmpty(); // 字符串检查
 
 // 字符串扩展
@@ -157,7 +157,7 @@ using GameFrameX.Foundation.Extensions;
 
 // 集合操作
 var list = new List<int> { 1, 2, 3, 4, 5 };
-var randomItem = list.Random(); // 随机获取元素
+var randomItem = list.RandomElement(); // 随机获取元素
 var isEmpty = list.IsNullOrEmpty(); // 检查是否为空
 
 // 字典扩展