JavaFX使用线程动态更快地填充GridPane

Question

我正在从我的数据库加载数据并在GridPane中动态创建按钮！但是有很多数据需要很长时间才能看到按钮！我正在使用线程从数据库加载数据，但似乎创建每个按钮并填满GridPane需要太长时间！

**我尝试创建硬编码数据并用它们填充GridPane只是为了测试但是又花了太长时间，所以我得出结论，使按钮花费太长时间显示的问题是填满GridPane，而不是加载数据来自DB！

有没有办法如何更快地填充GridPane？我使用2 for循环，一个用于行，另一个用于列，因此复杂度为（n平方）！

我能优化吗？

以下是我创建和填充GridPane的功能：

public void initialize() throws Exception {
    initializedProducts = new ArrayList<>();
    productsOnReceipt = new ArrayList<>();
    rows = new VBox();
    grandPrice = 0;

    //Initializing DatabaseConnections and accessors for each sections
    ProductDao burgerAccessor = new ProductDaoImpl();
    ProductDao burgerMenuAccessor = new ProductDaoImpl();
    ProductDao drinksAccessor = new ProductDaoImpl();

    Executor exec = Executors.newCachedThreadPool(runnable -> {
        Thread t = new Thread(runnable);
        t.setDaemon(true);
        return t;
    });

    initSections(burgers_section, 2, "PICI", exec, burgerAccessor);
    initSections(burgersMenu_section, 2, "TOPLI JADENJA", exec, burgerMenuAccessor);
    initSections(drinks_section, 1, "BEZALKOHOLNI PIJALOCI", exec, drinksAccessor);
}

初始化产品部分：

private void initSections(ScrollPane section, int rows, String category, Executor executor, ProductDao products) {
    GridPane grid = new GridPane();
    grid.setPadding(new Insets(5));
    grid.setHgap(20);
    grid.setVgap(-7);
    grid.setAlignment(Pos.TOP_LEFT);

    initProducts(grid, rows, category, executor, products);
    section.setContent(grid);
}

为每个部分初始化产品：

private void initProducts(GridPane grid, int rows, String category, Executor executor, ProductDao accessor) {
    Task<Products> productsResultTask = new Task<Products>() {
        @Override
        protected Products call() throws Exception {
            return accessor.getByCategory(category);
        }
    };

    productsResultTask.setOnFailed(e -> productsResultTask.getException().printStackTrace());

    productsResultTask.setOnSucceeded(e -> {
        if (productsResultTask.getValue() != null) {
            ArrayList<Product> products = productsResultTask.getValue().getProducts();
            int productCounter = 0;
            if (rows == 1) {
                for (int j = 0; j < products.size(); j++) {
                    initializeButton(category, products.get(productCounter), grid, 0, j);
                    initializedProducts.add(products.get(productCounter));
                    productCounter++;
                }
            } else {
                if (isEven(products.size())) {
                    for (int i = 0; i < rows; i++) {
                        for (int j = 0; j < products.size() / 2; j++) {
                            initializeButton(category, products.get(productCounter), grid, i, j);
                            initializedProducts.add(products.get(productCounter));
                            productCounter++;
                        }
                    }
                } else {
                    for (int i = 0; i < rows; i++) {
                        for (int j = 0; j < (products.size()-1) / 2; j++) {
                            initializeButton(category, products.get(productCounter), grid, i, j);
                            initializedProducts.add(products.get(productCounter));
                            productCounter++;
                        }
                    }
                    initializeButton(category, products.get(productCounter), grid, 0, ((products.size() - 1) / 2));
                    initializedProducts.add(products.get(productCounter));
                }
            }
        } else {
            System.out.println("Can't fetch products from DB!");
        }
    });
    executor.execute(productsResultTask);
}

**我不认为下面的initializeButton（）函数存在问题，但万一你可以看到工作流程！

初始化每个产品的每个按钮：

private void initializeButton(String category, Product product, GridPane grid, int i, int j) {
        Label bName = new Label(product.getName());
        bName.setStyle("-fx-font-size: 16;");
        bName.setPrefWidth(130);
        bName.setWrapText(true);
        bName.setStyle("-fx-text-alignment: center;");
        bName.setAlignment(Pos.CENTER);

        Button button = new Button();
        button.setPrefHeight(110);
        button.setPrefWidth(130);

        BackgroundImage backgroundImage = new BackgroundImage(new Image(getClass().getResource("/images/burgerm.png").toExternalForm()),
                BackgroundRepeat.NO_REPEAT, BackgroundRepeat.NO_REPEAT, BackgroundPosition.CENTER, new BackgroundSize(120, 100, false, false, false, false));
        Background background = new Background(backgroundImage);
        button.setBackground(background);
        button.setStyle("-fx-effect: dropshadow(three-pass-box, rgba(0,0,0,0.8), 10, 0, 0, 0);");
        button.setOnAction(event -> {
            if (category.equals("TOPLI JADENJA")) {
                askForDrink(product);
            } else {
                addToReceipt(product);
            }

        });

        VBox item = new VBox();
        item.getChildren().addAll(button, bName);
        item.setAlignment(Pos.TOP_CENTER);
        item.setSpacing(-8);
        item.setPrefWidth(110);
        grid.add(item, j, i);
    }

Answer 1

你在那里有一些潜在的性能杀手。

为什么一遍又一遍地加载backgroundImage。您可以在循环外部加载一次并重复使用它。

尽量将代码减少到最低限度。没有任何CSS和其他装饰的普通按钮然后比较性能。

你能提供一些数字吗？什么对你来说很慢？