Удаление нескольких идентичных элементов ComboBox

Question

У меня есть 4 ComboBox в каждом из них есть 5 элементов: «", "1", "2", "3", "4". Мне нужно сделать, если элемент «1» выбран в ComboBox, он автоматически исключается из другого. И если элемент «» выбран в ComboBox, то имя автоматически возвращается в другое. Такая же потребность в других предметах, кроме «».
Я пытался сделать это Вот код:

---

public void vubadditems(ComboBox<String> vub1,ComboBox<String> vub2,ComboBox<String> vub3,ComboBox<String> vub4){ 
    vub1.getItems().addAll(" ","1","2","3","4"); 
    vub2.getItems().addAll(" ","1","2","3","4"); 
    vub3.getItems().addAll(" ","1","2","3","4"); 
    vub4.getItems().addAll(" ","1","2","3","4"); 
} 
public void vubact(ComboBox<String> vub1,ComboBox<String> vub2,ComboBox<String> vub3,ComboBox<String> vub4){ 
    if(vub1.getSelectionModel().getSelectedItem()!=" "){  
    vub2.getItems().remove(vub1.getSelectionModel().getSelectedItem()); 
    vub3.getItems().remove(vub1.getSelectionModel().getSelectedItem()); 
    vub4.getItems().remove(vub1.getSelectionModel().getSelectedItem()); 
    s=vub1.getSelectionModel().getSelectedItem(); 
    }else{ 
    } 
    if(vub2.getSelectionModel().getSelectedItem()!=" "){  
     vub1.getItems().remove(vub2.getSelectionModel().getSelectedItem()); 
     vub3.getItems().remove(vub2.getSelectionModel().getSelectedItem()); 
     vub4.getItems().remove(vub2.getSelectionModel().getSelectedItem()); 
     } 
    if(vub3.getSelectionModel().getSelectedItem()!=" "){  
     vub1.getItems().remove(vub3.getSelectionModel().getSelectedItem()); 
     vub2.getItems().remove(vub3.getSelectionModel().getSelectedItem()); 
     vub4.getItems().remove(vub3.getSelectionModel().getSelectedItem()); 
     } 
    if(vub4.getSelectionModel().getSelectedItem()!=" "){  
     vub1.getItems().remove(vub4.getSelectionModel().getSelectedItem()); 
     vub2.getItems().remove(vub4.getSelectionModel().getSelectedItem()); 
     vub3.getItems().remove(vub4.getSelectionModel().getSelectedItem()); 
     } 
} 
public void vubq1ac(){ 
    vubact(q1vub1,q1vub2,q1vub3,q1vub4); 
} 
public void initialize(){ 
vubadditems(q1vub1,q1vub2,q1vub3,q1vub4); 
    vubadditems(q2vub1,q2vub2,q2vub3,q2vub4); 
    vubadditems(q3vub1,q3vub2,q3vub3,q3vub4); 
    vubadditems(q4vub1,q4vub2,q4vub3,q4vub4); 
    vubadditems(q5vub1,q5vub2,q5vub3,q5vub4); 
} 

---

q1vub1, q1vub2, q1vub3, q1vub4 загружается из FXML_fileComboBox «s
добавить vubq1ac в действие listiner на ComboBox
Но у меня есть некоторые исключения и неправильная работа программы Исключение:

