JFrame Taschenrechner

[marian04]

Hallo Leute,
ich bin neu im Forum und ein relativer Neuling in Java.
Um mich nicht zu übernehmen, wollte ich versuchen einen Taschenrechner auf JFrame Basis zu Programmieren.

Jetzt stellt sich mir die Frage, wie ich es hinbekomme, dass der Wert des Buttons auch in das Textfeld geschrieben wird bzw es so geht, dass er nur eine Zahl anhängt ?!

Aber schaut euch mal den Code an vllt wird es dann klar :

ActionListener A1=new ActionListener(){

    public void actionPerformed(ActionEvent e){
        String s =e.paramString();
        if (s.indexOf("cmd=1")!= 0) {txt1.setText(txt1.getText() + "1") ; }  ;
        if (s.indexOf("cmd=2")!= 0) {txt1.setText(txt1.getText() + "2") ; }  ;
         if (s.indexOf("cmd=3")!= 0) {txt1.setText(txt1.getText() + "3") ; }  ;
         if (s.indexOf("cmd=4")!= 0) {txt1.setText(txt1.getText() + "4") ; }  ;
         if (s.indexOf("cmd=5")!= 0) {txt1.setText(txt1.getText() + "5") ; }  ;
         if (s.indexOf("cmd=6")!= 0) {txt1.setText(txt1.getText() + "6") ; }  ;
         if (s.indexOf("cmd=7")!= 0) {txt1.setText(txt1.getText() + "7") ; }  ;
         if (s.indexOf("cmd=8")!= 0) {txt1.setText(txt1.getText() + "8") ; }  ;
                                                                          

        
        }
            };

Wenn ich hier versuche mit if / else,if zu arbeiten kommt immer ein Fehler : else without if.


Ich hoffe mich versteht hier jmd ^^


Mit freundlichen Grüßen Marian


Ps: Wen es interessiert der komplette Code steht im Anhang !

 

[andre111]

Lasst die Semikolons am Ende jeder Zeile nach den geschweiften Klammern weg, dann kannst dus mit else machen.

 

[marian04]

Danke, aber jetzt hab ich das Problem, dass jeder Button die 1 in das Textfeld schreibt, wo ist denn mein Denkfehler ?

ActionListener A1=new ActionListener(){

    public void actionPerformed(ActionEvent e){
        String s =e.paramString();
        if (s.indexOf("cmd=1")!= 0) {txt1.setText(txt1.getText() + "1") ; }
         else
           if (s.indexOf("cmd=2")!= 0) {txt1.setText(txt1.getText() + "2") ; }
          else
            if (s.indexOf("cmd=3")!= 0) {txt1.setText(txt1.getText() + "3") ; }
           else
            if (s.indexOf("cmd=4")!= 0) {txt1.setText(txt1.getText() + "4") ; }
            else
              if (s.indexOf("cmd=5")!= 0) {txt1.setText(txt1.getText() + "5") ; }
             else
              if (s.indexOf("cmd=6")!= 0) {txt1.setText(txt1.getText() + "6") ; }
               else
                if (s.indexOf("cmd=7")!= 0) {txt1.setText(txt1.getText() + "7") ; }
                else
                 if (s.indexOf("cmd=8")!= 0) {txt1.setText(txt1.getText() + "8") ; }



        }
  };

Mfg Marian

 

[Schandro]

Hast du dir überhaupt schonmal angeguckt was dir die paramString() Methode returnt?
Ein Beispiel:
"ACTION_PERFORMED,cmd=ok,when=1234991777515,modifiers=Button1"

Du must anders an die Info drankommen, welcher Button das Event ausgelöst hat. z.b.:
String actionCommand = ((JButton)event.getSource()).getActionCommand();
Genausogut könntest du auch den aufgemalten Text auslesen, ist aber nicht zu empfehlen weil du nur für Schöhnheitsänderungen dann deinen Code wieder umschreiben müsstest.

Grundsätzlich gibt es mit OOP aber sehr viel einfachere Wege, um 9 Buttons mit ähnlicher Funktion darzustellen, ohne 9 mal fast gleichen Code hinschreiben zu müssen.

Außerdem: Wenn du gucken willst ob ein String einen anderen enthält, nimm die contains-Methode oder schreib:
s1.indexOf(s2)!= -1

 

[Fu3L]

Allgemeiner Ratschlag: Niemals auf Klammern verzichten, auch wenn man sie weglassen kann. Deine Konstruktion mit den else is nur sehr schwer zu überblicken...

 

[marian04]

1.Hast du dir überhaupt schonmal angeguckt was dir die paramString() Methode returnt?


2.Du must anders an die Info drankommen, welcher Button das Event ausgelöst hat. z.b.:
String actionCommand = ((JButton)event.getSource()).getActionCommand();
Genausogut könntest du auch den aufgemalten Text auslesen, ist aber nicht zu empfehlen weil du nur für Schöhnheitsänderungen dann deinen Code wieder umschreiben müsstest.


3.Außerdem: Wenn du gucken willst ob ein String einen anderen enthält, nimm die contains-Methode oder schreib:
s1.indexOf(s2)!= -1

Hey, danke für die schnellen Antworten:
@ Schandro
zu 1. Diese Methode wurde mir von meinem Lehrer empfohlen, hatte ihn nach dem Regelunterricht gefragt ob sowas machbar sei und er sagte ich solle es damit versuchen..

zu 2. könntest du dies bitte an einer Zeile (wie im Code) fest machen, da ich dir zur Zeit leider nicht mehr folgen kann , dass wäre sehr nett !

@Fu3l

Okay danke für den Tipp werde ich beherzigen


mfg Marian

 

[Schandro]

guck dir mal die 1. Antwort von Gast, die in Wahrheit von mir stammt (scheiss Loginbug...), in diesem Thread an:
http://www.java-forum.org.server659...79028&highlight=setActionCommand(""

Ist ein gutes Beispiel dafür wie man die 9 Tasten für nen Tachenrechner einfacher programmieren kann.

 

[marian04]

hey,

hab ich gemacht und einfach mal versucht deinen Quelltext zu complimieren, geht uach, dann möchte ich starten und bekomme folgenden Error des Interpreten :

java.lang.Error: Do not use javax.swing.JFrame.add() use javax.swing.JFrame.getContentPane().add() instead
	at javax.swing.JFrame.createRootPaneException(JFrame.java:465)
	at javax.swing.JFrame.addImpl(JFrame.java:491)
	at java.awt.Container.add(Container.java:307)
	at Rechner.<init>(Rechner.java:16)
	at Rechner.main(Rechner.java:8)
Exception in thread "main"

Tut mir leid wenn ich mich sau dumm anstelle, aber iwie bekomm ich das net auf die Reihe !


mfg

 

[Schandro]

Welche Compiler Version hast du denn? Den Fehler hat ich noch nie, weil normalerweise die add Methode von JFrame an das ContentPane weitergeleitet wird. Du kannst einfach

.add

durch

.getContentPane().add

ersetzen, dann dürftes gehen

 

[marian04]

j2sdk1.4.2_19

Is die Version scheint jedenfalls so.

mfg