Swing paintComponent problem

[Tankpope]

Hallo,
ich bin grade ein bissel am Verzweifeln und finde meinen Fehler nicht. Warscheinlich sehe ich den Wald vor lauter Bäumen nicht.

Ich schreibe grade ein Programm bei dem bei einem Mausklick ein Kreuz auf ein JPanel gezeichnet werden soll.

Hierzu habe ich mir die Klasse DrawPoint geschrieben. Der im Konstuktor der Punkt für den Kreuz Mittelpunkt übergeben wird.

import java.awt.Graphics;
import java.awt.Point;
import javax.swing.JComponent;


public class drawPoint extends JComponent{

	private int X,Y;
	
	public drawPoint(Point p)
	{
        this.setBounds(p.x-5, p.y-5, 10, 10);
        this.setLocation(p);
        X = p.x;
        Y = p.y;
	}
	
	@Override
	protected void paintComponent(Graphics g) {
		super.paintComponent(g);
		g.drawLine(X-5, Y, X+5, Y);
		g.drawLine(X, Y-5, X, Y+5);
	}
	
}

Aufgerufen wird das ganze in einer Klasse die nur die Anwendungsmethoden enthält.

Zeichenbrett ist hierbei vom Typ JPanel
GUI ist vom Typ JFrame

	@Override
	public void mouseReleased(MouseEvent e) 
       {
		drawPoint dp = new drawPoint(e.getPoint());
		GUI.Zeichenbrett.add(dp);
		GUI.Zeichenbrett.repaint();
	}

Das Problem bei dem Ganzen ist das bei einem Mausklick die paintComponent Methode aufgerufen wir aber leider kein Kreuz gezeichnet wird. Setze ich die Backgoundcolor von dp wird an der Stelle wie es sein sollte ein Farbiges Rechteck in den angegebenen bounds gezeichnet, also legt er den drawpoint auf jeden fall an, paintComponent wird auch aufgerufen. Woran könnte es noch liegen das er meine drawLine anweisungen in paintComponent nicht ausführt.

schon mal thx
Michael

 

[hdi]

Das Problem ist, dass X und Y die Koordinaten im Koordinatensystem des Frames sind, aber das Graphics-Objekt, das in die paintComponent hineingereicht wird, malt immer innerhalb der eigenen Komponente, und jegliche Koordinaten beziehen sich auf das Koordinaten-System der Komponente.

zB du klickst bei (100,100). Dann platzierst du zwar im Frame an dieser Stelle die Komponente, aber beim Zeichnen malst du in der Komponente an der Stelle (100,100). Aber das Ding ist nur 10x10 Pixel groß.

Klar, was ich meine?
Entweder man nutzt X und Y in der paintComponent, in dem man direkt in ein großes Panel mit allen Kreuzen reinmalt (d.h. dein Kreuz weiss nicht selber wo es ist), oder man erzeugt für jedes Kreuz eine eigenständige Komponente. Du hast jetzt beides gemacht, und das funktioniert halt nicht weil du 2 verschiedene Koordinatensysteme vermischt

 @Override
    protected void paintComponent(Graphics g) {
        super.paintComponent(g);
        int h = getHeight(); 
        int w = getWidth(); 
        int centerx = w/2;
        int centery = h/2;
        g.drawLine(0, centery, w, centery);
        g.drawLine(centerx, 0, centerx, h);
    }

...und das dann dieses kleine Koordinatensystem im Fenster wiederum an der richtigen Stelle platziert ist, dafür hast du ja per setBounds mit X und Y gesorgt.

PS: Ein repaint() ist nicht nötig nach dem adden. Wobei beim null-Layout bin ich mir nicht sicher. Aber versuch's mal ohne.

PPS: Du verdrehst die Namenskonventionen von Klassen und Variablen.
Variablen schreibt man in camelCase, und Klassen in CamelCase, du hast es weitestgehend genau andersherum gemacht.

 

[André Uhres]

Ein kleines Beispiel:

import java.awt.*;
import java.awt.event.*;
import javax.swing.*;
public class DrawingDemo extends JFrame {
    private final JPanel panel;
    public DrawingDemo() {
        super("Drawing Demo");
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        setSize(400, 300);
        setLocationRelativeTo(null);
        panel = new JPanel(null);
        panel.setBackground(Color.WHITE);
        add(panel);
        panel.addMouseListener(new MouseAdapter() {
            @Override
            public void mousePressed(final MouseEvent e) {
                Point location = e.getPoint();
                createPoint(location);
            }
        });
    }
    public void createPoint(final Point location) {
        DrawPoint drawPoint = new DrawPoint(location);
        panel.add(drawPoint, 0);
        drawPoint.setForeground(Color.LIGHT_GRAY);
        drawPoint.requestFocusInWindow();
    }
    public static void main(final String[] args) {
        Runnable gui = new Runnable() {
            @Override
            public void run() {
                try {
                    UIManager.setLookAndFeel(UIManager.getSystemLookAndFeelClassName());
                } catch (Exception ex) {
                    ex.printStackTrace();
                }
                new DrawingDemo().setVisible(true);
            }
        };
        //GUI must start on EventDispatchThread:
        SwingUtilities.invokeLater(gui);
    }
}
class DrawPoint extends JComponent {
    private final JPopupMenu popup;
    private final int SIZE = 10;
    private final Point location;
    public DrawPoint(final Point location) {
        this.location = location;
        setBounds(location.x - SIZE / 2, location.y - SIZE / 2, SIZE + 1, SIZE + 1);
        setCursor(Cursor.getPredefinedCursor(Cursor.HAND_CURSOR));
        ToolTipManager.sharedInstance().registerComponent(this);
        popup = new JPopupMenu();
        popup.add(new JMenuItem("Test 1"));
        popup.add(new JMenuItem("Test 2"));
        setComponentPopupMenu(popup);
        addFocusListener(new FocusListener() {
            public void focusGained(final FocusEvent e) {
                repaint();
            }
            public void focusLost(final FocusEvent e) {
                repaint();
            }
        });
        addMouseListener(new MouseAdapter() {
            @Override
            public void mousePressed(final MouseEvent e) {
                getParent().setComponentZOrder(DrawPoint.this, 0);
                requestFocusInWindow();
            }
        });
    }
    @Override
    public String getToolTipText(final MouseEvent event) {
        return location.toString();
    }
    @Override
    protected void paintComponent(final Graphics g) {
        super.paintComponent(g);
        //set color:
        g.setColor(getForeground());
        if(hasFocus()){
            g.setColor(getForeground().darker().darker());
        }
        //paint cross:
        g.drawLine(0, SIZE / 2, SIZE, SIZE / 2);
        g.drawLine(SIZE / 2, 0, SIZE / 2, SIZE);
    }
}

 

[Tankpope]

Thx war eigendlich Ziemlich eindeutig woran es lag. Dum von mir. Aber Jetzt klapts.