paintComponent wird nicht aufgerufen

[kulturfenster]

Guten Abend,
Ich habe 2 Klassen,
Klasse A mit Frame, Panel, MouseListener,
Klasse B mit panel und paintComponent.

Nun möchte ich, dass ich 2x auf das Frame von Klasse A klicken kann und mir die Klasse B dann einen Strich zeichnet. Leider funktioniert das nur, wenn ich im Konstruktor der Klasse B ein Frame kreiere, welches ich dann gar nicht verwendet.

Hier nun die Klasse B:

public class PaintPanel extends JPanel
{
	private Line2D.Double line;
	
	public PaintPanel(Point2D.Double p1, Point2D.Double p2)
	{
		line = new Line2D.Double(p1,p2);
		JFrame frame = new JFrame("Tester");
/*		
		frame.add(this);	
		frame.setSize(200,200);
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		frame.setVisible(true);
*/
		repaint();
	}
	
	public void paintComponent(Graphics g)
	{
		super.paintComponent(g);
		Graphics2D g2 = (Graphics2D) g;
		g2.draw(line);
		System.out.println("zeichnet!");
	}
	
	public JPanel getPanel()
	{
		return this;
	}
}

Falls ich wie oben im Code das JFrame ausklammere, wird die paintComponent Methode nie aufgerufen. Sonst schon und das Zeichnen funktioniert wunderbar.

 frame.setVisible(false);

wäre hier wohl weniger angebracht.. :wink:

 

[Ebenius]

Bitte mach mal ein vollständiges, kompilier- und ausführbares Beispiel! Was Du da schreibst ergibt keinen Sinn.

 

[Ariol]

Die Methode heisst auch paintComponents()

 

[hdi]

Nein sie heisst paintComponent(Graphics)

 

[kulturfenster]

mit oder ohne s spielt keine Rolle, das Resultat bleibt gleich.

hier nun meine beiden vollständigen Klassen:

import javax.swing.JFrame;
import javax.swing.JPanel;

import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.BorderLayout;

import java.awt.Rectangle;

import java.awt.event.MouseListener;
import java.awt.event.MouseEvent;

import java.awt.geom.Point2D;
import java.awt.geom.Line2D;

import java.util.ArrayList;


public class Gui extends JFrame {

	private JFrame frame;
	private JPanel paintPanel;
	private JPanel mainPanel;
	
	
	private Point2D.Double p1;
	private Point2D.Double p2;

	final private ArrayList<Point2D.Double> aop = new ArrayList<Point2D.Double>();
		
	public Gui()
	{
		mainPanel = new JPanel();
		mainPanel.setLayout(new BorderLayout());
	
		class MousePressListener implements MouseListener
		{
	
			public void mousePressed(MouseEvent e)
			{
				int x = e.getX();
				int y = e.getY();
				p1 = new Point2D.Double(x,y);
				aop.add(p1);
				
				if (aop.size() == 2)
				{						
					PaintPanel pp = new PaintPanel(aop.get(0),aop.get(1));
					paintPanel = pp.getPanel();
					mainPanel.add(paintPanel, BorderLayout.CENTER);
				}
			}
	
			public void mouseReleased(MouseEvent e) {}
			public void mouseClicked(MouseEvent e) {}
			public void mouseEntered(MouseEvent e) {}
			public void mouseExited(MouseEvent e) {}
		}
		
		MouseListener listener = new MousePressListener();
		mainPanel.addMouseListener(listener);
		
		setFrame();	
	}
	
	public void setFrame()
	{		
		this.add(mainPanel);
		this.setTitle("Shape Printer");
		this.setSize(300,300);
		this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		this.setVisible(true);
		
	}

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

und noch diejenige, welches den Strich zeichnen soll.

import javax.swing.JFrame;
import javax.swing.JPanel;
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.Rectangle;

import java.awt.event.MouseListener;
import java.awt.event.MouseEvent;

import java.awt.geom.Point2D;
import java.awt.geom.Line2D;

import java.util.ArrayList;

public class PaintPanel extends JPanel
{
	private Line2D.Double line;
	
