Swing Manuellen Kreis zeichen

[derSoerrn]

Guten Abend zusammen,

mir fehlt gerade ein wenig die Logik wie ich eine Linie zeichne und per Tastendruck dazu auffordere eine RUNDE Kurve zu werden. Meine Gedanken bisher waren, dass ich einen Kreis nehme und ihn auf dem Panel erstmal nur nach rechts bewege (ohne das Panel zu löschen) und sobald man eine Taste drückt (idealer Weise 'links' / 'rechts' oder 'a' / 'd') sich die Linie nach oben bzw. nach unten bewegt. Allerdings kann ich ja nur mit int´s arbeiten, weshalb das alles dann ziemlich eckig aussieht (vgl. das angehängte Bild).

Ich habe mir 8 bools gemacht:

private boolean right, left, up, down, leftToTop, leftToBottom, rightToTop, rightToBottom

Mit diesen lege ich eben fest, in welche Richtung sich die Linie "bewegen" soll - so:

					if(right){
						lineX++;
						lineY += 0;
					} else if(left){
						lineX--;
						lineY += 0;
					} else if(up){
						lineX += 0;
						lineY--;
					} else if(down){
						lineX += 0;
						lineY--;
					} else if(rightToTop){
						lineX++;
						lineY--;
					} else if(leftToTop){
						lineX--;
						lineY--;
					} else if(leftToBottom){
						lineX--;
						lineY++;
					} else if(rightToBottom){
						lineX++;
						lineY++;
					}

Ich nehme erstmal am Anfang an, dass rechts true ist. Sobald ich dann einmal nach links drücke muss die Linie sich nach links-oben bewegen (also

rightToTop = true;

// rest false

)

Wenn ich dann wieder nach rechts gehe muss eben wieder right true werden und wenn ich weiter nach rechts gehe eben rightToBottom.

Ich habe mir überlegt einen Integer "count" zu initialisieren und ihn für rechts zu subtrahieren und für links zu addieren, wobei dann geprüft wird, ob er vor dem addieren/subtrahieren bereits bei 0/7 ist und ihn wenn es so ist jeweils auf 7 bzw. 0 zu setzen.

Jedoch ist dies doch ein ziemlich hässliche Lösung oder nicht? Würde mich sehr über Verbesserungsvorschläge freuen - habe nämlich wie ihr bestimmt merkt noch nie wirklich in JAVA gezeichnet.

Danke für Eure Hilfe

 

[misterbanana]

Hallo,

ich würde mit floats arbeiten und dann winkelfunktionen verwenden.
Du kannst ja anhand der x und y Koordinaten dann je nach eingabe, relativ fließend die
Richtung bestimmen. Ich hoffe du verstehs gerade was ich meine

mfg misterbanana

 

[derSoerrn]

Verstehen an sich schon. Allerdings weiß ich nicht genau wie ich das umsetzen soll. Hättest du ein Beispiel?

Momentan zeichne ich nämlich so:

public void paintComponent(Graphics g) {
			// super.paintComponent(g);
			g.setColor(Color.RED);
			g.fillArc((int) lineX, (int) lineY, 10, 10, 360, 360);
		}

 

[misterbanana]

das tupel (x,y) beschreibt eine richtung, so wie du es jetzt bereits machst.
dieses grundprinzip würde ich weiter verfolgen. Nun machst du das stufenlos
indem du bei tastendruck eine richtungsanderungsfunktion aufrufst.
die funktion sollte wohl sowas wie sin oder cos verwenden, die abhängig
von der aktuellen richtung (ggf. mittels counter) die neue richtung berechnet.
der wert der sinusfunktion schwankt ja zwischen -1 und 1
und das kannst du auf x und einmal auf y-achse anwenden.

mfg misterbanana

 

[misterbanana]

um das noch etwas konkreter zu beschreiben:

du hast 2 variablen für die aktuelle Position:
float actX, actY;

zusätzlich 2 variablen die dieser positon eine verschiebung beschreiben:
float deltaX, deltaY;

diese werden mit der methode, die geschrieben werden muss ermittelt


diese deltas müssen bei jedem Aufruf auf die position(actX, actY) addiert werden (delta-werte sind zb zwischen -1 und 1)

dann kannst du mittels typecast so die funktion aufrufen:

g.fillArc((int) actX, (int) actY, 10, 10, 360, 360);

es wird hierbei gerundet.

ich hoffe das hilft dir, sonst einfach fragen.

mfg misterbanana

 

[derSoerrn]

Also meinst du ungefähr so?

g.fillArc((int) (lineX + calcCos(count)), (int) (lineY + calcSin(count)), 10, 10, 360, 360);

Wenn ja, dann funktioniert das nicht

count zählt nun bis 360 hoch und wird dann wieder auf null gesetzt. Wenn es kleiner als 0 werden sollte, dann wird es wieder auf 360 gesetzt.

Und die Positionen werden jetzt jeweils mit sinus bzw cosinus von count addiert.

