Поток списка Java anyMatch, возвращающий неверный результат

Question

Поток списка Java anyMatch, возвращающий неверный результат

1

У меня есть проект "Проект" класса JPA, в котором есть список 'Employee' а getEmployeeList() возвращает список Employees.

public  class Project implements Serializable {

    @ManyToMany(targetEntity=Employee.class)
    private List<Employee> employeeList;

    public List<Employee> getEmployeeList() {
        return this.employeeList;
    }
}

public  class Employee implements Serializable {
    @Basic
    private EmployeeType type;

    public EmployeeType getType() {
        return this.type;
    }
}

public enum EmployeeType implements Serializable{
  EDITOR,
  AUDITOR,
  ADMIN,
  INACTIVE
}

У меня есть Сотрудник с типом "Аудитор", и когда я запускаю следующие

for(Employee e: p.getEmployeeList()){
     if(e.getType()==EmployeeType.AUDITOR)
         found=true;
 }

он возвращает true, который должен, но когда я запускаю

(p.getEmployeeList()).stream().anyMatch((e) -> (e.getType()==EmployeeType.AUDITOR))

он возвращает false

Что я делаю не так?

pallavt 09 авг. 2014, в 13:33

Источник

0

Он сравнивает перечисляемые переменные, так разве это не должно так работать? Код кажется мне нормальным, даже если в нем есть ненужные скобки. p.getEmployeeList().stream().anyMatch(e -> e.getType() == EmployeeType.AUDITOR) тоже p.getEmployeeList().stream().anyMatch(e -> e.getType() == EmployeeType.AUDITOR) .
Dici 09 авг. 2014, в 11:33
0

да, ты прав. Я собираюсь удалить комментарий.
ditkin 09 авг. 2014, в 11:34
0

Вы уверены, что p.getEmployeeList() не возвращает пустой список? Если это так, то found будет содержать свое начальное значение (которое, возможно, вы по ошибке инициализировали как true), не входя в цикл вообще, но anyMatch вернет false.
Eran 09 авг. 2014, в 12:27
0

мое начальное значение found - false, и я запустил цикл и anyMatch в одном наборе данных, поэтому он должен вернуть тот же результат.
pallavt 09 авг. 2014, в 13:27
2

Попробуйте добавить .peek(System.out::println) в поток перед вашим anyMatch() чтобы увидеть, что на самом деле подается.
Brian Goetz 09 авг. 2014, в 18:05
0

(p.getEmployeeList()).stream().peek(e -> System.out.println(e.getType())) или p.getEmployeeList().stream().forEach((Employee e) -> { System.out.println(e.getType());}); ничего не печатает, но p.getEmployeeList().size() печатает правильное значение. И я только что узнал, что p.getEmployeeList возвращает IndirectList, а IndirectList не имеет функции потока, но я не получаю никакой ошибки, утверждая, что функция не существует.
pallavt 10 авг. 2014, в 06:15
0

но если сделать ArrayList<Employee> employeeList = new ArrayList(p.getEmployeeList()); и вызовите System.out.println(employeeList.size()); , employeeList.stream().forEach((Employee e) -> System.out.println(e.getType())); и employeeList.stream().anyMatch(emp-> (emp.getType()==EmployeeType.AUDITOR));//returns true возвращает правильное значение.
pallavt 10 авг. 2014, в 06:24

Показать ещё 5 комментариев

Теги:

java

arraylist

java-8

jpa

java-stream

2 ответа

0

Обновите версию EclipseLink, эта ошибка уже исправлена!

Ошибка: https://bugs.eclipse.org/bugs/show_bug.cgi?id=433075

Обновление до версии 2.6.0 или новее: http://mvnrepository.com/artifact/org.eclipse.persistence/eclipselink/2.6.0

Marcelo C. 13 авг. 2015, в 19:03

Ещё вопросы

Он сравнивает перечисляемые переменные, так разве это не должно так работать? Код кажется мне нормальным, даже если в нем есть ненужные скобки. p.getEmployeeList().stream().anyMatch(e -> e.getType() == EmployeeType.AUDITOR) тоже p.getEmployeeList().stream().anyMatch(e -> e.getType() == EmployeeType.AUDITOR) .
да, ты прав. Я собираюсь удалить комментарий.
Вы уверены, что p.getEmployeeList() не возвращает пустой список? Если это так, то found будет содержать свое начальное значение (которое, возможно, вы по ошибке инициализировали как true), не входя в цикл вообще, но anyMatch вернет false.
мое начальное значение found - false, и я запустил цикл и anyMatch в одном наборе данных, поэтому он должен вернуть тот же результат.
Попробуйте добавить .peek(System.out::println) в поток перед вашим anyMatch() чтобы увидеть, что на самом деле подается.
(p.getEmployeeList()).stream().peek(e -> System.out.println(e.getType())) или p.getEmployeeList().stream().forEach((Employee e) -> { System.out.println(e.getType());}); ничего не печатает, но p.getEmployeeList().size() печатает правильное значение. И я только что узнал, что p.getEmployeeList возвращает IndirectList, а IndirectList не имеет функции потока, но я не получаю никакой ошибки, утверждая, что функция не существует.
но если сделать ArrayList<Employee> employeeList = new ArrayList(p.getEmployeeList()); и вызовите System.out.println(employeeList.size()); , employeeList.stream().forEach((Employee e) -> System.out.println(e.getType())); и employeeList.stream().anyMatch(emp-> (emp.getType()==EmployeeType.AUDITOR));//returns true возвращает правильное значение.

pallavt · Accepted Answer · 2014-08-11T03-26-00.000Z

Решение. Измените возвращаемый тип на arraylist

public ArrayList<Employee> getEmployeeList() {
    return new ArrayList<>(this.employeeList);
}