JavaFX：AnimationTimer和MenuBar之间的交互

Question

我正在开发一种软​​件，它从相机中获取图像并在JavaFX ImageView实时显示。 我有一个获取最后一个图像的线程（在本例中是一个BufferedImage ），以及一个将它分配给ImageView的AnimationTimer 。 我继续使用AnimationTimer的原因似乎是每次获得新图像时用Runnable填充平台。 刷新工作正常，FPS很不错。

但是，我注意到当AnimationTimer运行时，我的软件中的菜单栏显示不正确。 当我迷过其他菜单时，有些菜单项丢失了。 这张照片解释了它：

在左侧，您可以看到菜单通常的样子，在右侧，可以看到AnimationTimer运行时的样子。 如您所见，缺少“保存”菜单项，而是显示实时图像的背景。 此外，当我打开一个新窗口（在一个新Scene ）时，当我在任何类型的Node （一个按钮，一个复选框......）上移动时，背景变为黑色。 通过在初始化Scene时将depth-buffer boolean设置为true，我能够解决这个问题。 我不知道如何修复这个菜单栏错误，我认为这些错误表明我正在做的事情可能不对。

我想也许JavaFX应用程序线程已经饱和了要显示的新图像，并且基本上花费了太多时间来绘制其他元素（例如菜单项）。

问题：

1. 这真的是这个bug的来源吗？
2. 有没有办法改进我的代码，例如使用与AnimationTimer不同的东西？

---

这是一个重现错误的代码片段。 将start函数中的两个字符串更改为图像的路径。 图像应该相对较大（几MB）。

单击“开始”按钮以启动动画计时器。 然后尝试打开“文件”菜单，并将鼠标悬停在菜单项上。 该错误没有系统地出现，尝试重复上下移动鼠标，它应该出现在某个点上。

import java.awt.image.BufferedImage;
import java.io.File;
import java.io.IOException;
import java.util.logging.Level;
import java.util.logging.Logger;
import javafx.animation.AnimationTimer;
import javafx.application.Application;
import javafx.embed.swing.SwingFXUtils;
import javafx.scene.Scene;
import javafx.scene.control.Button;
import javafx.scene.control.Menu;
import javafx.scene.control.MenuBar;
import javafx.scene.control.MenuItem;
import javafx.scene.image.Image;
import javafx.scene.image.ImageView;
import javafx.scene.layout.BorderPane;
import javafx.scene.layout.Pane;
import javafx.stage.Stage;
import javax.imageio.ImageIO;

public class ImageRefresher extends Application {

    @Override
    public void start(Stage primaryStage) {

        //Here change the 2 Strings to a path to an image on your HDD
        //The bug appears more easily with large images (>3-4MB)
        String pathToImage1 = "/path/to/your/first/image";
        String pathToImage2 = "/path/to/your/second/image";

        try {
            //Image content (contains buffered image, see below)
            ImageContent image = new ImageContent(pathToImage1);

            //If this line is commented, the bug does not appear
            image.setImage(ImageIO.read(new File(pathToImage2)));

            //JavaFX class containing nodes (see below)
            MainWindow window = new MainWindow(image);

            Scene scene = new Scene(window.getPane(), 300, 250);
            primaryStage.setTitle("Menu refresh");
            primaryStage.setScene(scene);
            primaryStage.show();
        } catch (IOException ex) {
            Logger.getLogger(ImageRefresher.class.getName()).log(Level.SEVERE, null, ex);
        }

    }

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

    public class MainWindow {

        private BorderPane pane;
        private MenuBar menuBar;
        private ImageView displayImage;
        private Button startRefreshingButton;

        private ImageContent imageContent;
        private AnimationTimer animationTimer;

        public MainWindow(ImageContent imageContent) {
            this.imageContent = imageContent;

            //Builds the window's components
            buildGraphic();

            //The image is reset at each frame
            animationTimer = new AnimationTimer() {
                @Override
                public void handle(long now) {
                    displayImage.setImage(imageContent.getDisplayableImage());
                }
            };
        }