Jedoch steht der Punkt nun auf der Stelle.

 

[derSoerrn]

oh verdammt.. ich muss ja auch immer noch lineX/Y neu setzen

 

[misterbanana]

klappt's denn nun?

 

[derSoerrn]

Leider nicht, weil JAVA meint dass Math.sin(360) = 0.9589... sei und von 90 = 0.89399, was ja völliger Quatsch ist oder nicht? wie stelle ich das Bogenmaß aus, bzw wie rechne ich es in Grad um?

 

[misterbanana]

probier mal als datentyp double, statt float und gib bescheid ob es daran lag...

der rückgabewert von sin() ist nämlich double

 

[derSoerrn]

Ne das Problem ist, dass die Sinus Methode von Java Math im Winkel im Bogenmaß annimmt und nicht wie wir hier in "normalen" Gradzahlen. Muss mal eben noch herausfinden, wie ich das umrechnen kann

 

[derSoerrn]

So damit geht es:

public float sin(int i) {
			double intoRadians = Math.toRadians(i);
			float result = (float) Math.sin(intoRadians);
			if(i == 180 || i == 360){
				return 0;
			}
			return result;
		}

		public float cos(int i) {
			double intoRadians = Math.toRadians(i);
			float result = (float) Math.cos(intoRadians);
			if(i == 90 || i == 270){
				return 0;
			}
			return result;
		}

Allerdings reagiert die "Linie" etwas schwer fällig. Hast du auch noch ne Idee, wie man das besser lösen könnte?

 

[misterbanana]

du könntest das resultat multiplizieren, also statt x = x + 0.2345

x=x + (0.2345 * 2)

mfg misterbanana

 

[derSoerrn]

was soll ich da multiplizieren bzw wo?

 

[misterbanana]

in deinem fall konkret "results"

 

[derSoerrn]

Hä? da kommt doch schon das richtige Ergebnis raus

 

[misterbanana]

das würde als multiplikator wirken... dh statt
.->.

.------>.

also die richtung würde um den faktor effektiver sein, da größere zahl

mfg misterbanana

 

[derSoerrn]

Hmm. Ich lade dir morgen mal mein Miniprogammcode hier hin. Kannst es dir dann ja anschauen und mir präzise sagen was du für nicht so schön hältst, wenn du willst.

 

[derSoerrn]

So hier hast du mal das ganze Programm. Wenn du willst kannst du mir jetzt gerne alles sagen, was nicht so schön gelöst ist

package bewegtesZeichnen;

import java.awt.AWTException;
import java.awt.Color;
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.RenderingHints;
import java.awt.event.KeyEvent;
import java.awt.event.KeyListener;
import java.awt.event.WindowEvent;
import java.awt.event.WindowListener;
import java.awt.geom.Ellipse2D;

import javax.swing.JFrame;
import javax.swing.JPanel;
import javax.swing.SwingUtilities;

public class XXX {

	public static void main(String[] args) {
		new XXX();
	}

	private JFrame window = new JFrame();
	private Curve curve = new Curve();
	
	private Thread animationThread;

	private boolean playing = true;
	private boolean plus;
	private boolean minus;
	
	private float x = 2;
	private float y = 50;

	private int angle = 360;
	
	public XXX() {		
		window.setSize(1000, 800);
		window.setBackground(Color.BLACK);
		window.setLocationRelativeTo(null);
		window.add(curve);
		window.setFocusable(true);
		window.addKeyListener(curve);
		window.requestFocus();
		load();
		animationThread.start();
		window.requestFocus();
	}

	private void load() {
		// windowListener, um die noch laufenden Threads
		// beim schließen des Fensters zu beenden
		window.addWindowListener(new WindowListener() {

			@Override
			public void windowOpened(WindowEvent arg0) {
				// TODO Auto-generated method stub

			}

			@Override
			public void windowIconified(WindowEvent arg0) {
				// TODO Auto-generated method stub

			}

			@Override
			public void windowDeiconified(WindowEvent arg0) {
				// TODO Auto-generated method stub

			}

			@Override
			public void windowDeactivated(WindowEvent arg0) {
				// TODO Auto-generated method stub

			}

			@Override
			public void windowClosing(WindowEvent e) {
				// TODO Auto-generated method stub
				System.exit(0);
			}

			@Override
			public void windowClosed(WindowEvent arg0) {
				// TODO Auto-generated method stub
				System.exit(0);
			}

			@Override
			public void windowActivated(WindowEvent arg0) {
				// TODO Auto-generated method stub

			}
		});

		// Sichtbarkeit des Fensters
		SwingUtilities.invokeLater(new Runnable() {
			public void run() {
				window.setVisible(true);
			}
		});

		// Thread um das Programm am Laufen zu halten:
		animationThread = new Thread(new Runnable() {
			public void run() {
				curve.repaint();
				while (playing) {
					if (x < window.getWidth() - 30 && x > 0 && y > 0
						&& y < window.getHeight() - 30) {
						curve.repaint();
						if (plus) {
							if (angle < 360) {
								angle += 2;
							} else {
								angle = 0;
							}
						} else if (minus) {
							if (angle > 0) {
								angle -= 2;
							} else {
								angle = 360;
							}
						}
					System.out.println(angle);
					try {
						Thread.sleep(150);
					} catch (InterruptedException e) {
						e.printStackTrace();
					}
					}
				}
			}
		});
	}

