Est-ce un bon moyen de générer une chaîne de caractères aléatoires?

Question

J'ai trouvé ce snippet de code qui génère une chaîne de caractères aléatoires.

Mais y a-t-il une manière plus élégante/plus rapide/plus fiable de faire ceci? Cela semble reposer sur le fait que les chiffres 26 à 91 sont des caractères valides étant donné l'encodage actuel.

/// <summary> 
/// Generates a random string with the given length 
/// </summary> 
/// <param name="size">Size of the string</param> 
/// <param name="lowerCase">If true, generate lowercase string</param> 
/// <returns>Random string</returns> 
private string RandomString(int size, bool lowerCase) 
{ 
    StringBuilder builder = new StringBuilder(); 
    Random random = new Random(); 
    char ch; 

    for(int i = 0; i < size; i++) 
    { 
     ch = Convert.ToChar(Convert.ToInt32(Math.Floor(26 * random.NextDouble() + 65))); 
     builder.Append(ch); 
    } 

    if(lowerCase) 
     return builder.ToString().ToLower(); 

    return builder.ToString(); 
} 

Answer 1

Je préfère passer l'instance Random dans la méthode - alors vous pouvez réutiliser la même instance à plusieurs reprises, ce qui est important si vous devez générer beaucoup de chaînes aléatoires en succession rapide. Cependant, je voudrais aussi un peu quand même le modifie:

public const string LowerCaseAlphabet = "abcdefghijklmnopqrstuvwyxz"; 
public const string UpperCaseAlphabet = "ABCDEFGHIJKLMNOPQRSTUVWXYZ"; 

public static string GenerateUpperCaseString(int size, Random rng) 
{ 
    return GenerateString(size, rng, UpperCaseAlphabet); 
} 

public static string GenerateLowerCaseString(int size, Random rng) 
{ 
    return GenerateString(size, rng, LowerCaseAlphabet); 
} 

public static string GenerateString(int size, Random rng, string alphabet) 
{ 
    char[] chars = new char[size]; 
    for (int i=0; i < size; i++) 
    { 
     chars[i] = alphabet[rng.Next(alphabet.Length)]; 
    } 
    return new string(chars); 
} 

• Il n'y a pas besoin d'utiliser un StringBuilder lorsque vous connaissez la longueur finale
• L'utilisation Random.NextDouble() indique un manque de connaissance de la classe aléatoire. (En particulier Random.Next(int, int)
• Création d'un nouveau aléatoire sur chaque appel est susceptible d'entraîner des chaînes en double
• Appel Convert.ToInt32 et Convert.ToChar semble laid contre seulement coulée
• Basse-boîtier après semble inutile par rapport avec la cueillette des lettres minuscules pour commencer
• Fournir un alphabet à choisir est beaucoup plus flexible (avec des méthodes d'assistance pour les cas communs)

Answer 2

Si je devais le faire (et j'ai, b ut dans Java, squirreled quelque part), je fournirais un tableau de caractères permis, et utiliser un RNG simplement pour choisir l'index du caractère. Cela vous permet également de ne pas autoriser les caractères que vous ne voulez pas générer (si vous créez une clé de licence à saisie humaine, vous ne voulez pas générer de caractères qui peuvent être confondus entre eux; 0 et O, par exemple, ou 1 et l).

EDIT: ouais, comme ce que Jon a fait ...

Answer 3

Path.GetRandomFileName()

string randomName = Path.GetRandomFileName(); 
randomName = randomName.Replace(".", string.Empty); 

// take substring...