なんでNullPointerExceptionになるのかわからない。

Question

JAVAの初心者です。練習でJavaFXを使って選択ダイアログから選んだ画像ファイルを表示するプログラムを作成しているのですが、一応コードは書いたんですが、今度は「NullPointerException」エラーになって動きません。
なぜ、「NullPointerException」エラーになるのでしょうか？

又、「NullPointerException」エラーを回避するコツとかはあるのでしょうか？

package rensyuuyou;

import java.io.*;
import java.util.logging.Level;
import java.util.logging.Logger;

import javafx.application.Application;
import javafx.event.ActionEvent;
import javafx.event.EventHandler;
import javafx.geometry.Insets;
import javafx.geometry.Pos;
import javafx.scene.Scene;
import javafx.scene.control.Button;
import javafx.scene.layout.VBox;
import javafx.stage.Stage;
import javafx.stage.FileChooser;
import javafx.scene.image.Image;
import javafx.scene.image.ImageView;
import javafx.stage.FileChooser;
import javafx.scene.control.Alert;

public class Rensyuuyou extends Application {
     
    String filepath;
    
    @Override
    public void start(Stage stage) {
        stage.setTitle("練習用");
            stage.setWidth(2000);
                stage.setHeight(1000);
                
                  FileInputStream fis = null;
                    try{
                        fis = new FileInputStream(filepath);
                    }catch(IOException e){
                         Alert keikoku = new Alert(Alert.AlertType.ERROR);
                            keikoku.setContentText("ファイルが読み込めません");
                    }finally{
                        if(fis != null){
                            try {
                                fis.close();
                                } catch (Exception e) {
                            }
                        }
                    }
                            Image image = new Image(fis);
                                ImageView imageView = new ImageView(image);
                                    imageView.setFitHeight(500);
                                        imageView.setFitWidth(1000);
                                
                
                    Button botan = new Button("画像表示");
                        botan.setPrefSize(900, 30);
                            botan.setOnAction(new EventHandler<ActionEvent>() {

                @Override
                public void handle(ActionEvent event) {
                    FileChooser fileChooser = new FileChooser();
                         fileChooser.setTitle("画像を開く");  
                              fileChooser.getExtensionFilters().addAll(
                                  new FileChooser.ExtensionFilter
                                    ("イメージファイル", "*.jpg", "*.png"));
                                       File file = fileChooser.showOpenDialog(stage);
                                            filepath = file.getAbsolutePath();
                                            
            }
        });
                                          
                        VBox imageBox = new VBox();
                            imageBox.setAlignment(Pos.TOP_CENTER);
                                imageBox.setPadding(new Insets(40, 40, 40, 40));
                                    imageBox.getChildren().addAll(imageView);
                        
                            VBox hyouji = new VBox();
                                    hyouji.setAlignment(Pos.CENTER);//中央下に表示
                                            hyouji.setPadding(new Insets(30, 30, 30, 30));
                                                  hyouji.getChildren().addAll(imageBox,botan);
                                                
                                stage.setScene(new Scene(hyouji));
                                    stage.show();
    }

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

Comments

[swordone]

エラーログを貼りましょう

Answer 1

NullPointerExceptionはインスタンスを参照していない参照型変数を使用すると発生します。
NullPointerExceptionは大抵プログラマーのミスです。回避するではなく、発生させないようにしないといけません。

また、エラーが発生するならエラーログを質問に載せないと時間の無駄ですよ。

Answer 2

startメソッドはウィンドウが開くときに呼ばれてしまうメソッドです。
その先頭でいきなりfilepathを使ってファイルを開こうとしていますが、あなたはまだボタンを押しておらず、FileChooserでファイルを開くという手順を踏んでいません。なので、まだfilepathがnullなままでFileInputStreamを作ろうとしたため、NullPointerExceptionが発生しています。以下の部分をhandleの中に持っていったら動くと思います。

                FileInputStream fis = null;
                    try{
                        fis = new FileInputStream(filepath);
                    }catch(IOException e){
                         Alert keikoku = new Alert(Alert.AlertType.ERROR);
                            keikoku.setContentText("ファイルが読み込めません");
                    }finally{
                        if(fis != null){
                            try {
                                fis.close();
                                } catch (Exception e) {
                            }
                        }
                    }
                            Image image = new Image(fis);
                                ImageView imageView = new ImageView(image);
                                    imageView.setFitHeight(500);
                                        imageView.setFitWidth(1000);

Comments

[mr0237]

ありがとうこざいます。