Spørgsmål til random numre

Ja, du bør ikke oprette et Random-objekt hver gang du kalder funktionen. Flyt den uden for, sådan at den kun oprettes en gang i alt:

private Random NewRand = new Random();
public void createTrainData()
{
    double randNumber;
    for (int i=0; i<sizeX; i++)
    {
        randNumber[i] = NewRand.NextDouble();
    }
}

Læs i øvrigt arne_v's artikler sådan at du har bedre chancer for at undgå den slags fælder en anden gang:

http://www.eksperten.dk/artikler/680
http://www.eksperten.dk/artikler/686

Sorry, men jeg havde også lagt NewRand uden for, desværre en copy paste fejl. Det virker dog stadig ikke og jeg skal stadig benytte en Thread.Sleep() imellem funktionskaldene før det virker.


private Random NewRand = new Random();

public void createTrainData()
{
    double randNumber;
    for (int i=0; i<sizeX; i++)
    {
        randNumber[i] = NewRand.NextDouble();
    }
}

Det burde nu virke uden problemer.

Et spørgsmpål: Du definere:

double randNumber;

- inde i funktionen. Samtidigt referere du til et array med samme navn:

randNumber[i] = NewRand.NextDouble();

Dette burde give dig en kompiler-fejl, sådan at faktisk du slet ikke kan køre din kode.

Den indre skal vist slettes for den har slet ingen funktion alligevel:

private Random NewRand = new Random();

public void createTrainData()
{
    for (int i=0; i<sizeX; i++)
    {
        randNumber[i] = NewRand.NextDouble();
    }
}

Hehe. Det er en forkortet udgave af min funktion. Burde nok have lagt det hele med. Den er vist også kopieret forkert ind kan jeg se - sorry

Her er hele min kode. Som du kan se opretter jeg et antal objekter af typen CReplicate og gemmer dem i et array. Hvis jeg benytter Sleep(10) virker det ellers fungerer det ikke.

public void loadPatternReplicates()
{
    for(int i=0; i<numberOfPatternReplicates; i++)
    {
        CReplicate NewTrainData = new CReplicate(patternData, gridSize);
        patternReplicates[i] = NewTrainData;
        //System.Threading.Thread.Sleep(10);
        }
}

public class CReplicate
{
    private int[] patternData;
    private int[] data;
    private int patternNumber;   
    private int gridSize;
    private Random NewRand = new Random();

    public CReplicate(int[] x_patternData, int x_gridSize)
    {
        patternData = x_patternData;
        gridSize = x_gridSize;
        data = new int[gridSize];
        createTrainData();
    }

    public void createTrainData()
    {
        double limit = (double)1/(double)8;
        double randNumber;
        int pixel;
        int counter=0;

        for(int i=0; i<gridSize;i++)
        {
            randNumber = NewRand.NextDouble();
            pixel = patternData[i];

            if(randNumber >= limit)
            {
                data[counter] = pixel;
            }
            else
            {
                data[counter] = 1;
            }
        counter++;
        }
    }

    public int[] getData()
    {
        return data;
    }
}

Ahh, ok: Du bør heller ikke have din:

private Random NewRand = new Random();

- på den måde inden i din CReplicate klasse. For så kaldes den jo en gang for hver gang at du opretter et nyt CReplicate-objekt. Den bør faktisk kun kaldes *en* gang totalt.

En mulig metode er at definere den som værende static:

private static Random NewRand = new Random();

Tusind tak. Det virker nu. Svar og der er point

Svar :^)