How to iterate over a PriorityQueue?

2020-01-29 05:31发布

for (Event e : pq)

doesn't iterate in the priority order.

while(!pq.isEmpty()){
  Event e = pq.poll();
}

This works but empties the queue.

9条回答
2楼-- · 2020-01-29 05:58

From the Javadocs:

The Iterator provided in method iterator() is not guaranteed to traverse the elements of the PriorityQueue in any particular order. If you need ordered traversal, consider using Arrays.sort(pq.toArray()).

There are probably other equivalent mechanisms.

查看更多
该账号已被封号
3楼-- · 2020-01-29 05:59

A heap based priority queue only guarantees that the first element is the highest/lowest. There is no cheap (i.e. O(n)) way to get the elements in sorted form.

If you need to do this often, consider using a structure that maintains the elements in sorted form. For example, use java.util.TreeSet, and use either pollFirst() or pollLast() in place of peek() / poll()

查看更多
太酷不给撩
4楼-- · 2020-01-29 06:03

So taking the priorityQueue in a List and then sorting it is a good option as mentioned above. Here are some details why the iterator gives unexpected results:

The iterator does not return elements in the correct order because it prints from the underlying data structure (similar to ArrayList). The ArrayList has data stored in it in the same way the data is stored in an Array implementation of BinaryHeap. For example:

PriorityQueue<Integer> pq = new PriorityQueue<>();
ArrayList<Integer> test = new ArrayList(Arrays.asList(6,12,7,9,2));
test.forEach(x -> pq.add(x)); 
System.out.println("Priority Queue:- "+pq); [2, 6, 7, 12, 9]

where childOf(i) is 2*i+1 and 2*i+2 and parentOf(i) is (i-1)/2

查看更多
登录 后发表回答