我正在从我的DB加载数据,并在GridPane中动态创建按钮!但是有很多数据需要很长时间才能看到按钮!我正在使用线程从DB加载数据,但似乎创建每个按钮并填充GridPane需要太长时间!
**我试图创建硬编码数据,并与他们只是为了测试填充网格窗格,但同样需要太长的时间,所以与此我得出结论,使按钮需要太长时间显示的问题是填充网格窗格,而不是从DB加载数据!
有什么方法可以更快地填充GridPane吗?我使用2个for循环,一个用于行,另一个用于列,所以复杂度是(n-square)!
我可以优化它吗?
以下是我创建和填充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);
}
**我不认为下面的初始化按钮()函数有问题,但以防万一你可以看到工作是如何流动的!
为每个产品初始化每个按钮:
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和其他装饰的普通按钮,然后比较性能。
你能提供几个数字吗?什么对你来说很慢?