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

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

ただいまの
回答率

90.53%

  • Java

    15553questions

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

  • JavaFX

    444questions

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

JavaFXにて既存の画像がある場合は、次に画像を読み込む際はその画像をクリアしてからダイアログから新たに選択した画像を表示させたい

受付中

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,502

mr0237

score 144

JAVAの初心者です。練習でJavaFXを使って既に読み込んである画像をクリアして新たにダイアログから選択した画像を表示させるプログラムを作成しているのですが、どのようにコードを書いたらベストなのか(初心者なので)思いつきません。

if文を使って書こうと思いますが、なんか他に良いのが思いつきません。ネットで探しても見つかりませんでした。

なんかいいアドバイスがあったら教えてください。よろしくお願いします。

いままでできたソースコードを上げておきます

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 {
                                } 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ページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

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

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

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

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

    詳細な説明はこちら

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

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

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

回答 1

0

ImageViewはイメージを生成する際に作るのではなく、初めから作ってしまう方法が簡単かと思います。表示すべきImageを指定しなくてもImageView自体は生成でき、そのまま画面上に配置してかまいません。...(A)

またImageViewへのImageの設定は、そのImageViewに画像が指定ずみであってもなくても気にする必要はありません。単に表示させたいImageを好きな時に設定してやりさえすればOKです。...(B)
画面の更新はJavaFXランタイムが自動的に行ってくれます。

大雑把にいえばJavaFXは画面に表示すべきもの(この場合はImage)をプロパティーへ設定しさえすれば画面の更新はプログラマーが気にしなくても自動的に行われます。その辺りは古いGUIシステムであるswingと使い勝手が大きく異なる点といえましょう。


追記:簡単にコードで表すと次のようなイメージになります。
質問者さんはif文による切り分けが必要と考えておられたようですが、そのような必要はありません。

public class Rensyuuyou extends Application {
  VBox imageBox = new VBox();
  ImageView imageView = new ImageView();

  @Override
  public void start(Stage stage) {
    ...
    imageBox.getChildren().add(imageView); // <=「上の回答の(A)」
    button.setOnAction(this::onOpenButton); // <=※メソッド参照(後述)
    ...
  }

  void onOpenButton(ActionEvent event) {
    // FileChooserでファイルを選びそこからイメージを生成する
    // 以降の処理は初回のボタンクリックでも2回目以降のボタンクリックでも処理を
    // 変える必要はない。
    Image image = ...;
    imageView.setImage(image); // <= 「上の回答の(B)」
    ...
  }
}


※メソッド参照:
本件とは関係ないことですがボタンへのアクションハンドラーの設定コードをメソッド参照を用いる方法に書き換えました。メソッド参照やlambda式はJava8以降でしか使えませんが、JavaFXを使うならJava8以降が推奨バージョンだと思いますのでlambda式やメソッド参照を用いすっきりしたコードにすることをお勧めしたいと思いこうしました。

投稿

編集

  • 回答の評価を上げる

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

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

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

  • 回答の評価を下げる

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

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

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

  • 2017/02/13 22:56

    すいません。言っている意味が分かりません。つまり、Imageクラスを書く必要はないということでしょうか?

    キャンセル

  • 2017/02/13 23:46

    「大雑把にいえばJavaFXは画面に表示すべきもの(この場合はImage)をプロパティーへ設定しさえすれば」という説明が混乱されたのでしょうか?自分が申し上げたかったのは、
    「ImageViewのインスタンスは最初から生成して画面上へ配置しておく」
    「ファイルからImageのインスタンスを生成したら、それが初回でも2回目でも処理を変える必要はなく単にImageViewのimageプロパティーへ設定すればよい」
    ということです。

    キャンセル

  • 2017/02/15 23:25

    すいません。似た質問になりますが、具体的にどこの部分をどのように書けばいいのでしょうか? よろしくお願いします。

    キャンセル

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

  • Java

    15553questions

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

  • JavaFX

    444questions

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

  • トップ
  • Javaに関する質問
  • JavaFXにて既存の画像がある場合は、次に画像を読み込む際はその画像をクリアしてからダイアログから新たに選択した画像を表示させたい