C# mehrere Zufallszahlen

Werbeanzeige

ERROR

Alter Hase

Beiträge: 417

Wohnort: Paderborn

Beruf: Informatik Student

21.03.2012, 21:57

C# mehrere Zufallszahlen

Hallihallo,

Zu Übung habe ich heute mal angefangen einen Rechentrainer, den wir in der Schule mal in VB gemacht haben, in C# zu programmieren. Leider stoss ich ziemlich schnell auf ein Problem, dass ich auch noch langer recherche nicht ordentlich lösen konnte.

Ich will 2(oder mehr) Zufallszahlen erzeugen, die aber NICHT gleich sind. Leider kommen bei all meinen versuchen immer 2 gleiche raus. Kann mir vielleicht juemand einen Tipp/Denkstanstoss/Beispielscode geben?

So sieht es bisher aus:

C#-Quelltext

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;

namespace Rechentrainer_test
{
    public partial class Form1 : Form
    {
        public Form1()
        {
            InitializeComponent();
        }
        private int RandomNumber(int min, int max)
        {
            Random rnd = new Random();
            return rnd.Next(min, max);
        }

        private void btnStart_Click(object sender, EventArgs e)
        {
            int a = RandomNumber(1, 10);
            lbl1.Text = Convert.ToString(a);

            double b = RandomNumber(1,10);
            lbl2.Text = Convert.ToString(b);
        }
    }
}

Zum Seitenanfang

Sound~Wave

Alter Hase

Beiträge: 449

21.03.2012, 22:00

Ich kenne mich in C# nicht aus, aber kann es sein, dass du den Zufallsgenerator ohne Seed verwendest und jedes mal in der Funktion neu initialisierst? Zufallszahlen des PC's sind nämlich nicht echt, sondern berechnet und benötigen daher einen Seed.

Zum Seitenanfang

BlueCobold

Community-Fossil

Beiträge: 10 738

Beruf: Teamleiter Mobile Applikationen & Senior Software Engineer

21.03.2012, 22:01

Du hast da einen grundlegenden Fehler, nämlich den, dass Dein Generator vermutlich immer mit der selben Zahl initialisiert wird (der Zeit). Um das zu verhindern musst Du entweder eine Instanz halten, die mehrfach verwendet wird oder sie mit anderen Werten initialisieren.

Beispiel, was es eventuell schon lösen könnte:

C-/C++-Quelltext

using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;

namespace Rechentrainer_test
{
    public partial class Form1 : Form
    {
        Random rnd = new Random();

        public Form1()
        {
            InitializeComponent();
        }
        private int RandomNumber(int min, int max)
        {
            return rnd.Next(min, max);
        }

Übrigens macht die Methode "RandomNumber" keinen Sinn. Wieso? Weil "RandomNumber(0,15)" länger ist als "rnd.Next(0,15)". Wozu also solchen Quatsch kapseln?

Teamleiter von Rickety Racquet (ehemals das "Foren-Projekt") und von Marble Theory

Willkommen auf SPPRO, auch dir wird man zu Unity oder zur Unreal-Engine raten, ganz bestimmt.[/Sarkasmus]

Zum Seitenanfang

Sound~Wave

Alter Hase

Beiträge: 449

21.03.2012, 22:02

Also im Prinzip genau das, was ich schrieb...

Zum Seitenanfang

BlueCobold

Community-Fossil

Beiträge: 10 738

Beruf: Teamleiter Mobile Applikationen & Senior Software Engineer

21.03.2012, 22:04

Nicht ganz. Du bist teilweise davon ausgegangen, dass es kein Seed gibt. Das gibt es aber auch dann, wenn man nichts angibt, nämlich die Systemzeit.

Teamleiter von Rickety Racquet (ehemals das "Foren-Projekt") und von Marble Theory

Willkommen auf SPPRO, auch dir wird man zu Unity oder zur Unreal-Engine raten, ganz bestimmt.[/Sarkasmus]

Zum Seitenanfang

Spiele Programmierer

Alter Hase

Beiträge: 1 223

Wohnort: Deutschland Bayern

Beruf: Schüler

21.03.2012, 22:08

Du generierst immer wieder ein neues "Random"-Objekt und verwirfst es wieder.
Du musst dir die Instanz schon merken:

C#-Quelltext

private readonly Random Rnd = new Random();
public Form1()
{
    InitializeComponent();
}
private void btnStart_Click(object sender, EventArgs e)
{
    int a = Rnd.Next(1, 10);
    lbl1.Text = Convert.ToString(a);

    double b = Rnd.Next(1, 10);
    lbl2.Text = Convert.ToString(b);
}

Ganz nebenbei, du willst beim zweiten vermutlich auch Kommazahlen, sonst hättest du ja nicht "Double" da.
Momentan generierst du aber eine Ganzzahl und wandelst sie bloß in eine Gleitkommazahl um. Richtig wäre dann "Rnd.NextDouble() * 9.0d + 1.0d"

Edit: Da waren wohl welche schneller.

Zum Seitenanfang

Legend

Alter Hase

Beiträge: 731

Beruf: Softwareentwickler

22.03.2012, 11:24

Theoretisch hättest du zwei unterschiedliche Zahlen bekommen können, wenn zwischen den beiden Aufrufen von RandomNumber grad die Sekunde umspringt. Aber das ist sehr unwahrscheinlich!

"Wir müssen uns auf unsere Kernkompetenzen konzentrieren!" - "Juhu, wir machen eine Farm auf!"

Netzwerkbibliothek von mir, C#, LGPL: https://sourceforge.net/projects/statetransmitt/

Zum Seitenanfang

Sylence

Community-Fossil

Beiträge: 1 663

Beruf: Softwareentwickler

22.03.2012, 12:18

Hätte man auch durch einen kurzen blick in die Doku herausfinden können:

Zitat von »"http://msdn.microsoft.com/en-us/library/h343ddh9.aspx"«

As a result, different Random objects that are created in close succession by a call to the default constructor will have identical default seed values and, therefore, will produce identical sets of random numbers.