Probleme mit repaint / paint

[granini]

Hallo zusammen,

ich habe ein kleines Testprogramm mit welchem ich Punkte bei Mausklick in eine ArrayList speichere und auf einer Zeichenfläche ausgeben möchte. Nur klappt bei mir das ganze nicht wenn ich repaint und die Methode paint nutzen möchte. Wer kann mir sagen woran es scheitert?

public class Flaeche {
    
    JFrame frame = new JFrame("Test");
    public ArrayList<Point> pts = new ArrayList<Point>();
    public Point p;
    
    public Flaeche(){
	
	JMenuBar menu = new JMenuBar();
	JMenu go = new JMenu("Go");
	JMenuItem go1 = new JMenuItem("Go");
	go.add(go1);
	menu.add(go);
	
	go1.addActionListener(new ActionListener() {
		public void actionPerformed(ActionEvent e) {
			newPoints(frame);
		}
	});
	frame.setJMenuBar(menu);
	frame.setDefaultCloseOperation(JFrame.DO_NOTHING_ON_CLOSE);
	frame.setSize(1024, 768);
	frame.setLocation(150, 100);
	frame.setVisible(true);
	
    }
    
    public void newPoints(JFrame f){
	f.getContentPane().addMouseListener(new MouseAdapter(){
	    public void mousePressed(MouseEvent e){
		addP(e.getX(),e.getY());
		repaint();
	//	Graphics g = ((JPanel) e.getComponent()).getGraphics();
		
	//	g.setColor(Color.BLACK);
	//	g.fillRect(p.x, p.y, 7, 7);
	    }
	});
    }
    public void paint(Graphics g){
	super.paint(g);
	g.setColor(Color.BLACK);
	g.fillRect(p.x, p.y, 7, 7);
	
    }
    public void addP(int x, int y){
	p = new Point(x,y);
	pts.add(p);
    }  
}

Viele Grüße
granini

 

[Marco13]

Bisher gibt es keinen Anlass, paint aufzurufen - und bei dem super.paint müßte bisher ein Fehler sein...

public class Flaeche extends JPane {
    
    public Flaeche(){
        ...
	frame.setSize(1024, 768);

        frame.getContentPane().add(this);

        ...
    }
    
    public void paintComponent(Graphics g){
	super.paintComponent(g);
	g.setColor(Color.BLACK);

        // Hier mit einer for-Schleife über alle punkt gehen
	g.fillRect(p.x, p.y, 7, 7);
	
    }
}

 

[granini]

Danke erstmal. Ich habe die Anpassungen vorgenommen. Aber nach wie vor wird beim Aufrufen der Methode repaint() die Methode paintComponent nicht angesprochen..

 public void newPoints(JFrame f){
	f.getContentPane().addMouseListener(new MouseAdapter(){
	    public void mousePressed(MouseEvent e){
		addP(e.getX(),e.getY());
		repaint();
	    }
	});
    }
    public void paintComponent(Graphics g){
	super.paintComponent(g);
	System.out.println("test");
	g.setColor(Color.BLACK);
	for (int i = 0; i < pts.size();i++ ){
	    p = pts.get(i);
	    g.fillRect(p.x, p.y, 7, 7);
	}

 

[Marco13]

Ja, der MouseListener hing am ContentPane - er müßte aber an der Zeichenfläche hängen. (Die MouseEvents kommen ja nicht mehr bei der ContentPane an, wenn in (""über"") der ContentPane die Zeichenfläche liegt.

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

public class Flaeche extends JPanel
{

    JFrame frame = new JFrame("Test");
    public ArrayList<Point> pts = new ArrayList<Point>();
    public Point p;

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

    public Flaeche()
    {

        JMenuBar menu = new JMenuBar();
        JMenu go = new JMenu("Go");
        JMenuItem go1 = new JMenuItem("Go");
        go.add(go1);
        menu.add(go);

        go1.addActionListener(new ActionListener()
        {
            public void actionPerformed(ActionEvent e)
            {
                newPoints();
            }
        });
        frame.setJMenuBar(menu);
        frame.setDefaultCloseOperation(JFrame.DO_NOTHING_ON_CLOSE);
        frame.setSize(1024, 768);
        frame.getContentPane().add(this);
        frame.setLocation(150, 100);
        frame.setVisible(true);

    }

    public void newPoints()
    {
        addMouseListener(new MouseAdapter()
        {
            public void mousePressed(MouseEvent e)
            {
                addP(e.getX(), e.getY());
                repaint();
                //  Graphics g = ((JPanel) e.getComponent()).getGraphics();

                //  g.setColor(Color.BLACK);
                //  g.fillRect(p.x, p.y, 7, 7);
            }
        });
    }

    public void paintComponent(Graphics g)
    {
        super.paintComponent(g);
        System.out.println("test");
        g.setColor(Color.BLACK);
        for (int i = 0; i < pts.size(); i++)
        {
            p = pts.get(i);
            g.fillRect(p.x, p.y, 7, 7);
        }
    }

    public void addP(int x, int y)
    {
        p = new Point(x, y);
        pts.add(p);
    }
}

EDIT: BTW: Das hier
// Graphics g = ((JPanel) e.getComponent()).getGraphics();
läßt du mal schön bleiben!

 

[granini]

Alles klar. Danke!

Kannst du mir noch kurz was zu deinem Edit sagen. Unsauber, ein Do not oder warum ist das nicht zu empfehlen? Funktion war zumindest gegeben.. ;-)

 

[Marco13]

getGraphics auf einer Component ist in den ALLERmeisten Fällen falsch (nur ganz selten z.B. im Zusammenhang mit "active rendering" sinnvoll). Die Faustregel ist: ALLES was gezeichnet wird, muss von der paintComponent-Methode aus gezeichnet werden (und zwar in das Graphics, das man dort übergeben bekommt).

BTW: "Unsauber" sind auch am zuletzt geposteten noch einige Punkte. Z.B. sollte das GUI im Event-Dispatch-Thread gebaut werden, und... wenn man 10x "Go" macht, werden 10 Listener hinzugefügt und so...

 

[André Uhres]

//	Graphics g = ((JPanel) e.getComponent()).getGraphics();

Ungeachtet dessen, wie ein Malantrag ausgelöst wird, benutzt Swing einen "callback" ("Wiederholungsbesuch") Mechanismus zum Zeichnen. Das heißt, daß ein Programm den Darstellungscode der Komponente innerhalb einer bestimmten überschriebenen Methode setzen sollte, und das Toolkit ruft diese Methode auf, wenn es Zeit ist zu malen. Die zu überschreibende Methode ist in javax.swing.JComponent:

protected void paintComponent(Graphics g)

Wenn Swing diese Methode aufruft, ist der Parameter "Graphics g" mit dem passenden Zustand für das Zeichnen auf dieser bestimmten Komponente vorkonfiguriert:

* Die Farbe des Graphics Objektes wird auf die foreground-Eigenschaft der Komponente eingestellt.
* Die Schriftart des Graphics Objektes wird auf die font-Eigenschaft der Komponente eingestellt.
* Die "translation" des Graphics Objektes wird so eingestellt, daß die Koordinate (0,0) die obere linke Ecke der Komponente darstellt.
* Das "clip"-Rechteck des Graphics Objektes wird auf den Bereich der Komponente eingestellt, der neu gezeichnet werden muss.

Programme müssen dieses Graphics Object verwenden (oder ein von ihm abgeleitetes) um die Oberfläche darzustellen. Sie sind frei, den Zustand des Graphics Objektes so zu ändern, wie es benötigt wird.