JButton ableiten

[bliet]

Hallo,

ich bin neu hier und hoffe ihr könnte mir weiter helfen

Ich habe eine Klasse, welche von JButton erbt.
Mehrere davon will ich nun in ein Panel mit einem GridLayout darstellen.

Wenn ich bei folgenden Code JButtons erzeuge funktioniert auch alle wunderbar.
Wenn ich nun aber meine Klasse "Feld" nehmen, welche von JButton erbt, werden alle Button oben links übereinander dargestellt.

pane = new JPanel();
buttons = new Feld[width][height];
pane.setLayout(new GridLayout(width,height,gap,gap));

for(int x=0; x < width; x++){
	for(int y=0; y < height; y++){
		buttons[x][y] = new Feld(""+x+y);
		buttons[x][y].setPos(x, y);
		pane.add(buttons[x][y]); 
}
}
this.setSize(width * 100, height * 100); 
this.add(pane);

Die Klasse Feld:

public class Feld extends JButton{
	private static final long serialVersionUID = 1L;
		
	public Feld(String Name){
		super(Name);
	}
	
	private int X;
	private int Y;
	
	public void setPos(int newX, int newY) {
		X = newX;
		Y = newY;
	}
	
	public int getX(){
		return X;
	}
	
	public int getY(){
		return Y;
	}

Ich habe schon Google zu rate gezogen, aber nichts passendes gefunden ;-)
Ich hoffe die Code-Ausschnitte reichen. Der obere Code wird im Konstruktor der Klasse "Flaeche", welche von JFrame erbt, ausgeführt.

Wieso geht das mit JButtons, aber nicht mit meiner davon geerbten Klasse?

Danke für eure Hilfe.

Mit freundlichen Grüßen
Benjamin

 

[SlaterB]

getX() + getY() sind wichtige Funktionen in GUI-Komponenten,
du überschreibst sie und läßt sie falsche Werte zurückgeben, benenne deine Methoden anders,
getPosX() z.B. müsste noch frei sein

 

[Quaxli]

Du solltest Dir angwöhnen kompilierbare Beispiel zu posten, dann wird Dir schneller geholfen bzw. überhaupt....

Das Ding hast du selbst verbrochen, durch die Methode setPos(x,y)! Dort änderst Du die Positon der Buttons und zwar anhand der Nummern der Array-Felder. Bei einem kleinen Array mit 5x5 Feldern werden die Buttons dann jeweils mit 1 Pixel Abweichung übereinander gezeichnet.

Was diese Methode soll verstehe ich nicht. Entweder setzt man die Position der Buttons selbst, dann aber mit einem Null-Layout ODER man überläßt es dem Layout-Manager, dann benötigt man dieses setPos nicht.

 

[Quaxli]

Hab' mal ein KSKB gebastelt.

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

public class Test extends JPanel{

	private static final long	serialVersionUID	= 1L;
	JFrame frame;
	 Feld[][] buttons;
	 int width = 5;
	 int height = 5;
	 
	 public static void main(String [] args){
		 new Test();
	 }
	 
	 public Test(){
		 
		 frame = new JFrame("Test");
		 frame.setLocation(100,100);
		 frame.setDefaultCloseOperation(JFrame.EXIT_ON_CLOSE);
		 
		 buttons = new Feld[width][height];
		 setLayout(new GridLayout(width,height,10,10));
     setPreferredSize(new Dimension(300,300));
     
		for (int x = 0; x < width; x++) {
			for (int y = 0; y < height; y++) {
				buttons[x][y] = new Feld("" + x + y);
				buttons[x][y].setPos(x, y); 
				add(buttons[x][y]);
			}
		}

		 frame.add(this);
		 frame.pack();
		 frame.setVisible(true);
		 
	 }
}

class Feld extends JButton{
	
    private static final long serialVersionUID = 1L;
    int x;
    int y;
        
    public Feld(String Name){
        super(Name);
    }

		public void setPos(int x, int y) {
			this.x = x;
			this.y = y;
		}
    
}

 

[SlaterB]

@Quaxli setPos() und entsprechende Werte in Subklasse zu speichern ist an sich nicht störend,

die Info ist gewiss nicht fürs Layout gedacht sondern logische Speicherung für spätere Auswertung,
ansonsten müsste der Button-Listener erst das Array durchlaufen (oder gar die Position im Layout irgendwie zurückrechnen) um herauszufinden, ob nun gerade der 5. Button der ersten Reihe gedrückt wurde oder ein anderer

 

[Quaxli]

Stimmt. Habe ich beim Erstellen des KSKB dann auch gemerkt.
Danke für die Richtigstellung.

 

[SlaterB]

Danke für die Richtigstellung.

bestimmt nicht, aber trotzdem nötig

 

[bliet]

Wow, super wie schnell hier geantwortet wird

Habe die Methoden umbenannt. Jetzt läuft es

Bin noch aus der .Net Welt gewöhnt, dass man auf so was vom Editor hingewiesen wird :-D

 

[SlaterB]

ne gute IDE wie Eclipse würde das machen bei passenden Einstellungen

 

[bliet]

Ich benutzt Eclipse. Wo kann ich das denn einstellen?

 

[SlaterB]

anbei ein Screenshot für Projekt-Properties, gibt auch ähnliche Einstellungen allgemein in den Preferences für gesamten Workspace,
Java Compiler - Error/ Warnings

wenn gesuchte Einstellung nicht vorhanden, dann wahrscheinlich zu alte Eclipse-Version

 

[bliet]

und nochmals Danke