Problem beim erstellen eines JPanel in einem JFrame

[Protex]

hi, ich würde gerne in einer JFrame einen JPanel setzen. Da ich in der JPanel div. operationen möglich sein sollen dachte ich mir ich schreibe ne klasse "ZeichneBrett" und implementiere in diese klasse die gewünschten listener.. aber wenn ich dann in der main methode die klasse "ZeichneBrett" aufrufe erscheint gar nichts..

Dies wäre der Quelltext:

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

public class GehVersuch {

	
	public static class ZeichneBrett extends JPanel implements MouseListener, MouseMotionListener{
		
		JLabel label;
		
		public void raster(){
			
			JPanel panel = new JPanel();
			label = new JLabel("Hier sollen mal die Koordinaten stehen");
			
			
			panel.setLayout(new SpringLayout());
			SpringLayout layout = new SpringLayout();
			
			panel.add(label);
			layout.putConstraint(SpringLayout.EAST, label, 10, SpringLayout.EAST, panel);
			
			panel.addMouseListener(this);
			panel.addMouseMotionListener(this);
			
			panel.setSize(600, 400);
			panel.setVisible(true);
			
		}	
				
		
		
		public void mouseClicked(MouseEvent e){			
		}
		public void mouseReleased(MouseEvent e){			
		}
		public void mousePressed(MouseEvent e){			
		}
		public void mouseExited(MouseEvent e){			
		}
		public void mouseEntered(MouseEvent e){			
		}
		public void mouseDragged(MouseEvent e){		
		}
		public void mouseMoved(MouseEvent e){
			
			int x = e.getX();
			int y = e.getY();
			
			label.setText("x: " + x + " y: " + y);
		
		}
	}	
	
	public static void main(String[] args){
		
		JFrame frame = new JFrame("Hauptfenster");
		ZeichneBrett brett = new ZeichneBrett();
		frame.setLayout(null);

		brett.setBounds(0, 0, 600, 400);
		
		frame.add(brett);
		
		
		frame.pack();
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		frame.setVisible(true);		
	}
}

ne ahnung was ich hier falsch mache?

 

[VfL_Freak]

Moin,

Du musst Dein Panel auch dem Frame hinzufügen !
Hab's so auf die Schnelle zumindest nicht gesehen ...

Gruß
Klaus

 

[diggaa1984]

joa,

in raster() folgendes am Ende aufrufen:

panel.setPreferredSize(new Dimension(600, 400)); //statt setSize
this.add(panel); //fuegt panel der Instanzt von ZeichneBrett hinzu

und in der main fehlte schlichtweg der Aufruf zum Rastern, allerdings kannst du da auch die layout-anweisungen rausnehmen.

public static void main(String[] args){
        
        JFrame frame = new JFrame("Hauptfenster");
        ZeichneBrett brett = new ZeichneBrett();

        brett.raster();
        frame.add(brett);
        
        
        frame.pack();
        frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
        frame.setVisible(true);     
}

Besser wäre es aber gleich noch, pro Klasse eine Datei anzulegen und nicht mit static zu hantieren

 

[Camino]

Ich denke, da sind mehrere grundsätzliche Fehler.

Du erstellst in der main-Methode einen JFrame und ein ZeichneBrett, welches du dem JFrame hinzufügst. ZeichneBrett ist von JPanel abgeleitet (da sist auch richtig so), aber es gibt keinen Konstruktor für ZeichneBrett. Stattdessen gibt es eine Methode raster, in welcher ein weiteres JPanel erstellt wird, welches aber nie erscheint, da dir Methode raster nie aufgerufen wird.

An deiner Stelle würde ich erstmal das Panel im Konstruktor der Klasse ZeichneBrett erstellen und dann dem JFrame hinzufügen. Dem JFrame würde ich auch nicht das null-Layout verpassen, sondern das JPanel einfach so hinzufügen. Landet dann im CENTER-Bereich des Frames.

 

[Protex]

hi danke

ich hab das jetzt einwenig umgeändert, bin aber auf weitere hindernisse gestoßen.. :-/
ich kann die hintergrundfarbe, der methode rasterZeichnen nicht ändern.. sie ist immer grau...
und wenn ich panel.getWidth() oder .getHeight() eingebe, bekomme ich als rückgabe wert 0..

eine frage habe ich noch, die frage der fragen: wie kann ich die einzelnen linien segmente anwählen? also das raster...
muss ich da einen mouselistener dran hängen irgendwie?

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