	public PaintPanel(Point2D.Double p1, Point2D.Double p2)
	{
		line = new Line2D.Double(p1,p2);
		JFrame frame = new JFrame("Tester");
	/*
		frame.add(this);	
		frame.setSize(200,200);
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		frame.setVisible(true);
*/
		repaint();
	}
	
	public void paintComponents(Graphics g)
	{
		super.paintComponent(g);
		Graphics2D g2 = (Graphics2D) g;
		g2.draw(line);
		System.out.println("zeichnet!");
	}
	
	public JPanel getPanel()
	{
		return this;
	}
}

 

[hdi]

Achso gibt es Leute die dafür paintComponents() nutzen?
Ist paintComponent nicht besser? Keine Ahnung, es ist halt JComponent, ist für ein Panel
vllt besser als eine Container-Methode?

Wo isn da der Unterschied eigentlich ???:L

 

[Ebenius]

paintComponent ohne 's'. Die mit s zeichnet die Kinder. Das will man nicht überschreiben.

 

[Ariol]

Ok, ich bring die immer durcheinander, aber JFrame hat nach dem hier keine paintComponent()...

http://java.sun.com/javase/6/docs/api/javax/swing/JFrame.html

Daher hab ich das wieder..

 

[Ebenius]

Ok, ich bring die immer durcheinander, aber JFrame hat nach dem hier keine paintComponent()...

http://java.sun.com/javase/6/docs/api/javax/swing/JFrame.html

Augen auf: Gezeichnet wird hier in einem JPanel (siehe oben), welches ─ im Gegensatz zu JFrame ─ von JComponent erbt.

 

[Ebenius]

Zurück zum Problem... Ich habe Deinen Code kaputt gemacht. Jetzt funktioniert's:

import java.awt.BorderLayout;
import java.awt.event.MouseEvent;
import java.awt.event.MouseListener;
import java.awt.geom.Point2D;
import java.util.ArrayList;

import javax.swing.JFrame;
import javax.swing.JPanel;

public class Gui extends JFrame {

  private JFrame frame;
  private JPanel paintPanel;
  private JPanel mainPanel;

  private Point2D.Double p1;
  private Point2D.Double p2;

  final private ArrayList<Point2D.Double> aop =
        new ArrayList<Point2D.Double>();

  public Gui() {
    mainPanel = new JPanel();
    mainPanel.setLayout(new BorderLayout());

    class MousePressListener implements MouseListener {

      public void mousePressed(MouseEvent e) {
        int x = e.getX();
        int y = e.getY();
        p1 = new Point2D.Double(x, y);
        aop.add(p1);

        if (aop.size() == 2) {
          PaintPanel pp = new PaintPanel(aop.get(0), aop.get(1));
          paintPanel = pp.getPanel();
          mainPanel.add(paintPanel, BorderLayout.CENTER);
          mainPanel.revalidate();
          mainPanel.repaint();
        } else if (paintPanel != null) {
          paintPanel.repaint();
        }
      }

      public void mouseReleased(MouseEvent e) {}

      public void mouseClicked(MouseEvent e) {}

      public void mouseEntered(MouseEvent e) {}

      public void mouseExited(MouseEvent e) {}
    }

    MouseListener listener = new MousePressListener();
    mainPanel.addMouseListener(listener);

    setFrame();
  }

  public void setFrame() {
    this.add(mainPanel);
    this.setTitle("Shape Printer");
    this.setSize(300, 300);
    this.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    this.setVisible(true);

  }

  public static void main(String args[]) {

    new Gui();
  }
}

import java.awt.Color;
import java.awt.Graphics;
import java.awt.Graphics2D;
import java.awt.geom.Line2D;
import java.awt.geom.Point2D;

import javax.swing.JFrame;
import javax.swing.JPanel;

public class PaintPanel extends JPanel {

