Alles ignorieren ausser Zahlen

[Reality]

Hi,
ich will nen kleinen Taschenrechner bauen und da kann ich kein "+" in eine Variable speichern, also habe ich folgenden Ansatz gemacht:

  StringBuffer get= new StringBuffer();
  for(short i=0; i<tfEingabe.getText().length(); i++) 
    if(tfEingabe.getText().charAt(i)>='0' && tfEingabe.getText().charAt(i)<='9')
      get.append(i);
  
  int rechnung= Integer.parseInt(get.toString());
  tfEingabe.setText(rechnung+" klappt");

Das ist natürlich nur ein Ansatz.
Mein Problem ist, dass entweder die falschen Zahlen ausgegeben werden oder sie werden nicht komplett ausgegeben.

Sieht jemand den Fehler?!

Liebe Grüße
Reality

 

[K-Man]

Die for-Schleife musst du zunächst mal rückwerts laufen lassen, sonst ist dein Ergebnis spiegelverkehrt...
Zu deinem anderen Problem schau ich mal schnell nach

Edit:
Vergiss meinen Beitrag...

 

[Reality]

Hi,
ich glaube da irrst du dich.

Liebe Grüße
Reality

 

[K-Man]

Man sieht den Wald vor lauter Bäumen nicht
Du darfst nicht i adden, sondern deinen char aus dem Textfeld

   StringBuffer get= new StringBuffer();
  for(short i=0; i<tfEingabe.getText().length(); i++)
    if(tfEingabe.getText().charAt(i)>='0' && tfEingabe.getText().charAt(i)<='9')
      get.append(tfEingabe.getText().charAt(i));
 
  int rechnung= Integer.parseInt(get.toString());
  tfEingabe.setText(rechnung+" klappt");

 

[K-Man]

Übrigens noch einen Tipp für größere Schleifen...:

for(short i=0, n=tfEingabe.getText().length(), i <n; i++) // va bei großen Mengen viel effizienter...

 

[Reality]

Oh man, warum bin ich so blöd? :cry:

Danke!

Liebe Grüße
Reality

 

[K-Man]

Oh man, warum bin ich so blöd? :cry:

Danke!

Liebe Grüße
Reality

Kein Problem, aber solche Fehler passieren immer wieder Denk dir nix...

 

[Reality]

Eins ist mir aufgefallen.
Manchmal kommt eine NumberFormatException wie z.B. in diesem Fall:

Ich habe folgendes eingegeben:

sdf489sd498498a4sd6

java.lang.NumberFormatException: For input string: "48949849846"

at java.lang.NumberFormatException.forInputString(NumberFormatException.java:48)

at java.lang.Integer.parseInt(Integer.java:480)

at java.lang.Integer.parseInt(Integer.java:518)

at Taschenrechner.berechnen(Taschenrechner.java:29)

at Taschenrechner.actionPerformed(Taschenrechner.java:37)

at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:1786)

at javax.swing.AbstractButton$ForwardActionEvents.actionPerformed(AbstractButton.java:1839)

at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:420)

at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:258)

at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(BasicButtonListener.java:245)

at java.awt.Component.processMouseEvent(Component.java:5100)

at java.awt.Component.processEvent(Component.java:4897)

at java.awt.Container.processEvent(Container.java:1569)

at java.awt.Component.dispatchEventImpl(Component.java:3615)

at java.awt.Container.dispatchEventImpl(Container.java:1627)

at java.awt.Component.dispatchEvent(Component.java:3477)

at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:3483)

at java.awt.LightweightDispatcher.processMouseEvent(Container.java:3198)

at java.awt.LightweightDispatcher.dispatchEvent(Container.java:3128)

at java.awt.Container.dispatchEventImpl(Container.java:1613)

at java.awt.Window.dispatchEventImpl(Window.java:1606)

at java.awt.Component.dispatchEvent(Component.java:3477)

at java.awt.EventQueue.dispatchEvent(EventQueue.java:456)

at java.awt.EventDispatchThread.pumpOneEventForHierarchy(EventDispatchThread.java:201)

at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:151)

at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:145)

at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:137)

at java.awt.EventDispatchThread.run(EventDispatchThread.java:100)

java.lang.NumberFormatException: For input string: "48949849846"

at java.lang.NumberFormatException.forInputString(NumberFormatException.java:48)

at java.lang.Integer.parseInt(Integer.java:480)

at java.lang.Integer.parseInt(Integer.java:518)

at Taschenrechner.berechnen(Taschenrechner.java:29)

at Taschenrechner.actionPerformed(Taschenrechner.java:37)

at javax.swing.AbstractButton.fireActionPerformed(AbstractButton.java:1786)

at javax.swing.AbstractButton$ForwardActionEvents.actionPerformed(AbstractButton.java:1839)

at javax.swing.DefaultButtonModel.fireActionPerformed(DefaultButtonModel.java:420)

at javax.swing.DefaultButtonModel.setPressed(DefaultButtonModel.java:258)

at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(BasicButtonListener.java:245)

at java.awt.Component.processMouseEvent(Component.java:5100)

at java.awt.Component.processEvent(Component.java:4897)

at java.awt.Container.processEvent(Container.java:1569)

at java.awt.Component.dispatchEventImpl(Component.java:3615)

at java.awt.Container.dispatchEventImpl(Container.java:1627)

at java.awt.Component.dispatchEvent(Component.java:3477)

at java.awt.LightweightDispatcher.retargetMouseEvent(Container.java:3483)

at java.awt.LightweightDispatcher.processMouseEvent(Container.java:3198)

at java.awt.LightweightDispatcher.dispatchEvent(Container.java:3128)

at java.awt.Container.dispatchEventImpl(Container.java:1613)

at java.awt.Window.dispatchEventImpl(Window.java:1606)