        private void buildGraphic() {
            pane = new BorderPane();
            menuBar = new MenuBar();

            Menu menu = new Menu("File");
            menu.getItems().addAll(new MenuItem("Save"),
                    new MenuItem("Open"),
                    new MenuItem("Close"));
            menuBar.getMenus().add(menu);

            displayImage = new ImageView();

            startRefreshingButton = new Button("Start");
            startRefreshingButton.setOnAction((event) -> {
                animationTimer.start();
            });

            pane.setTop(menuBar);
            pane.setCenter(displayImage);
            pane.setBottom(startRefreshingButton);
        }

        public Pane getPane() {
            return pane;
        }
    }

    public class ImageContent {

        private BufferedImage imageContent;

        //Initializes bufferedimage with the path specified
        public ImageContent(String pathToImage) throws IOException {
            imageContent = ImageIO.read(new File(pathToImage));
        }

        public void setImage(BufferedImage newImage) {
            imageContent = newImage;
        }

        //Function called by the animation timer to
        //get a JavaFX image from a bufferedimage
        public Image getDisplayableImage() {
            return SwingFXUtils.toFXImage(imageContent, null);
        }
    }
}

Answer 1

我想问题是，因为你每帧重新绘制图像，所以你将菜单弹出窗口覆盖在图像上。 这似乎是一个错误，但你也要求FX应用程序线程的工作量超出你的需要。

理想情况下，您应该找到一种方法来检查是否确实存在新图像，并且只有在确实存在新文件时才更新图像。 （考虑使用java.nio.file.Path来表示文件并调用Files.getLastModifiedTime(path) 。）

另一种避免使用太多Platform.runLater(...)调用来淹没FX应用程序线程的方法，请参阅限制javafx gui更新

Answer 2

最后我没有在jira上提出任何问题，因为我能够解决我的问题。 问题来自于我调用SwingFXUtils.toFXImage(imageContent, null) 。 我在每一帧都返回了这个函数的结果，我不确定细节，但这可能每次创建一个新对象。 避免这种情况的一种简单方法是将WritableImage作为参数传递，并将ImageView的ImageProperty值绑定到它。

如果我采用上面发布的MCVE，这可能是这样的（没有测试，可能是更清晰的解决方案）：

    public class MainWindow {

        private BorderPane pane;
        private MenuBar menuBar;
        private ImageView displayImage;
        private Button startRefreshingButton;

        private ImageContent imageContent;
        private AnimationTimer animationTimer;

        // Here's the value to bind
        private ObservableValue<WritableImage> imageProperty;

        public MainWindow(ImageContent imageContent) {
            //initialization stuff 

            this.imageProperty = new ObservableValue<>(imageContent.getWritableImage());
            displayImage.imageProperty().bind(imageProperty);

            //The image is reset at each frame
            animationTimer = new AnimationTimer() {
                @Override
                public void handle(long now) {
                    SwingFXUtils.toFXImage(imageContent.getBufferedImage(), image.getWritableImage());
                }
            };
        }
    }


    public class ImageContent {

        private BufferedImage imageContent;
        private WritableImage writableImage;

        //Initializes bufferedimage with the path specified
        public ImageContent(String pathToImage) throws IOException {
            imageContent = ImageIO.read(new File(pathToImage));

            //Get the width and height values from your image
            int width = imageContent.getWidth();
            int height = imageContent.getHeight();
            writableImage = new WritableImage(width, height);
        }

        public void setImage(BufferedImage newImage) {
            imageContent = newImage;
        }

        public WritableImage getWritableImage() {
            return writableImage;
        }

        public BufferedImage getBufferedImage() {
            return imageContent;
        }
    }

但是，现在这似乎是内存密集型的，我会调查一下。