Java 8 - компаратор с вложенными объектами - PullRequest
0 голосов
/ 29 октября 2018

Я пишу компаратор для сравнения двух объектов сотрудников.

Здесь я пытаюсь сравнить два объекта сотрудника на основе их отдела, затем их имя и идентификатор соответственно.

Проблема, с которой я здесь сталкиваюсь, заключается в сравнении с примитивами, и их обертки просты, но когда я пытаюсь сравнить двух сотрудников на основе их отдела, я получаю следующую ошибку компиляции:

Тип Employee не определяет getDept (T), который применим здесь

Согласно моему пониманию, даже Отдел getDept () должен быть расширен как

  • getDept (это)

    во время вызова в качестве функции и предоставления сведений об отделе.

Код выглядит следующим образом:

Employee.java

package com.deloitte.javatut.pojo;

public class Employee {

    public Employee() {
        // TODO Auto-generated constructor stub
    }

    private String emptName;
    private Long empId;
    private Department dept;

    public String getEmptName() {
        return emptName;
    }

    public void setEmptName(String emptName) {
        this.emptName = emptName;
    }

    public Long getEmpId() {
        return empId;
    }

    public void setEmpId(Long empId) {
        this.empId = empId;
    }

    public Department getDept() {
        return dept;
    }

    public void setDept(Department dept) {
        this.dept = dept;
    }

}

Department.java

package com.deloitte.javatut.pojo;

Департамент общественного класса {

public Department() {
    // TODO Auto-generated constructor stub
}

private String deptName;
private Long deptId;

public String getDeptName() {
    return deptName;
}

public void setDeptName(String deptName) {
    this.deptName = deptName;
}

public Long getDeptId() {
    return deptId;
}

public void setDeptId(Long deptId) {
    this.deptId = deptId;
}

@Override
public int hashCode() {
    final int prime = 31;
    int result = 1;
    result = prime * result + ((deptId == null) ? 0 : deptId.hashCode());
    result = prime * result + ((deptName == null) ? 0 : deptName.hashCode());
    return result;
}

@Override
public boolean equals(Object obj) {
    if (this == obj)
        return true;
    if (obj == null)
        return false;
    if (getClass() != obj.getClass())
        return false;
    Department other = (Department) obj;
    if (deptId == null) {
        if (other.deptId != null)
            return false;
    } else if (!deptId.equals(other.deptId))
        return false;
    if (deptName == null) {
        if (other.deptName != null)
            return false;
    } else if (!deptName.equals(other.deptName))
        return false;
    return true;
}

}

Логика сравнения:

public static void main(String[] args) {
    // TODO Auto-generated method stub
    Department dept = new Department();
    dept.setDeptId(1L);
    dept.setDeptName("IT");
    Employee emp = new Employee();
    emp.setEmpId(2L);
    emp.setEmptName("John Doe");
    emp.setDept(dept);
    Employee emp2 = new Employee();
    emp2.setEmpId(4L);
    emp2.setEmptName("John Doe 2");
    emp2.setDept(dept);
    Function<Employee, Department> deptFunction = Employee::getDept;
    Comparator<Employee> empComparator = Comparator.comparing(Employee::getDept)
            .thenComparing(Employee::getEmpId).thenComparing(Employee::getEmptName);

}

1 Ответ

0 голосов
/ 29 октября 2018

Department не реализует Comparable<Department>, поэтому Java не считает, что это сопоставимо.

Чтобы сделать эту работу, вы можете либо Department внедрить Comparable<Department>:

class Department implements Comparable<Department> {
   // ...
   public int compareTo(Department other) {
       // By "compare by department", you probably meant comparing by the department name, right?
       // If not, implement your own compareTo
       return getName().compareTo(other.getName()); 
   }
}

В качестве альтернативы, сравните то, что действительно сопоставимо в comparing:

Comparator<Employee> empComparator = Comparator.comparing(x -> x.getDept().getName())
            .thenComparing(Employee::getEmpId).thenComparing(Employee::getEmptName);
...