JavaFXでテキストファイルから読み取った文字列をラベルに表示しようとするとエラーになる

Question

ファイルから読み取った文字列をSystem.out.printlnで表示するのにはエラーが出ないのですが、ラベルにSetTextで表示させようとするとエラーがでます。

package money;

import java.io.BufferedReader;
import java.io.File;
import java.io.FileNotFoundException;
import java.io.FileReader;
import java.io.IOException;
import java.net.URL;
import java.util.ResourceBundle;
import javafx.event.ActionEvent;
import javafx.fxml.FXML;
import javafx.fxml.Initializable;
import javafx.scene.control.Label;
import javafx.scene.control.TextField;



public class FXMLDocumentController implements Initializable {
        @FXML
    private Label nowmoney;
    {
try{
      File file = new File("/Users/uron/Documents/test.txt");

      if (checkBeforeReadfile(file)){
        BufferedReader br = new BufferedReader(new FileReader(file));

        String str;
        while((str = br.readLine()) != null){
          System.out.println(str);
          nowmoney.setText(str);
        }

        br.close();
      }else{
        System.out.println("ファイルが見つからないか開けません");
      }
    }catch(FileNotFoundException e){
      System.out.println(e);
    }catch(IOException e){
      System.out.println(e);
    }
  }

  private static boolean checkBeforeReadfile(File file){
    if (file.exists()){
      if (file.isFile() && file.canRead()){
        return true;
      }
    }

    return false;
    }

    


    
    
    




//    @FXML
//    private void handleButtonAction(ActionEvent event) {
//        System.out.println("You clicked me!");
//        label.setText("Hello World!");
//    }

    @Override
    public void initialize(URL url, ResourceBundle rb) {
        // TODO
    }

}

エラーは

Exception in Application start method
java.lang.reflect.InvocationTargetException
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at com.sun.javafx.application.LauncherImpl.launchApplicationWithArgs(LauncherImpl.java:389)
	at com.sun.javafx.application.LauncherImpl.launchApplication(LauncherImpl.java:328)
	at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
	at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
	at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
	at java.lang.reflect.Method.invoke(Method.java:498)
	at sun.launcher.LauncherHelper$FXHelper.main(LauncherHelper.java:767)
Caused by: java.lang.RuntimeException: Exception in Application start method
	at com.sun.javafx.application.LauncherImpl.launchApplication1(LauncherImpl.java:917)
	at com.sun.javafx.application.LauncherImpl.lambda$launchApplication$155(LauncherImpl.java:182)
	at java.lang.Thread.run(Thread.java:745)
Caused by: javafx.fxml.LoadException: 
file:/Users/uron/NetBeansProjects/Money/dist/run1859694132/Money.jar!/money/FXMLDocument.fxml:10

	at javafx.fxml.FXMLLoader.constructLoadException(FXMLLoader.java:2601)
	at javafx.fxml.FXMLLoader.loadImpl(FXMLLoader.java:2579)
	at javafx.fxml.FXMLLoader.loadImpl(FXMLLoader.java:2441)
	at javafx.fxml.FXMLLoader.loadImpl(FXMLLoader.java:3214)
	at javafx.fxml.FXMLLoader.loadImpl(FXMLLoader.java:3175)
	at javafx.fxml.FXMLLoader.loadImpl(FXMLLoader.java:3148)
	at javafx.fxml.FXMLLoader.loadImpl(FXMLLoader.java:3124)
	at javafx.fxml.FXMLLoader.loadImpl(FXMLLoader.java:3104)
	at javafx.fxml.FXMLLoader.load(FXMLLoader.java:3097)
	at money.Money.start(Money.java:22)
	at com.sun.javafx.application.LauncherImpl.lambda$launchApplication1$162(LauncherImpl.java:863)
	at com.sun.javafx.application.PlatformImpl.lambda$runAndWait$175(PlatformImpl.java:326)
	at com.sun.javafx.application.PlatformImpl.lambda$null$173(PlatformImpl.java:295)
	at java.security.AccessController.doPrivileged(Native Method)
	at com.sun.javafx.application.PlatformImpl.lambda$runLater$174(PlatformImpl.java:294)
	at com.sun.glass.ui.InvokeLaterDispatcher$Future.run(InvokeLaterDispatcher.java:95)
