Und schon wieder die Linien (Problem beim überzeichnen)

[Zallesandro]

Hallo ich wollte mit meinem Programm ein Linien zeichen, wo Linien gezogen werden können, vergleichbar zum Konstruktionstool in Word, um halt später damit Analysen durchführen zu können. Das funktioniert zwar wunderbar aber nur für eine Linie Hab mich schon hier durchgeblättert im Forum nur bin ich nicht schlauer geworden :###
Hier ist erstmal der Code, ich benutze eine Canvasoberfläche zum Zeichnen:

public class MyCanvas extends Canvas implements MouseListener,
        MouseMotionListener {
int width, height;
private Point p1, p2;
Graphics MyGraphics;
MyCanvas(int width, int height) {
        this.width = width;
        this.height = height;
        reshape(0, 0, width, height);
        addMouseListener(this);
        addMouseMotionListener(this);

    }
 public void paint(Graphics g) {
        MyGraphics = getGraphics();

        if (p1 != null && p2 != null) {
            Graphics2D g2d = (Graphics2D) g;
            g2d.setStroke(stroke);
            g2d.drawLine(p1.x, p1.y, p2.x, p2.y);
            //Anzeigen der Nummer: aus Array...
            g2d.drawString("Nr. xx", p2.x + 10, p2.y + 10);
        }
    }
public void mousePressed(MouseEvent e) {
        //Durch drücken wird der erste Punkt geholt
        p1 = e.getPoint();
    }public void mouseDragged(MouseEvent e) {
        //Solange Maus gehalten, wird gezeichnet
        p2 = e.getPoint();
        this.update(MyGraphics);
    }

Das Problem ist das die ganze Oberfläche jedesmal neu gezeichnet wird, sobalg die Maus gezogen wird. das update brauch ich da sonst sehr viele linien gezeichnet werden. Hab mir überlegt die Linien in ein Vektor oder einen Baum zu packen und die dann immer zeichnen zulassen,da ich eh eine Zwischenspeicherung brauche, um die Nummer der Linie anzeigen zu lassen. Stehe aber irgendwie auf dem Schlauch.

 

[André Uhres]

Mit dem Array müsste es gehen: in paint durchläufst du das Array in zeichest alle Linien die drin stehen.

 

[Marco13]

Was du da mit dem "MyGraphics" machst ist so nicht richtig. Du solltest in mouseDragged NUR repaint aufrufen.

 

[Zallesandro]

Ich will ja die linie beim Zeichnen auch sehen, wenn ich halt nur repaint bei mouse dragged einsetze, zeichnet er zwar die Linie beim loslassen, nur bewege ich die Maus "blind". Speichere jetzt die Punkte p1 und p2 in einem Array ab mit denen ich in der Paintmethode dann wieder die Linien neu zeichne. Weiß vllt jemand einen besseren weg, denn ich will auf diese Linien später im Code noch zugreifen können.

Point[] erster;
Point[] zweiter;

public void addGraphik(Point p1, Point p2) {
        erster[zaehler] = p1;
        zweiter[zaehler] = p2;
}
public void printGraphic() {
        for(int i=0;i<zaehler;i++) {
            MyGraphics.drawLine(erster[i].x, erster[i].y, zweiter[i].x,
                                zweiter[i].y);
            MyGraphics.drawString("Nr. "+(i+1), zweiter[i].x + 10, zweiter[i].y + 10);
}

Der Zähler wird beim loslassen hochgesetzt und falsche "Klicks" (wenn jemand wild durch die gegend klickt) abgefangen. Danke

 

[Marco13]

Erstmal vorneweg: repaint reicht

import java.awt.*;
import java.awt.event.*;



public class MyCanvas extends Canvas implements MouseListener,
        MouseMotionListener {

public static void main(String args[])
{
    Frame f = new Frame();
    f.add(new MyCanvas(400,400));
    f.setSize(400,400);
    f.setVisible(true);
}

int width, height;
private Point p1, p2;
MyCanvas(int width, int height) {
        this.width = width;
        this.height = height;
        reshape(0, 0, width, height);
        addMouseListener(this);
        addMouseMotionListener(this);

    }
public void paint(Graphics g) {

        if (p1 != null && p2 != null) {
            Graphics2D g2d = (Graphics2D) g;
            //g2d.setStroke(stroke);
            g2d.drawLine(p1.x, p1.y, p2.x, p2.y);
            //Anzeigen der Nummer: aus Array...
            g2d.drawString("Nr. xx", p2.x + 10, p2.y + 10);
        }
    }
public void mousePressed(MouseEvent e) {
        //Durch drücken wird der erste Punkt geholt
        p1 = e.getPoint();
    }
    public void mouseDragged(MouseEvent e) {
        //Solange Maus gehalten, wird gezeichnet
        p2 = e.getPoint();
        repaint();
    }

public void mouseEntered(MouseEvent e) {}
public void mouseExited(MouseEvent e) {}
public void mouseMoved(MouseEvent e) {}
public void mouseReleased(MouseEvent e) {}
public void mouseClicked(MouseEvent e) {}


}

Zur eigentlichen Frage: Funktionieren tut das ja mit den zwei Punkten - es ist nur nicht besonders schön. Du könntest eine Klasse "Line" erstellen, die zwei Punkte enthält. Die Linien kannst du dann in einer ArrayList<Line> speichern. Da kanst du sie auch einfach hinzufügen, und musst nicht am Anfang eine feste Arraygröße vorgeben.