Nullpointer aber wo?

[Jibos]

hi, ich bekomme in der folgenden Methode bei "String ertragString = fieldErtragErträge.getText();" einen Nullpointer und weiß nicht warum:

  public String berechneGewinn(String gewinn)   {
     
      String ertragString = fieldErtragErträge.getText();
      String aufwandString = fieldAufwandAufwendungen.getText();
     
      ertrag = Double.parseDouble(ertragString);
      aufwand = Double.parseDouble(aufwandString);
     
      double zwischenergebnis = ertrag - aufwand;
     
      return gewinn = Double.toString(zwischenergebnis);
     
   }

Weiter oben habe ich dem TextField fieldErtragErträge schon den Wert 0 zugeordnet.

fieldErtragErträge.setText("0");

Leider weiß ich deswegen nicht, warum ich dann einen Nullpointer bekomme?

 

[Flown]

Wir auch nicht, wenn du deinen StackTrace nicht postest!

 

[JCODA]

Hast du das TextField initalisiert?
Also ist fieldErtragErträge null?

 

[Jibos]

Wir auch nicht, wenn du deinen StackTrace nicht postest!

java.lang.NullPointerException
at application.MainWindowController.berechneGewinn(MainWindowController.java:166)
at application.MainWindowController.einzahlen(MainWindowController.java:246)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
at java.lang.reflect.Method.invoke(Unknown Source)
at sun.reflect.misc.Trampoline.invoke(Unknown Source)
at sun.reflect.GeneratedMethodAccessor1.invoke(Unknown Source)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(Unknown Source)
at java.lang.reflect.Method.invoke(Unknown Source)
at sun.reflect.misc.MethodUtil.invoke(Unknown Source)
at javafx.fxml.FXMLLoader$MethodHandler.invoke(Unknown Source)
at javafx.fxml.FXMLLoader$ControllerMethodEventHandler.handle(Unknown Source)
at com.sun.javafx.event.CompositeEventHandler.dispatchBubblingEvent(Unknown Source)
at com.sun.javafx.event.EventHandlerManager.dispatchBubblingEvent(Unknown Source)
at com.sun.javafx.event.EventHandlerManager.dispatchBubblingEvent(Unknown Source)
at com.sun.javafx.event.CompositeEventDispatcher.dispatchBubblingEvent(Unknown Source)
at com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(Unknown Source)
at com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(Unknown Source)
at com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(Unknown Source)
at com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(Unknown Source)
at com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(Unknown Source)
at com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(Unknown Source)
at com.sun.javafx.event.EventUtil.fireEventImpl(Unknown Source)
at com.sun.javafx.event.EventUtil.fireEvent(Unknown Source)
at javafx.event.Event.fireEvent(Unknown Source)
at javafx.scene.Node.fireEvent(Unknown Source)
at javafx.scene.control.Button.fire(Unknown Source)
at com.sun.javafx.scene.control.behavior.ButtonBehavior.mouseReleased(Unknown Source)
at com.sun.javafx.scene.control.skin.BehaviorSkinBase$1.handle(Unknown Source)
at com.sun.javafx.scene.control.skin.BehaviorSkinBase$1.handle(Unknown Source)
at com.sun.javafx.event.CompositeEventHandler$NormalEventHandlerRecord.handleBubblingEvent(Unknown Source)
at com.sun.javafx.event.CompositeEventHandler.dispatchBubblingEvent(Unknown Source)
at com.sun.javafx.event.EventHandlerManager.dispatchBubblingEvent(Unknown Source)
at com.sun.javafx.event.EventHandlerManager.dispatchBubblingEvent(Unknown Source)
at com.sun.javafx.event.CompositeEventDispatcher.dispatchBubblingEvent(Unknown Source)
at com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(Unknown Source)
at com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(Unknown Source)
at com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(Unknown Source)
at com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(Unknown Source)
at com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(Unknown Source)
at com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(Unknown Source)
at com.sun.javafx.event.EventUtil.fireEventImpl(Unknown Source)
at com.sun.javafx.event.EventUtil.fireEvent(Unknown Source)
at javafx.event.Event.fireEvent(Unknown Source)
at javafx.scene.Scene$MouseHandler.process(Unknown Source)
at javafx.scene.Scene$MouseHandler.access$1500(Unknown Source)
at javafx.scene.Scene.impl_processMouseEvent(Unknown Source)
at javafx.scene.Scene$ScenePeerListener.mouseEvent(Unknown Source)
at com.sun.javafx.tk.quantum.GlassViewEventHandler$MouseEventNotification.run(Unknown Source)
at com.sun.javafx.tk.quantum.GlassViewEventHandler$MouseEventNotification.run(Unknown Source)
at java.security.AccessController.doPrivileged(Native Method)
at com.sun.javafx.tk.quantum.GlassViewEventHandler.lambda$handleMouseEvent$354(Unknown Source)
at com.sun.javafx.tk.quantum.QuantumToolkit.runWithoutRenderLock(Unknown Source)
at com.sun.javafx.tk.quantum.GlassViewEventHandler.handleMouseEvent(Unknown Source)
at com.sun.glass.ui.View.handleMouseEvent(Unknown Source)
at com.sun.glass.ui.View.notifyMouse(Unknown Source)
at com.sun.glass.ui.win.WinApplication._runLoop(Native Method)
at com.sun.glass.ui.win.WinApplication.lambda$null$148(Unknown Source)
at java.lang.Thread.run(Unknown Source)

