Fehler in einer Schleifenschachtelung

[Droggelbecher]

Hallo Leute ich hab eine Methode geschrieben die mir 4 Sinuskurve ausgeben soll und diese in 4 verschiedenen Farben. soweit eigentlich kein Ding aber das soll natürlich alles in Schleifen gepackt werden und da gibt es irgentwo einen fehler in meiner denkweise oder so jedenfalls gibt er mir immer nur die letzt der Kurven aus

public void draw(Graphics g){
		//Farben
		Color red	= new Color	(255,0,0);
		Color blue 	= new Color	(0,0,255);
		Color black = new Color (0, 0, 0);
		Color green = new Color	(0,255,0);
		int j=0;
		do {
			if (j==0)
				g.setColor(red);
				double a=50, omega=0.04, omega0=0.25;
			if (j==1)
				g.setColor(blue);
				a=250; omega=0.02; omega0=-0.25;				
			if (j==2)
				g.setColor(black);
				a=150;omega=0.02; omega0=-0.25;
			if (j==3)
				g.setColor(green);
				a=50; omega=0.04; omega0=-0.75;

			for (int i=-500; i<1000;i++){
				double f= a* Math.sin(omega*i+omega0);
				int hohe = new Double(f).intValue();
				g.fillOval(i,hohe,3,3);
			
				j++;
			}
		}while (j<4);
	}
}

ich hoffe ihr könnt mir weiterhelfen denn ich muss das morgen abgeben
schon einmaldanke im Voraus

Droggelbecher

 

[srea]

public void draw(Graphics g){
		//Farben
		Color red	= new Color	(255,0,0);
		Color blue 	= new Color	(0,0,255);
		Color black = new Color (0, 0, 0);
		Color green = new Color	(0,255,0);
		int j=0;
		do {
			if (j==0) {
				g.setColor(red);
				double a=50, omega=0.04, omega0=0.25;
                        }
			if (j==1) {
				g.setColor(blue);
				a=250; omega=0.02; omega0=-0.25;	
                        }			
			if (j==2) {
				g.setColor(black);
				a=150;omega=0.02; omega0=-0.25;
                        }
			if (j==3) {
				g.setColor(green);
				a=50; omega=0.04; omega0=-0.75;
                        }

			for (int i=-500; i<1000;i++){
				double f= a* Math.sin(omega*i+omega0);
				int hohe = new Double(f).intValue();
				g.fillOval(i,hohe,3,3);
			
				j++;
			}
		}while (j<4);
	}
}

Versuch mal die Sachen, die beim jeweiligen If ausgeführt werden sollen in {} zu setzen, sonst zählt das If nur für den ersten Befehl und er setzt jedesmal das letzt a, omega und omega0.

 

[AlexSpritze]

Das j++; muss raus aus der for-Schleife:

}
    j++;        
        }while (j<4);

 

[Blakh]

Du hast anscheinend Klammern vergessen?

...

if (j==0) {
   g.setColor(red);
   double a=50, omega=0.04, omega0=0.25;
}

...

Edit ... hm waren wieder paar schneller

Und das j innerhalb der for-schleife ist natürlich auch falsch ...

Du kannst die while auch als for-schleife machen.

for (int j = 0; j < 4; j++) {
....
}

 

[Michael...]

Würde es auch als for-Schleife machen, ausserdem könnte man das ganze noch etwas übersichtlicher und ohne diese if Abfragen machen:

private Color[] color = new Color[] {Color.RED, Color.BLUE, Color.BLACK, Color.GREEN};
private int[] a = {50, 250, 150, 50};
private double[] omega = {0.04, 0.02, 0.02, 0.04};
private double[] omega0 = {0.25, -0.25, -0.25, -0.75};

public void draw(Graphics g){
	for (int kurve=0; kurve<color.length; kurve++) {
		g.setColor(color[kurve]);
		for (int x=-500; x<1000; x++) {
			double y = a[kurve] * Math.sin(omega[kurve]*x + omega0[kurve]);
 			g.fillOval(x, (int)y, 3, 3);
		}
	}
}

 

[Droggelbecher]

Danke euch für die schnellenAntworten