Caused by: java.lang.NullPointerException
	at money.FXMLDocumentController.<init>(FXMLDocumentController.java:34)
	at sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method)
	at sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62)
	at sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45)
	at java.lang.reflect.Constructor.newInstance(Constructor.java:423)
	at java.lang.Class.newInstance(Class.java:442)
	at sun.reflect.misc.ReflectUtil.newInstance(ReflectUtil.java:51)
	at javafx.fxml.FXMLLoader$ValueElement.processAttribute(FXMLLoader.java:927)
	at javafx.fxml.FXMLLoader$InstanceDeclarationElement.processAttribute(FXMLLoader.java:971)
	at javafx.fxml.FXMLLoader$Element.processStartElement(FXMLLoader.java:220)
	at javafx.fxml.FXMLLoader$ValueElement.processStartElement(FXMLLoader.java:744)
	at javafx.fxml.FXMLLoader.processStartElement(FXMLLoader.java:2707)
	at javafx.fxml.FXMLLoader.loadImpl(FXMLLoader.java:2527)
	... 14 more
Exception running application money.Money
Java Result: 1
Deleting directory /Users/uron/NetBeansProjects/Money/dist/run1859694132
jfxsa-run:
ビルド成功(合計時間: 1秒)

Answer 1

FXMLやそれをロードする部分のコードが記載されてないのでjava-beginnerさんがおっしゃるようなfx:idの不一致の問題があるかどうかはわかりません。しかしそれ以前にコントローラーの作り方に問題があると思います。

FXMLをロードする際には次のように動きます。

1. ApplicationクラスのstartメソッドなどからFXMLLoader#loadを呼び出す
2. FXMLLoaderはFXML中の記述に基づいてコントローラーのインスタンスを生成

本件の場合はFXMLDocumentControllerのインスタンスがここで作られます。
3. FXMLLoaderはFXML中に記述されたノードの階層を生成
この際にfx:idの属性を見つけるとコントローラーの@FXMLアノテーションに基づき対応するコントローラーインスタンスのフィールドに該当ノードのインスタンスを自動設定します。
4. 全てのノード階層が完成したらコントローラーインスタンスに対してinitializeメソッドが呼び出される
5. FXMLLoader#loadの処理終わり

さて、質問者さんのコードではコントローラーのインスタンス初期化ブロックで@FXMLで定義したノードにnowmoney.setText(str);のようにしてアクセスしています。これは先に述べたように2で行われます。もうお分かりと思いますがコントローラーのインスタンス生成直後に@FXML指定したフィールドに値が自動設定されているわけではありません。インスタンス生成が終わった後に3で設定されます。

対処方法：
コントローラーの初期化コードをインスタンス初期化ブロックからinitializeメソッドへ全て移動するとうまくいくと思います。FXMLローダーを使う限りコントローラーの初期化はinitializeメソッドだけでやると話は単純になると思います。

Answer 2

xmlファイルに記述しているfx:idがjavaファイルと異なるのではないでしょうか？

以下のように試した結果、setTextでエラーは起きませんでした。

<?xml version="1.0" encoding="UTF-8"?>

<?import javafx.scene.control.Label?>
<?import javafx.scene.layout.BorderPane?>
<?import javafx.scene.layout.Pane?>


<BorderPane maxHeight="-Infinity" maxWidth="-Infinity"
  minHeight="-Infinity" minWidth="-Infinity" prefHeight="400.0"
  prefWidth="600.0" xmlns="http://javafx.com/javafx/8.0.65" xmlns:fx="http://javafx.com/fxml/1"
  fx:controller="test60751.AppController">
  <center>
    <Pane prefHeight="200.0" prefWidth="200.0" BorderPane.alignment="CENTER">
      <children>
        <Label fx:id="nowmoney" layoutX="58.0" layoutY="36.0"
          prefHeight="59.0" prefWidth="200.0" text="Label" />
      </children>
    </Pane>
  </center>
</BorderPane>

package test60751;

import java.net.URL;
import java.util.ResourceBundle;

import javafx.fxml.FXML;
import javafx.fxml.Initializable;
import javafx.scene.control.Label;

public class AppController implements Initializable {

    @FXML
    private Label nowmoney;

    @Override
    public void initialize(URL location, ResourceBundle resources) {
        System.out.println("Start");
        nowmoney.setText("Hello.");
    }

}