MainWindowController.java:166 ist dabei "String ertragString = fieldErtragErträge.getText();" und
MainWindowController.java:246 is ein Bezug auf den return Wert der Methode. Ich habe es extra schon verkürtzt in meine Problembeschreibung gepackt, wei ldie Zeile ja bei einem Auschnitt sonst nicht stimmen.

Hast du das TextField initalisiert?
Also ist fieldErtragErträge null?

Ja, ich habe dem TextField per "fieldErtragErträge.setText("0");" die 0 zugewiesen, oder ist das falsch?

 

[kneitzel]

Kannst Du uns mehr Code zeigen? Wo setzt Du fieldErtragErträge?

 

[Jibos]

Also, dann poste ich mal die ganze Klasse. Die Zeile kommt ihr ja dem StackTrace entnehmen. Ich weiß leider nicht wo es her kommt:

public class MainWindowController {
  
   public Main main;
  
   private MenuItem auftragErstellen;
  
   //Tabelle
  
  private static Connection con;

  private static Statement stat;

  private PreparedStatement prep;

  private ObservableList<UserData> data;
  
   @FXML private TextField fieldBankvermögen;
  
   //Kostenrechnungen
  
   private String gewinn = "0";
   private double ertrag = 0;
   private double aufwand = 0;
  
   private String umsatz = "0";
   private double verkaufspreis;
   private double verkaufsmenge;
  
   private String bankkonto = "0";
   private String kasse = "0";
  
   //Startseite
  
   @FXML private TextField fieldBankkonto;
   @FXML private TextField fieldKasse;
   @FXML private TextField fieldForderungen;
   @FXML private TextField fieldVerbindlichkeiten;
  
   //Einzahlung tätigen
  
   @FXML private TextField fieldBetrag;
   @FXML private TextField fieldBetreff;
   @FXML private TextField fieldKontoart;
   @FXML private TextField fieldKapitalherkunft;
   @FXML private DatePicker fieldDatumEinzahlung;
  
  
   //Erträge/Aufwendungen
  
   @FXML private TextField fieldErtragErträge;
   @FXML private TextField fieldAufwandAufwendungen;
   @FXML private TextField fieldVerkaufspreis;
   @FXML private TextField fieldVerkaufsmenge;
  
   //Tabelle Überblick
  
   @FXML TableView tableÜberblick;
  
   @FXML
   private TableColumn tableDatumÜberblick;
   @FXML
   private TableColumn tableBankkontoÜberblick;
   @FXML
   private TableColumn tableKasseÜberblick;
   @FXML
   private TableColumn tableGewinnÜberblick;
   @FXML
   private TableColumn tableUmsatzÜberblick;
  