  private Line2D.Double line;

  public PaintPanel(Point2D.Double p1, Point2D.Double p2) {
    line = new Line2D.Double(p1, p2);
  }

  @Override
  public void paintComponent(Graphics g) {
    super.paintComponent(g);
    Graphics2D g2 = (Graphics2D) g;
    g2.setColor(Color.BLUE);
    g2.draw(line);
    System.out.println("zeichnet!");
  }

  public JPanel getPanel() {
    return this;
  }

  public static void main(String[] args) {
    PaintPanel pp =
          new PaintPanel(new Point2D.Double(10, 20), new Point2D.Double(50,
                60));
    JFrame frame = new JFrame("tester2");
    JPanel panel = new JPanel();
    panel = pp.getPanel();
    frame.add(panel);
    frame.setSize(200, 300);
    frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
    frame.setVisible(true);
  }

}

Wichtig sind Zeilen 38 bis 41! [ edit ]Bezieht sich auf den ersten Code-Block in dieser Antwort!

Ebenius

 

[Ariol]

Ok, ich bring die immer durcheinander, aber JFrame hat nach dem hier keine paintComponent()...

http://java.sun.com/javase/6/docs/api/javax/swing/JFrame.html

Augen auf: Gezeichnet wird in einem JPanel, welches -- im Gegensatz zu JFrame -- von JComponent erbt.

Aaaaargh


Wenn man mal da nachsieht wo man soll..... :###

 

[André Uhres]

Etwas vereinfacht könnte es auch so gehen:

import java.awt.*;
import java.awt.event.*;
import java.util.*;
import java.util.List;
import javax.swing.*;
public class Gui extends JFrame {
    public Gui() {
        super("Shape Printer");
        setSize(300, 300);
        setLocationRelativeTo(null);
        setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        add(new PaintPanel(), BorderLayout.CENTER);
    }
    public static void main(final String[] args) {
        Runnable gui = new Runnable() {
            public void run() {
                new Gui().setVisible(true);
            }
        };
        //GUI must start on EventDispatchThread:
        SwingUtilities.invokeLater(gui);
    }
}
class PaintPanel extends JPanel {
    final private List<Point> aop;
    public PaintPanel() {
        aop = new ArrayList<Point>();
        MouseListener listener = new MousePressListener();
        addMouseListener(listener);
    }
    @Override
    public void paintComponent(final Graphics g) {
        super.paintComponent(g);
        g.setColor(Color.BLUE);
        Point p1 = null;
        for (Point p : aop) {
            if (p1 == null) {
                p1 = p;
            } else {
                g.drawLine((int) p1.x, (int) p1.y, (int) p.x, (int) p.y);
                p1 = null;
            }
        }
    }
    class MousePressListener extends MouseAdapter {
        @Override
        public void mousePressed(final MouseEvent e) {
            aop.add(new Point(e.getX(), e.getY()));
            repaint();
        }
    }
}

 

[Ebenius]

Da fällt mir grad ein, dass es ─ in beiden Beispielen ─ immer nur eine Linie geben wird. Wer findet raus warum?

 

[André Uhres]

Da fällt mir grad ein, dass es ─ in beiden Beispielen ─ immer nur eine Linie geben wird. Wer findet raus warum?

In meinem gibt's soviel Linien wie die Anzahl der Mausklicks geteilt durch zwei.

 

[Ebenius]

Da fällt mir grad ein, dass es ─ in beiden Beispielen ─ immer nur eine Linie geben wird. Wer findet raus warum?

In meinem gibt's soviel Linien wie die Anzahl der Mausklicks geteilt durch zwei.

Stimmt... Ich sollte in's Bett gehen, wenn ich das nicht gleich sehe.

 

[Ebenius]

André... Dann antworte doch hier noch: Quiz.

 

[André Uhres]

André... Dann antworte doch hier noch: Quiz.

Du bist ja immer noch nicht im Bett :lol: