如何覆盖compareto方法以订购员工薪水?

时间:2019-10-27 15:17:12

标签: java compareto

我有2个班级:人员和员工以及人机界面 在我的Person类中,我有一个compareTo方法(人类h) 分别为该人的年龄分配+ 1,-1和0。我班上的员工= 公共类员工扩展人员实施人员= 我也有一个compareTo方法,如果年龄相同(用于排序),则该方法需要考虑员工的薪水。

我不太确定该如何解决? 我能够为Persons类创建compreTo,但是我不确定如何在此处对人和雇员进行排序。

谢谢您的帮助。

我已经在Employee类中尝试过此操作:

compareTo (Human h) {
Employee e = (Employee)h;

if (super.compareTo(h) == 0 && getSalary ()< e.getSalary())
    return -1;
    else if (super.compareTo(h) == 0 && getSalary () == e.getSalary())
        return 0;
    else 
        return 1;

}

此方法有效,但是我希望能够使用instanceof解决此问题:

     public int compareTo(Human h) {

    // TODO Auto-generated method stub

     if (getAge() < h.getAge()) {
          return -1;
        } else if (getAge() > h.getAge()) {
          return 1;
        } else {
            Employee e = (Employee)h;
          // age is identical: compare salary
          if (getSalary() < e.getSalary()) {
            return -1;
          } else if (getSalary() > e.getSalary()) {
            return 1;
          } else {
            return 0;
          }
        }
      }

下面,我证明了我认为对此问题必要的代码量:

public interface Human extends Comparable <Human>{
//extends = is a


int getAge();
String getName();

}



public class Person implements Human {
    private int age; 
private String name;
    public int compareTo(Human h) {


    //System.out.println(this.age + ". " +h.getAge());
    if (h.getAge() > getAge())
        return -1;
    else if (getAge() == h.getAge())
        return 0;
    else 
        return 1;    
}

public class Employee extends Person implements Human{


private int salary;
private String employer; 

public int compareTo(Human h) {
 ???
}



  public static void main(String[] args) {
  ArrayList<Human> p = new ArrayList<Human>();
    p.add(new Person("A", 1));
    p.add(new Employee("B", 31, "E1", 45000));
    p.add(new Person("C", 122));
    p.add(new Employee("D", 3, "E2", 54321));
    p.add(new Person("E", 21));
    p.add(new Employee("F", 31, "E1", 21000));
    p.add(new Employee("G", 31, "E1", 38000));
    System.out.println(p);
    Collections.sort(p);
    System.out.println(p); }

这是我要测试的:

non sorted: [Person:[A, 1], Employee:[B, 31][E1, 45000], Person:[C, 122], Employee:[D, 3][E2, 54321], Person:[E, 21], Employee:[F, 31][E1, 21000], Employee:[G, 31][E1, 38000]]

sorted: [Person:[A, 1], Employee:[D, 3][E2, 54321], Person:[E, 21], Employee:[F, 31][E1, 21000], Employee:[G, 31][E1, 38000], Employee:[B, 31][E1, 45000], Person:[C, 122]]

任何帮助将不胜感激。

2 个答案:

答案 0 :(得分:0)

为了确保正确的订购; compareTo方法需要满足Comparable接口指定的contract

不幸的是,无法扩展Person;在compareTo中覆盖Employee,以比较薪水,同时保留合同。

一个简单的解决方案是将比较器传递给Collections.sort();确保集合的所有元素都使用相同的比较器实现:

Comparator.comparingInt(Human::getAge).thenComparingInt(h -> h instanceof Employee ? ((Employee) h).getSalary() : 0)

答案 1 :(得分:0)

您可以通过简单地在Person和Employee中实现compareTo方法来实现此目的,

// In Person class
@Override
public int compareTo(Human h) {
    return age - h.getAge();
}

还有

// In Employee class:
@Override
public int compareTo(Human h) {
    int result = super.compareTo(h);
    if ((result == 0) && (h instanceof Employee)) {
        result = salary - ((Employee) h).salary;
    }
    return result;
}

干杯!