Как сгенерировать случайную строку из букв и цифр в C#?

Проблема с классом Random и её решение

Ваш код действительно содержит несколько типичных ошибок начинающих разработчиков. Давайте разберём их по порядку.

1. Создание нового Random при каждом вызове

Класс Random использует системное время для инициализации seed-значения. Если вы создаёте несколько экземпляров подряд (с интервалом менее 1 мс), они получат одинаковое начальное значение, что приведёт к генерации одинаковых последовательностей. Решение — использовать один статический экземпляр Random.

private static readonly Random _random = new Random();

public static string GenerateString(int length)
{
    const string chars = "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789";
    char[] result = new char[length];
    
    lock (_random) // Потокобезопасность
    {
        for (int i = 0; i < length; i++)
        {
            result[i] = chars[_random.Next(chars.Length)];
        }
    }
    
    return new string(result);
}

2. Конкатенация строк в цикле

Использование result += chars[index] в цикле — очень неэффективно, так как строка в C# неизменяема. Каждая операция конкатенации создаёт новый объект строки. Лучше использовать StringBuilder или массив символов char[], как показано выше.

3. Недостаточная случайность

Класс Random не подходит для криптографических целей (пароли, токены). Он использует псевдослучайный генератор, который предсказуем. Для безопасной генерации используйте System.Security.Cryptography.RandomNumberGenerator.

using System.Security.Cryptography;
using System.Text;

public static string GenerateCryptographicString(int length)
{
    const string chars = "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789";
    byte[] randomBytes = new byte[length];
    
    using (var rng = RandomNumberGenerator.Create())
    {
        rng.GetBytes(randomBytes);
    }
    
    char[] result = new char[length];
    for (int i = 0; i < length; i++)
    {
        // Используем остаток от деления для выбора символа
        result[i] = chars[randomBytes[i] % chars.Length];
    }
    
    return new string(result);
}

Расширенное решение с LINQ

Современный C# позволяет сделать код более элегантным:

public static string GenerateString(int length)
{
    const string chars = "abcdefghijklmnopqrstuvwxyzABCDEFGHIJKLMNOPQRSTUVWXYZ0123456789";
    return new string(Enumerable.Repeat(chars, length)
        .Select(s => s[_random.Next(s.Length)])
        .ToArray());
}

Рекомендации

• Для обычных задач (например, ID в базе данных) используйте Random со статическим экземпляром и блокировкой.
• Для паролей, токенов, ключей шифрования — обязательно RandomNumberGenerator.
• Всегда задавайте длину строки явно, не используйте магические числа.
• Рассмотрите использование GUID, если нужны уникальные идентификаторы.

public static string GenerateString(int length)
{
    // Guid даёт 32 символа, обрезаем до нужной длины
    return Guid.NewGuid().ToString("N").Substring(0, Math.Min(length, 32));
}

public static string GenerateString(int length)
{
    return Path.GetRandomFileName().Replace(".", "").Substring(0, Math.Min(length, 11));
}