Array von Klassen

Das Problem verursacht wird, aufgrund der Tatsache, dass Sie nicht tatsächlich die Schaffung neuer ClassA Objekte, Sie sind nur mit den gleichen man zu jedem array-index. Sie schien auch falsch ClassA an einigen stellen, wie Class ich bin mir nicht sicher, wie Sie Sie zusammenstellen, wie es war. Bewegen Sie Ihre Klasse die Initialisierung in der Schleife, wo du bist Zuordnung zu arrayOfClass.

public static void Main()
{
    ClassA[,] arrayOfClass = new ClassA[6,6];
    //ClassA classA= new Class();
    int i, j;
    for (i = 0; i < 6; i++)
    {
        for (j = 0; j < 6; j++)
        {
            classA = new ClassA();
            classA.randomize();
            arrayOfClass[i, j] = classA;
        }
    }
    CheckClassA(arrayOfClass);
}

Wenn dies geschieht, führen Sie in ein weiteres Problem von allen der Rückgabe der gleichen Anzahl für einen ganz anderen Grund.

Es produziert die gleiche Anzahl, da jeder ClassA-in ein sehr enger Zeitraum und Random wird initialisiert mit dem gleichen seed.

Dies ist aus der Random() default-Konstruktor-Seite auf MSDN:

Die Standard-seed-Wert wird abgeleitet aus der system-Uhr und hat endliche Auflösung. Als ein Ergebnis, verschiedene Zufällige Objekte, die erstellt werden, in enger Folge durch einen Aufruf der default-Konstruktor wird haben identisch Standard-seed-Werte und daher produzieren identische Sätze von Zufallszahlen. Dieses problem kann vermieden werden durch die Verwendung eines einzelnen Random-Objekt zu erzeugen Zufallszahlen. Sie können auch es umgehen, ändern Sie den seed-Wert, der zurückgegeben wird, durch die system-Uhr und dann explizit die Bereitstellung dieser neue Ausgangswert für die Zufällige(Int32) - Konstruktor. Weitere Informationen finden Sie in der Random(Int32) - Konstruktor.

Also im Grunde, weil Random ist pseudo-random-es ist initialisiert, indem eine Zahl, die basierend auf der Systemuhr. Ihre beste Weg, um dieses ist, dass dieselben Random Objekt zwischen allen Instanzen des ClassA, entweder, indem Sie es static oder durch die übergabe einer Instanz von Random in der randomize Methode.

Sollte dies das problem beheben (so dass es static):

class ClassA
{
    public int number = -1;

    public static Random rnd = new Random();
    public void randomize()
    {
         var random = rnd;
         number = rnd.next(4)+2;
    }

}

InformationsquelleAutor Daniel Imms

Das eigentliche Problem Sie haben, ist, dass dein array ist immer das gleiche Objekt, für jeden Eintrag. Sie müssen new eine neue ClassA jedes mal, wenn Sie Durchlaufen, sonst bist du nur gehen zu zeigen, bis zum letzten .Anzahl.

Natürlich, das ist neben dem kleinen wäre mit einer new Random() jedes mal", was Sie sollte machen statischen etwa so:

public static void Main()
{
        ClassA[,] arrayOfClass = new ClassA[6,6];
        ClassA classA;
        int i, j;
        for (i = 0; i < 6; i++)
        {
                for (j = 0; j < 6; j++)
                {
                    classA = new ClassA(); //MISSING THIS LINE CAUSED ARRAY TO ONLY HAVE LAST ENTRY
                    classA.randomize();

                    arrayOfClass[i, j] = classA;
                }
        }
        CheckClassA(arrayOfClass);
        Console.ReadLine();
    }
    private static void CheckClassA(ClassA[,] arrayOfClass)
    {
            int i,j;
            for(i=0;i<6;i++)
            {
                for (j = 0; j < 6; j++)
                {
                    Console.WriteLine("Randoming : " + arrayOfClass[i, j].number);
                }
            }
    }

     public class ClassA
     {
         public int number = -1;

         public static Random rnd = new Random();
         public void randomize()
         {
             number = rnd.Next(4) + 2;
         }

     }
}

InformationsquelleAutor Rob G