Java выдает IndexOutOfBounds при редактировании наблюдаемого списка
Мой учитель научил меня, что когда я обновляю объект в своем списке наблюдаемых, он автоматически обнаруживает изменения и уведомляет об этом наблюдателя. Однако, когда я пытаюсь обновить человека в списке, он выдает исключение индекса за пределами. Я не могу понять, что происходит не так.
private ObservableList<ClubPersoon> personen;
public void updatePersoon(ClubPersoon editPerson) {
GenericDaoJpa.openPersistency();
GenericDaoJpa.em.getTransaction().begin();
UpdatePersoonValues(editPerson);
GenericDaoJpa.em.getTransaction().commit();
GenericDaoJpa.closePersistency();
}
private void UpdatePersoonValues(ClubPersoon currentLid) {
int index = personen.indexOf(personen.stream().filter(p -> p.getId() == currentLid.getId()).findFirst());
personen.get(index).setVoornaam(currentLid.getVoornaam());
personen.get(index).setAchternaam(currentLid.getAchternaam());
personen.get(index).setEmail(currentLid.getEmail());
personen.get(index).setEmailOuders(currentLid.getEmailOuders());
personen.get(index).setEmail(currentLid.getEmail());
personen.get(index).setTelefoonNummer(currentLid.getTelefoonNummer());
personen.get(index).setGeboorteDatum(currentLid.getGeboorteDatum());
personen.get(index).setBackupTelefoon(currentLid.getBackupTelefoon());
personen.get(index).setScore(currentLid.getScore());
personen.get(index).setGeslacht(currentLid.getGeslacht());
personen.get(index).setRijksregisternummer(currentLid.getRijksregisternummer());
personen.get(index).setRol(currentLid.getRol());
personen.get(index).setGraad(currentLid.getGraad());
personen.get(index).setLocatie(currentLid.getLocatie());
personen.get(index).setVerwachteDagen(currentLid.getVerwachteDagen());
personen.get(index).setActiviteiten(currentLid.getActiviteiten());
}
Это путь ошибки:
Caused by: java.lang.ArrayIndexOutOfBoundsException: -1
at java.base/java.util.Vector.elementData(Vector.java:761)
at java.base/java.util.Vector.get(Vector.java:782)
at javafx.base/com.sun.javafx.collections.ObservableListWrapper.get(ObservableListWrapper.java:89)
at domein.Club.UpdatePersoonValues(Club.java:143)
at domein.Club.updatePersoon(Club.java:80)
at domein.DomeinController.updatePersoon(DomeinController.java:113)
at gui.PersoonDetailController.handleBtnOpslaan(PersoonDetailController.java:186)
Ответы 1
Как указано в документации List.indexOf(Object), этот метод возвращает -1, если List не содержит объект, переданный в качестве аргумента. Попытка получить доступ к List через индекс -1 всегда будет приводить к IndexOutOfBoundsException (или ArrayIndexOutOfBoundsException, поскольку вы, кажется, оборачиваете Vector).
Тем не менее, способ, которым вы в настоящее время реализовали свой метод, всегда приведет к возврату -1. Обратите внимание на подпись indexOf; он принимает Object, а не E. Это означает, что вы можете передать любой объект этому методу, и ваш код будет скомпилирован. Теперь взгляните на возвращаемый тип Stream.findFirst() — он возвращает Optional<T>. Другими словами, ваш код эквивалентен:
Optional<ClubPerson> optional = personen.stream().filter(p -> p.getId() == currentLid.getId()).findFirst();
int index = personen.indexOf(optional);
...
Поскольку personen является ObservableList<ClubPerson>, он не может содержать элементы Optional<ClubPerson>.
Не совсем понятно, почему вы пытаетесь найти ClubPerson в списке, а затем запрашиваете индекс этого элемента только для того, чтобы просто снова получить доступ к списку с указанным индексом, чтобы изменить ClubPerson, который вы уже нашли. Почему бы просто не изменить ClubPerson напрямую? Вы даже можете использовать Optional.ifPresent(Consumer).
personen.stream().filter(p -> p.getId() == currentLid.getId()).findFirst().ifPresent(p -> {
p.setVoornaam(currentLid.getVoornaam());
p.setAchternaam(currentLid.getAchternaam());
p.setEmail(currentLid.getEmail());
p.setEmailOuders(currentLid.getEmailOuders());
p.setEmail(currentLid.getEmail());
p.setTelefoonNummer(currentLid.getTelefoonNummer());
p.setGeboorteDatum(currentLid.getGeboorteDatum());
p.setBackupTelefoon(currentLid.getBackupTelefoon());
p.setScore(currentLid.getScore());
p.setGeslacht(currentLid.getGeslacht());
p.setRijksregisternummer(currentLid.getRijksregisternummer());
p.setRol(currentLid.getRol());
p.setGraad(currentLid.getGraad());
p.setLocatie(currentLid.getLocatie());
p.setVerwachteDagen(currentLid.getVerwachteDagen());
p.setActiviteiten(currentLid.getActiviteiten());
});
Если элемент, отсутствующий в списке, является ошибкой, вы можете вместо этого использовать что-то вроде Optional.orElseThrow(Supplier) или Optional.orElseThrow() (Java 10+).
ClubPerson p = personen.stream().filter(p -> p.getId() == currentLid.getId())
.findFirst()
.orElseThrow(); // throws NoSuchElementException if element not found
// set properties of "p"...
Проверьте
indexперед использованием...