Frage zu drawLine

[smau]

Hallo zusammen,

ich zeichne mit der Methode drawLine in ein Bild rein und rufe diese Methode mehrmals auf.

Mein Problem ist, dass die Linie immer oben links im Bild anfängt und am Ende auch dort endet. Wie kann man das umgehen, also das die Linie bei meinem ersten Punkt auch anfängt?

Danke schonmal im Voraus!

 

[L-ectron-X]

Hmm. Was genau willst du machen, ich habs noch nicht verstanden. :bahnhof:

 

[smau]

Ich zeichne in ein Bild Punkte aus einem Array rein die mitten im Bild liegen mittels drawLine. Das funktioniert auch, jedoch startet und endet die Linie oben links im Bild immer.

Die soll aber an meinem ersten Punkt starten und an dem letzten enden.

 

[hdi]

ich schätze du vergisst vor dem Neuzeichnen das alte Bild zu löschen:

protected void paintComponent(Graphics g){
    super.paintComponent(g); // <-- löscht die aktuelle Zeichnung
    // jetzt kannst du ne neue Linie malen
}

edit: Wenn es nicht daran liegt zeig doch bitte mal deinen Code (inkl. Inhalt des Arrays)

 

[smau]

Mein Quellcode sieht so aus:

Graphics2D g = (Graphics2D) bild.getGraphics();
                g.drawLine(x[0][i],y[i][0],x[0][i+1],y[i+1][0]);
                g2.dispose();

 

[L-ectron-X]

getGraphics() aufzurufen ist fast immer ein Fehler.
Etwas mehr vom Quellcode könnte helfen, das Problem zu beseitigen.

 

[Michael...]

Was steckt denn in x[0] und y[0] und warum benutzt Du ein mehrfach verschachteltes Array?
verwendest Du eine Schleife oder stehen da nur die drei Zeilen in Deiner paint()
Lass Dir doch mal die Werte beim Zeichnen ausgeben.

Sehe gerade Du rufst getGraphics() auf - ist nicht umbedingt zu empfehlen. Eventuell packst Du mal das wesentliche in ein vollständiges Programm und postest dieses hier.

 

[smau]

In den Arrays stecken mehrere Werte die ich zeichne und mit dem nächsten Wert verbinde.

Graphics2D g = (Graphics2D) bild.getGraphics();
       
            for (int i = 0; i <= ende; i++) {
                g.drawLine(x[0][i],y[i][0],x[0][i+1],y[i+1][0]);
            }
        g.dispose();

EDIT: Dieser Quellcode ist in einer Methode geschrieben und diese Methode ruf ich mehrmals auf!

 

[eRaaaa]

getGraphics() aufzurufen ist fast immer ein Fehler.

wieso? gibt es eine quelle mit plausibler erklärung, oder kannst du das kurz erläutern? habe das letztens nämlich auch genutzt und würde dann schon gerne wissen wollen, was daran schlecht ist ;/

 

[Michael...]

wieso? gibt es eine quelle mit plausibler erklärung, oder kannst du das kurz erläutern? habe das letztens nämlich auch genutzt und würde dann schon gerne wissen wollen, was daran schlecht ist ;/

Es besteht das Risiko, dass das Objekt zum Zeitpunkt des Methodenaufrufs nicht sichtbar ist und dann liefert getGraphics() null. Gilt aber glaube ich nur für Komponenten - bin da aber kein Fachmann

@smau falls Du ein BufferedImage verwendest: hier ruft man eher createGraphics() anstelle von getGraphics() auf
Zum Zeichnen selbst: Wenn ich das richtig verstehe zeichnest Du anhand von Koordinaten eine Polylinie!?!
Wenn dann würde ich die Punktkoordinaten in einem einfach Array ablegen und so zeichen:

int[] x = new int[]{10, 50, 20};
int[] y = new int[]{10, 30, 80};
for (int i = 0; i <=x.length ; i++) {
                g.drawLine(x[i], y[i] , x[i+1], y[i+1]);
}

[edit] ansonsten kann ich anhand des kurzen Quellcodes nicht viel dazu sagen, besser wäre ein komplettes Programm in Kurzform (KSKB)

 

[Marco13]

wieso? gibt es eine quelle mit plausibler erklärung, oder kannst du das kurz erläutern? habe das letztens nämlich auch genutzt und würde dann schon gerne wissen wollen, was daran schlecht ist ;/

