Funktion eines JButton in einen Vektor verlagern

[fast35]

Hallo zusammen,

ich habe folgendes Problem:
- Ich habe mehrere Ein- und Ausschalt-Button die eig immer das gleiche machen und möchte die Funktion z.B. eines JButton in einen Vektor verlagern, also immer wenn ich die Ein-Taste betätige soll es vom Vektor die genau Funktion finden und übergeben.

1. Ist sowas überhaupt möglich? Wenn, ja, wie?
2. Wie ist es mit Verlagerung in eine Datenbank, ist sowas generell möglich?

Mein Ziel ist es meine Klassen und meine Code-Zeilen zu reduzieren.

private JButton getBtnRelay1() {
		if (btnRelay1 == null) {
			btnRelay1 = new JButton("Ein");
			btnRelay1.setFont(new Font("Tahoma", Font.PLAIN, 8));
			btnRelay1.addActionListener(new ActionListener (){
				public void actionPerformed(ActionEvent arg0) {
					Serielle kommPort1 = new Serielle();
					kommPort1.openCOMBOARD(JComboBox_RelayBoard.getSelectedItem().toString());
					
					
					kommPort1.initRelayBoard();
					
					boolean b = true;
					kommPort1.setRelay(1,b);
					kommPort1.close();
				}
			});
			btnRelay1.setBounds(28, 73, 50, 23);
		}
		return btnRelay1;
	}

Ich freu mich für eure Antworten. Danke

 

[turtle]

Ich verstehe das Problem nicht richtig.

Ist sowas überhaupt möglich? Wenn, ja, wie?

Wenn du meinst, das du den Code zu Buttons nur einmal schreiben möchtest, lautet die Antwort: Ja.

Der Listener, also der Code, der aufgerufen wird, wenn auf den Button geklickt wird, ist ja auch "nur" eine Klasse:

new ActionListener (){
                public void actionPerformed(ActionEvent arg0) {
                    Serielle kommPort1 = new Serielle();
                    kommPort1.openCOMBOARD(JComboBox_RelayBoard.getSelectedItem().toString());
                    
                    
                    kommPort1.initRelayBoard();
                    
                    boolean b = true;
                    kommPort1.setRelay(1,b);
                    kommPort1.close();
                }
            }

Also, kannst du eine Instanz dieser Klasse anlegen und für einen Button über addActionListener() registrieren.

Du kannst alles auch mit mehreren Buttons machen und im ActionEvent abfragen, welcher Button der Auslöser war.

Somit kannst du auch eine Funktion schreiben, die nach einem gewissen Kriterium (alle Ein-Buttons) mit dem entsprechenden ActionListener "verdrahtest".

Wie ist es mit Verlagerung in eine Datenbank, ist sowas generell möglich?

Da das Reagieren auf einen Button ja Java-Code erfordert, ist eine Verlagerung in eine DB nicht möglich.

Wenn ich dich aber falsch verstanden habe, ignorier diese Antwort

 

[fast35]

Hallo turtle,

Du hast es richtig verstanden, ich möchte für mehrere
Buttons nur einmal den Code schreiben. Ich habe zuviele
Zeilen an code, die ich reduzieren möchte.
Das hört sich alles schon gut an, aber es gibt noch ein Problem.
Z.B. Mein Button getBtnRelay1 schaltet auf einer Platine Relais Nr.1 ein.
Button2-Relais2,..Button6-Relais6.
Also, muss ich eine Instanz anlegen und für jeden Button addActionListenen() registrieren.
Das alles in eine Funktion und dort wird festgelegt, durch ActionEvent, Button 3 wurde gedrückt, schicke ActionListener() für Button3, Schalte Relais3 ein.
Habe ich das richtig verstanden?

 

[Sen-Mithrarin]

nein .. da hast du was nicht ganz verstanden

du willst code nur genau ein mal schreiben
die antwort : ja, genau das geht auch
und es geht sogar noch weiter : du brauchst dafür auch sogar nur ein objekt ... musst also nur einmal NEW schreiben

du hast jetzt noch den denkfehler das du von deinem aktuellen code ausgehst und glaubst du müsstest für jedes relais extra code schreiben

man kann das ganze aber sehr eleganz lösen

setActionCommand(String)

hier kannst du nötige parameter für jeden button einzeln vergeben die dann im event ausgelesen werden können


am ende hast du also z.b. sowas


JButton button1=new JButton();
button1.setActionCommand("1/on");
button1.addActionListener(listenerInstance);
JButton button2=new JButton();
button2.setActionCommand("2/on");
button2.addActionListener(listenerInstance);
// ...

und im listener dann sowas

String command=actionEvent.getActionCommand();
int relay=Integer.parseInt(command.split("/")[0]);
boolean on="on".equalsIgnoreCase(command.split("/")[1]) ? true : false;

setRelay(relay, on);


klar geht das ganze noch kürzer mit nem loop ... dann spart man sich die haufen button-variablen ... aber nicht alles auf einmal =D
(heißt also : man könnte das wofür du sicher über die hälfte deiner code-zeilen "verschwendest" sauber in 5 bis 10 zeilen packen und es würde immer noch genau so funktionieren wie jetzt ... nur halt deutlich "aufgeräumter")


zu Vektor : du solltest lieber ArrayList statt Vektor nutzen ...
sie machen zwar am ende das gleiche ... aber eben nicht 100% das selbe ...
google liefert dazu sehr ausführliche informationen warum ArrayList einem Vektor immer vorzuziehen ist

 

[fast35]

Danke euch, hat mir sehr weitergeholfen :toll:
Jetzt muss ich alles noch umsetzen, dann passt alles.

Später mit einem loop muss ich mir noch anschauen, wie gesagt
erstmal das und code-zeilen reduzieren.