Блог

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

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

Есть ли какой-нибудь способ быстрее заполнить область сетки? Я использую 2 цикла for, один для строк, а другой для столбцов, поэтому сложность составляет (n-квадрат)!

Могу ли я это оптимизировать?

Вот мои функции для создания и заполнения области сетки:

 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);
    }
  

Здесь есть несколько потенциальных факторов, снижающих производительность.

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

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

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