【发布时间】:2014-04-07 09:24:50
【问题描述】:
我有以下问题:我正在实现一种算法,该算法本质上是通过时间执行扫描 (https://en.wikipedia.org/wiki/Sweep_line_algorithm) 以更新数据结构。对于相关的时间点,我会生成适当的事件并将其存储在std::priority_queue 中:在每个步骤中,我都会弹出具有最小时间值的事件,对其进行处理并在必要时将其他一些事件推送到队列中。不幸的是,我的代码运行得很慢。我用 gprof 看了一下,似乎算法在std::priority_queue::pop 中花费了大约 60% 的执行时间。有什么方法可以让操作更高效吗?
编辑:循环本质上是这样的:
typedef std::priority_queue<Event, vector<Event>, greater<Event> > EventQueue;
while(!events.empty())
{
Event e = events.top();
events.pop();
const DirectedEdge &edge = e.current_edge->get_edge();
if(e.type == Event::LEAVING)
{
Event ee = e;
ee.type = Event::ENTERING;
++ee.current_edge;
if(!ee.current_edge)
{
ee.path->set_arrival_time(ee.time);
}
else
{
ee.current_edge->set_departure_time(ee.time);
ee.time += get_delay(ee.current_edge->get_edge());
events.push(ee);
}
}
else
{
Event ee = e;
ee.type = Event::LEAVING;
const FlowFloat time = queues[edge.get_index()].insert_traveler(e.time, e.path);
ee.time = time;
events.push(ee);
}
}
【问题讨论】:
-
您需要展示代码的相关部分才能获得有用的答案。
-
如果将优先级队列实现为最小堆,则应该是最坏情况 log(n) 来弹出优先级队列的最小值。从根目录删除并向下冒泡。
-
那段时间实际上是在
pop还是在pop以及从pop调用的东西?pop所做的主要事情是比较和交换Event对象并最终销毁一个对象,因此您应该检查这些操作的性能。 -
好吧,在 pop() 中,大约 50% 的时间用于调用 pop_heap() 并在其中调用 __adjust_heap
-
嗯,队列大小可能高达 10000。这可能是个问题吗?
标签: c++ performance algorithm priority-queue