Button Klasse mit allem drum und dran leider fehlt doch was ^^

[Cem]

Hallo alle zusammen

Ich sitze gerade vor einem Problem welches mich ein wenig verzweifeln lässt.

Ich Habe mir eine - so zu sagen abstrakte - Klasse gebaut die folgendes soll.

==> : - einen Button erstellen
- diesem Button soll gleich eine Position, Größe und Text zugeteilt werden
- ( so und nun kommt mein Prob ) er soll auch gleich einen ActionListener bekommen

das Ganze könnte ich jetzt natürlich so gestalten ^^

Hier die Klasse

class initJButtons extends JButton {

		/**
		 * the constructor of the initButtons class
		 * 
		 * @param xPos
		 * @param yPos
		 * @param width
		 * @param heigth
		 * @param text
		 * @param panel
		 */
		public initJButtons(int xPos, int yPos, int width, int heigth,
				String text, JPanel panel,ActionListener al ) {

			this.setBounds(xPos, yPos, width, heigth);
			this.addActionListener(al);
			this.setText(text);
			
			panel.add(this);

		}

	}

und hier der erstellende Konstruktor

public void initGuiStones(){
		
		buttonPanel = new initJPanels(1, 500, 600, 100, Color.BLACK);
		button = new initJButtons(1, 1, 100, 30, "click mich", buttonPanel, new ActionListener() {
			
			public void actionPerformed(ActionEvent e) {
				
				if(e.getSource().equals(button)){
					System.out.println("hallo");
				}
				
				
			}
		});
	}

Nun ist mir das aber zu unsauber jedes mal 10 Zeilen oder mehr code extra zu haben in meiner Methode in der eigentlich nur Konstruktoren stehen sollten.
Denn das ganze sieht nach 10 Buttons, nicht mehr so Sauber aus wie am Anfang und ich verliere den Überblick ( was nebenbei gesagt auch der Grund war warum ich das so programieren will, denn es soll als Basis für weitere fenster Dienen )

Jetzt zu meiner Idee die noch nicht funktioniert

Ich schiebe den Code erstmal vor und erkläre hinterher mein Problem genauer

so sollte es so ungefair ausehen.

hier wieder die Buttonklasse, das wichtige ist hier der parameter initinitButtonListener bl
und die Zeile this.addButtonListener(bl);

class initJButtons extends JButton {

		/**
		 * the constructor of the initButtons class
		 * 
		 * @param xPos
		 * @param yPos
		 * @param width
		 * @param heigth
		 * @param text
		 * @param panel
		 */
		public initJButtons(int xPos, int yPos, int width, int heigth,
				String text, JPanel panel,initButtonListener bl ) {

			this.setBounds(xPos, yPos, width, heigth);
			
			this.addButtonListener(bl);
			
			this.setText(text);
			
			panel.add(this);

		}

	}

So und jetzt zu der initButtonListener Klasse die ich mir in etwa so vorgestellt habe

class initButtonListener {
		
		public initButtonListener(int i, final initJButtons button) {
			
			switch (i) {
			case 1:
				
				new ActionListener() {
					
					@Override
					public void actionPerformed(ActionEvent e) {
						
						if(e.getSource().equals(button)){
							System.out.println("hallo");
						}		
					}
				};
				
				break;

			default:
                                     
                  System.out.println("This ActionListener doesn´t exist. Make sure you implement a new Actionlistener "); 

				break;
			}	
			
		}
			
	}

Ich weis jetzt aber nicht wie ich es in meiner erstellenden Methode so hinbekomme das alles Richtig ist und Funktioniert wie es soll

hier wieder die erstellende Methode, wie ich sie mir vorstelle

public void initGuiStones(){
		
		buttonPanel = new initJPanels(1, 500, 600, 100, Color.BLACK);
		button = new initJButtons(1, 1, 100, 30, "click mich", buttonPanel, new initButtonListener(1, button));
			
		
	}

Ich hoffe ich habe alles ausführlich genug beschrieben damit man es gleich verstehen kann

Danke schon einmal im vorraus

MfG Cem

P.S. wer Rechtschreibfehler findet, darf sie behalten

 

[Michael...]

Was erwartetest Du denn, dass die Klasse initButtonListener (KlassenNamen schreibt man üblicherweise gross) machen soll?
Und was hast Du eigentlich vor?

Die Tatsache, dass Du Position und Grösse den Buttons übergibt's, lasst darauf schliessen, dass Du keinen LayoutManager verwendest, wovon ich eher abraten würde.

 

[Cem]

Also Klassennamen schreibt man immer Groß sry hab ich vergessen ^^ wird verbessert!

So das ist richtig, ich verwende keine L-Manager, weil ich bei meinen letzten programmen gemerkt habe das es einfach besser funktioniert ohne welche eingebaut zu haben. Übrigens beim verwenden eines drag an drop addons der Bausteine sind auch keine Layouts von nöten.

So was ich machen will ist einfach das ich, wenn ich einen neuen Button erstellen will, einfach eine neue Instanz meiner Klasse InitButton erstelle die mich, mit den Parametern zwingt die Position auf dem Panel zu bestimmen, die Größe zu bestimmen und auf welchem Panel er sein soll.
Das war der einfache Teil, jetzt will ich aber noch einen weiteren Parameter, der mich zwingt die Klasse InitButtonListener aufzurufen( mit ihrem Konstruktor ), die wiederrum den Parameter i und die Instanz des Buttons dem dieser AcL. zugewiesen werden soll, beinhaltet.

das i ist für die switch case Abfrage

