Problem mit NullPointerException

[gast²]

import java.awt.*;
import java.awt.event.*;

public class rechner3 extends Frame {

  // Objekte
  Label zahl1, zahl2, ergebnis;
  TextField zahl1_feld, zahl2_feld, ergebnis_feld;
  Button berechnen;
  Choice auswahl;

  public rechner3()
  {
    setTitle("Rechner");
    setSize(340,220);

    // Fenster schliessen
    addWindowListener(new WindowAdapter()
     { public void windowClosing(WindowEvent e){System.exit(0);}});


    //ActionListener
    class berechnen_exe implements ActionListener
    {
      public void actionPerformed(ActionEvent ereignis)
      {
        String Z1 = zahl1_feld.getText();
        String Z2 = zahl2_feld.getText();
        int erg=0;

        if (auswahl.getSelectedItem() == "Addieren")
          erg = Integer.parseInt(Z1) + Integer.parseInt(Z2);

        else if (auswahl.getSelectedItem() == "Subtrahieren")
          erg = Integer.parseInt(Z1) - Integer.parseInt(Z2);

        else if (auswahl.getSelectedItem() == "Dividieren")
          erg = Integer.parseInt(Z1) / Integer.parseInt(Z2);

        else if (auswahl.getSelectedItem() == "Multiplizieren")
          erg = Integer.parseInt(Z1) * Integer.parseInt(Z2);

        ergebnis_feld.setText(Integer.toString(erg));
      }
    }

    //Label und TextField für Zahl1
    zahl1 = new Label("Zahl 1");
    zahl1.setBounds(25,50,50,15); //x,y,b,h
    add(zahl1);

    zahl1_feld = new TextField("0");
    zahl1_feld.setBounds(90,48,100,20); //x,y,b,h
    add(zahl1_feld);

    //Label und TextField für Zahl2
    zahl2 = new Label("Zahl 2");
    zahl2.setBounds(25,90,50,15); //x,y,b,h
    add(zahl2);

    zahl2_feld = new TextField("0");
    zahl2_feld.setBounds(90,88,100,20); //x,y,b,h
    add(zahl2_feld);

    //Label und TextField fürs Ergebnis
    ergebnis = new Label("Ergebnis");
    ergebnis.setBounds(25,130,60,15); //x,y,b,h
    add(ergebnis);

    ergebnis_feld = new TextField();
    ergebnis_feld.setBounds(90,128,100,20); //x,y,b,h
    add(ergebnis_feld);

    //Choice

    Choice auswahl = new Choice();
    auswahl.addItem("Addieren");
    auswahl.addItem("Subtrahieren");
    auswahl.addItem("Dividieren");
    auswahl.addItem("Multiplizieren");
    auswahl.setBounds(210,88,100,20); //x,y,b,h
    add(auswahl);


    //Buttons fürs Berechnen
    berechnen = new Button();
    berechnen.setLabel("Berechnen");
    berechnen.setBounds(210,127,100,20); //x,y,b,h
    add(berechnen);
    berechnen.addActionListener(new berechnen_exe());


    setLayout(new BorderLayout());


    //Fenster anzeigen
    setVisible(true);
  }

  public static void main(String[] args)
  {
  new rechner3();
  }

}

Es wird zwar erfolgreich compeliert, aber wenn ich im Programm selber den "Berechnen"-Button drücke, kommt im Java-Editor undten so eine Fehlermeldung;

Exception occurred during event dispatching:
java.lang.NullPointerException
 at rechner3$1$berechnen_exe.actionPerformed(rechner3.java:31)
 at java.awt.Button.processActionEvent(Button.java:301)
 at java.awt.Button.processEvent(Button.java:274)
 at java.awt.Component.dispatchEventImpl(Component.java:2393)
 at java.awt.Component.dispatchEvent(Component.java:2306)
 at java.awt.EventQueue.dispatchEvent(EventQueue.java:295)
 at java.awt.EventDispatchThread.pumpOneEventForComponent(EventDispatchThread.java, Compiled Code)
 at java.awt.EventDispatchThread.pumpEventsForComponent(EventDispatchThread.java:89)
 at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:84)
 at java.awt.EventDispatchThread.run(EventDispatchThread.java:76)
Exception occurred during event dispatching:
java.lang.NullPointerException
 at rechner3$1$berechnen_exe.actionPerformed(rechner3.java:31)
 at java.awt.Button.processActionEvent(Button.java:301)
 at java.awt.Button.processEvent(Button.java:274)
 at java.awt.Component.dispatchEventImpl(Component.java:2393)
 at java.awt.Component.dispatchEvent(Component.java:2306)
 at java.awt.EventQueue.dispatchEvent(EventQueue.java:295)
 at java.awt.EventDispatchThread.pumpOneEventForComponent(EventDispatchThread.java, Compiled Code)
 at java.awt.EventDispatchThread.pumpEventsForComponent(EventDispatchThread.java:89)
 at java.awt.EventDispatchThread.pumpEvents(EventDispatchThread.java:84)
 at java.awt.EventDispatchThread.run(EventDispatchThread.java:76)

Irgendwas funktioniert mit meinem Button nicht, mein Lehrer konnte mir auch nicht helfen :bahnhof:

Edit von L-ectron-X: Titel geändert.

 

[MASTERmind]

eine null pointer exception heißt, dass du auf etwas zugreifst, dass noch gar nicht existiert!



mal die frage, warum du das nicht mit swing componenten machst???

  // Objekte
  Label zahl1, zahl2, ergebnis;
  TextField zahl1_feld, zahl2_feld, ergebnis_feld;
  Button berechnen;
  Choice auswahl;

Bin mir nicht sicher aber das sind glaube ich Objektreferenzvariablen, oder???

 

[KSG9|sebastian]

steht doch da..

aber erstmal:

1. klassen werden groß geschrieben
2. ist es unnötig ne innere klasse zu machen, so geht es einfacher und schneler:

button.addActionListener(){new ActionListener(){
 public void actionPerformed(ActionEvent e){

 }
});

3. Strings vergleicht man mit equals ( if(a.equals("test") ), wie oft muss man das noch sagen...
4. deine ChoiceBox (auswahl) ist "null", d.h. sie ist nicht erzegt

=> Choice c = new Choice()

..was auch immer Choice ist

 

[The_God]

wie oft muss man das noch sagen, Strings vergleicht man auch mit compareTo, da weiß man dann auch ob der verglichene String kleiner größer oder gleich ist ...

 

[Snape]

Ist aber fast nie notwendig zu wissen.

 

[Guest]

Hallo,

@The_God: Sorry, aber der Tipp war ein Griff ins Klo, denn die compareTo()-Implementierung ist komplexer und i.d.R. langsamer als die equals()-Implementierung in java.util.String. Das liegt u.a. daran, dass compareTo() auch zwei unterschiedlich lange Strings vollständig vergleichen muss, während equals() das als erstes prüft und direkt false zurück gibt.

Also: compareTo() nur verwenden, wenn man an kleiner/größer auch interessiert ist.

Hmm, wenn ich mir Deinen NickName ansehe, war das jetzt wohl Ketzerei :wink:

Gruß,
Karl

 

[gast²]

@MASTERmind

Wir machen das in der Schule und sollen halt erstmal mit AWT machen.

@KSG9|plak
Ein Mitschüler hat es auch so gemacht, wie ich das auch geschrieben habe. Danke für deine Antwort. Wird mir bestimmt weiterhelfen.


Danke nochmal an euch alle!

 

[gast²]

irgendwie weiß nicht immer noch nicht wo mein Fehler ist ???:L

 

[Karl]

Hallo,

sorry, da haben wir Dich ja völlig vergessen bei der Diskutiererei ;-)

Also, das Problem liegt hier

[...]
    Choice auswahl = new Choice();
    auswahl.addItem("Addieren");
[...]

Du hast damit die Variable auswahl lokal ein zweites mal deklariert und dann initialisiert. Die globale
Variable auswahl wird niemals initialisiert - daher der Nullpointer beim späteren Zugriff.

Ändere die Stelle einfach in ...

[...]
    auswahl = new Choice(); //nicht erneut deklarieren
    auswahl.addItem("Addieren");
[...]

... und Dein Programm läuft.

Gruß,
Karl

 

[Guest]

vielan danke für deine hilfe karl :applaus:

 

[Dominicana]

3. Strings vergleicht man mit equals ( if(a.equals("test") ), wie oft muss man das noch sagen...

kurzer Ausschnitt eines Original Borland Codes :

String pkdId = request.getParameter("_id");
File f = null;
If (pkgId == null) {
    return;
}
try {
    f = new File(Base_Dir + SEP + pkgId);
    process(f);
} finally {
    if (f != null) {
        f.delete();
    }
}

Hab es selbst getestet:

    void iconEinfuegenButton_actionPerformed(ActionEvent actionEvent, JButton button) {
        FileDialog fdg = new FileDialog(this, "Cover einfügen", FileDialog.LOAD);
        fdg.setSize(300, 200);
        fdg.show();
        String dir = fdg.getDirectory();
        String file = fdg.getFile();
        if (file == "" || file == null) {

        } else {
            button.removeAll();
            ImageIcon neuesIcon = new ImageIcon("" + dir + file);
            int x = neuesIcon.getIconWidth();
            int y = neuesIcon.getIconHeight();
            int z = (int) (y * 100) / x;
            neuesIcon.setImage(neuesIcon.getImage().getScaledInstance(100, z, 0));
            button.setIcon(neuesIcon);
            button.setText("");
            button.setName("" + dir + file);
            button.validate();
        }
    }

Wenn ich stattdessen if (file.equals("") { schreibe, bekomme ich nullPointerExceptions....
Also wie ist es denn nun richtig???

 

[SebiB90]

also file == null und file.equals("") sind 2 ganz unterschiedliche sachen, beim ersten wird überprüft ob die Variable leer ist ganz leer also gar kein string da ist, beim 2. wird überprüft ob ein der inhalt des String leer ist, aber ein string ist vorhanden in der variable file da man auf die methoden equals() zugreift.
du bekommst jetzt ein fehler weil file anscheind null ist also ganz leer aber auf eine methode zugreifen willst, da du aber kein object hast kannste auf keine methode zugreifen
also erst auf null testen wenn dann ein object vorhanden ist auf "" testen