Прокрутка и сортировка TableView приводят к неправильному стилю строк через RowFactory

В вашем коде есть несколько недоразумений. Первый касается ячеек и повторного использования (TableRow есть Cell). Ячейки можно повторно использовать произвольно и потенциально часто (особенно во время прокрутки пользователем), чтобы остановить отображение одного элемента и отобразить другой.

В вашем коде, если строка используется для отображения недопустимой сущности, слушатель в строке itemProperty вызовет исполняемый объект в фоновом потоке, который в какой-то момент установит состояние псевдокласса в true.

Однако, если ячейка впоследствии повторно используется для отображения допустимого элемента, следующий исполняемый объект, который выполняется, не меняет состояние псевдокласса. Таким образом, это состояние остается истинным, а цвет строки остается красным.

Следовательно, строка становится красной, если в какой-то момент она когда-либо отображала недопустимый элемент. (Нет, если в настоящее время он отображает недопустимый элемент.) Если вы достаточно прокрутите, в конечном итоге все ячейки станут красными.

Во-вторых, вы не должен обновляете любой пользовательский интерфейс, являющийся частью графа сцены, из любого потока, кроме потока приложения FX. Кроме того, некоторые другие операции, такие как создание экземпляров Window (Tooltip является подклассом Window), должны выполняться в потоке приложения FX. Обратите внимание, что это включает в себя изменение свойств модели, привязанных к пользовательскому интерфейсу, включая свойства, используемые в столбцах таблицы. Вы нарушаете это в своем validationThread, где вы создаете Tooltip, устанавливаете его в строке и изменяете состояние псевдокласса, и все это в фоновом потоке.

Здесь хорошим подходом является использование API параллелизма JavaFX. Используйте Task, которые, насколько это возможно, используют только неизменяемые данные и возвращают неизменяемое значение. Если вам нужно обновить свойства, которые отображаются в пользовательском интерфейсе, используйте Platform.runLater(...), чтобы запланировать эти обновления в потоке приложения FX.

С точки зрения дизайна MVC, хорошей практикой для вашего класса (ов) модели является хранение всех данных, необходимых представлению. Ваш дизайн сталкивается с проблемой, потому что нет реального места, где хранится статус проверки. Более того, статус проверки на самом деле больше, чем просто «действительный» или «недействительный»; существует фаза, когда поток выполняется, но не завершен, и статус проверки неизвестен.

Вот мое решение, которое решает эти проблемы. Я предполагаю:

1. У вашей сущности есть понятие достоверности.
2. Установление действительности объекта — длительный процесс.
3. Срок действия зависит от одного или нескольких свойств, которые могут измениться во время отображения пользовательского интерфейса.
4. Действительность должна устанавливаться «лениво» по мере необходимости.
5. Пользовательский интерфейс предпочитает не отображать «неизвестную» достоверность, и если отображается сущность, достоверность которой неизвестна, ее следует установить и отобразить повторно.

Я создал перечисление для ValidationStatus, которое имеет четыре значения:

public enum ValidationStatus {
    VALID, INVALID, UNKNOWN, PENDING ;
}

UNKNOWN указывает, что достоверность неизвестна и проверка не запрашивалась; PENDING указывает, что проверка была запрошена, но еще не завершена.

Затем у меня есть оболочка для вашего объекта, которая добавляет статус проверки в качестве наблюдаемого свойства. Если свойство, от которого зависит проверка в базовой сущности, изменяется, проверка сбрасывается на UNKNOWN.

import javafx.beans.property.ObjectProperty;
import javafx.beans.property.SimpleObjectProperty;

public class ValidatingTestEntity {

    private final TestEntity entity ;
    private final ObjectProperty<ValidationStatus> validationStatus = new SimpleObjectProperty<>(ValidationStatus.UNKNOWN);


    public ValidatingTestEntity(TestEntity entity) {
        this.entity = entity;

        entity.firstNameProperty().addListener((obs, oldName, newName) -> setValidationStatus(ValidationStatus.UNKNOWN));
    }


    public TestEntity getEntity() {
        return entity;
    }

    public ValidationStatus getValidationStatus() {
        return validationStatus.get();
    }

    public ObjectProperty<ValidationStatus> validationStatusProperty() {
        return validationStatus;
    }