   URL url;
   ResourceBundle rb;
  
   public void setMain(Main main)   {
   this.main = main;
   fillFields();
  
   }
  
   public void tabelleErstellen()   {
    
     initialize(url, rb);
    
   }
  
   public void auftragErstellenMenu()   {
    
     try   {
      
  FXMLLoader fxmlLoader = new FXMLLoader(getClass().getResource("/application/auftragErstellen.fxml"));
  Parent root1 = (Parent) fxmlLoader.load();
  Stage stage = new Stage();
  stage.initModality(Modality.APPLICATION_MODAL);
  stage.setTitle("Neuer Auftrag");
  stage.setScene(new Scene(root1)); 
  stage.show();
  }
    
     catch(Exception e) {
      
    e.printStackTrace();
    
    }
    
   }
  
   public void einzahlungTätigenMenu()   {
    
     try   {
      
  FXMLLoader fxmlLoader = new FXMLLoader(getClass().getResource("/application/einzahlungTätigen.fxml"));
  Parent root2 = (Parent) fxmlLoader.load();
  Stage stage = new Stage();
  stage.initModality(Modality.APPLICATION_MODAL);
  stage.setTitle("Neuer Auftrag");
  stage.setScene(new Scene(root2)); 
  stage.show();
  }
    
     catch(Exception e) {
      
    e.printStackTrace();
    
    }
    
    
   }
  
   public String getDateAsString() {
    
      DateFormat formatter = new SimpleDateFormat("dd.MM.yyyy");
      return formatter.format(new Date());
     }
  
   public String berechneGewinn(String gewinn)   {
    
      String ertragString = fieldErtragErträge.getText();
      String aufwandString = fieldAufwandAufwendungen.getText();
    
      ertrag = Double.parseDouble(ertragString);
      aufwand = Double.parseDouble(aufwandString);
    
      double zwischenergebnis = ertrag - aufwand;
    
      return gewinn = Double.toString(zwischenergebnis);
    
   }
  
   public String berechneUmsatz(String umsatz)   {
    
     verkaufspreis = Double.parseDouble(fieldVerkaufspreis.getText());
     verkaufsmenge = Double.parseDouble(fieldVerkaufsmenge.getText());
    
     double zwischenergebnis = verkaufspreis * verkaufsmenge;
    
     return umsatz = Double.toString(zwischenergebnis);
    
   }
  
   public String berechneBankkonto(String bankkonto)   {
    
     String einzahlBetrag = fieldBetrag.getText();
    
     double zwischenergebnis = Double.parseDouble(bankkonto) + Double.parseDouble(einzahlBetrag);
    
     bankkonto = Double.toString(zwischenergebnis);
  
     return bankkonto;
   }
  
   public void fillFields()   {
    
     fieldErtragErträge.setText("0");
     fieldAufwandAufwendungen.setText("0");
     fieldVerkaufspreis.setText("0");
     fieldVerkaufsmenge.setText("0");
     fieldBankkonto.setText("0");
     fieldKasse.setText("0");
     fieldForderungen.setText("0");
     fieldVerbindlichkeiten.setText("0");
    
    
   }
  
   public String berechneKasse(String kasse)   {
  
    
    
     return kasse;
   }
  

   @FXML
   public void einzahlen()    {

     Connection c = null;
     String date = fieldDatumEinzahlung.getValue().format(DateTimeFormatter.ofPattern("yyyy-MM-dd"));

  try {
    
      try {
       Class.forName("org.sqlite.JDBC");
     } catch (ClassNotFoundException e) {
       // TODO Auto-generated catch block
       e.printStackTrace();
     }
      c = DriverManager.getConnection("jdbc:sqlite:test.db");
      
        prep = c.prepareStatement("insert into überblick values(?,?,?,?,?);");
      
        prep.setString(1, date);
      
        prep.setString(2, berechneBankkonto(bankkonto));
      
        prep.setString(3, berechneKasse(kasse));
      
        prep.setString(4, berechneGewinn(gewinn));
      
        prep.setString(5, berechneUmsatz(umsatz));
      
        prep.execute();
      
        data.add(new UserData(date, bankkonto, kasse, gewinn, umsatz));
      
      
      
     } catch (Exception e) {
       // TODO Auto-generated catch block
       e.printStackTrace();
     }
  
     //updateTabelle(url, rb);
      
            
    
   }
  
