C#: Random für Zufallsbuchstabe erzeugt gleiches Ergebnis

[AFU]

Hallo,

wo liegt mein Fehler? Ich hätte gerne zufällig 36 Zeichen langen String, der sich aus dem angegeben String zufällig zusammensetzt. Was mache ich falsch?

//Rextester.Program.Main is the entry point for your code. Don't change it.

//Compiler version 4.0.30319.17929 for Microsoft (R) .NET Framework 4.5



using System;

using System.Collections.Generic;

using System.Linq;

using System.Text.RegularExpressions;



namespace Rextester

{

    public class Program

    {

        public static void Main(string[] args)

        {

            //Your code goes here

            

            string zeichen = "ABCDEFGHIJKLMNOPQRSTUVWXYZabcdefghijklmnopqrstuvwxyz";

            

            int zufallsbuchstabe = 0;

            string final ="";

            

            

            for (int i=0; i<36; i++)             {

            

            Random zufall = new Random();

            zufallsbuchstabe = zufall.Next(0, 51);

            

                

            final = final + zeichen.Substring(zufallsbuchstabe, 1);

            //int position = zeichen.IndexOf(zufallszahl);

            //final = zeichen.Substring(position, 1);

                

            

                

                }

                

            Console.WriteLine("Zufall! " + final );

        

    }

}

}

[Nuebel]

Was funktioniert denn da nicht?

Habe keinen C#-Compiler zur Hand. Der folgende Code ist in Java und funktioniert:

import java.util.Random;



public class RandomString {

    public static void main(String[] args) {

	    char[] alphabet = (

            "ABCDEFGHIJKLMNOPQRSTUVWXYZ"

          + "abcdefghijklmnopqrstuvwxyz").toCharArray();



        final int alphabetLength = alphabet.length;



        Random random = new Random();



        final int length = 36;

        StringBuilder randomStr = new StringBuilder(length);



        for (int i = 0; i < length; ++i) {

            randomStr.append(alphabet[random.nextInt(alphabetLength)]);

        }



        System.out.println(randomStr.toString());

    }

}

Aufgrund der Ähnlichkeit zu C#, solltest du es übertragen können.

[Brainy]

Erzeuge das Random-Objekt mal außerhalb der for-Schleife (wie Nuebel es getan hat).

[Manipulate]

Könntest du nicht auch theoretisch
final = final + zeichen.Substring(zufall, zufall+1);
machen?

edit:
Ach Moment, hab länger nicht mehr mit random gearbeitet, weiß nicht mehr genau wie das funktioniert. xD

[Brainy]

Könntest du nicht auch theoretisch
final = final + zeichen.Substring(zufall, zufall+1);
machen?

edit:
Ach Moment, hab länger nicht mehr mit random gearbeitet, weiß nicht mehr genau wie das funktioniert. xD

Der zweite Parameter bei der Methode Substring (das gilt für die vom Fragestellenden benutze Programmiersprache C#) bestimmt die Anzahl (Länge) der Zeichen in der Teilzeichenfolge.

Betrachten wir mal folgendes Beispiel:

String abcde = "abcde";

In C# liefert abcde.Substring(2,3) "cde".
In Java liefert abcde.substring(2,3) "c".

[Nuebel]

In C# liefert abcde.Substring(2,3) "cde".
In Java liefert abcde.substring(2,3) "c".

Ich hasse das, wenn man bei Methoden, die "Range-Argumente" entgegennehmen nicht sofort erkennen kann, ob die obere Schranke in- oder exklusiv ist.

[Manipulate]

@Brainy
Achso, bin mit C# nicht vertraut, habe angenommen es wäre genauso wie in Java. Danke für die Aufklärung.

Aber in dem Fall könnte man doch statt wie bei mir zufall+1 einfach 1 hinschreiben, hätte das nicht den selben Effekt?
edit: oh, hat er getan, ich halte mich dann doch raus XD

[AFU]

Thx to all, speziell zu Nuebel und Brainy, der Tipp mit dem Erzeugen außerhalb der For-Schleife hat die Lösung gebracht

[DMW007]

Ergänzend noch die Erklärung, warum die Variante aus dem Eingangspost nicht das erwartete Ergebnis bringt: Der Standardkonstruktor der Random-Klasse erzeugt einen Seed-Wert für die Zufallszahlen, der von der Systemuhrzeit abgeleitet werden. Es gibt auch eine Überladung, um einen eigenen Seed festlegen zu können. Die beste Lösung ist aber, wie schon hier vorgeschlagen, die Random-Instanz außerhalb der Klasse zu erzeugen. Nicht nur wegen des Seed-Problemes, sondern auch vom generellen Entwicklungsmuster und der Performance her: Die Variante des TE erzeugt für JEDEN Durchlauf der Schleife eine Objektinstanz, in dem Beispiel also 36 Stück. Das sind 35 mehr als nötig. In dem simplen Mini-Programm ist das zwar egal, aber bei größeren Programmen und komplexeren Klassen kann das schon was ausmachen. Vor allem bei Client-Server Anwendungen.

Man sollte es generell vermeiden, Objekte in einer Schleife zu Instanzieren. Ein Klassiker sind z.B. Datenbankabfragen in einer Schleife. Da ist der Performance-Overhead noch viel schlimmer, vor allem wenn ein ORM dazwischen steckt. Möglicherweise ist das da dem Entwickler gar nicht bewusst, wenn er mit der Architektur noch nicht so vertraut ist. Wenn man so etwas tut, empfehle ich daher, noch mal darüber nachzudenken, ob das wirklich sein muss. In manchen Ausnahmefällen kann das sinnvoll sein. Oft ist es das aber nicht und verursacht die genannten Probleme.

Dieses Verhalten der Random-Klasse von C# ist übrigens auch in der MSDN dokumentiert: https://msdn.microsoft.com/de-de/lib...v=vs.110).aspx