public class GehVersuch {

	
	public static class ZeichneBrett extends JPanel implements MouseListener, MouseMotionListener{
		
		JLabel label;
		Dimension d = Toolkit.getDefaultToolkit().getScreenSize();
		
		public void raster(){
					
			JPanel panel = new JPanel();
			label = new JLabel("Hier sollen mal die Koordinaten stehen");
			JLabel label2 = new JLabel();
			panel.setLayout(null);
			panel.setBackground(Color.white);
			rasterZeichnen raster = new rasterZeichnen();
			
			label.setBounds(525,380,100,20);
			panel.add(label);
			
			panel.addMouseListener(this);
			panel.addMouseMotionListener(this);
			
			panel.setPreferredSize(new Dimension(600, 400));
			this.add(panel);
			
			int x = panel.getWidth();   // ich bekomme für x und y = 0
			int y = panel.getHeight();
			
			label2.setText("Breite/Höhe = " + x + "/" + y);
			label2.setBounds(5,0,100,20);
			panel.add(label2);
			
			raster.setBounds(0, 0, 600, 400);
			panel.add(raster);
			
			
			panel.setVisible(true);
			
		}	
				
		
		
		public void mouseClicked(MouseEvent e){			
		}
		public void mouseReleased(MouseEvent e){			
		}
		public void mousePressed(MouseEvent e){			
		}
		public void mouseExited(MouseEvent e){			
		}
		public void mouseEntered(MouseEvent e){			
		}
		public void mouseDragged(MouseEvent e){		
		}
		public void mouseMoved(MouseEvent e){
			
			int x = e.getX();
			int y = e.getY();
			
			label.setText("x: " + x + " y: " + y);
		
		}
		
		class rasterZeichnen extends JPanel{
			
			public void paintComponent(Graphics g){
				
				super.paintComponents(g);
				super.setBackground(Color.white);	// kann Hintergrundfarbe nicht ändern
			
				int rasterX = 20;
				int rasterY = 20;
			
				for(int x=0; x <= d.width; x = x + rasterX){
					for(int y=0; y <= d.height; y = y + rasterY){
					
						g.drawLine(x, y, x, y);	
					}			
				}
			}
		}
	}	
	
	public static void main(String[] args){
		
		JFrame frame = new JFrame("Hauptfenster");
		ZeichneBrett brett = new ZeichneBrett();
		
		
		brett.raster();		
		frame.add(brett);

				
		frame.pack();
		frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		frame.setVisible(true);
		frame.setLocationRelativeTo(null);
	}
}

 

[Camino]

Noch ein paar Tipps zu deinem Code:

Deine Klasse ZeichneBrett erbt ja schon von JPanel, ist also quasi selbst eins. Das heisst, du bräuchtest in deiner Klasse ZeichneBrett eigentlich gar keine neues JPAnel erstellen, sondern könntest direkt deine Komponenten zu ZeichneBrett hinzufügen. Und das static würdeich bei der Klasse auch rausnehmen.

Und ich würde dir auch raten, deine Klassen in eigene Dateien zu stecken, dann wird das vielleicht auch übersichtlicher (für dich und auch für uns). Vermutlich kommt da in Zukunft ja auch noch einiges an Code dazu...

 

[beastofchaos]

Also, um sowas zu prüfen, brauchst du nur einen MouseListener, der in der Methode mousePressed, die Koordinaten der Maus mit denen der Linien vergleicht und dann entsprechend ietwas macht. So eine Linie "auswählen" geht theoretisch nicht, da es ja keine eigene Komponente ist. Da gäbs ein paar Ideen bei mir, weil ich selber grad ein riesiges Grafikprogramm schreibe, aber ich rate dir nochmal, deinen Code zu überarbeiten, ein paar Beispiele zu machst, wo alles leicht zu lösen ist, damit du ein "gefühl" für Java bekommst. Man kann dir hier nicht immer sagen, was man beim Code noch besser machen kann

Und wenn du genügend Tutorials angeschaut und genug getestet hast, kommst du wieder als der weise Yedi

Gruß, Thomas

 

[Protex]

@Camino: hab das noch nicht so ganz raus mit dem vererben und so.. aber werd ich beherzigen
@beastofchaos: ja.. hast recht.. leider ist das teil meiner bak arbeit, scheint als habe ich mir da etwas viel vorgenommen

danke trotzdem