   public void updateTabelle(URL url, ResourceBundle rb)   {
    
     Connection c = null;
    
     try {  

     Class.forName("org.sqlite.JDBC");
     c = DriverManager.getConnection("jdbc:sqlite:test.db");
  stat = c.createStatement();

  data = FXCollections.observableArrayList();

  ResultSet rs = c.createStatement().executeQuery("select * from überblick");

  while (rs.next()) {

  data.add(new UserData(rs.getString("Datum"), rs.getString("Bankkonto"), rs.getString("Kasse"), rs.getString("Gewinn"), rs.getString("Umsatz")));

  }

  tableDatumÜberblick.setCellValueFactory(new PropertyValueFactory("Datum"));

  tableBankkontoÜberblick.setCellValueFactory(new PropertyValueFactory("Bankkonto"));
  
  tableKasseÜberblick.setCellValueFactory(new PropertyValueFactory("Kasse"));
  
  tableGewinnÜberblick.setCellValueFactory(new PropertyValueFactory("Gewinn"));
  
  tableUmsatzÜberblick.setCellValueFactory(new PropertyValueFactory("Umsatz"));

  tableÜberblick.setItems(null);

  tableÜberblick.setItems(data);

  
  } catch (Exception e) {

  e.printStackTrace();

  System.out.println("Error on Output Data");

     }
   }
    
   public void initialize(URL url, ResourceBundle rb) {
    
     Connection c = null;
    
     try {  

     Class.forName("org.sqlite.JDBC");
     c = DriverManager.getConnection("jdbc:sqlite:test.db");
  stat = c.createStatement();

  //stat.executeUpdate("drop table if exists user");
  stat.executeUpdate("create table if not exists überblick(Datum varchar(50),Bankkonto varchar(50),Kasse varchar(50),Gewinn varchar(50),Umsatz varchar(50));");

  data = FXCollections.observableArrayList();

  ResultSet rs = c.createStatement().executeQuery("select * from überblick");

  while (rs.next()) {

  data.add(new UserData(rs.getString("Datum"), rs.getString("Bankkonto"), rs.getString("Kasse"), rs.getString("Gewinn"), rs.getString("Umsatz")));

  }

  tableDatumÜberblick.setCellValueFactory(new PropertyValueFactory("Datum"));

  tableBankkontoÜberblick.setCellValueFactory(new PropertyValueFactory("Bankkonto"));
  
  tableKasseÜberblick.setCellValueFactory(new PropertyValueFactory("Kasse"));
  
  tableGewinnÜberblick.setCellValueFactory(new PropertyValueFactory("Gewinn"));
  
  tableUmsatzÜberblick.setCellValueFactory(new PropertyValueFactory("Umsatz"));

  tableÜberblick.setItems(null);

  tableÜberblick.setItems(data);

  System.out.println("Table Created");

  } catch (Exception e) {

  e.printStackTrace();

  System.out.println("Error on Building Data");

     }
   }
  
   public static class UserData {

    
    private SimpleStringProperty tableDatumÜberblick;
    
    private SimpleStringProperty tableBankkontoÜberblick;
    
    private SimpleStringProperty tableKasseÜberblick;
    
    private SimpleStringProperty tableGewinnÜberblick;
    
    private SimpleStringProperty tableUmsatzÜberblick;


