TableView中的JavaFX内存泄漏与动态TableColumns

时间:2015-03-18 23:53:35

标签: memory-leaks javafx tableview javafx-8 tablecolumn

我正在使用jdk1.8.0_40

我想创建一个可以动态更改列的TableView,但它会泄漏Old / Tenured gen中的内存并最终挂起JVM。

这是一个演示泄漏的简单程序(不包括导入):

public class Main extends Application {
    @Override
    public void start(Stage primaryStage) {    
        TableView<ObservableList<String>> tableView = new TableView<>();
        Button button = new Button("button");

        button.setOnAction((actionEvent) -> {
            // Clearing items and/or columns here doesn't remove the leak
            tableView.getItems().clear();
            tableView.getColumns().clear();

            for (int g = 0; g < 50; g++) { // iterate 50 times to see the leak quickly
                TableColumn<ObservableList<String>, String> tableColumn = new TableColumn<>("column");
                tableView.getColumns().add(tableColumn);
            }
                tableView.getItems().add(FXCollections.observableArrayList("1"));

            // clearing the items here removes the leak somehow, but also makes the table useless
            /* tableView.getItems().clear(); */
        });

        primaryStage.setScene(new Scene(new VBox(tableView, button), 800, 600));
        primaryStage.show();
    }    
    public static void main(String[] args) { launch(args); }
}

在上面的代码中,第二次按下按钮应该清除所有项目和列,但确实如此,但某些参考文件仍会存在。

我已尝试过的一些事情:

  • 尝试搜索类似的问题,但没有发现任何相关内容。
  • 尝试将所有引用存储在ArrayList中并显式删除() - 而不是clear() - 没有工作。
  • 尝试使用ListView,效果很好,但它不是一个表,不能有列。
  • 尝试使用不同的类组合(例如StringProperty而不是String),这并不令人惊讶也没有用。

我最近刚开始学习JavaFX,所以我可能只是遗漏了一些明显的东西,如果是这样的话 - 对于一个愚蠢的问题感到抱歉。

如何解决此漏洞?

如果它无法修复,它是一个错误,还是这个预期的行为,我不应该动态创建TableColumn?

2 个答案:

答案 0 :(得分:2)

要回答我自己的问题,以防有人也遇到这个问题 - 这是一个错误。

我提交了一份JIRA错误报告,有关更多信息,可能会有更新,请参阅https://javafx-jira.kenai.com/browse/RT-40325

答案 1 :(得分:0)

如果您怀疑存在泄漏,请先通过使用分析器识别它来修复它。

我重写了你的样本(没有显示任何数据btw),似乎没有泄漏。试试这个并检查控制台输出。

import javafx.application.Application;
import javafx.beans.property.SimpleStringProperty;
import javafx.beans.property.StringProperty;
import javafx.collections.FXCollections;
import javafx.collections.ObservableList;
import javafx.scene.Scene;
import javafx.scene.control.Button;
import javafx.scene.control.TableColumn;
import javafx.scene.control.TableView;
import javafx.scene.layout.VBox;
import javafx.stage.Stage;

public class MainLeak extends Application {

    private static final int iterations = 100;
    private static final int columns = 50;
    private static final int rows = 100;

    @Override
    public void start(Stage primaryStage) {

        TableView<ObservableList<StringProperty>> table = new TableView<>();

        Button button = new Button("button");

        button.setOnAction((actionEvent) -> {

            addData( table, iterations);


        });

        addData( table, iterations);

        primaryStage.setScene(new Scene(new VBox(table, button), 800, 600));
        primaryStage.show();
    }

    private void addData( TableView table, int iterations) {

        for (int i = 0; i < iterations; i++) {

            final int index = i;

            addData(table);

            System.gc();

            System.out.println(index + ": free: " + Runtime.getRuntime().freeMemory() / 1024 + " kb, max: " + Runtime.getRuntime().maxMemory() / 1024 + " kb");

        }

    }

    private void addData(TableView table) {

        table.getItems().clear();
        table.getColumns().clear();

        for( int col=0; col < columns; col++) {
            table.getColumns().add(createColumn(col));
        }

        for( int row=0; row < rows; row++) {
            ObservableList<StringProperty> data = FXCollections.observableArrayList();
            for( int col=0; col < columns; col++) {
                data.add(new SimpleStringProperty(String.valueOf( row + " / " + col)));
            }
            table.getItems().add(data);
        }
    }

    private TableColumn<ObservableList<StringProperty>, String> createColumn(final int columnIndex) {

        TableColumn<ObservableList<StringProperty>, String> column = new TableColumn<>();
        String title = "Column " + columnIndex;
        column.setText(title);
        column.setCellValueFactory(cellDataFeatures -> cellDataFeatures.getValue().get(columnIndex));

        return column;
    }

    public static void main(String[] args) {
        launch(args);
    }
}
相关问题