3

在过去的几个月里,我一直在从事一个 Visual C++ 项目,以从相机中获取图像并进行处理。直到今天,更新数据大约需要 65 毫秒,但现在它突然显着增加。发生的情况是:我启动我的程序并在前 30 次左右的迭代中按预期执行,然后循环时间突然从 65 毫秒增加到 250 毫秒。

奇怪的是,在对每个函数计时后,我发现导致速度变慢的代码部分相当基本,并且一个多月没有修改过。进入其中的数据在每次迭代时都保持不变且相同,但最初小于 1 毫秒的执行时间突然增加到 170 毫秒,而其余代码仍按预期执行(时间方面)。

基本上,我一遍又一遍地调用同一个函数,对于前 30 次调用,它按应有的方式执行,之后它会无缘无故地变慢。还可能值得注意的是,这是执行时间的突然变化,而不是逐渐增加。

这可能是什么原因造成的?该代码泄漏了一些内存(约 50 kb/s),但还不足以保证突然减速 4 倍。如果有人有任何想法,我很想听听他们的意见!

编辑:哇,真快!这是减慢速度的代码(减去一些数学)。我知道这是一个函数,如果增加行数,计算时间会迅速增加。这里的关键是,对于相同的数据,在 30 次迭代后速度会变慢。

void CameraManager::IntersectLines()
{

    // Two custom classes
    TMaths maths;
    TLine line1, line2;

    while(lines.size()>0)
    {

        // Save the current line
        line1 = lines[0];

        // Then remove it from the list
        lines.erase(lines.begin());

        CvMat* aPoint;
        for (int i = 0; i<lines.size(); i++)
        {

            line2 = lines[i];

            aPoint = cvCreateMat(1, 4, CV_32FC1);

            // Calculate the point of intersection
            maths.Intersect(line1.xyz, line2.xyz, line1.uvw, line2.uvw, aPoint);

            // Add the point to the list
            points.push_back(aPoint);
            }

        }

    }

}

4

7 回答 7

9

是否有可能在泄漏一定数量的内存后,您的计算机必须开始分页输入/输出内容?即使是简单的功能,这也肯定会减慢速度。

在不知道该函数的作用的情况下,很难确切地说出可能导致问题的原因。

编辑:正如问题评论中所建议的那样,泄漏一定量的内存也可能会开始将 CPU 缓存中的东西敲出,这也会减慢速度。要么修复内存泄漏,要么在这里发布代码供我们查看,这将是一个好主意。

编辑 2:您在该循环中调用了几个函数。除了简单的算术计算之外,他们还会做其他事情吗?

于 2010-06-15T18:27:42.270 回答
5

如果它泄漏了足够的内存来填满一个页面(50KB/s 可能就足够了),那么 Windows 将不得不切换页面来处理数据。当这种情况发生时,程序变得更加低效。

于 2010-06-15T18:28:47.807 回答
2

显然有些事情发生了变化。尝试将您的代码恢复到减速之前的状态。如果它再次变快,请关注代码更改。如果速度很慢,请在代码之外寻找问题。诸如数据库,操作系统等之类的东西。

于 2010-06-15T18:31:25.693 回答
2

如果您有内存泄漏,那么您的代码中肯定发生了一些不好的事情。首先解决这个问题,你很有可能会发现你的问题,或者至少消除一个相当明显的嫌疑人。

正如其他人所提到的,如果您提供一些代码,那么尝试帮助您会容易得多......

[编辑] 我猜你的cvCreateMat函数分配了一些内存?那会在任何地方得到释放吗?

于 2010-06-15T18:35:51.613 回答
1

分析代码,然后您无需猜测答案。

当然,无论如何这可能会改变代码的性能,但它是最直接的方法,可以准确地查看代码正在发生的事情......

(根据此问题的当前最佳答案:Profiling in Visual Studio 2008 PRO,您需要 VS 2008 的“团队”版本才能使用内置分析器,否则您需要使用外部分析器)

于 2010-06-15T19:48:56.057 回答
1

需要擦除容器中的每一行吗?

    // Then remove it from the list
    lines.erase(lines.begin());

根据容器的不同,擦除元素(一次一个)可能会很慢,尤其是对于大型容器。我建议在计算过程中使用起始索引。在调用函数之前推进索引。索引和迭代器的更新比擦除元素更快。计算完成后,您可以删除或擦除整个容器(clear如果容器有,请参阅方法)。该clear方法可能比一次擦除一个元素更快。

于 2010-06-15T19:06:14.800 回答
0

如果您想要任何有用的答案,您需要使您的问题更加具体。

于 2010-06-15T18:29:04.807 回答