at java.awt.Component.dispatchEvent(Component.java:3477)

at java.awt.EventQueue.dispatchEvent(EventQueue.java:456)

at java.awt.EventDispatchThread.pumpOneEventForHierarchy(EventDispatchThread.java:201)

at java.awt.EventDispatchThread.pumpEventsForHierarchy(EventDispatchThread.java:151)

at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:145)

at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:137)

at java.awt.EventDispatchThread.run(EventDispatchThread.java:100)

Wieso kommt das?!

EDIT: Kommt eigentlich nur, wenn der String zu lang ist, denke ich.

Liebe Grüße
Reality

 

[Illuvatar]

Die größte Zahl, die ein int aufnehmen kann, ist Integer.MAX_VALUE, also 2147483647 oder 2^31-1.

 

[Reality]

Achso, danke. Ich dachte der Wert wäre weit aus höher.

Liebe Grüße
Reality

 

[Reality]

Hi,
ich bin´s nochmal.
Wie kann ich das Pluszeichen das in einem String oder Char gespeichert ist für Integer gültig machen?
Bei char geht es zwar, aber es gibt mir nur den ASCII-Code heraus.

Liebe Grüße
Reality

 

[Reality]

Wieso geht das eigentlich nicht auf?!

String get= tfEingabe.getText();
  StringBuffer form= new StringBuffer();
  int rechnung= 0;
  
  for(short i=0; i<get.length(); i++){
    if (get.charAt(i) >= '0' && get.charAt(i) <= '9'){
      form.append(tfEingabe.getText().charAt(i));
    }
      
    else if (get.charAt(i)=='+')
      rechnung += Integer.parseInt(form.toString());
  }
  
  if(rechnung!=0)
    tfEingabe.setText(Integer.toString(rechnung));
  
  else
    tfEingabe.setText(form.toString());

Liebe Grüße
Reality

 

[K-Man]

Weil du nicht addierst, sondern nur den String in einen int umwandelst...
Also Eingabe; 10 + 15
StringBuffer enthält 1015:

Integer.parseInt(form.toString()) ergibt also: 1015...da wird nix addiert.

Mach es mit einem Stack Kannst es auch gleich üben
Wandle deine Eingabe in postFix um und packe die Werte und Rechenzeichen in zwei Stacks.
Dann gehst du von oben nach unten durch und führst das letzte Rechenzeichen an den letzten beiden Werten aus und fügst das Ergebnis wieder in den Stack...so macht es der Computer....es gibt sicher einige Links zu diesem Thema, komme aber grad von ner Disco und bin jetzt nicht mehr in der Lage zu suchen :lol:

 

[Reality]

Hi,
doch, addieren tu ich schon mit dem Code:

rechnung += Integer.parseInt(form.toString());

Nur ist rechnung eben vorher als 0 initialisiert und darum bringt es nichts. :bloed:

Das mit dem Stack kann ich erst später ausprobieren, weil das in meinem Buch erst nach ca. 12 Seiten drankommt.

Liebe Grüße
Reality

 

[Jens]

Ich glaube, du musst den Stringbuffer nach dem addieren leeren:

form.setLength(0);

Danach kann dein Stringbuffer die nächsten Zahlen aufnehmen.

Viele Grüße
Jens

 

[Reality]

Hi Jens,
geht leider auch nicht. Trotzdem danke!

Liebe Grüße
Reality

 

[Jens]

Na Hossa.
Habs aber so getestet. Ging astrein.

probier nochmal.

Grüße
Jens

 

[Jens]

Hi Reality.

Hier der getestete Code:

String get= tfEingabe.getText(); 
StringBuffer form= new StringBuffer(); 
int rechnung= 0; 
	  
for(short i=0; i<get.length(); i++) 
{ 
    if (get.charAt(i) >= '0' && get.charAt(i) <= '9') { 
        form.append(tfEingabe.getText().charAt(i)); 
    } else {
        if (get.charAt(i)=='+') 
        rechnung += Integer.parseInt(form.toString()); 
        form.setLength(0);
    }
} 
		
if (form.length() > 0)
    rechnung += Integer.parseInt(form.toString());
		
if(rechnung!=0) 
    tfEingabe.setText(Integer.toString(rechnung)); 
else 
    tfEingabe.setText(form.toString());

Ich vermute, dass deine Klammersetzung nicht ganz koscha ist.
Bei mir funzte es mit deinem Code zunächst auch nicht. Erst, als ich die Klammern wie oben gesetzt hatte, lief alles problemlos.

Viele Grüße
Jens

 

[Reality]

Hi Jens,
danke so funktioniert es!
Du hast mir jedoch diesen Code vorenthalten:

  if (form.length() > 0)
    rechnung += Integer.parseInt(form.toString());

Das wird jedoch schwierig, wenn ich nun auch "-", "/" und "*" verwende.
Aber danke für den Ansatz!

Liebe Grüße
Reality

 

[Kaffeetrinker]

Hi Reality.

Inzwischen hab ich mich im Board registriert. Meine letzten Postings habe ich als 'Jens' geschrieben.

Besonders schwer wirds, wenn du auch noch Klammern und Potenzen rechnen willst.

Ich würde den String zunächst zerlegen, Klammern zusammenfassen, Punkt- vor Strichrechnung und danach alles nach und nach zusammenrechnen. Puuh. Oder nur eine einzelne Operation zulassen. :autsch:

Viel Erfolg trotzdem.
Viele Grüße
Jens

 

[Beni]

Falls du nach einer komplizierten Lösung suchst: hier

 

[Reality]

Hi, danke für eure Antworten!
Jetzt wo ich so überlege... Wäre das alles nicht mit ArrayList viel einfacher?!

Liebe Grüße
Reality