TornadoFX связывает грязные свойства разных view-моделей

Question

У меня есть приложение tornadoFX по шаблону MVVM с моделью:

data class Person (
    val name: String,
    val cars: List<Car>
)

data class Car (
    val brand: String,
    val model: String
)

Приложение определяет следующий вид:

Существует просмотр списка, в котором перечислены все лица. Кроме того, listView представляет собой подробное представление с текстовым полем для имени человека и табличное представление для автомобилей этого человека.
Двойной щелчок по записи автомобиля в таблице открывает диалоговое окно, в котором можно редактировать свойства автомобиля.

Я хочу, чтобы, если я открою детали машины и отредактирую запись, изменения будут отражены в виде таблицы. Так как я не могу изменить Car-модель (которая является неизменным типом), добавив fx-свойства, я придумал следующую view-модель:

class PersonViewModel(): ItemViewModel<Person> {
    val name = bind(Person::name)
    val cars = bind { SimpleListProperty<CarViewModel>(item?.cars?.map{CarViewModel(it)}?.observable()) }

    override fun onCommit {
        // create new person based on ViewModel and store it
    }
}

class CarViewModel(item: Car): ItemViewModel<Car> {
    val brand = bind(Car::name)
    val model = bind(Car::model)

    init {
        this.item = item
    }
}

Таким образом, если дважды щелкнуть запись автомобиля в табличном представлении и открыть подробное представление автомобиля, обновленная информация об автомобиле будет непосредственно отражена в табличном представлении.

Моя проблема в том, что я не могу найти способ связать грязные свойства всех моих CarViewModel в таблице с PersonViewModel. Поэтому, если я меняю автомобиль, PersonViewModel не помечается как грязный.

Есть ли способ связать пакостные свойства PersonViewModel и CarViewModel? (А также свяжите их, если выбран другой человек).

Или есть даже лучший способ определить мои модели вида?

Answer 1

Я внес изменение в структуру, чтобы привязки ViewModel к спискам могли наблюдать события ListChange.Это позволяет вам вызвать грязное состояние свойства списка, изменяя список каким-либо образом.Простое изменение свойства внутри элемента в списке не вызовет его, поэтому в следующем примере я просто получаю индекс Car перед фиксацией и переназначаю Car на тот же индекс.Это вызовет событие ListChange, которое теперь слушает фреймворк.

Важное действие происходит в функции сохранения диалогового окна автомобиля:

button("Save").action {
    val index = person.cars.indexOf(car.item)

    car.commit {
        person.cars[index] = car.item
        close()
    }
}

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

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

class Person(name: String, cars: List<Car>) {
    val nameProperty = SimpleStringProperty(name)
    var name by nameProperty

    val carsProperty = SimpleListProperty<Car>(FXCollections.observableArrayList(cars))
    var cars by carsProperty
}

class PersonModel : ItemViewModel<Person>() {
    val name = bind(Person::nameProperty)
    val cars: SimpleListProperty<Car> = bind(Person::carsProperty)
}


class Car(brand: String, model: String) {
    val brandProperty = SimpleStringProperty(brand)
    var brand by brandProperty

    val modelProperty = SimpleStringProperty(model)
    var model by modelProperty
}

class CarModel(car: Car? = null) : ItemViewModel<Car>(car) {
    val brand = bind(Car::brandProperty)
    val model = bind(Car::modelProperty)
}

class DataController : Controller() {
    val people = FXCollections.observableArrayList<Person>()

    init {
        people.add(
                Person("Person 1", listOf(Car("BMW", "M3"), Car("Ford", "Fiesta")))
        )
    }
}

class PersonMainView : View() {
    val data: DataController by inject()
    val selectedPerson: PersonModel by inject()

    override val root = borderpane {
        center {
            tableview(data.people) {
                column("Name", Person::nameProperty)
                bindSelected(selectedPerson)
            }
        }
        right(PersonEditor::class)
    }
}

class PersonEditor : View() {
    val person: PersonModel by inject()
    val selectedCar : CarModel by inject()

    override val root = form {
        fieldset {
            field("Name") {
                textfield(person.name).required()
            }
            field("Cars") {
                tableview(person.cars) {
                    column("Brand", Car::brandProperty)
                    column("Model", Car::modelProperty)
                    bindSelected(selectedCar)
                    onUserSelect(2) {
                        find<CarEditor>().openModal()
                    }
                }
            }
            button("Save") {
                enableWhen(person.dirty)
                action {
                    person.commit()
                }
            }
        }
    }
}

class CarEditor : View() {
    val car: CarModel by inject()
    val person: PersonModel by inject()

    override val root = form {
        fieldset {
            field("Brand") {
                textfield(car.brand).required()
            }
            field("Model") {
                textfield(car.model).required()
            }
            button("Save").action {
                val index = person.cars.indexOf(car.item)

                car.commit {
                    person.cars[index] = car.item
                    close()
                }
            }
        }
    }
}

Эта функция доступна в TornadoFX 1.7.17-SNAPSHOT.