JavaFXにてボタンを押してダイアログを表示することはできたが、今度は画像が表示されない

Question

JAVAの初心者です。練習でJavaFXを使ってボタンを押して選択ダイアログを出して、画像を表示するプログラムを作りたいのですが、一応コードは選択ダイアログから画像を選択すると、画像を表示するところまで書いたのですが、今度は画像が表示されません。画像を表示するにはどうすればいいのでしょうか？

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;
        VBox imageBox = new VBox();
       
    
    @Override
    public void start(Stage stage) {
        stage.setTitle("練習用");
            stage.setWidth(2000);
                stage.setHeight(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();
                /*
                画像を表示する
                */
                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);
                        
                            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

[KSwordOfHaste]

mr0237さんの質問のコードは字下げが毎回乱れているのですが(^^; も少し字下げを配慮したほうがいいと思います。

Answer 1

コードが違ってたようなので改めてコメントします
fisをcloseしてからImageをfisから作っているように見えます。

File file = dialog.showOpenDialog();
try (FileInputStream fis = new FileInputStream(file)) {
  image = new Image(fis);
  ImageView imageView = new Image(image);
  ...
}

のようにしかるべき場所でImageを作れば解決すると思います。

なおJava7からtry-resource文が利用できますのでこれをうまく利用してはいかがでしょう？

class XX implements AutoClosable { }

try (XX x = new XX(...)) {
  // x の有効範囲(スコープ)はtryブロックの中だけ
}
// tryブロックのスコープ外に出るとxは自動的にcloseされる
...

try-resourceを使うと明示的にcloseする必要もなくストリームを保持する変数の有効範囲(tryブロック)を抜けると同時に確実にcloseするようなコードを表現できます。こうすると今回の問題のように「変数が参照可能な場所（スコープ内）で実際にはcloseされてしまっているストリームを誤って参照してしまう」といったようなバグの可能性を排除できます。

---

コード変更前の回答：

filepathに何も代入しないまま new FileInputStream(filepath)しているように見えます。これだとNull Pointer Exceptionが発生するような気がします。

今度は画像が表示されません

もしNPEが発生しているなら「画像が表示されない」ではなくNPEがこの行で発生すると書いた方がよいと思います。

Comments

[mr0237]

うまくいきました。ありがとうこざいました。