为my PriorityQueue实现自定义比较器

Question

我试图解决以下leetcode问题：

给定一个排序数组，两个整数k和x，查找数组中与x最近的k个元素。结果也应该按升序排序。如果有一个领带，较小的元素总是首选。

示例1：输入：[1,2,3,4,5]，k=4，x=3

产出：[1,2,3,4]

示例2：输入：[1,2,3,4,5]，k=4，x=-1

产出：[1,2,3,4]

目前我的错误解决方案如下：

class Solution {
    public List<Integer> findClosestElements(int[] arr, int k, int x) {
        PriorityQueue<Integer> pq = new PriorityQueue<>(arr.length, (a,b) -> a == b ? a - b : Math.abs(a-x) - Math.abs(b-x));

       for(int i=0; i<arr.length; i++) {
           pq.add(arr[i]);

       }



        ArrayList ints = new ArrayList<>();

        for(int i=0;i<k;i++) {
            ints.add(pq.poll());


        }
        return ints;
    }
}

问题在于我传递给构造函数的比较器。其思想是，我希望我的比较器根据任何整数I和输入x之间的最小距离对整数进行排序，然后从队列中轮询k元素。我如何实现一个比较器函数，以这种方式对元素进行排序？

Answer 1

在您的实现中，您没有检查两个整数距离X相同的情况。
以下比较器实现将给出正确的结果：

PriorityQueue<Integer> pq = new PriorityQueue<>(arr.length, 
                (a,b) -> {
                    int comp = Integer.compare(Math.abs(a - x), Math.abs(b - x));
                    if(comp==0) {return Integer.compare(a, b);}
                    return comp;
                });

Answer 2

在这里，问题不在于您的优先级队列，而是我们需要两个具有不同排序格式的结果。您的队列将给出前K个元素，但它永远不会按升序排列，因为它根据元素与“X”的距离排列元素，因此对于给定的X=4，元素3和5都在同一级别，因此您的结果将有像[4,3,5这样的数据]。

最好对结果列表进行单独排序。

执行集合。排序（整数）并返回结果。

Answer 3

我会利用默认的Integer.compare方法。基本上，你想要的是首先检查绝对差异的比较，如果是平局，做一个正常的比较。

static int compare(int x, int a, int b) {
    int comp = Integer.compare(Math.abs(a - x), Math.abs(b - x));
    if (comp == 0) {
        return Integer.compare(a, b);
    }
    return comp;
}

这使得编写实际的优先级队列实现变得非常简洁

static List<Integer> findClosestElements(int[] arr, int k, int x) {
    PriorityQueue<Integer> queue = new PriorityQueue<>(
        arr.length, (a,b) -> compare(x, a, b));
    Arrays.stream(arr).forEach(queue::add);
    return queue.stream().limit(k).sorted().collect(Collectors.toList());
}