Générateur de chaîne aléatoire retournant la même chaîne

Question

J'ai développé un générateur de chaînes aléatoires mais il ne se comporte pas tout à fait comme je l'espérais. Mon objectif est de pouvoir exécuter ceci deux fois et générer deux chaînes aléatoires distinctes de quatre caractères. Cependant, il génère juste une chaîne aléatoire de quatre caractères deux fois.

Voici le code et un exemple de sa sortie :

private string RandomString(int size)
{
    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);
    }

    return builder.ToString();
}

// obtenir la 1ère chaîne aléatoire 
string Rand1 = RandomString(4);

// obtenir la 2ème chaîne aléatoire 
string Rand2 = RandomString(4);

// créer la chaîne aléatoire complète
string docNum = Rand1 + "-" + Rand2;

...et la sortie ressemble à ceci : UNTE-UNTE ...mais elle devrait ressembler à quelque chose comme ceci UNTE-FWNU

Comment puis-je m'assurer que deux chaînes aléatoires distinctes soient générées ?

Answer 1

Vous créez l'instance Random dans la méthode, ce qui fait qu'elle renvoie les mêmes valeurs lorsqu'elle est appelée rapidement. Je ferais quelque chose comme ceci :

private static Random random = new Random((int)DateTime.Now.Ticks);//mercipour McAden
private string RandomString(int size)
    {
        StringBuilder builder = new StringBuilder();
        char ch;
        for (int i = 0; i < size; i++)
        {
            ch = Convert.ToChar(Convert.ToInt32(Math.Floor(26 * random.NextDouble() + 65)));                 
            builder.Append(ch);
        }

        return builder.ToString();
    }

// obtenir le 1er string aléatoire 
string Rand1 = RandomString(4);

// obtenir le 2ème string aléatoire 
string Rand2 = RandomString(4);

// créer le numéro de document aléatoire complet
string docNum = Rand1 + "-" + Rand2;

(version modifiée de votre code)

Answer 2

Vous instanciez l'objet Random à l'intérieur de votre méthode.

L'objet Random est initialisé à partir de l'horloge système, ce qui signifie que si vous appelez votre méthode plusieurs fois rapidement, il utilisera toujours la même graine, ce qui signifie qu'il générera la même séquence de nombres aléatoires, ce qui signifie que vous obtiendrez la même chaîne.

Pour résoudre le problème, déplacez votre instance de Random en dehors de la méthode elle-même (et tant que vous y êtes, vous pourriez vous débarrasser de cette séquence folle d'appels à Convert, Floor et NextDouble):

private readonly Random _rng = new Random();
private const string _chars = "ABCDEFGHIJKLMNOPQRSTUVWXYZ";

private string RandomString(int size)
{
    char[] buffer = new char[size];

    for (int i = 0; i < size; i++)
    {
        buffer[i] = _chars[_rng.Next(_chars.Length)];
    }
    return new string(buffer);
}

Answer 3

Une implémentation très simple qui utilise Path.GetRandomFileName():

using System.IO;   
public static string RandomStr()
{
    string rStr = Path.GetRandomFileName();
    rStr = rStr.Replace(".", ""); // Pour supprimer le .
    return rStr;
}

Maintenant il suffit d'appeler RandomStr().

Answer 4

Tant que vous utilisez Asp.Net 2.0 ou une version ultérieure, vous pouvez également utiliser l'appel à la bibliothèque - System.Web.Security.Membership.GeneratePassword, cependant cela inclura des caractères spéciaux.

Pour obtenir 4 caractères aléatoires avec un minimum de 0 caractères spéciaux-

Membership.GeneratePassword(4, 0)

Answer 5

Juste pour les personnes qui passent et qui veulent avoir une chaîne aléatoire en une seule ligne de code

int yourRandomStringLength = 12; //maximum: 32
Guid.NewGuid().ToString("N").Substring(0, yourRandomStringLength);

PS: Veuillez garder à l'esprit que yourRandomStringLength ne peut pas dépasser 32 car Guid a une longueur maximale de 32.