質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.52%

  • Java

    13779questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

  • JavaFX

    396questions

    JavaFXとは、Java仮想マシン上で動作するリッチインターネットアプリケーション (RIA) のGUIライブラリです。Swingとは異なり、FXMLと呼ばれる XMLとCSSを併用してデザインを記述します。

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

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 617

mr0237

score 143

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);
    }

}

(追記)すいません。違うソースコードを上げてしまいました。アップしたのでよろしくお願いします。

  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • KSwordOfHaste

    2017/02/10 23:26

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

    キャンセル

回答 1

checkベストアンサー

0

コードが違ってたようなので改めてコメントします
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がこの行で発生すると書いた方がよいと思います。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/02/12 16:26

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

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.52%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • Java

    13779questions

    Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

  • JavaFX

    396questions

    JavaFXとは、Java仮想マシン上で動作するリッチインターネットアプリケーション (RIA) のGUIライブラリです。Swingとは異なり、FXMLと呼ばれる XMLとCSSを併用してデザインを記述します。

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