    public void setValidationStatus(ValidationStatus validationStatus) {
        this.validationStatus.set(validationStatus);
    }
}

ValidationService предоставляет сервис для проверки объектов в фоновом потоке, обновляя соответствующие свойства с результатом. Это управляется через пул потоков и с помощью JavaFX Tasks. Это просто имитирует вызов базы данных, засыпая в течение случайного промежутка времени, а затем возвращая чередующиеся результаты.

Когда задача меняет состояние (т. е. по мере прохождения своего жизненного цикла), свойство проверки объекта обновляется: UNKNOWN, если задача не завершается нормально, PENDING, если задача находится в незавершенном состоянии, и либо VALID, либо INVALID, в зависимости от по результату задания, если задание выполнено успешно.

import javafx.application.Platform;
import javafx.concurrent.Task;

import java.util.concurrent.Executor;
import java.util.concurrent.Executors;
import java.util.concurrent.ThreadLocalRandom;

public class ValidationService {

    private final Executor exec = Executors.newFixedThreadPool(Runtime.getRuntime().availableProcessors() * 2 + 1,
            r -> {
                Thread thread = new Thread(r);
                thread.setDaemon(true);
                return thread;
            }
    );

    public Task<Boolean> validateEntity(ValidatingTestEntity entity) {

        // task runs on a background thread and should not access mutable data,
        // so make final copies of anything needed here:
        final String firstName = entity.getEntity().getFirstName();
        final int code =entity.getEntity().getC();

        Task<Boolean> task = new Task<Boolean>() {
            @Override
            protected Boolean call() throws Exception {
    
                try {
                    Thread.sleep(500 + ThreadLocalRandom.current().nextInt(500));
                } catch (InterruptedException exc) {
                    // if interrupted other than being cancelled, reset thread's interrupt status:
                    if (! isCancelled()) {
                        Thread.currentThread().interrupt();
                    }
                }

                boolean result = code % 2 == 0;
                return result;
            }
        };

        task.stateProperty().addListener((obs, oldState, newState) ->
            entity.setValidationStatus(
                    switch(newState) {
                        case CANCELLED, FAILED -> ValidationStatus.UNKNOWN;
                        case READY, RUNNING, SCHEDULED -> ValidationStatus.PENDING ;
                        case SUCCEEDED ->
                                task.getValue() ? ValidationStatus.VALID : ValidationStatus.INVALID ;
                    }
            )
        );


        exec.execute(task);

        return task ;
    }
}

Это TableRow реализация. У него есть прослушиватель, который наблюдает за статусом проверки текущего элемента, если он есть. Если элемент изменяется, прослушиватель удаляется из старого элемента (если он есть) и прикрепляется к новому элементу (если он есть). Если изменяется элемент или изменяется состояние проверки текущего элемента, строка обновляется. Если новый статус проверки — UNKNOWN, в службу отправляется запрос на проверку текущего элемента. Существует два состояния псевдокласса: недействительный (красный) и неизвестный (оранжевый), которые обновляются каждый раз при изменении элемента или его статуса проверки. Всплывающая подсказка устанавливается, если элемент недействителен, в противном случае устанавливается значение null.

import javafx.beans.value.ChangeListener;
import javafx.css.PseudoClass;
import javafx.scene.control.TableRow;
import javafx.scene.control.Tooltip;

public class ValidatingTableRow extends TableRow<ValidatingTestEntity> {

    private final ValidationService validationService ;

    private final PseudoClass pending = PseudoClass.getPseudoClass("pending");
    private final PseudoClass invalid = PseudoClass.getPseudoClass("invalid");

    private final Tooltip tooltip = new Tooltip();

    private final ChangeListener<ValidationStatus> listener = (obs, oldStatus, newStatus) -> {
         updateValidationStatus();
    };

    public ValidatingTableRow(ValidationService validationService){
        this.validationService = validationService ;
        itemProperty().addListener((obs, oldItem, newItem) -> {
            setTooltip(null);
            if (oldItem != null) {
                oldItem.validationStatusProperty().removeListener(listener);
            }
            if (newItem != null) {
                newItem.validationStatusProperty().addListener(listener);
            }
            updateValidationStatus();
        });
    }

