时间戳

首页 > 解决方案 > 我的比较器的比较方法不起作用

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

问题描述

我有一个 Customer 对象类，它有一些变量，并且正在实现一个 Comparator 已经关于这些变量之一。但是我需要为不同的变量 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(customers);

它不会激活下面的公共 int compare(Customer customer_1, Customer customer_2)。

坦率地说，我不知道它在排序中用作比较器的是什么。有谁知道如何解决这个问题并按姓氏排序？

哦，一旦退货，如何设法从购买的 100(0-99) 件商品到退货清单中的 103(0-102) 件商品？不知道这是怎么回事。

修复了这部分，我切换了 for 循环阅读购买，然后浏览所有客户的列表并找到匹配项，反之亦然。

任何帮助表示赞赏。

提前致谢。

标签： javasortingcollectionscomparator

解决方案

你没有使用你的compare方法。

    Collections.sort(customers);

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

    Collections.sort(customers, this);

现在您CompareByLastName将用作排序中的比较器。

一些旁白：

  • 您在比较器类中使用排序方法的设计是非常规的。不过，它应该可以工作。

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

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

推荐阅读