L-ectron-X's Hinweis bezog sich auf Components. Bei dem Aufruf [c]bild.getGraphics()[/c] kann man davon ausgehen, dass du das auf einem BufferedImage aufrufst, und dort ist das in Ordnung. Bei sowas wie [c]panel.getGraphics()[/c] (d.h. Aufruf auf einem JPanel o.ä.) würden die von Michael... angedeuteten Probleme auftreten.

 

[smau]

Es besteht das Risiko, dass das Objekt zum Zeitpunkt des Methodenaufrufs nicht sichtbar ist und dann liefert getGraphics() null. Gilt aber glaube ich nur für Komponenten - bin da aber kein Fachmann

@smau falls Du ein BufferedImage verwendest: hier ruft man eher createGraphics() anstelle von getGraphics() auf
Zum Zeichnen selbst: Wenn ich das richtig verstehe zeichnest Du anhand von Koordinaten eine Polylinie!?!
Wenn dann würde ich die Punktkoordinaten in einem einfach Array ablegen und so zeichen:

int[] x = new int[]{10, 50, 20};
int[] y = new int[]{10, 30, 80};
for (int i = 0; i <=x.length ; i++) {
                g.drawLine(x[i], y[i] , x[i+1], y[i+1]);
}

[edit] ansonsten kann ich anhand des kurzen Quellcodes nicht viel dazu sagen, besser wäre ein komplettes Programm in Kurzform (KSKB)

Mehr habe ich in der Methode nicht drinstehene, ausser den Kopf der Methode. Und ja ich benutze BufferedImage.

Ist es ein Unterschied, ob ich ein zweidimensionales Array benutze oder ein eindimensionales?

 

[Michael...]

Ist es ein Unterschied, ob ich ein zweidimensionales Array benutze oder ein eindimensionales?

letzteres ist einfacher ;-)
[Ernsthaft] Verstehe nur nicht, was Du mit was Du da genau machst

Hier mal ein KSKB meinerseits, hab hier keine Linien die bei 0,0 starten oder enden. Eventuell hast Du ja in Deinem Array einen 0en drinstehen?

import java.awt.*;
import java.awt.image.*;
import javax.swing.*;

public class DrawImageDemo extends JFrame {
	public DrawImageDemo () {
		BufferedImage image = new BufferedImage(200, 200, BufferedImage.TYPE_INT_RGB);
		Graphics2D g = image.createGraphics();
		int[] x = new int[]{10, 25, 50, 75, 100, 125, 150, 175};
		int[] y = new int[]{10, 50, 30, 80, 80, 65, 85, 150};
		for (int i=1; i<Math.min(x.length, y.length); i++) 
			g.drawLine(x[i-1], y[i-1], x[i], y[i]);
		g.dispose();
		this.getContentPane().add(new JLabel(new ImageIcon(image)), BorderLayout.CENTER);
	}
	
	public static void main(String[] args) {
		JFrame frame = new DrawImageDemo();
		frame.setBounds(0, 0, 500, 300);
		frame.setLocationRelativeTo(null);
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		frame.setVisible(true);
	}
}

PS: so schaut heute in etwa der Kursverlauf meiner Aktien aus;(

 

[smau]

g.drawLine(x[i-1], y[i-1], x[i], y[i]);

habe meine ersten Werte jetzt auch auf [i-1] gemacht und schon ist der Startpunkt richtig. Jetzt geht nur noch von meinem letzten Punkt aus eine Linie nach oben links.

 

[Michael...]

g.drawLine(x[i-1], y[i-1], x[i], y[i]);

habe meine ersten Werte jetzt auch auf [i-1] gemacht und schon ist der Startpunkt richtig. Jetzt geht nur noch von meinem letzten Punkt aus eine Linie nach oben links.

Das müsste bei Deiner urspr. Schleife aber auf einen Fehler laufen, da Du ja bei i=0 anfängst?!?

Lass doch einfach mal in Deiner Schleife die Werte ausgeben, damit Du überhaupt mal siehst welche Koordinaten zum Zeichnen verwendet werden:

System.out.println("Linie von " + x[0][i] + "|" + y[i][0]  + " nach "  + x[0][i+1] + "|" + y[i+1][0]);