Kleine Panels auf einem großen Panel frei platzieren

[RedNifre]

Hallo,

Ich habe gerade ein paar Probleme, eine GUI so hinzubiegen, wie ich sie gerne hätte. (Was GUIs betrifft bin ich Anfänger)

Das Ziel:
In der GUI ist eine große Fläche, auf die man per Mausklick Kreise setzen kann. Später soll man diese auch verschieben können.

Meine Idee:
Da man die Kreise später noch auswählen, verschieben, umfärben etc. können soll dachte ich mir, dass ich sie am besten als eigene Panels implementiere, die auf dem großen Hintergrundpanel liegen.

Die Probleme:
Man sieht nichts. /
Ich weiß auch nicht genau, an welcher Stelle im kleinen Panel der Code zum Zeichnen hin soll. Ich habe mal etwas Debug-Code in "paint", "repaint" und "paintComponent" geschrieben, aber anscheinend wird keine dieser Methoden aufgerufen.

Wie schaffe ich es, dass die kleinen Panels sagen wir mal 20x20 Pixel groß mit einem schwarzen Kreis in der Mitte angezeigt werden?

 

[SlaterB]

poste deinen bisherigen Code, dann kann man ihn zurechtbiegen,

von null auf oder in Textform alle relevanten Befehle aufzuführen ist nicht zielführend

 

[RedNifre]

Okay.
Um das Verständnis zu erleichtern hier noch ein paar Details: Das Programm soll ein Graph-Editor werden. Meine ursprüngliche Version bestand nur aus einem großen Panel, welches in einer riesigen Methode alles direkt gemalt hat, woraus der Graph, bestand, etwa so:

 [b]ALTER CODE[/b]
