Problem Math.random in for-Schleife

[Fuchsteufel]

Hallo,

ich hab da mal eine Frage zu einem Stückchen Code, den ich grade geschrieben hab, Ziel ist es, Kennlinien, die ich mittels Sensoren aufnehme, in einem Koordinatensystem anzeigen zu lassen. Um zunächst mal die Graphen in den Griff zu bekommen, wollte ich die Y-Koordinaten per Math.Random erzeugen. Das ganze sieht folgendermaßen aus:

//Y-Koordinaten als Zufallswert
	for(int i=0; i<20; i++)
      	{
		yKoords[i] = (int) (Math.random()*20+1);

		System.out.println(yKoords[i]);
	}

Die Ausgabe in der Konsole bringt mir dabei 40 statt 20 Werte. Wenn ich die Koords einzeln angebe, funktionierts. Gibts da eine Besonderheit in random, die ich nicht kenne?

Gruß
Micha

 

[SlaterB]

der geposte Code enthält nix spannendes,
vielleicht wird die Schleife 2x ausgeführt oder ähnliches,

du kannst mehr Information erlangen, indem zu z.B. auch i mit ausgibst,
oder hier mehr Code posten

oder ersetze auch testweise
> yKoords = (int) (Math.random()*20+1);
durch
> yKoords = i;

wenn sich die Anzahl der Ausgaben nicht ändern, wovon ich ausgehe, kann es offensichtlich nichts mit Random zu tun haben

 

[njans]

Vielleicht hast du auch irgendwo noch eine andere Zuweisung, die da Werte überschreibt.

 

[Fuchsteufel]

Achja, das hab ich ganz vergessen:

Wenn ich die Anweisung mit dem random durch ein yKoords=i+1 ersetze, dann funktionierts perfekt. Mit dem Random allerdings, das habe ich gerade getestet, durchläuft er die Schleife 2 mal und ich komme nicht dahinter, warum er das macht.

Hier mal der ganze Code bitte nicht über die magic numbers in der Initialisierung des Koordinatensystems wundern, das wird noch dynamisch, ist nur für Testzwecke, Anfängersachen, wissenschon :

import java.awt.Graphics;
import javax.swing.JFrame;

public class Linie extends JFrame {

    public void paint(Graphics g) {

	//Koordinatensystem zeichnen:	
	g.drawLine(100, 100, 100, 300);
	g.drawLine(100, 300, 300, 300);
	
	//Skalierung zeichnen, X-Achse
	g.drawLine(100, 300, 100, 302);
	g.drawLine(110, 300, 110, 302);
	g.drawLine(120, 300, 120, 302);
	g.drawLine(130, 300, 130, 302);
	g.drawLine(140, 300, 140, 302);
	g.drawLine(150, 300, 150, 302);
	g.drawLine(160, 300, 160, 302);
	g.drawLine(170, 300, 170, 302);
	g.drawLine(180, 300, 180, 302);
	g.drawLine(190, 300, 190, 302);
	g.drawLine(200, 300, 200, 302);
	g.drawLine(210, 300, 210, 302);
	g.drawLine(220, 300, 220, 302);
	g.drawLine(230, 300, 230, 302);
	g.drawLine(240, 300, 240, 302);
	g.drawLine(250, 300, 250, 302);
	g.drawLine(260, 300, 260, 302);
	g.drawLine(270, 300, 270, 302);
	g.drawLine(280, 300, 280, 302);
	g.drawLine(290, 300, 290, 302);
	g.drawLine(300, 300, 300, 302);

	//Skalierung zeichnen, Y-Achse
	g.drawLine(98, 100, 100, 100);
	g.drawLine(98, 110, 100, 110);
	g.drawLine(98, 120, 100, 120);
	g.drawLine(98, 130, 100, 130);
	g.drawLine(98, 140, 100, 140);
	g.drawLine(98, 150, 100, 150);
	g.drawLine(98, 160, 100, 160);
	g.drawLine(98, 170, 100, 170);
	g.drawLine(98, 180, 100, 180);
	g.drawLine(98, 190, 100, 190);
	g.drawLine(98, 200, 100, 200);
	g.drawLine(98, 210, 100, 210);
	g.drawLine(98, 220, 100, 220);
	g.drawLine(98, 230, 100, 230);
	g.drawLine(98, 240, 100, 240);
	g.drawLine(98, 250, 100, 250);
	g.drawLine(98, 260, 100, 260);
	g.drawLine(98, 270, 100, 270);
	g.drawLine(98, 280, 100, 280);
	g.drawLine(98, 290, 100, 290);
	g.drawLine(98, 300, 100, 300);

	

	//###############
	//Graph zeichnen
	//###############

	//variablen anlegen
	int xUrsprung = 100;
	int yUrsprung = 300;
	int[] xKoords = new int[20];
	int[] yKoords = new int[20];
	

	//X-Koordinaten, Schrittweite 1, entspricht 10 Pixeln
	for(int i=0; i<20; i++)
      	{
		xKoords[i] = i;
	}

	//Y-Koordinaten als Zufallswert
	for(int i=0; i<20; i++)
      	{
		yKoords[i] = (int) (Math.random()*20+1);
		System.out.println("i:" + i);
		System.out.println("Koord:" + yKoords[i]);
	}


	

	//Graph zeichnen, Schrittweite 1 entspricht 10 Pixeln
	for(int i=1; i<20; i++)
	{
		g.drawLine(10*xKoords[i-1]+xUrsprung, yUrsprung-10*yKoords[i-1], 10*xKoords[i]+xUrsprung, yUrsprung-10*yKoords[i]);	
	}

	
	setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    }
    
    public static void main(String[] args) {
        Linie fenster = new Linie();
        fenster.setSize(400, 400);
        fenster.setLocation(150, 150);
        fenster.setVisible(true);
    }
}

Dann schonmal vielen Dank.
Grüße
Micha

 

[Dekker]

Oder mach mal ein zweites Sysout da drein. Vielleicht gibst du den Array an einer anderen Stelle nochmal aus.

Edit: Ok, sollte vorher Thread refreshen vorm Posten :E

 

[njans]

Was hast du denn da alles in der paint() Methde?
Nehm doch mal die Berechnung der yKoords aus der paint raus und lagere die aus, in eine Methode, die nicht andauert aufgerufen wird.^^

 

[SlaterB]

bei mir macht es keinen Unterschied (Random oder i+1), und wie gesagt wurde wird die paint-Methode unbeherrschbar beliebig oft aufgerufen,
z.B. auch beim Umschalten zwischen Konsole und GUI hin und her,
ist für gewünschte Begrenzung unbrauchbar

 

[Fuchsteufel]

zunächst mal danke

Ja, ist mir auch grade aufgefallen, dass die Berechnung keinen Unterschied macht, das dachte ich lediglich vorher, da die Punkte im Graph alle übereinander lagen und daher nicht zu sehen waren.

Nur bin ich grade aufgeschmissen, wie ich das auslagere, damit die Werte auch zur Verfügung stehen. Ohweh
Warum macht er das denn genau 2 mal?

 

[njans]

z.B. in den Konstruktor packen