有很多解决方案，这里有一个：如果你只想要数字，那么就删除字母，然后方法会收到随机数和结果长度。

public String GenerateRandom(Random oRandom, int iLongitudPin)
{
    String sCharacters = "123456789ABCDEFGHIJKLMNPQRSTUVWXYZ123456789";
    int iLength = sCharacters.Length;
    char cCharacter;
    int iLongitudNuevaCadena = iLongitudPin; 
    String sRandomResult = "";
    for (int i = 0; i < iLongitudNuevaCadena; i++)
    {
        cCharacter = sCharacters[oRandom.Next(iLength)];
        sRandomResult += cCharacter.ToString();
    }
    return (sRandomResult);
}

每次执行时

Random random = new Random (15);

执行数百万次并不重要，您将始终使用相同的种子。

如果您使用

Random random = new Random ();

如果黑客猜到了种子，并且你的算法与系统的安全性有关，你会得到不同的随机数序列——你的算法被破坏了。如果你处决了骡子。在该构造函数中，种子由系统时钟指定，如果在极短的时间（毫秒）内创建了多个实例，则它们可能具有相同的种子。

如果你需要安全的随机数，你必须使用这个类

System.Security.Cryptography.RGCryptoServiceProvider

public static int Next(int min, int max)
{
    if(min >= max)
    {
        throw new ArgumentException("Min value is greater or equals than Max value.");
    }
    byte[] intBytes = new byte[4];
    using(RNGCryptoServiceProvider rng = new RNGCryptoServiceProvider())
    {
        rng.GetNonZeroBytes(intBytes);
    }
    return  min +  Math.Abs(BitConverter.ToInt32(intBytes, 0)) % (max - min + 1);
}

用法：

int randomNumber = Next(1,100);

您可以使用以下代码：

public static class ThreadSafeRandom
{
    private static readonly Random _global = new Random();
    private static readonly ThreadLocal<Random> _local = new ThreadLocal<Random>(() =>
    {
        int seed;
        lock (_global)
        {
            seed = _global.Next();
        }
        return new Random(seed);
    });

    public static Random Instance => _local.Value;
}

此代码可以按原样使用，也可以通过NuGet包ThreadSafeRandomizer使用。

编辑：从.NET 6.0开始，您可以改用Random.Shared.Next（）。您仍然可以使用上面的包，它可以在上面的代码或Random.Shared与预处理器指令之间进行选择。

每次执行新的Random（）时，都会使用时钟进行初始化。这意味着在一个紧密的循环中，你会多次获得相同的值。您应该保留一个Random实例，并在同一实例上继续使用Next。

//Function to get a random number 
private static readonly Random random = new Random(); 
private static readonly object syncLock = new object(); 
public static int RandomNumber(int min, int max)
{
    lock(syncLock) { // synchronize
        return random.Next(min, max);
    }
}

编辑（见评论）：为什么我们这里需要锁？

基本上，Next将更改Random实例的内部状态。如果我们同时从多个线程执行这一操作，你可能会认为“我们只是让结果变得更加随机”，但我们实际上正在做的事情可能会破坏内部实现，我们也可能会从不同的线程获得相同的数据，这可能是一个问题，也可能不是。不过，内部情况的保证是更大的问题；因为Random不保证线程安全。因此，有两种有效的方法：

同步，以便我们不会同时从不同的线程访问它每个线程使用不同的随机实例

两者都可以；但同时从多个调用方互斥一个实例只是自找麻烦。

锁实现了这些方法中的第一种（也是更简单的）；然而，另一种方法可能是：

private static readonly ThreadLocal<Random> appRandom
     = new ThreadLocal<Random>(() => new Random());

这是每个线程，因此不需要同步。

Mark的解决方案可能非常昂贵，因为它每次都需要同步。

我们可以通过使用线程特定的存储模式来避免同步的需要：

public class RandomNumber : IRandomNumber
{
    private static readonly Random Global = new Random();
    [ThreadStatic] private static Random _local;

    public int Next(int max)
    {
        var localBuffer = _local;
        if (localBuffer == null) 
        {
            int seed;
            lock(Global) seed = Global.Next();
            localBuffer = new Random(seed);
            _local = localBuffer;
        }
        return localBuffer.Next(max);
    }
}

衡量这两种实现，您应该会看到显著的差异。

1） 正如马克·格雷维尔所说，尝试使用一个随机生成器。将此添加到构造函数System.Environment.TickCount中总是很酷的。

2） 一个提示。假设您要创建100个对象，并假设每个对象都应该有自己的随机生成器（如果您在很短的时间内计算随机数的负载，则很方便）。如果要在循环中执行此操作（生成100个对象），可以这样做（以确保完全随机性）：

int inMyRandSeed;

for(int i=0;i<100;i++)
{
   inMyRandSeed = System.Environment.TickCount + i;
   .
   .
   .
   myNewObject = new MyNewObject(inMyRandSeed);  
   .
   .
   .
}

// Usage: Random m_rndGen = new Random(inMyRandSeed);

干杯