Exception in thread "JavaFX Application Thread" java.lang.IndexOutOfBoundsException 
at com.sun.javafx.scene.control.ReadOnlyUnbackedObservableList.subList(ReadOnlyUnbackedObservableList.java:136) 
at javafx.collections.ListChangeListener$Change.getAddedSubList(ListChangeListener.java:242) 
at com.sun.javafx.scene.control.behavior.ListViewBehavior.lambda$new$177(ListViewBehavior.java:269) 
at javafx.collections.WeakListChangeListener.onChanged(WeakListChangeListener.java:88) 
at com.sun.javafx.collections.ListListenerHelper$Generic.fireValueChangedEvent(ListListenerHelper.java:329) 
at com.sun.javafx.collections.ListListenerHelper.fireValueChangedEvent(ListListenerHelper.java:73) 
at com.sun.javafx.scene.control.ReadOnlyUnbackedObservableList.callObservers(ReadOnlyUnbackedObservableList.java:75) 
at javafx.scene.control.MultipleSelectionModelBase.clearAndSelect(MultipleSelectionModelBase.java:378) 
at javafx.scene.control.ListView$ListViewBitSetSelectionModel.clearAndSelect(ListView.java:1403) 
at com.sun.javafx.scene.control.behavior.CellBehaviorBase.simpleSelect(CellBehaviorBase.java:256) 
at com.sun.javafx.scene.control.behavior.CellBehaviorBase.doSelect(CellBehaviorBase.java:220) 
at com.sun.javafx.scene.control.behavior.CellBehaviorBase.mousePressed(CellBehaviorBase.java:150) 
at com.sun.javafx.scene.control.skin.BehaviorSkinBase$1.handle(BehaviorSkinBase.java:95) 
at com.sun.javafx.scene.control.skin.BehaviorSkinBase$1.handle(BehaviorSkinBase.java:89) 
at com.sun.javafx.event.CompositeEventHandler$NormalEventHandlerRecord.handleBubblingEvent(CompositeEventHandler.java:218) 
at com.sun.javafx.event.CompositeEventHandler.dispatchBubblingEvent(CompositeEventHandler.java:80) 
at com.sun.javafx.event.EventHandlerManager.dispatchBubblingEvent(EventHandlerManager.java:238) 
at com.sun.javafx.event.EventHandlerManager.dispatchBubblingEvent(EventHandlerManager.java:191) 
at com.sun.javafx.event.CompositeEventDispatcher.dispatchBubblingEvent(CompositeEventDispatcher.java:59) 
at com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(BasicEventDispatcher.java:58) 
at com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(EventDispatchChainImpl.java:114) 
at com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(BasicEventDispatcher.java:56) 
at com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(EventDispatchChainImpl.java:114) 
at com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(BasicEventDispatcher.java:56) 
at com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(EventDispatchChainImpl.java:114) 
at com.sun.javafx.event.BasicEventDispatcher.dispatchEvent(BasicEventDispatcher.java:56) 
at com.sun.javafx.event.EventDispatchChainImpl.dispatchEvent(EventDispatchChainImpl.java:114) 
at com.sun.javafx.event.EventUtil.fireEventImpl(EventUtil.java:74) 
at com.sun.javafx.event.EventUtil.fireEvent(EventUtil.java:54) 
at javafx.event.Event.fireEvent(Event.java:198) 
at javafx.scene.Scene$MouseHandler.process(Scene.java:3757) 
at javafx.scene.Scene$MouseHandler.access$1500(Scene.java:3485) 
at javafx.scene.Scene.impl_processMouseEvent(Scene.java:1762) 
at javafx.scene.Scene$ScenePeerListener.mouseEvent(Scene.java:2494) 
at com.sun.javafx.tk.quantum.GlassViewEventHandler$MouseEventNotification.run(GlassViewEventHandler.java:380) 
at com.sun.javafx.tk.quantum.GlassViewEventHandler$MouseEventNotification.run(GlassViewEventHandler.java:294) 
at java.security.AccessController.doPrivileged(Native Method) 
at com.sun.javafx.tk.quantum.GlassViewEventHandler.lambda$handleMouseEvent$354(GlassViewEventHandler.java:416) 
at com.sun.javafx.tk.quantum.QuantumToolkit.runWithoutRenderLock(QuantumToolkit.java:389) 
at com.sun.javafx.tk.quantum.GlassViewEventHandler.handleMouseEvent(GlassViewEventHandler.java:415) 
at com.sun.glass.ui.View.handleMouseEvent(View.java:555) 
at com.sun.glass.ui.View.notifyMouse(View.java:937) 
at com.sun.glass.ui.win.WinApplication._runLoop(Native Method) 
at com.sun.glass.ui.win.WinApplication.lambda$null$148(WinApplication.java:191) 
at java.lang.Thread.run(Unknown Source) 

Answer 1

Если вы будете следовать своему текущему подходу, ваш код будет довольно длинным и довольно сложным для отладки. В настоящее время кажется, что:

1. Вы только удаляете и никогда не возвращаете товар.
2. Вы не сохраняете ранее выбранное значение. Представьте себе, что в вашем первом ComboBox «1» был выбран (и впоследствии исключен из других компонентов), и вы решили изменить его на «». Вы не можете получить предыдущее значение непосредственно из ComboBox, потому что .getSelectedItem() вернет только текущее значение, которое будет «уже».

Представляю вам мой подход и кратко расскажу, как это работает.

public class JavaFXApplication extends Application { 

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

    @Override 
    public void start(Stage primaryStage) { 
     FlowPane root = new FlowPane(); 

     ArrayList<String> items = new ArrayList<>(); 
     Collections.addAll(items, " ", "1", "2", "3", "4"); 

     List<ComboBox> comboBoxList = new ArrayList<>(); 
     for (int i = 0; i < 4; i++) { 
      ComboBox c = new ComboBox(FXCollections.observableArrayList(items)); 
      c.setValue(c.getItems().get(0)); 
      c.valueProperty().addListener(removeFromOtherCombo(comboBoxList, c)); 
      comboBoxList.add(c); 
     } 

     root.getChildren().addAll(comboBoxList); 
     Scene scene = new Scene(root, 200, 100); 

     primaryStage.setScene(scene); 
     primaryStage.show(); 
    } 

    public static ChangeListener removeFromOtherCombo(List<ComboBox> boxes, ComboBox current) { 
     return (o, oldVal, newVal) -> { 
      if (newVal.equals(oldVal)) { 
       return; 
      } 

      if (!oldVal.equals(" ")) { 
       boxes.stream() 
         .filter(c -> !c.equals(current)) 
         .forEach((ComboBox) -> { 
          ComboBox.getItems().add(oldVal); 
         }); 
      } 

      if (!newVal.equals(" ")) { 
       boxes.stream() 
         .filter(c -> !c.equals(current)) 
         .forEach((ComboBox) -> { 
          ComboBox.getItems().remove(newVal); 
         }); 
      } 
     }; 
    } 
} 

Для каждого ComboBox я создаю ChangeListener, что знает о всех других ComboBox и ComboBox, к которому он назначен.

c.valueProperty().addListener(removeFromOtherCombo(comboBoxList, c)); 

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

Если старое значение равно новому значению, нам нечего делать.

if (newVal.equals(oldVal)) { 
    return; 
} 

Если старое значение является числом (например, мы изменяем от «1» до «2»), то мы должны вернуть ранее выбранный номер каждому ComboBox, кроме одного, который фильтруют.

if (!oldVal.equals(" ")) { 
    boxes.stream() 
      .filter(c -> !c.equals(current)) 
      .forEach((ComboBox) -> { 
       ComboBox.getItems().add(oldVal); 
      }); 
} 

Такой же подход применяется, когда мы изменяем к ряду, но на этот раз мы должны удалить его из всех ComboBox, кроме одного.