JavaFX点击按钮无法触发TableView的RowFactory设置问题

我帮你排查出了问题的核心，并且整理了修复方案，让你点击"Set Order"按钮后能正常启用表格行的拖拽重排功能。

问题根源 #

1. TableView行不会自动刷新：当你设置新的rowFactory后，TableView不会自动替换已有的行，旧行还是用原来的默认工厂创建的，没有绑定拖拽事件。
2. 冗余的重复代码：你同时在按钮的onAction和pressedProperty监听器里写了完全一样的拖拽逻辑，这不仅没必要，还可能导致逻辑冲突。

修复步骤 #

1. 把拖拽行的工厂逻辑抽成独立方法，避免重复代码，提升可读性；
2. 设置完rowFactory后，调用tableView.refresh()强制TableView重新创建所有行，让新的拖拽逻辑生效；
3. 移除冗余的pressedProperty监听器，只保留按钮点击事件的处理。

完整修复代码 #

import java.util.function.Function;
import javafx.application.Application;
import javafx.beans.property.SimpleStringProperty;
import javafx.beans.property.StringProperty;
import javafx.beans.value.ObservableValue;
import javafx.event.ActionEvent;
import javafx.event.EventHandler;
import javafx.scene.Scene;
import javafx.scene.control.Button;
import javafx.scene.control.TableColumn;
import javafx.scene.control.TableRow;
import javafx.scene.control.TableView;
import javafx.scene.input.ClipboardContent;
import javafx.scene.input.DataFormat;
import javafx.scene.input.Dragboard;
import javafx.scene.input.TransferMode;
import javafx.scene.layout.BorderPane;
import javafx.scene.layout.VBox;
import javafx.stage.Stage;

public class TableViewDragRows extends Application {
    private static final DataFormat SERIALIZED_MIME_TYPE = new DataFormat("application/x-java-serialized-object");
    public TableView<Person> tableView;

    @Override
    public void start(Stage primaryStage) {
        tableView = new TableView<>();
        tableView.getColumns().add(createCol("First Name", Person::firstNameProperty, 150));
        tableView.getColumns().add(createCol("Last Name", Person::lastNameProperty, 150));
        tableView.getColumns().add(createCol("Email", Person::emailProperty, 200));

        tableView.getItems().addAll(
                new Person("Jacob", "Smith", "jacob.smith@example.com","1"),
                new Person("Isabella", "Johnson", "isabella.johnson@example.com","2"),
                new Person("Ethan", "Williams", "ethan.williams@example.com","3"),
                new Person("Emma", "Jones", "emma.jones@example.com","4"),
                new Person("Michael", "Brown", "michael.brown@example.com","5")
        );

        Button order = new Button("Set Order");
        order.setOnAction(new EventHandler<ActionEvent>() {
            @Override
            public void handle(ActionEvent e) {
                // 设置带拖拽功能的行工厂
                tableView.setRowFactory(tv -> createDraggableRow());
                // 强制刷新表格，让新的行工厂生效
                tableView.refresh();
            }
        });

        Button button = new Button("Get Order");
        button.setOnAction(new EventHandler<ActionEvent>() {
            @Override
            public void handle(ActionEvent e) {
                for(Person p: tableView.getItems()){
                    System.out.println(p.getid());
                }
            }
        });

        VBox vbox = new VBox();
        vbox.getChildren().addAll(tableView, button, order);

        Scene scene = new Scene(new BorderPane(vbox), 600, 400);
        primaryStage.setScene(scene);
        primaryStage.show();
    }

    // 抽离拖拽行的创建逻辑到独立方法
    private TableRow<Person> createDraggableRow() {
        TableRow<Person> row = new TableRow<>();

        row.setOnDragDetected(event -> {
            if (! row.isEmpty()) {
                Integer index = row.getIndex();
                Dragboard db = row.startDragAndDrop(TransferMode.MOVE);
                db.setDragView(row.snapshot(null, null));
                ClipboardContent cc = new ClipboardContent();
                cc.put(SERIALIZED_MIME_TYPE, index);
                db.setContent(cc);
                event.consume();
            }
        });

        row.setOnDragOver(event -> {
            Dragboard db = event.getDragboard();
            if (db.hasContent(SERIALIZED_MIME_TYPE)) {
                if (row.getIndex() != ((Integer)db.getContent(SERIALIZED_MIME_TYPE)).intValue()) {
                    event.acceptTransferModes(TransferMode.COPY_OR_MOVE);
                    event.consume();
                }
            }
        });

        row.setOnDragDropped(event -> {
            Dragboard db = event.getDragboard();
            if (db.hasContent(SERIALIZED_MIME_TYPE)) {
                int draggedIndex = (Integer) db.getContent(SERIALIZED_MIME_TYPE);
                Person draggedPerson = tableView.getItems().remove(draggedIndex);

                int dropIndex ;
                if (row.isEmpty()) {
                    dropIndex = tableView.getItems().size() ;
                } else {
                    dropIndex = row.getIndex();
                }

                tableView.getItems().add(dropIndex, draggedPerson);
                event.setDropCompleted(true);
                tableView.getSelectionModel().select(dropIndex);
                event.consume();
            }
        });
        return row ;
    }

    private TableColumn<Person, String> createCol(String title, Function<Person, ObservableValue<String>> mapper, double size) {
        TableColumn<Person, String> col = new TableColumn<>(title);
        col.setCellValueFactory(cellData -> mapper.apply(cellData.getValue()));
        col.setPrefWidth(size);
        return col ;
    }

    public class Person {
        private final StringProperty firstName = new SimpleStringProperty(this, "firstName");
        private final StringProperty lastName = new SimpleStringProperty(this, "lastName");
        private final StringProperty email = new SimpleStringProperty(this, "email");
        private final StringProperty id = new SimpleStringProperty(this, "id");

        public Person(String firstName, String lastName, String email, String id) {
            this.firstName.set(firstName);
            this.lastName.set(lastName);
            this.email.set(email);
            this.id.set(id);
        }

        public final StringProperty idProperty() {
            return this.id;
        }

        public final String getid() {
            return this.idProperty().get();
        }

        public final StringProperty firstNameProperty() {
            return this.firstName;
        }

        public final String getFirstName() {
            return this.firstNameProperty().get();
        }

        public final void setFirstName(final String firstName) {
            this.firstNameProperty().set(firstName);
        }

        public final StringProperty lastNameProperty() {
            return this.lastName;
        }

        public final String getLastName() {
            return this.lastNameProperty().get();
        }

        public final void setLastName(final String lastName) {
            this.lastNameProperty().set(lastName);
        }

        public final StringProperty emailProperty() {
            return this.email;
        }

        public final String getEmail() {
            return this.emailProperty().get();
        }

        public final void setEmail(final String email) {
            this.emailProperty().set(email);
        }
    }

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

关键说明 #

• tableView.refresh()：这个方法会让TableView销毁现有行，重新调用你设置的rowFactory创建每一行，这样新的拖拽事件处理器才能绑定到所有行上。
• 抽离独立方法：把拖拽行的逻辑单独拿出来，不仅让代码更整洁，后续如果需要关闭拖拽功能，只需要调用tableView.setRowFactory(null)再刷新即可恢复默认行样式。

内容的提问来源于stack exchange，提问作者Anish Damineni