    private void updateValidationStatus() {

        if (getItem() == null) {
            pseudoClassStateChanged(pending, false);
            pseudoClassStateChanged(invalid, false);
            setTooltip(null);
            return ;
        }
        ValidationStatus validationStatus = getItem().getValidationStatus();
        if ( validationStatus == ValidationStatus.UNKNOWN) {
            validationService.validateEntity(getItem());
        }
        if (validationStatus == ValidationStatus.INVALID) {
            tooltip.setText("Invalid entity: "+getItem().getEntity().getFirstName() + " " +getItem().getEntity().getC());
            setTooltip(tooltip);
        } else {
            setTooltip(null);
        }
        pseudoClassStateChanged(pending, validationStatus == ValidationStatus.PENDING);
        pseudoClassStateChanged(invalid, validationStatus == ValidationStatus.INVALID);
    }
}

Вот Entity, то же самое, что и в вопросе:

import javafx.beans.property.IntegerProperty;
import javafx.beans.property.SimpleIntegerProperty;
import javafx.beans.property.SimpleStringProperty;
import javafx.beans.property.StringProperty;

public class TestEntity
{

    public TestEntity(String firstName, String lastName, int c)
    {
        setFirstName(firstName);
        setLastName(lastName);
        setC(c);
    }

    private StringProperty firstName = new SimpleStringProperty();
    private StringProperty lastName = new SimpleStringProperty();
    private IntegerProperty c = new SimpleIntegerProperty();

    public String getFirstName() {
        return firstName.get();
    }

    public StringProperty firstNameProperty() {
        return firstName;
    }

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

    public String getLastName() {
        return lastName.get();
    }

    public StringProperty lastNameProperty() {
        return lastName;
    }

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

    public int getC() {
        return c.get();
    }

    public IntegerProperty cProperty() {
        return c;
    }

    public void setC(int c) {
        this.c.set(c);
    }
}

А вот и класс приложения. Я добавил возможность редактировать первое имя, что позволяет вам увидеть, как элемент возвращается к неизвестному, а затем восстанавливает свою действительность (вам нужно быстро изменить выбор после фиксации редактирования).

import javafx.application.Application;
import javafx.scene.Scene;
import javafx.scene.control.TableColumn;
import javafx.scene.control.TableView;
import javafx.scene.control.cell.TextFieldTableCell;
import javafx.scene.layout.VBox;
import javafx.stage.Stage;

public class TableViewProblemMain extends Application
{
    public static void main(String[] args)
    {
        launch(args);
    }

    @Override
    public void start(Stage stage)
    {
        TableView<ValidatingTestEntity> tableView = new TableView();
        tableView.setEditable(true);

        TableColumn<ValidatingTestEntity, String> column1 = new TableColumn<>("First Name");
        column1.setCellValueFactory(cellData -> cellData.getValue().getEntity().firstNameProperty());
        column1.setEditable(true);
        column1.setCellFactory(TextFieldTableCell.forTableColumn());

        TableColumn<ValidatingTestEntity, String> column2 = new TableColumn<>("Last Name");
        column2.setCellValueFactory(cellData -> cellData.getValue().getEntity().lastNameProperty());

        TableColumn<ValidatingTestEntity, Number> column3 = new TableColumn<>("C");
        column3.setCellValueFactory(cellData -> cellData.getValue().getEntity().cProperty());

        tableView.getColumns().addAll(column1, column2, column3);

        ValidationService service = new ValidationService();

        tableView.setRowFactory(tv -> new ValidatingTableRow(service));


        for (int i = 0; i < 300; i++)
        {
            tableView.getItems().add(new ValidatingTestEntity(
                    new TestEntity("Fname" + i, "Lname" + i, i)));
        }


        VBox vbox = new VBox(tableView);

        Scene scene = new Scene(vbox);
        scene.getStylesheets().add(this.getClass().getResource("/style.css").toExternalForm());
      
        stage.setScene(scene);
        stage.show();
    }
}

Наконец, для полноты, таблица стилей:

.table-row-cell:invalid {
    -fx-background-color: rgba(240, 116, 116, 0.18);
}
.table-row-cell:pending {
    -fx-background-color: rgba(240, 120, 0, 0.18);
}