  private UserData(String tableDatumÜberblick, String tableBankkontoÜberblick, String tableKasseÜberblick, String tableGewinnÜberblick, String tableUmsatzÜberblick) {

     this.tableDatumÜberblick = new SimpleStringProperty(tableDatumÜberblick);
    
     this.tableBankkontoÜberblick = new SimpleStringProperty(tableBankkontoÜberblick);
    
     this.tableKasseÜberblick = new SimpleStringProperty(tableKasseÜberblick);
    
     this.tableGewinnÜberblick = new SimpleStringProperty(tableGewinnÜberblick);
    
     this.tableUmsatzÜberblick = new SimpleStringProperty(tableUmsatzÜberblick);
    
  }
  
  public String getTableDatumÜberblick() {
    
     return tableDatumÜberblick.get();
    
  }
  
  public void setTableDatumÜberblick(String Datum) {
    
     tableDatumÜberblick.set("Datum");
    
  }
  
  public StringProperty tableDatumÜberblickProperty() {
    
     return tableDatumÜberblick;
    
  }
  
  public String getTableBankkontoÜberblick() {
    
     return tableBankkontoÜberblick.get();
    
  }
  
  public void setTableBankkontoÜberblick(String Bankkonto) {
    
     tableBankkontoÜberblick.set("Bankkonto");
    
  }
  
  public StringProperty tableBankkontoÜberblickProperty() {
    
     return tableBankkontoÜberblick;
    
  }
  
  public String getTableKasseÜberblick() {
    
     return tableKasseÜberblick.get();
    
  }
  
  public void setTableKasseÜberblick(String Kasse) {
    
     tableKasseÜberblick.set("Kasse");
    
  }
  
  public StringProperty tableKasseÜberblickProperty() {
    
     return tableKasseÜberblick;
    
  }
  
  public String getTableGewinnÜberblick() {
    
     return tableGewinnÜberblick.get();
    
  }
  
  public void setTableGewinnÜberblick(String Gewinn) {
    
     tableGewinnÜberblick.set("Gewinn");
    
  }
  
  public StringProperty tableGewinnÜberblickProperty() {
    
     return tableGewinnÜberblick;
    
  }
  
  public String getTableUmsatzÜberblick() {
    
     return tableUmsatzÜberblick.get();
    
  }
  
  public void setTableUmsatzÜberblick(String Umsatz) {
    
     tableUmsatzÜberblick.set("Umsatz");
    
  }
  
  public StringProperty tableUmsatzÜberblickProperty() {
    
     return tableUmsatzÜberblick;
    
  } 
  
  public String getDatum() {    
  
     return tableDatumÜberblick.get();
  
  }
  
  public String getBankkonto() {    
  
     return tableBankkontoÜberblick.get();
  
  }
  
  public String getKasse() {    
  
     return tableKasseÜberblick.get();
  
  }
  
  public String getGewinn() {    
  
     return tableGewinnÜberblick.get();
  
  }
  
