Addition zweier Zahlen

[Johannes Z]

Ich soll für Informatik in der Schule ein Programm Fenster erstellen mit 3 Textfeldern und einem Butten. Wenn man den Butten drückt sollen die Zahlen (nur ganzzahlig und positiv) aus Textfeld 1 und 2 addiert werden und ins Dritte geschrieben werden. Ich hab das bis jetzt so weit:

package arbeit;
import java.awt.BorderLayout;
import javax.swing.JPanel;
import javax.swing.JFrame;
import java.awt.Dimension;
import javax.swing.JTextField;
import java.awt.Rectangle;
import java.awt.event.ActionEvent;
import java.awt.event.ActionListener;
import javax.swing.JButton;
import javax.swing.JTextPane;
public class Oberfläche extends JFrame {
 private static final long serialVersionUID = 1L;
 private JPanel jContentPane = null;
 private JTextField Eingabe1 = null;
 private JTextField Eingabe2 = null;
 private JTextField Ausgabe = null;
 private JButton Plus = null;
 /**
  * This is the default constructor
  */
 public Oberfläche() {
  super();
  initialize();
 }
 /**
  * This method initializes this
  *
  * @return void
  */
 private void initialize() {
  this.setSize(388, 231);
  this.setContentPane(getJContentPane());
  this.setTitle("JFrame");
 }
 /**
  * This method initializes jContentPane
  *
  * @return javax.swing.JPanel
  */
 private JPanel getJContentPane() {
  if (jContentPane == null) {
   jContentPane = new JPanel();
   jContentPane.setLayout(null);
   jContentPane.add(getEingabe1(), null);
   jContentPane.add(getEingabe2(), null);
   jContentPane.add(getAusgabe(), null);
   jContentPane.add(getPlus(), null);
  }
  return jContentPane;
 }
 /**
  * This method initializes Eingabe1 
  *  
  * @return javax.swing.JTextField 
  */
 private JTextField getEingabe1() {
  if (Eingabe1 == null) {
   Eingabe1 = new JTextField();
   Eingabe1.setBounds(new Rectangle(30, 30, 211, 31));
  }
  return Eingabe1;
 }
 /**
  * This method initializes Eingabe2 
  *  
  * @return javax.swing.JTextField 
  */
 private JTextField getEingabe2() {
  if (Eingabe2 == null) {
   Eingabe2 = new JTextField();
   Eingabe2.setBounds(new Rectangle(30, 90, 211, 31));
  }
  return Eingabe2;
 }
 /**
  * This method initializes Ausgabe 
  *  
  * @return javax.swing.JTextField 
  */
 private JTextField getAusgabe() {
  if (Ausgabe == null) {
   Ausgabe = new JTextField();
   Ausgabe.setBounds(new Rectangle(30, 150, 331, 31));
  }
  return Ausgabe;
 }
 /**
  * This method initializes Plus 
  *  
  * @return javax.swing.JButton 
  */
 private JButton getPlus() {
  if (Plus == null) {
   Plus = new JButton();
   Plus.setBounds(new Rectangle(300, 75, 46, 46));
   Plus.setText("+");
   ActionListener al = new ActionListener() {
    @Override
    public void actionPerformed(ActionEvent e) {
     String meinWert1 = Eingabe1.getText();
     int Zahl1 = Integer.parseInt(meinWert1);
     
     String meinWert2 = Eingabe2.getText();
     int Zahl2 = Integer.parseInt(meinWert2);
     
     int Ausgabe1 = Zahl1+Zahl2 ;
     
     String asdf = (Integer(Ausgabe1)).toString();
     
     Ausgabe.setText(asdf);
     
          
     

     
     
    }
    private Object Integer(int ausgabe1) {
     // TODO Auto-generated method stub
     return null;
    }
   };
   Plus.addActionListener(al);
   
  }
  return Plus;
 }
}  //  @jve:decl-index=0:visual-constraint="10,10"

Wenn ich es dann ausfüre kommt diese Meldung:

