なぜJavaFX TableViewから選択した項目のリストをコピーする必要がありますか？

Question

私はObservableListに裏打ちされたテーブルビューがあります。

private ObservableList<Renderer> renderers = FXCollections.observableArrayList(); 

@FXML 
private TableView<Renderer> renderersTable; 
@FXML 
private TableColumn<Renderer, String> nameColumn; 
@FXML 
private TableColumn<Renderer, Boolean> approvedColumn; 

@FXML 
private void initialize() { 
    nameColumn.setCellValueFactory(new PropertyValueFactory<>("name")); 
    approvedColumn.setCellValueFactory(new PropertyValueFactory<>("approved")); 
    renderersTable.setItems(this.renderers); 
} 

レンダラオブジェクトは非常に単純であり、次のようになります。このコードの一部で

@Data 
@NoArgsConstructor 
@EqualsAndHashCode(callSuper = false) 
public class Renderer extends Model { 
    private String name; 
    private boolean approved; 
    // ... 
} 

を私がテーブルビューから選択されたすべてのアイテムを取得します

複数の選択を有効にして、実際には選択したレンダラーが適切なカウントを表示していますが、ループは一度実行する。

private void approveSelectedRenderers() {   
    // Get all the selected renderers but copy them. 
    List<Renderer> selectedRenderers = new ArrayList<>();   
    selectedRenderers.addAll(renderersTable.getSelectionModel().getSelectedItems()); 

    for (Renderer renderer : selectedRenderers) { 
     renderer.setApproved(true); 
     renderers.set(renderers.indexOf(renderer), renderer); 
    } 
} 

それが正しい動作し、すべての項目が処理されます。

代わりにした場合、私はこのように、コピーを作成します。どうしたの？これに対処する適切な方法は何ですか？

Answer 1

他の質問から判断すると、実際にはTableViewのバッキング（観測可能な）データ構造が変更されています。つまり、選択した（観測可能な）リスト全体をリセットして、バッキングデータ構造のレンダラーをリセットします。私はちょっと驚いたそれはConcurrentModificationExceptionをスローしません。

解決策：選択したリストをループするときにバッキングデータ構造を変更しないでください。更新後にリセットする必要もありません。値の型（boolean）をSimpleBooleanPropertyに変更すると、CellFactoryを使用して直接TableViewに渡すことができます。 SimpleBooleanPropertyの値を変更すると、自動的にテーブルが更新されます。

SimpleBooleanPropertyを使用することができない場合は、TableView#refresh()メソッドを使用することもできます。ただし、変更された要素だけではなく、TableView全体が再投入されることに注意してください。

さらに読む：Simular SO question