我的比较器的比较方法不起作用

Question

我有一个客户对象类，它有一些变量，并且已经实现了一个关于其中一个变量的比较器。然而，我需要为不同的变量last_name实现另一个比较器。

因为我的Customer类中不能有2个compareTo（）方法，所以我决定在这里专门创建一个Comparing类

public class CompareByLastName implements Comparator<Customer> {

    private List<Purchase> purchases;
    private List<Customer> customers;

    public CompareByLastName(List<Purchase> purchases, List<Customer> customers) {
        this.purchases = purchases;
        this.customers = customers;
    }

    /**
     * @param descending
     * @return will be a sorted, in ascending, or descending, array of customer's according to their authors.
     */
    public List<Purchase> sortByLastName(boolean descending){

        List<Purchase> return_List = new LinkedList<Purchase>();

        Collections.sort(customers);

        if(descending == true) {
            Collections.reverse(customers);
        }

        for(Customer customer : customers) {
            for(Purchase purchase_info : purchases) {
                if(customer.getId() == purchase_info.getCustomer_id()) {
                    return_List.add(purchase_info);
                }
            }
        }

        return return_List;
    }

    @Override
    public int compare(Customer customer_1, Customer customer_2) {

        int result = customer_1.getLastName().compareTo(customer_2.getLastName());

        if(result < 0) {
            return -1;
        }
        else if(result > 0) {
            return 1;
        }
        else {
            return 0;
        }
    }   
}

但一旦触及Collections.sort（客户）；

它不会激活下面的公共int比较（Customer Customer_1，Customercustomer_2）。

坦白地说，我不知道在这种情况下它作为比较器有什么用；有人知道如何解决这个问题，让它按姓氏排序吗？

哦，一旦它返回了一些，如何设法从购买的100(0-99)个项目到返回列表的103(0-102)个项目？不知道这是怎么发生的。

修复了这一部分，我将for循环切换为Purchase，然后浏览所有客户的列表并找到匹配项，而不是反过来。

任何帮助，不胜感激。

先谢谢你。

Answer 1

您没有使用比较方法。

    Collections.sort(customers);

上面的行按客户的自然顺序（由 Customer.compareTo（） 定义）对客户进行排序，而不是根据您的比较器。相反，您可以这样做：

    Collections.sort(customers, this);

现在，您的< code>CompareByLastName将在排序中用作比较器。

几个旁白:

>

在comparator类中使用sort方法的设计是非常规的。不过，它应该管用

在compare方法中，不需要if-else构造。以下简单得多的实现就足够了：

   return customer_1.getLastName().compareTo(customer_2.getLastName());