如何在C#中设置随机数最大值等于输入的参与者数量？

核心问题分析 #

你遇到的错误主要来自两点：

• Random.Next(int minValue, int maxValue) 的第二个参数是排他上限，生成的数范围是 [minValue, maxValue)，直接传入参与者数量的话，最大只能生成到 numberofParticipants - 1，无法覆盖目标最大值。
• 缺少输入验证，若用户输入非整数、0或负数，会触发转换失败或 Next 方法参数非法的错误，也就是你遇到的“非32位有符号整数”类问题。

修复步骤及代码 #

1. 修正随机数范围

把 rand.Next(1, 20) 改成 rand.Next(1, numberofParticipants + 1)，这样就能生成 1 到 numberofParticipants 的完整整数序列。

2. 添加输入验证

确保用户输入的是有效的正整数，避免无效输入导致的异常。

3. 优化随机数生成效率（可选但推荐）

原代码用 do-while 循环结合 Contains 判断去重，当参与者数量较多时，重复尝试的次数会大幅增加，效率低下。推荐用洗牌算法：先创建包含1到目标数的有序列表，再随机打乱顺序，一次性生成不重复的随机序列。

修改后的完整代码：

using System;
using System.Collections.Generic;
using System.Linq;

Console.WriteLine("Please Enter the number of participants!");

// 输入验证：确保输入是有效的正整数
if (!int.TryParse(Console.ReadLine(), out int numberofParticipants) || numberofParticipants <= 0)
{
    Console.WriteLine("Invalid input! Please enter a positive integer.");
    Console.ReadLine();
    return;
}

var rand = new Random();

// 方法一：洗牌算法（高效推荐）
List<int> listNumbers = Enumerable.Range(1, numberofParticipants).ToList();
// 打乱列表顺序
for (int i = listNumbers.Count - 1; i > 0; i--)
{
    int j = rand.Next(i + 1);
    (listNumbers[i], listNumbers[j]) = (listNumbers[j], listNumbers[i]);
}

// // 方法二：修复原do-while逻辑（适合理解基础逻辑）
// List<int> listNumbers = new List<int>();
// int number;
// for (int i = 0; i < numberofParticipants; i++)
// {
//     do
//     {
//         number = rand.Next(1, numberofParticipants + 1);
//     } while (listNumbers.Contains(number));
//     listNumbers.Add(number);
// }

// 输出结果
listNumbers.ForEach(Console.WriteLine);

Console.ReadLine();

代码说明 #

• 输入验证：用 int.TryParse 替代 Convert.ToInt32，避免输入非整数时抛出异常；同时判断参与者数量是否大于0，保证后续逻辑合法。
• 洗牌算法：通过交换列表元素位置实现随机打乱，时间复杂度为O(n)，比原方法的O(n²)效率高很多，适合参与者数量较多的场景。
• 原逻辑修复：保留了修复后的do-while版本作为注释，方便你理解基础的去重逻辑。

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