	private class Curve extends JPanel implements KeyListener {
		Graphics2D g2d;
		
		private float height = 10f;
		private float width = 10f;

		public Curve() {
			
		}

		public void paintComponent(Graphics g) {
			//super.paintComponent(g);
			
			g2d = (Graphics2D) g;
			g2d.setColor(Color.ORANGE);
			g2d.setRenderingHint(RenderingHints.KEY_ANTIALIASING,
					RenderingHints.VALUE_ANTIALIAS_ON);
			Ellipse2D.Float circle= new Ellipse2D.Float(getXPos(), getYPos(),
					width, height);
			g2d.fill(circle);
		}

		public float getXPos() {
			float posX = x + cos(angle);
			x = posX;
			return posX;
		}
		
		public float getYPos() {
			float posY = y + sin(angle);
			y = posY;
			return posY;
		}

		@Override
		public void keyPressed(KeyEvent e) {
			// TODO Auto-generated method stub
			System.out.println(e.VK_LEFT + " Taste: " + e.getKeyChar());
			if (e.getKeyChar() == 'a' || e.getKeyChar() == 'A'
					|| e.getKeyChar() == e.VK_LEFT) {
				plus = false;
				minus = true;
			} else if (e.getKeyChar() == 'd' || e.getKeyChar() == 'D'
					|| e.getKeyChar() == e.VK_RIGHT) {
				plus = true;
				minus = false;
			}
		}

		@Override
		public void keyReleased(KeyEvent e) {
			// TODO Auto-generated method stub
			if (e.getKeyChar() == 'a' || e.getKeyChar() == 'A'
					|| e.getKeyChar() == e.VK_LEFT) {
				minus = false;
			} else if (e.getKeyChar() == 'd' || e.getKeyChar() == 'D'
					|| e.getKeyChar() == e.VK_RIGHT) {
				plus = false;
			}
		}

		@Override
		public void keyTyped(KeyEvent e) {

		}

		public float sin(int i) {
			double intoRadians = Math.toRadians(i);
			float result = (float) Math.sin(intoRadians);
			if (i == 180 || i == 360) {
				return 0;
			}
			return result;
		}

		public float cos(int i) {
			double intoRadians = Math.toRadians(i);
			float result = (float) Math.cos(intoRadians);
			if (i == 90 || i == 270) {
				return 0;
			}
			return result;
		}
	}
}

 

[misterbanana]

Hallo,

public float getXPos() {
            float posX = x + cos(angle);
            x = posX;
            return posX;
}

kann vereinfacht werden zu:

public float getXPos() {
            x += cos(angle);
            return x;
}

An dieser Stelle kannst du auch du auch die intensität der richtungsänderung anpassen:

public float getXPos() {
            x += (cos(angle) * 2);
            return x;
}

Für y dann dasselbe.

Damit würde der kreis einen größeren umfang haben. Da dies antiproportional wirkt müsstest du also
für einen kleineren Umfang Werte im bereich ]0,1[ wählen.

Ansonsten läuft das ja soweit.

mfg misterbanana

 

[derSoerrn]

Ok stimmt: jedes mal beim Methodenaufruf ein neues float Objekt zu erstellen ist nicht so effizient. Werde das mal ändern.
Sonst meinst du ist alles programmiertechnisch in Ordnung oder würdest du noch mehr ändern?

 

[misterbanana]

So an sich sieht das, soweit ich mir das angeguckt habe, schonmal ganz gut aus.
Du solltest dir aber auf jeden Fall mal javadoc angucken.
How to Write Doc Comments for the Javadoc Tool
Dann kann man als Außenstehender besser überblicken was wofür gedacht ist (Rückgabewert etc.).

Wenn du das in ner IDE bearbeitest (zb. eclipse), dann ploppt da auch immer schön auf, was die Methoden für Parameter erwarten und
so weiter.

Sonst wäre das Thema für mich erstmal abgeschlossen, falls keine weiteren Fragen bestehen

Und immer schön weiter rumprobieren

mfg misterbanana

 

[derSoerrn]

Das mit den Kommentaren hätte ich natürlich machen sollen. Hatte aber keine Lust, weil ich das Programm ja eig nur für mich selbst geschrieben habe und da schien es mir unnötig für die 200 Zeilen Code Kommentare zu schreiben :bae:

Das Thema hier wäre dann auch für mich abgeschlossen. Hab aber schon wieder ein neues Wenn du willst kannst du gerne mal reingucken

Hier ;D