public class GPanel extends JPanel implements MouseListener {
	[...]
	public void paintComponent(Graphics comp) {
	[...]
			Graphics2D comp2D = (Graphics2D)comp;
	[...]
				comp2D.drawLine(
							(int)(offX+ squareSize*(sol.nodes.elementAt(e).getX() + SHADOWOFFSET)),
							(int)(offY+ squareSize*(sol.nodes.elementAt(e).getY() + SHADOWOFFSET)),
							(int)(offX+ squareSize*(sol.nodes.elementAt((e+1)%sol.nodes.size()).getX() + SHADOWOFFSET)),
							(int)(offY+ squareSize*(sol.nodes.elementAt((e+1)%sol.nodes.size()).getY() + SHADOWOFFSET))
					);
[...]

Weil mich diese Herangehensweise irgendwie in eine Sackgasse gebracht hat versuche ich es jetzt anders. Ich habe jetzt eine eigene Knotenklasse, die von JPanel erbt und deren Instanzen auf dem GPanel liegen sollen. Hier mal beide Klassen, wobei der Code noch ganz am Anfang steht.

public class GPanel extends JPanel {
	[...]
	public GPanel () {
		setLayout(null);
		
		
		Dimension d = new Dimension(100,100);
		setPreferredSize(d);
		setMinimumSize(d);  // soll es auf feste größe setzen
		setMaximumSize(d);
		

// ist vielleicht quatsch:
		insets = this.getInsets();
		this.setSize(300 + insets.left + insets.right,
		              125 + insets.top + insets.bottom);
		
		
		addMouseListener(new MouseListener() {

			public void mouseClicked(MouseEvent e) {
				// Problem: Can't click while moving the mouse (Windows)
				// use mousePressed instead
			
			}

			public void mouseEntered(MouseEvent e) {
				// TODO Auto-generated method stub
				
			}

			public void mouseExited(MouseEvent e) {
				// TODO Auto-generated method stub
				
			}

			public void mousePressed(MouseEvent e) {
				System.out.println("Test");
				System.out.println("getX: "+e.getX() +"\ngetXOnScreen: "+e.getXOnScreen()
					+"\ngetY: "+	e.getY() +"\ngetYOnScreen: " +e.getYOnScreen());
				
				
				addNode(e.getX()<<4, e.getY()<<4); 
					
				
				
				//repaint();
				
				
			}

			public void mouseReleased(MouseEvent e) {
				// TODO Auto-generated method stub
				
			}
			
			
			
			
		});

	[...]

	public void addNode(int nodeX, int nodeY) {
		Node newnode = new Node(nodeX,nodeY);
		
		
		//newnode.setBounds(10, 20, 30, 40);
		
		nodes.add(newnode);
		
		
		add(newnode);
		
		Dimension size = new Dimension(20,20);
		
		newnode.setBounds(25 + insets.left, 5 + insets.top, 20,20);
		
		validate();
		newnode.repaint();
		this.repaint();
		
	//	newnode.setBounds(10, 20, 30, 40);
	//	newnode.repaint();
	//	repaint();
		
		
		//insertComponent(newnode);
		
		
		//this.validate();
		
		
		//this.updateUI();
		//SwingUtilities.updateComponentTreeUI(this);
		
		
		
	
	}
	
}

In der Knoten-Panel-Klasse Node habe ich momentan nur einfache Debug-Messages in die Methoden paint, repaint und paintComponent geschrieben, welche aber nicht ausgegeben werden. Mir ist das Verhältnis dieser Methoden untereinander auch nicht klar. Wo sollte ich denn den Knoten-Zeichnen-Code am besten hinschreiben?

 

[Gelöschtes Mitglied 9001]

Hallo,

ich habe mir Deinen Code zwar noch nicht bis ins Detail angesehen, aber zumindest Deine letzte Frage kann ich beantworten:
Für eigene Zeichenoperationen wird die paint-Methode überschrieben. Dort hinein kommt Dein Zeichencode.

 

[SlaterB]

setBounds hilft nur beim null-Layout, dabei muss nicht die Komponente selber das null-Layout haben,
sondern natürlich der Container, in dem man die Komponente einfügt

und wenn du noch Probleme hast, die Komponente überhaupt zu sehen,
dann würde ich

comp2D.drawLine(5,5,20,20); schreiben und nicht 3 Zeilen verrücken Code, der 10 Fehler versteckt enthalten könnte

auch MouseListener usw. ist doch nciht nötig,
mit kleinen kurzen Programm anfangen

 

[RedNifre]

Danke für den Tipp! Die beiden anderen Methoden (repaint und paintCompontent) habe ich erstmal gelöscht.
Momentan sieht die paint-Methode im Node nur so aus:

public class Node extends JPanel{
	[...]
	public void paint() {
		System.err.println("== sapperlot");
	}

Wird aber leider nicht auf der Kommandozeile ausgegeben. :-/

 

[SlaterB]

hast du mein Posting gelesen? es fehlt in deinem Code übrigens noch ne main-Operation, ein JFrame usw

und verwende
public void paintComponent(Graphics g) {
super.paintComponent(g)

// dein Code
}

 

[RedNifre]

hast du mein Posting gelesen?

Nein, hatte deinen Kommentar nicht gesehen, da du ihn geschrieben hast während ich gerade meinen schrieb. :wink:

und wenn du noch Probleme hast, die Komponente überhaupt zu sehen,
dann würde ich

comp2D.drawLine(5,5,20,20); schreiben und nicht 3 Zeilen verrücken Code, der 10 Fehler versteckt enthalten könnte

Das große Panel sehe ich ja. Das Problem ist halt, dass im kleinen Panel die paint-Methode garnicht aufgerufen wird.

auch MouseListener usw. ist doch nciht nötig,
mit kleinen kurzen Programm anfangen

Dochdoch, das ist schon nötig, schließlich sollen die kleinen Panels ja da erstellt werden wo man hinklickt.

es fehlt in deinem Code übrigens noch ne main-Operation, ein JFrame usw

Nein, fehlt nicht, das steht einfach nur ganz woanders. Das GPanel wird schon korrekt in der GUI angezeigt. Mir geht es nur um die kleineren Nodes (extends JPanel), die ich per Mausklick frei auf dem GPanel platziert können möchte.

und verwende
public void paintComponent(Graphics g) {
super.paintComponent(g)

// dein Code
}

Habe ich gerade gemacht. Was bewirkt das?

 

[SlaterB]

das ist einfach nur der Standard beim Überschreiben

http://java.sun.com/docs/books/tutorial/uiswing/painting/step2.html

ich wies darauf hin, weil ich den paint-Tipp von Rajmund eben nicht für den Standard hielt
ist evtl. Geschmackssache


> Nein, fehlt nicht, das steht einfach nur ganz woanders.

wie gesagt: wenn du hier ein Problem ausführlich besprechen willst,
wäre es von Vorteil, allen Code zu posten und dabei natürlich möglichst wenig,
kleine neue komplette Testklassen unabhängig von deinem restlichen Programm, welches zu 90% erstmal gar nicht interessiert

> Dochdoch, das ist schon nötig, schließlich sollen die kleinen Panels ja da erstellt werden wo man hinklickt.

aber nicht im ersten Schritt wenn du das JPanel noch nicht mal sehen kannst..

 

[Marco13]

Ham jetzt grad keine Lust, länger zu antworten, aber als kurzen Einwurf: Eine Knoten-Klasse (die erstmla nicht viel merh macht, als einen Kreis und vielleicht ein bisserl Text zu zeichnen) von einem JPanel abzuleiten ist Humbug.

 

[RedNifre]

Danke für die Hilfen, ich habs jetzt geschafft; die kleinen Panels sind sichtbar.

Ich hab auch herausgefunden, was das super.paintComponent(comp); tut: Weil die Superklasse das JPanel ist, füllt dieser Aufruf das Panel komplett grau aus. Da ich aber keine Kreise auf grauen Quadraten zeichnen möchte lasse ich diesen Aufruf besser weg.

Jetzt habe ich aber schon das nächste Problem: Ich möchte die Knoten im Bild (Die kleinen Panels) anklickbar machen. Dafür habe ich auch hier einen MouseListener angehängt. Leider landen die Mausklicks aber immer beim Container, also dem GPanel (Hintergrundfläche, auf der die Nodes liegen).
Was muss ich tun, damit ich die Panels im Container anklicken kann, anstatt den Container selbst?[/i]

 

[RedNifre]

@Marco13:
Was wäre hier der bessere Ansatz?