Exception in thread "AWT-EventQueue-0" java.lang.NumberFormatException: For input string: ""
at java.lang.NumberFormatException.forInputString(Unknown Source)
at java.lang.Integer.parseInt(Unknown Source)
at java.lang.Integer.parseInt(Unknown Source)
at arbeit.Oberfläche$1.actionPerformed(Oberfläche.java:103)
at javax.swing.AbstractButton.fireActionPerformed(Unknown Source)
at javax.swing.AbstractButton$Handler.actionPerformed(Unknown Source)
at javax.swing.DefaultButtonModel.fireActionPerformed(Unknown Source)
at javax.swing.DefaultButtonModel.setPressed(Unknown Source)
at javax.swing.plaf.basic.BasicButtonListener.mouseReleased(Unknown Source)
at java.awt.Component.processMouseEvent(Unknown Source)
at javax.swing.JComponent.processMouseEvent(Unknown Source)
at java.awt.Component.processEvent(Unknown Source)
at java.awt.Container.processEvent(Unknown Source)
at java.awt.Component.dispatchEventImpl(Unknown Source)
at java.awt.Container.dispatchEventImpl(Unknown Source)
at java.awt.Component.dispatchEvent(Unknown Source)
at java.awt.LightweightDispatcher.retargetMouseEvent(Unknown Source)
at java.awt.LightweightDispatcher.processMouseEvent(Unknown Source)
at java.awt.LightweightDispatcher.dispatchEvent(Unknown Source)
at java.awt.Container.dispatchEventImpl(Unknown Source)
at java.awt.Window.dispatchEventImpl(Unknown Source)
at java.awt.Component.dispatchEvent(Unknown Source)
at java.awt.EventQueue.dispatchEventImpl(Unknown Source)
at java.awt.EventQueue.access$000(Unknown Source)
at java.awt.EventQueue$1.run(Unknown Source)
at java.awt.EventQueue$1.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.AccessControlContext$1.doIntersectionPrivilege(Unknown Source)
at java.security.AccessControlContext$1.doIntersectionPrivilege(Unknown Source)
at java.awt.EventQueue$2.run(Unknown Source)
at java.awt.EventQueue$2.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)
at java.security.AccessControlContext$1.doIntersectionPrivilege(Unknown Source)
at java.awt.EventQueue.dispatchEvent(Unknown Source)
at java.awt.EventDispatchThread.pumpOneEventForFilters(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForFilter(Unknown Source)
at java.awt.EventDispatchThread.pumpEventsForHierarchy(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.pumpEvents(Unknown Source)
at java.awt.EventDispatchThread.run(Unknown Source)

Ich weiß auch nicht ob es schon einen Beitrag zu dem Problem gibt weil ich keine Ahnung hab was das Problem(e) ist.



MfG Johannes

 

[jgh]

anstatt:

[java=109] String asdf = (Integer(Ausgabe1)).toString();
[/code]

das hier...und diese Exception sollte nicht mehr fliegen.
[JAVA=109]String asdf = Ausgabe1 + "";[/code]

 

[Johzi]

Jetzt funktionier es Vielen Dank.

Ich hab noch eine kleine Frage wie kann ich brüfen ob der eingegbene String eine gültige Zahl ist?

 

[Spin]

 try {
     int x = Integer.parseInt(input);
     System.out.println(x);
 }
 catch(NumberFormatException nFE) {
     System.out.println("Not an Integer");
}

Es geht auch mit der Klasse Character, aber das ist wohl zu viel für deine Zwecke

grüße Spin

 

[VfL_Freak]

Moin,

Ich soll für Informatik in der Schule ein Programm Fenster erstellen mit 3 Textfeldern und einem Butten. Wenn man den Butten drückt ...

kleiner Tipp am Rande für Diene nächste Klausur :

es heißt immer noch "Button" und nicht "Butten" (auch wenn sich durchaus so spricht)

Gruß
Klaus

 

[AmunRa]

String asdf = Ausgabe1 + "";

Das ist sehr unsauber und sollte man daher eher nicht machen. Dafür gibst eigentlich

String asdf = String.valueOf(Ausgabe1);

Und in Java sollte man Variablenname in mit einem kleinen Buchstaben beginnen. Also "lowerCamelCase".

 

[AmunRa]

Da man auch Exceptions eigentlich nicht zur Flusssteuerung verwenden sollte, gibt es noch diese Möglichkeit zu überprüfen ob eine String nur eine Zahl ist.

public boolean isInteger(String zahl){
   return zahl.matches("[-]?[0-9]+");
}

Diese Methode überprüft ob es sich um einen ganzzahligen Wert handelt, wobei auch negative Zahlen erlaubt sind. Nicht überprüft wird, ob diese Zahl auch in einem int platz hat.

 

[Landei]

Das ist sehr unsauber und sollte man daher eher nicht machen.

Inwiefern ist das "unsauber"? Es liefert das korrekte Ergebnis, wird vom Compiler optimiert und die "Absicht" des Programmierers ist auf den ersten Blick zu erkennen (sicher wäre es schöner, wenn man als Verkettungs-Operator in Java nicht ausgerechnet + verwendet hätte, aber dem ist nunmal so).

 

[AmunRa]

Es wird eben nicht wirklich weg optimiert.

Gab dazu schon einige Threads z.B. http://www.java-forum.org/java-basics-anfaenger-themen/84198-int-string-umwandeln.html

und

das Problem ist eben, dass der Compiler zumindest bei mir im Bytecode, tatsächlich versucht einen leeren String zu erzeugt und daher 3 Strings erzeugt werden , wo eigentlich nur einer von Nöten wäre.
während bei der String.valueOf Variante wirklich nur diese Methode aufgerufen wird.

 

[Landei]

OK, dann habe ich den Compiler überschätzt. Hoffentlich basteln sie diese Optimierung bald ein...