JavaFX Заполняет GridPane динамически быстрее, используя потоки - PullRequest
1 голос
/ 13 марта 2019

Я загружаю данные из своей БД и динамически создаю кнопки внутри GridPane!Но с большим количеством данных это занимает слишком много времени, чтобы увидеть кнопки!Я использую потоки для загрузки данных из БД, но кажется, что создание каждой кнопки и заполнение GridPane занимает слишком много времени!

** Я пытался создать жестко закодированные данные и заполнять их GridPane только для тестированияно опять же это занимает слишком много времени, поэтому я прихожу к выводу, что проблема, из-за которой кнопки отображаются слишком долго, заключается в заполнении GridPane, а не в загрузке данных из БД!

Есть ли способ заполненияGridPane быстрее?Я использую 2 для циклов, один для строк, а другой для столбцов, поэтому сложность равна (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);
    }

1 Ответ

0 голосов
/ 13 марта 2019

У вас есть несколько потенциальных убийц производительности.

Почему вы загружаете backgroundImage снова и снова.Вы можете загрузить его только один раз за пределы цикла и использовать его повторно.

Попробуйте уменьшить свой код до абсолютного минимума.Простые кнопки без каких-либо CSS и других украшений, а затем сравнить производительность.

Не могли бы вы предоставить несколько цифр.Что у тебя медленно?

...