JPanel: Zeichnen funktioniert nicht - Nur grauer Hintergrund

[AlexDozer]

Hi, ich soll ein Programm schreiben was Linien zeichnet wenn man die Maus drückt. Also vom letzten Mausklick zum aktuellen Mausklick.

Im Prinzip nicht schwer. Mein Problem ist jetzt das das JPanel nach überschreiben der paintComponent-Methode grau ist und nichts passiert. Wenn ich die Methode weglasse ist es schwarz wie im Konstruktor definiert.

Was mache ich falsch?

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

public class Punkte extends JPanel {
	
	public class Maus extends MouseAdapter {
		
		public void mousePressed (MouseEvent e)  {
			
			x2 = e.getX();
			y2 = e.getY();
						
			repaint();
		}
	}
			
	public int x1,x2,y1,y2 = 0;
		
	Punkte () {
		
		setBackground(Color.black);
		setForeground(Color.yellow);
	}
		
	public static void main(String [] args) {
		
		Punkte meinePunkte = new Punkte();
		
		JFrame frame = new JFrame("Punkte verbinden");
			
		frame.setSize(500, 500);
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		frame.add(meinePunkte);
		frame.setVisible(true);
	}
	
	public void paint(Graphics g){
				
		g.drawLine(x1, y1, x2, y2);
		x1=x2;
		y1=y2;
	
	}
}

Schon mal vielen Dank im Voraus!


Gruß Alex

 

[SlaterB]

public class Punkte
    extends JPanel
{

    public class Maus
        extends MouseAdapter
    {
        public void mousePressed(MouseEvent e)
        {
            System.out.println("geht");
            x1 = x2; // !!!<-----------------
            y1 = y2;
            x2 = e.getX();
            y2 = e.getY();

            repaint();
        }
    }

    public int x1, x2, y1, y2 = 0;

    Punkte()
    {
        addMouseListener(new Maus()); // !!!<-----------------
        setBackground(Color.black);
        setForeground(Color.yellow);
    }

    public static void main(String[] args)
    {

        Punkte meinePunkte = new Punkte();

        JFrame frame = new JFrame("Punkte verbinden");

        frame.setSize(500, 500);
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.add(meinePunkte);
        frame.setVisible(true);
    }

    public void paint(Graphics g)
    {
        super.paint(g);  // !!!<-----------------
        g.drawLine(x1, y1, x2, y2);


    }
}

 

[doctus]

1. die zu überschreibende methode ist paintComponent, nicht paint
2. im konstruktor addMouseListener(new Maus()); aufrufen, sonst kommen MouseEvents nicht an.

lg doctus

 

[AlexDozer]

Vielen Dank euch erstmal. Hab das ganze jetzt mal geändert.

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

public class Punkte extends JPanel {
	
	public class Maus extends MouseAdapter {
		
		public void mousePressed (MouseEvent e)  {
			x1 = x2;
            y1 = y2;
            x2 = e.getX();
			y2 = e.getY();
						
			repaint();
		}
	}
			
	public int x1,x2,y1,y2 = 0;
		
	Punkte () {
		addMouseListener(new Maus()); 
		setBackground(Color.black);
		setForeground(Color.yellow);
	}
		
	public static void main(String [] args) {
		
		Punkte meinePunkte = new Punkte();
		
		JFrame frame = new JFrame("Punkte verbinden");
			
		frame.setSize(500, 500);
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		frame.add(meinePunkte);
		frame.setVisible(true);
						
	}
	
	protected void paintComponent(Graphics g){
		
		super.paint(g);
		g.drawLine(x1, y1, x2, y2); 
		
		
	}
}

Jetzt bringt er mir aber ne Exception: #

at Punkte.paintComponent(Punkte.java:42)
	at javax.swing.JComponent.paint(Unknown Source)

Wieso kennt er die Quelle nicht?

 

[SlaterB]

weil du den Source-Code der Java-API nicht eingebunden hast,

-----

so erzeugst du eine Endlosschleife,
wenn du paintComponent() überschreibst, dann selbstverständlich
super.paintComponent(g);
aufrufen

 

[AlexDozer]

Vielen Dank! Es funktioniert

Es wird zwar jedesmal der alte Strich gelöscht aber das liegt wahrscheinlich daran das ich ja jedesmal die paintComponent-Methode der Elternklasse aufrufe. Da werd ich noch ne If-Abfrage einbauen müssen. Oder geht das eleganter?

Gruß Alex

 

[AlexDozer]

Hab es so gelöst:

		if (x1 == 0 && y1 == 0)
			super.paintComponent(g);

 

[SlaterB]

einfach über den vorherigen Zustand überzumalen ist nie eine gute Idee,
wer weiß was da zwischendurch passiert, Überdeckung durch andere Fenster usw.

zwei Wege:
1.)
alle Striche in einer Liste merken und jedes Mal Bild von Anfang an neu aufbauen
2.)
auf ein Image malen und auf dem Bildschirm immer das Image zeichen,
da geht das sukzessive Zeichnen, da kann dir niemand dazwischenfunken
und jedes normale paint ist korrekt vollständig (es wird das ganze Image auf den Bildschirm gemalt)

erfordert natürlich höhere Kenntnisse

 

[AlexDozer]

Alles klar. Werd ich mal probieren.

Ich hab das ganze jetzt nochmal mit Canvas versucht. Hier wird ja anscheinend immer das Canvas komplett neu gezeichnet d.h. die alten Striche werden immer entfernt. Ist es hier möglich dies zu unterbinden oder komme ich hier an die von dir genannten Methoden nicht vorbei?

 

[AlexDozer]

Habs selber herausgefunden. Man muss die Methode update() von Canvas überschreiben. Dann wird nicht jedesmal alles neu gezeichnet:

	public void update (Graphics g){
		paint(g);
	}