Calculator - Buttons zusammenfassen

[beezie]

Hallo,

hätte zwei Fragen, und hoffe ihr könnt mir helfen.

Ich habe einen Taschenrechner programmiert, aber er reagiert nur auf einen Mausklick, kann ich irgendwas eingeben damit er auf die Tastatur auch "hört"?

Und ich habe verschiedene Buttons, die aber alle den gleichen Code haben, kann ich das zusammenfassen um Platz usw.. zu sparen.
Hier mal der Code:

[HIGHLIGHT="Java"]private void buttonPlusActionPerformed(java.awt.event.ActionEvent evt) {
pruefung = false;
if (operator == 0) {
operands.add(Double.parseDouble(jTextPane1.getText()));
}
else {
/*
* Ich hole mir den Text vom Textfeld
* Der Text wird in ein Double umgewandelt
* Von der ArrayList wird die Zahl, die an oberster Stelle steht, dazugezählt.
*
* */
if (operator == '+') {
operands.add(operands.remove(operands.size()-1) + Double.parseDouble(jTextPane1.getText()));
}

if (operator == '-') {
operands.add(operands.remove(operands.size()-1) - Double.parseDouble(jTextPane1.getText()));
}
if (operator == '*') {
operands.add(operands.remove(operands.size()-1) * Double.parseDouble(jTextPane1.getText()));
}
if (operator == '/') {
operands.add(operands.remove(operands.size()-1) / Double.parseDouble(jTextPane1.getText()));
}

jTextPane1.setText("" + operands.get(operands.size()-1));
}
operator = '+';
}

private void buttonMinusActionPerformed(java.awt.event.ActionEvent evt) {
pruefung = false;
if (operator == 0) {
operands.add(Double.parseDouble(jTextPane1.getText()));
}
else {

if (operator == '+') {
operands.add(operands.remove(operands.size()-1) + Double.parseDouble(jTextPane1.getText()));
}

if (operator == '-') {
operands.add(operands.remove(operands.size()-1) - Double.parseDouble(jTextPane1.getText()));
}
if (operator == '*') {
operands.add(operands.remove(operands.size()-1) * Double.parseDouble(jTextPane1.getText()));
}
if (operator == '/') {
operands.add(operands.remove(operands.size()-1) / Double.parseDouble(jTextPane1.getText()));
}

jTextPane1.setText("" + operands.get(operands.size()-1));
}
operator = '-';
}

private void buttonMultiplikationActionPerformed(java.awt.event.ActionEvent evt) {
pruefung = false;
if (operator == 0) {
operands.add(Double.parseDouble(jTextPane1.getText()));
}
else {

if (operator == '+') {
operands.add(operands.remove(operands.size()-1) + Double.parseDouble(jTextPane1.getText()));
}

if (operator == '-') {
operands.add(operands.remove(operands.size()-1) - Double.parseDouble(jTextPane1.getText()));
}
if (operator == '*') {
operands.add(operands.remove(operands.size()-1) * Double.parseDouble(jTextPane1.getText()));
}
if (operator == '/') {
operands.add(operands.remove(operands.size()-1) / Double.parseDouble(jTextPane1.getText()));
}

jTextPane1.setText("" + operands.get(operands.size()-1));
}
operator = '*';
}[/HIGHLIGHT]


Danke im Vorraus,

liebe grüße

 

[Hadernlump]

füge deinem Fenster einen KeyListener hinzu, der zu den betätigten zahlen und rechenzeichen die gewünschten aktionen ausführt. Meines Wissens gibt es keine möglichkeit den Buttons einen Shortcut zuzuweisen.

Gruß Hadernlump

 

[beezie]

das heißt ich muss für alle buttons plus minus, mal, geteilt die funktion extra hinschreiben obwohl es die gleich funktion ist?

 

[Hadernlump]

Du könntest die funktionalität in eine eigene Methode (oder Klasse) schreiben und die Listener nur noch die benötigten aufrufe machen lassen

[HIGHLIGHT="Java"]
public class Funktion
{
public void add(char c)
{
...
}
}

...

Funktion f = new Funktion();

ActionListener al = new ActionListener()
{
public void actionPerformed(ActionEvent arg0)
{
String text = ((JButton)arg0.getSource()).getText();
// Angenommen jeder Button hat nur das eine Char das die
Funktionalität beschreibt (+, - 0, 1, ...)
char c = text.toCharArray()[0];

f.add(c);
}
}

KeyListener kl = new KeyAdapter()
{
public void keyTyped(KeyEvent arg0)
{
char c = arg0.getKeyChar();
f.add(c);
}
}

};
[/HIGHLIGHT]

Ich hoffe man kann verstehen was ich damit sagen will

Gruß Hadernlump

 

[Marco13]

Ja neee... ein bißchen Inspiration kannst du dir für die allgemeine Struktur (redundanzvermeidung) schonmal hier holen http://www.java-forum.org/java-faq-beitraege/49255-componenten-und-listener.html

In diesem Fall wäre es aber vielleicht sogar günstiger oder eleganter, Actions zu verwenden. Dann kannst du eine ActionMap erstellen, die die Tasten auf die passenden Actions mapps, so dass bei einem Muasklick auf den Button oder einem Tastendruch wirklich dieselbe Action ausgeführt wird... Siehe How to Use Key Bindings (The Java™ Tutorials > Creating a GUI with JFC/Swing > Using Other Swing Features) und How to Use Actions (The Java™ Tutorials > Creating a GUI with JFC/Swing > Using Other Swing Features)

 

[beezie]

Ja ich hab mir gedacht, dass ich für diese Rechnungen eine Methode anlege.
So brauche ich die Buttons doch nur mit der Methode zu verknüpfen oder?
Aber ich weiß nicht mehr so genau wie ich anfangen soll.

Hätte mir gedacht mit

private void Funktion () {
  pruefung = false;
   if (operator == 0) {
    operands.add(Double.parseDouble(jTextPane1.getText()));
  } 
  else {
    
    if (operator == '+') {
      operands.add(operands.remove(operands.size()-1) + Double.parseDouble(jTextPane1.getText()));
    }
   
    if (operator == '-') {
      operands.add(operands.remove(operands.size()-1) - Double.parseDouble(jTextPane1.getText()));
    }
    if (operator == '*') {
      operands.add(operands.remove(operands.size()-1) * Double.parseDouble(jTextPane1.getText()));
    }
    if (operator == '/') {
      operands.add(operands.remove(operands.size()-1) / Double.parseDouble(jTextPane1.getText()));
    }

    jTextPane1.setText("" + operands.get(operands.size()-1));
  }
  operator = '/';
}

Kann das so richtig sein?

lg

 

[Marco13]

Das einzig spezifische an der Methode is das
operator = '/';
am Ende. Das '/' könnte man als Argument übergeben ... oder so... es gibt da 100 Möglichkeiten...

 

[beezie]

ist die Funktion, die ich geschrieben habe dann richtig? oder muss ich irgendwas ändern sodass ich sie auf meine buttons anwenden kann?
lg

 

[Marco13]

Naja, die Funktion würde dann von der "actionPerformed"-Methode einer Action aus aufgerufen werden. Ob sie "richtig" ist... nun... im Mathematischen Sinne? Sicher nicht... Aber... es würde grundsätzlich genausogut oder schlecht funktionieren, wie vorher: Überall da, wo du bisher all das, was jetzt in der Methode steht, hingeschrieben hattest, kannst du jetzt dann einfach
Funktion();
oder besser
funktion('+');
oder noch besser
applyOperation('+');
oder so hinschreiben...