class initButtonListener {
        
        public initButtonListener(int i, final initJButtons button) {
            
            switch (i) {
            case 1:
                
                new ActionListener() {
                    
                    @Override
                    public void actionPerformed(ActionEvent e) {
                        
                        if(e.getSource().equals(button)){
                            System.out.println("hallo");
                        }       
                    }
                };
                
                break;
 
            default:
                                     
                  System.out.println("This ActionListener doesn´t exist. Make sure you implement a new Actionlistener "); 
 
                break;
            }   
            
        }
            
    }

Jetzt weis ich aber nicht wie ich meiner abstrakten Klasse InitJButton diesen Actionlistener über die Klasse InitButtonListener zuweisen kann den ich kann ja nicht einfach

public initJButtons(int xPos, int yPos, int width, int heigth,
                String text, JPanel panel, --> initButtonListener bl <-- ) {
.
.
.
   this.addButtonListener(bl);
.
.
.
}

Machen dazu fehlt mir die methode addButtonListener die das kann

Es ist wie wenn ich einen Button habe, dem ich über eine Zwischenklasse, einen Actionlistener zuweisen will.

So, ich hoffe ich hab es jetzt einigermaßen deutlich gemacht, wo mein Problem liegt, wenn nicht einfach reklamieren nehme jede Kritik positiv auf, ich will ja was lernen

MfG Cem

 

[Michael...]

Prinzipiell könnte man die Listener Klasse wie folgt gestalten, wobei Du eigentlich ja nur eine Methode benötigst.

Allerdings sollte man sich dieses Vorgehen überlegen.
- brauch jeder Button einen eigenen ActionListener oder kann man nicht einen für mehrere verwenden
- muss man so diverse anonyme ActionListener vorhalten
- so ganz verstanden habe ich auch nicht worum es Dir da geht. Geht's nur darum den Code übersichtlicher zu strukturieren? Das könnte auch ganz schnell das Gegenteil bewirken.

class initButtonListener {

	public initButtonListener(int i, final initJButtons button) {
		switch (i) {
		case 1:
			button.addActionListener(new ActionListener() {
				public void actionPerformed(ActionEvent e) {
					System.out.println("hallo");
				}
			});
			break;
		default:
			System.out.println("This ActionListener doesn´t exist. Make sure you implement a new Actionlistener ");
			break;
		}
	}
}

 

[Cem]

Ja ich könnte es auch mit einem ActionListener machen mit hilfe einer if - Abfrage nur hätte ich es gerne so das jeder button seinen eigenen Ac.L hat damit das ganze strukturierter aussieht ( ich zumindest finde es sieht so besser aus muss man nicht verstehen )

die Idee ist gut danke aber ich bekomme eine java.lang.NullPointerExeption in der Zeile

--> button.addActionlistener(new ActionListener...

Ich glaube das hängt mit meiner Klasse InitJButtons zusammen, da die ja nicht weis, was sie mit dem übergebenen Parameter anfangen soll

class InitJButtons extends JButton {

		/**
		 * the constructor of the initButtons class
		 * 
		 * @param xPos
		 * @param yPos
		 * @param width
		 * @param heigth
		 * @param text
		 * @param panel
		 */
		public initJButtons(int xPos, int yPos, int width, int heigth,
				String text, JPanel panel,initButtonListener bl ) {

			this.setBounds(xPos, yPos, width, heigth);
			this.setText(text);
			panel.add(this);

		}

	}

hier müsste doch dann auch noch etwas stehen wie this.addButtonListener(bl) oder ?

nur wie soll ich so etwas wie addButtonListener machen und was soll das sein eine Methode in InitButtonListener oder auch was ganz andres ?

MfG Cem und danke für die bisherige Hilfe

 

[Michael...]

Naja button ist vermutlich null die Klasse initButtonListener passt ja nicht zu Deinem anderen Code.
Wenn überhaupt, dann würde ich keine Klasse sondern entsprechende Methoden zur Verfügung stellen (und diese eventuell in einer eigenen Klasse kapseln).
Eher so:

class Constructor extends JPanel {
	private JButton button2;

	public Constructor() {
		this.setLayout(null);
		this.add(createButton("Button1", 10, 10, 50, 26, 0));
		this.add(button2 = createButton("Button2", 10, 50, 50, 26, 1));
	}

	private JButton createButton(String text, int x, int y, int w, int h, int i) {
		JButton button = new JButton(text);
		button.setBounds(x, y, w, h);
		button.addActionListener(createActionListener(i));
		return button;
	}

	private ActionListener createActionListener(int index) {
		switch (index) {
		case 0:
			return new ActionListener() {
				public void actionPerformed(ActionEvent e) {
					System.out.println(e.getSource());
				}
			};
		case 1:
			return new ActionListener() {
				public void actionPerformed(ActionEvent e) {
					System.out.println(e.getActionCommand());
				}
			};
		}
		return null;
	}
}

 

[Ebenius]

Ich möchte nur mal am Rande anmerken, dass mir die numerischen Identifizierer (also [c]index[/c]) als Referenz eher misfallen. Da wird der Code meiner Meinung nach eher unübersichtlicher als besser. Wenn man's so macht, sollte man doch eher Enums oder Strings nutzen, da kann man beim Lesen wenigstens was mit anfangen. In aller Regel sind solche [c]switch[/c]/[c]case[/c]-Anweisungen ein Indiz für unschönen Code:

switch(magicNumber) {
  case 0:
  case 1:
  case 2:
  case 3:
}

Just my $0.02
Ebenius