Parallel.ForEachAsync中Random.Next()持续生成相同值的原因排查

核心原因：Random类非线程安全 #

.NET的Random实例内部维护着一个种子状态，当多个线程同时调用Next()方法时，会并发修改这个状态，直接导致状态被破坏——极端情况下状态会被重置到固定值，之后每次调用Next()都会返回相同结果（比如你遇到的0）。

你的代码在方法开头创建了全局共享的rnd实例，然后在Parallel.ForEachAsync的并行循环里共用它，完全踩中了Random的线程安全陷阱。

可行解决方案 #

1. 每个线程使用独立的Random实例 #

在并行循环的委托内创建专属的Random实例，用Guid生成唯一种子避免多线程初始化时种子重复：

await Parallel.ForEachAsync(items, async (item, ct) =>
{
    var threadRnd = new Random(Guid.NewGuid().GetHashCode());
    var room1Index = threadRnd.Next(0, slot1.RoomList.Count);
    var room2Index = threadRnd.Next(0, slot2.RoomList.Count);

    while (room1Index == room2Index)
    {
        room2Index = threadRnd.Next(0, slot2.RoomList.Count);
    }
    // 后续业务逻辑
});

2. 使用ThreadLocal实现线程隔离 #

提前创建ThreadLocal<Random>，确保每个线程拿到专属实例，避免重复初始化：

var threadLocalRnd = new ThreadLocal<Random>(() => 
    new Random(Guid.NewGuid().GetHashCode()));

await Parallel.ForEachAsync(items, async (item, ct) =>
{
    var rnd = threadLocalRnd.Value;
    var room1Index = rnd.Next(0, slot1.RoomList.Count);
    var room2Index = rnd.Next(0, slot2.RoomList.Count);

    while (room1Index == room2Index)
    {
        room2Index = rnd.Next(0, slot2.RoomList.Count);
    }
    // 后续业务逻辑
});

threadLocalRnd.Dispose(); // 记得释放资源

3. .NET 6+ 直接使用Random.Shared（推荐） #

.NET 6引入了线程安全的Random.Shared静态实例，内部已处理线程同步，直接替换原有rnd即可：

var room1Index = Random.Shared.Next(0, slot1.RoomList.Count);
var room2Index = Random.Shared.Next(0, slot2.RoomList.Count);

while (room1Index == room2Index)
{
    room2Index = Random.Shared.Next(0, slot2.RoomList.Count);
}

额外提示 #

你代码中Next(0, slot1.RoomList.Count - 1)的写法有问题：Next方法的上限是排他的，若RoomList计数为4，这个写法只会生成0、1、2三个索引，最后一个元素（索引3）永远不会被选中，应改为Next(0, slot1.RoomList.Count)。

内容的提问来源于stack exchange，提问作者coolblue2000