  public String getUmsatz() {    
  
     return tableUmsatzÜberblick.get();
  
  }
   }

 

[JCODA]

Du hast auf deinem Textfield eine Methode aufgerufen, aber hast du irgendwo

fieldErtragErträge = new TextField();

geschrieben?

 

[kneitzel]

Ok, ich vermute, dass das Problem über den Umlaut im Namen kommt und der FXML Loader daher diese Variable nicht initialisiert. Ersetze das "ä" doch bitte durch "ae" (Sowohl in der Klasse als auch in deiner fxml Datei).

@JCODA - Der FXML Loader lädt eine fxml Datei und erstellt die darin beschriebenen Elemente. Und dann wird der Controller erstellt und die Variablen initialisiert. Dazu steht das ein @FXML als Attribut davor.

 

[Jibos]

Nein, ich habe nur diese Methode zum testweise befüllen von Werten benutzt:

public void fillFields()   {
    
     fieldErtragErträge.setText("0");
     fieldAufwandAufwendungen.setText("0");
     fieldVerkaufspreis.setText("0");
     fieldVerkaufsmenge.setText("0");
     fieldBankkonto.setText("0");
     fieldKasse.setText("0");
     fieldForderungen.setText("0");
     fieldVerbindlichkeiten.setText("0");
    
    
   }
[/java]

 

[kneitzel]

Aber ich vermute, dass Du diese Funktion nirgends aufrufst. Ansonsten würdest Du da ja ebenso eine NPE bekommen.

Und ich habe auf den ersten Blick nicht den genauen Überblick, was Du treibst. Du hast da offensichtlich viel zusammen gemischt in eine Klasse, die das alles sehr unübersichtlich macht:
Ich sehe in der Klasse:
- Elemente eines Controllers
- Du öffnest in zwei Methoden ein neues Fenster. Diese Fenster haben dann wieder diese Klasse als Controller? Du bist Dir bewusst, dass da jeweils eine neue Instanz des Controllers erstellt wird?
Daher mag es sein, dass da noch andere Probleme enthalten sind. Wenn Du aber in einer @FXML Methode, die von einem Event aufgerufen wird, etwas machst, dann sollten die Controls initialisiert worden sein so die Namen soweit zugeordnet werden konnten.

 

[Jibos]

Aber ich vermute, dass Du diese Funktion nirgends aufrufst. Ansonsten würdest Du da ja ebenso eine NPE bekommen.

Und ich habe auf den ersten Blick nicht den genauen Überblick, was Du treibst. Du hast da offensichtlich viel zusammen gemischt in eine Klasse, die das alles sehr unübersichtlich macht:
Ich sehe in der Klasse:
- Elemente eines Controllers
- Du öffnest in zwei Methoden ein neues Fenster. Diese Fenster haben dann wieder diese Klasse als Controller? Du bist Dir bewusst, dass da jeweils eine neue Instanz des Controllers erstellt wird?
Daher mag es sein, dass da noch andere Probleme enthalten sind. Wenn Du aber in einer @FXML Methode, die von einem Event aufgerufen wird, etwas machst, dann sollten die Controls initialisiert worden sein so die Namen soweit zugeordnet werden konnten.

Das ich ein weiteres Fenster in der gleichen Controller Klasse aufrufen kann, hab ic hvon Stackoverflow.

Aber es lag escheinend wirklich daran, da ich jedes TextField per "private TextField fieldErtragErträge = new TextField("0");" Das new und so hatte ich vergessen. Lag wohl daran... Danke

 

[kneitzel]

Also wenn Du den FXML Loader verwendest, dann erstellst Du keine Controlls von Hand. Der Loader nimmt die fxml Datei und erstellt dann alle controls. Und durch die fx:id= Attribute in der fxml Datei gibst Du an, welche Variablen die Instanz bekommen sollen. Diese Variablen haben dann ein @FXML Attribut.

Natürlich kannst Du aus einem Controller ein neues Fenster öffnen. Das ist kein Problem. Aber man muss etwas aufpassen, dass man Dinge, die nicht zusammen gehören, nicht zusammen fasst. Und ein Problem, dass daraus entstehen kann ist dann eine Unübersichtlichkeit. Kleines Beispiel:
Du hast eine Klasse Controller. Diese enthält (wie bei Dir) sowohl Elemente, die Du selbst initialisierst und Elemente, die der Loader initialisiert.

Also sowas:

Controller controller = new Controller(); // Deine Instanz. In der kannst Du nun Variablen setzen.
controller.setMyValue(someValue); // myValue ist nun hier gesetzt.

FXMLLoader fxmlLoader =new FXMLLoader(getClass().getResource("..."));
Parent root =(Parent) fxmlLoader.load();
// Jetzt wurde eine neue Instanz von Controller erzeugt. 
// In dieser neuen Instanz ist myValue nicht gesetzt.
// In unserer Instanz controller sind die @FXML Elemente nicht initialisiert

Daher macht es Sinn, hier eine saubere Trennung zu haben. Wenn der FXML Loader den Controller initialisieren soll, dann erstelle ich keine eigene Instanz davon.

 

[kneitzel]

BTW: Am Umlaut kann es nicht liegen. Der FXMLLoader funktioniert auch einwandfrei mit Umlauten. Das habe ich bei mir einmal geprüft. Somit ist meine Vermutung diesbezüglich falsch.