Improved RandomHelper

Directory.Packages.props

@@ -48,6 +48,7 @@
     <PackageVersion Include="JetBrains.Annotations" Version="2024.2.0" />
     <PackageVersion Include="LdapForNet" Version="2.7.15" />
     <PackageVersion Include="LibGit2Sharp" Version="0.30.0" />
+    <PackageVersion Include="ListShuffle" Version="2.1.0" />
     <PackageVersion Include="Magick.NET-Q16-AnyCPU" Version="13.4.0" />
     <PackageVersion Include="MailKit" Version="4.8.0" />
     <PackageVersion Include="Markdig.Signed" Version="0.37.0" />
@@ -168,6 +169,7 @@
     <PackageVersion Include="System.Text.Json" Version="9.0.0-rc.2.24473.5" />
     <PackageVersion Include="System.Threading.Tasks.Extensions" Version="4.5.4" />
     <PackageVersion Include="System.IdentityModel.Tokens.Jwt" Version="8.1.0" />
+    <PackageVersion Include="ThreadSafeRandomizer" Version="1.2.0" />
     <PackageVersion Include="TimeZoneConverter" Version="6.1.0" />
     <PackageVersion Include="Unidecode.NET" Version="2.1.0" />
     <PackageVersion Include="xunit" Version="2.9.2" />

framework/src/Volo.Abp.Core/Volo.Abp.Core.csproj

@@ -1,4 +1,4 @@
-<Project Sdk="Microsoft.NET.Sdk">
+<Project Sdk="Microsoft.NET.Sdk">
 
   <Import Project="..\..\..\configureawait.props" />
   <Import Project="..\..\..\common.props" />
@@ -16,6 +16,7 @@
     <RootNamespace />
   </PropertyGroup>
   <ItemGroup>
+    <PackageReference Include="ListShuffle" />
     <PackageReference Include="System.Collections.Immutable" />
     <PackageReference Include="Microsoft.Extensions.Localization" />
     <PackageReference Include="Microsoft.Extensions.DependencyInjection" />
@@ -32,6 +33,7 @@
     <PackageReference Include="System.Linq.Queryable" />
     <PackageReference Include="JetBrains.Annotations" />
     <PackageReference Include="Nito.AsyncEx.Context" />
+    <PackageReference Include="ThreadSafeRandomizer" />
   </ItemGroup>
   <ItemGroup Condition=" '$(TargetFramework)' == 'netstandard2.0' ">
     <PackageReference Include="System.Threading.Tasks.Extensions" />

framework/src/Volo.Abp.Core/Volo/Abp/RandomHelper.cs

@@ -1,7 +1,8 @@
 using System;
 using System.Collections.Generic;
-using System.Linq;
 using JetBrains.Annotations;
+using ListShuffle;
+using ThreadSafeRandomizer;
 
 namespace Volo.Abp;
 
@@ -11,8 +12,6 @@ namespace Volo.Abp;
 /// </summary>
 public static class RandomHelper
 {
-    private static readonly Random Rnd = new Random();
-
     /// <summary>
     /// Returns a random number within a specified range.
     /// </summary>
@@ -25,10 +24,7 @@ public static class RandomHelper
     /// </returns>
     public static int GetRandom(int minValue, int maxValue)
     {
-        lock (Rnd)
-        {
-            return Rnd.Next(minValue, maxValue);
-        }
+        return ThreadSafeRandom.Instance.Next(minValue, maxValue);
     }
 
     /// <summary>
@@ -42,10 +38,7 @@ public static int GetRandom(int minValue, int maxValue)
     /// </returns>
     public static int GetRandom(int maxValue)
     {
-        lock (Rnd)
-        {
-            return Rnd.Next(maxValue);
-        }
+        return ThreadSafeRandom.Instance.Next(maxValue);
     }
 
     /// <summary>
@@ -54,10 +47,7 @@ public static int GetRandom(int maxValue)
     /// <returns>A 32-bit signed integer greater than or equal to zero and less than <see cref="int.MaxValue"/>.</returns>
     public static int GetRandom()
     {
-        lock (Rnd)
-        {
-            return Rnd.Next();
-        }
+        return ThreadSafeRandom.Instance.Next();
     }
 
     /// <summary>
@@ -91,18 +81,9 @@ public static T GetRandomOfList<T>([NotNull] IList<T> list)
     /// <param name="items">items</param>
     public static List<T> GenerateRandomizedList<T>([NotNull] IEnumerable<T> items)
     {
-        Check.NotNull(items, nameof(items));
-
-        var currentList = new List<T>(items);
-        var randomList = new List<T>();
-
-        while (currentList.Any())
-        {
-            var randomIndex = RandomHelper.GetRandom(0, currentList.Count);
-            randomList.Add(currentList[randomIndex]);
-            currentList.RemoveAt(randomIndex);
-        }
+        var list = new List<T>(items);
+        list.Shuffle();
 
-        return randomList;
+        return list;
     }
 }