tableView 中的行为重复和反转行 (javafx)

Question

我在 tableview 的更新中将代码简化为以下内容。

出于某些原因，我没有像人们期望的那样使用 JavaFx 的属性，也许这种行为与此有关。来自 JTable 和观察者模式，我想尝试如何在 javafx 中实现它。然而，我的 table 的数据都很好，但是当我尝试更改背景颜色时，更多的行出现在 table 的范围之外。我已经检查了 Observable 列表的大小，它符合预期。

我无法真正看到或理解 datas.set 是如何产生这种行为的，这是我的代码遗漏了什么，还是 Javafx tableview 的预期行为。

基本上使用的逻辑是我从另一个class接收更新（我在这里用线程模拟），然后我将用更新的数据更新可观察列表的相应行。

谢谢。

package application;

import javafx.application.Application;
import javafx.scene.Scene;
import javafx.scene.layout.Pane;
import javafx.stage.Stage;


public class Main extends Application {
    @Override
    public void start(Stage primaryStage) {
        try {
            Pane root = new Pane();
            root.getChildren().add(new TableModel().getTable());
            Scene scene = new Scene(root,400,800);
            scene.getStylesheets().add(getClass().getResource("application.css").toExternalForm());
            primaryStage.setScene(scene);
            primaryStage.show();
        } catch(Exception e) {
            e.printStackTrace();
        }
    }

    public static void main(String[] args) {
        launch(args);
    }
}

package application;

import java.util.Random;

import javafx.application.Platform;
import javafx.beans.property.SimpleIntegerProperty;
import javafx.beans.property.SimpleObjectProperty;
import javafx.beans.value.ObservableValue;
import javafx.collections.FXCollections;
import javafx.collections.ObservableList;
import javafx.scene.control.TableCell;
import javafx.scene.control.TableColumn;
import javafx.scene.control.TableColumn.CellDataFeatures;
import javafx.scene.control.TableView;
import javafx.util.Callback;

public class TableModel{

    private TableView<TableData> table = new TableView<>();
    private ObservableList<TableData> datas;

    @SuppressWarnings("unchecked")
    public TableModel() {
        datas = FXCollections.observableArrayList();

        final TableColumn<TableData, Integer> cId = new TableColumn<>("ID");
        cId.setCellValueFactory(
                new Callback<TableColumn.CellDataFeatures<TableData, Integer>, ObservableValue<Integer>>() {
                    @Override
                    public ObservableValue<Integer> call(CellDataFeatures<TableData, Integer> param) {
                        return new SimpleObjectProperty<Integer>(param.getValue().getId());
                    }
                });

        cId.setCellFactory(e -> new TableCell<TableData, Integer>() {
            @Override
            public void updateItem(Integer item, boolean empty) {
                // Always invoke super constructor.
                super.updateItem(item, empty);

                if (item == null || empty) {
                    setText(null);
                } else {
                    setText(String.valueOf(item));
                    if (getIndex() == 0) {
                        this.setStyle("-fx-background-color: green;");
                    }else {
                         this.setStyle("-fx-background-color: red;");
                    }
                }
            }
        });


        table.getColumns().addAll(cId);
        table.setColumnResizePolicy(TableView.CONSTRAINED_RESIZE_POLICY);
        table.setItems(datas);
        table.setPrefSize(150, 450);

            datas.add(0, new TableData(0));
            datas.add(1, new TableData(0));
            datas.add(2, new TableData(0));
            datas.add(3, new TableData(0));
            datas.add(4, new TableData(0));


        new Thread(()->{
            while(true) {
                Random rand = new Random(10);
                update(rand.nextInt());
                try {
                    Thread.sleep(3000);
                } catch (InterruptedException e1) {
                    e1.printStackTrace();
                }
            }
        }).start(); ;

    }



    public TableView<TableData> getTable() {
        return table;
    }

    public void update(int num) {
        TableData data = new TableData(num);
                Platform.runLater(() -> {
                    datas.set(0, data);
                });
    }

    private class TableData {
        private SimpleIntegerProperty cId = new SimpleIntegerProperty();

        public TableData(Integer conid) {
            this.cId.setValue(conid);
        }

        public Integer getId() {
            return cId.get();
        }

    }
}

输出datas.set(0, data);

没有datas.set(0, data);

的输出

Answer 1

您的代码有 4 个问题：

第 1 期

final TableColumn<TableData, Integer> cId = new TableColumn<>("ID");
cId.setCellValueFactory(
        new Callback<TableColumn.CellDataFeatures<TableData, Integer>, ObservableValue<Integer>>() {
            @Override
            public ObservableValue<Integer> call(CellDataFeatures<TableData, Integer> param) {
                return new SimpleObjectProperty<Integer>(param.getValue().getId());
            }
        });

这可以防止属性的更改自动触发更新。例如

table.getItems().get(0).cId.set(5);

不会导致更新 table。要修复此 return 属性本身，而不是在 cellValueFactory 时用属性的值初始化的新创建的属性执行。

final TableColumn<TableData, Number> cId = new TableColumn<>("ID");
cId.setCellValueFactory(
        new Callback<TableColumn.CellDataFeatures<TableData, Number>, ObservableValue<Number>>() {

            @Override
            public ObservableValue<Number> call(TableColumn.CellDataFeatures<TableData, Number> param) {
                return param.getValue().cId;
            }
        });

第 2 期

@Override
public void updateItem(Integer item, boolean empty) {
    // Always invoke super constructor.
    super.updateItem(item, empty);

    if (item == null || empty) {
        setText(null);
    } else {
        setText(String.valueOf(item));
        if (getIndex() == 0) {
            this.setStyle("-fx-background-color: green;");
        }else {
             this.setStyle("-fx-background-color: red;");
        }
    }
}

鉴于单元格在填充值后可能会变空这一事实，此实现是不正确的。您应该确保恢复 "empty" 外观，以防单元格变空。这需要你清除样式属性:

@Override
protected void updateItem(Number item, boolean empty) {
    // Always invoke supertype's implementation
    super.updateItem(item, empty);

    if (item == null || empty) {
        setText(null);
        setStyle(null);
    } else {
        setText(item.toString());
        if (getIndex() == 0) {
            this.setStyle("-fx-background-color: green;");
        } else {
            this.setStyle("-fx-background-color: red;");
        }
    }
}

第 3 期和第 4 期

new Thread(()->{
    while(true) {
        Random rand = new Random(10);
        update(rand.nextInt());
        try {
            Thread.sleep(3000);
        } catch (InterruptedException e1) {
            e1.printStackTrace();
        }
    }
}).start();

即使关闭 GUI 以防止程序终止，该线程仍将保持活动状态。您可以使用守护线程轻松解决此问题。

此外 Random 使用种子 return 确定性结果进行初始化。由于您在循环的每次迭代中都使用相同的种子重新初始化 Random，因此值 rand.nextInt() returns 始终相同。您需要将 Random 的初始化移到循环之外。

Thread thread = new Thread(() -> {
    Random rand = new Random(10);
    while (true) {
        update(rand.nextInt());
        try {
            Thread.sleep(3000);
        } catch (InterruptedException e1) {
            e1.printStackTrace();
        }
    }
});
thread.setDaemon(true);
thread.start();

tableView 中的行为重复和反转行 (javafx)

behaviour duplicate and inverse rows in a tableView (javafx)

javafx

tableview

observer-pattern

第 1 期

第 2 期

第 3 期和第 4 期