19

我想对特定方法进行一些性能测量，但我想平均完成所需的时间。（这是一个 C# Winforms 应用程序，但这个问题很可能适用于其他框架。）

我有一个秒表，我在方法开始时重置并在结束时停止。我想将最后 10 个值存储在列表或数组中。添加的每个新值都应将最旧的值从列表中移出。

我会定期调用另一种方法来平均所有存储的值。

我认为这个构造是一个循环缓冲区是否正确？

如何创建具有最佳性能的缓冲区？现在我有以下内容：

List<long> PerfTimes = new List<long>(10);

// ...

private void DoStuff()
{
    MyStopWatch.Restart();
    // ...
    MyStopWatch.Stop();
    PerfTimes.Add(MyStopWatch.ElapsedMilliseconds);
    if (PerfTimes.Count > 10) PerfTimes.RemoveAt(0);
}

不知何故，这似乎效率低下，但也许不是。

建议？

4

8 回答 8

22

您可以创建一个自定义集合：

class SlidingBuffer<T> : IEnumerable<T>
{
    private readonly Queue<T> _queue;
    private readonly int _maxCount;

    public SlidingBuffer(int maxCount)
    {
        _maxCount = maxCount;
        _queue = new Queue<T>(maxCount);
    }

    public void Add(T item)
    {
        if (_queue.Count == _maxCount)
            _queue.Dequeue();
        _queue.Enqueue(item);
    }

    public IEnumerator<T> GetEnumerator()
    {
        return _queue.GetEnumerator();
    }

    IEnumerator IEnumerable.GetEnumerator()
    {
        return GetEnumerator();
    }
}

您当前的解决方案有效，但效率低下，因为删除 a 的第一项List<T>很昂贵。

于 2011-06-17T22:54:34.420 回答
10 
private int ct = 0;
private long[] times = new long[10];

void DoStuff ()
{
   ...
   times[ct] = MyStopWatch.ElapsedMilliseconds;
   ct = (ct + 1) % times.Length; // Wrap back around to 0 when we reach the end.
}

这是一个简单的圆形结构。这不需要其他解决方案所具有的链表节点的数组复制或垃圾收集。

于 2011-06-17T23:33:00.213 回答
3

为了获得最佳性能，您可能只使用 long 数组而不是列表。

我们曾经有一个类似的要求来实现下载时间估计器，并且我们使用循环缓冲区来存储最后N几秒的速度。

我们对整个时间的下载速度不感兴趣，只是根据最近的活动大致预计需要多长时间，但不是最近到数据会到处乱跳（例如，如果我们刚刚使用最后一秒计算）。

我们对整个时间范围不感兴趣的原因是下载可能会在半小时内达到 1M/s，然后在接下来的十分钟内切换到 10M/s。尽管您现在的下载速度非常快，但前半小时会严重降低平均速度。

我们创建了一个循环缓冲区，每个单元格保存 1 秒内下载的数量。循环缓冲区大小为 300，允许 5 分钟的历史数据，并且每个单元都初始化为零。在您的情况下，您只需要十个单元格。

我们还维护了一个总数（缓冲区中所有条目的总和，因此最初也是零）和计数（显然最初是零）。

每一秒，我们都会计算出自上一秒以来下载了多少数据，然后：

  • 从总数中减去当前单元格。
  • 将当前图形放入该单元格并推进单元格指针。
  • 将当前数字加到总数中。
  • 如果还不是 300，则增加计数。
  • 根据总数/计数更新显示给用户的数字。

基本上，在伪代码中：

def init (sz):
    buffer = new int[sz]
    for i = 0 to sz - 1:
        buffer[i] = 0 
    total = 0
    count = 0
    index = 0
    maxsz = sz

def update (kbps):
    total = total - buffer[index] + kbps   # Adjust sum based on deleted/inserted values.
    buffer[index] = kbps                   # Insert new value.
    index = (index + 1) % maxsz            # Update pointer.
    if count < maxsz:                      # Update count.
        count = count + 1
    return total / count                   # Return average.

这应该很容易适应您自己的要求。总和是“缓存”信息的一个很好的功能，它可以让你的代码更快。我的意思是：如果你需要计算总和或平均值，你只能在数据发生变化时才能计算出来，并使用最少的必要计算。

另一种方法是在请求时将所有十个数字相加，这在将另一个值加载到缓冲区时会比单个减法/加法要慢。

于 2011-06-17T22:55:06.983 回答
1

您可能希望改为使用 Queue 数据结构。你可以使用一个简单的线性列表，但它完全是低效的。可以使用圆形数组，但您必须不断调整它的大小。因此，我建议您使用队列。

于 2011-06-17T22:57:03.173 回答
1

我需要在一个数组中保留 5 个最后的分数，我想出了这个简单的解决方案。希望它会帮助一些人。

void UpdateScoreRecords(int _latestScore){
        latestScore = _latestScore;
        for (int cnt = 0; cnt < scoreRecords.Length; cnt++) {
            if (cnt == scoreRecords.Length - 1) {
                scoreRecords [cnt] = latestScore;
            } else {
                scoreRecords [cnt] = scoreRecords [cnt+1];
            }
        }
    }
于 2017-05-28T14:10:00.660 回答
0

对我来说似乎没问题。改用 LinkedList 怎么样？使用 List 时，如果您删除第一项，则所有其他项都必须退回一项。使用 LinkedList，您可以以很少的成本在列表中的任何位置添加或删除项目。但是，我不知道这会有多大的不同，因为我们只谈论十个项目。

链表的权衡是您无法有效地访问列表的随机元素，因为链表必须基本上“沿着”列表“行走”，传递每个项目，直到它到达您需要的项目。但是对于顺序访问，链表很好。

于 2011-06-17T22:49:44.913 回答
0

对于java来说，可能是这样的

import java.util.Iterator;
import java.util.LinkedList;
import java.util.Queue;

public class SlidingBuffer<T> implements Iterable<T>{
    private Queue<T> _queue;
    private int _maxCount;

    public SlidingBuffer(int maxCount) {
        _maxCount = maxCount;
        _queue =  new LinkedList<T>();
    }

    public void Add(T item) {
        if (_queue.size() == _maxCount)
            _queue.remove();
        _queue.add(item);
    }

    public Queue<T> getQueue() {
        return _queue;
    }

    public Iterator<T> iterator() {
        return  _queue.iterator();
    }
}

可以这样开始

public class ListT {

    public static void main(String[] args) {
        start();
    }

    private static void start() {
        SlidingBuffer<String> sb = new SlidingBuffer<>(5);
        sb.Add("Array1");
        sb.Add("Array2");
        sb.Add("Array3");
        sb.Add("Array4");
        sb.Add("Array5");
        sb.Add("Array6");
        sb.Add("Array7");
        sb.Add("Array8");
        sb.Add("Array9");

        //Test printout
        for (String s: sb) {
            System.out.println(s);
        }
    }
}

结果是

数组5

阵列6

数组7

数组8

数组9

于 2018-12-05T22:00:15.307 回答
0

在最新答案发布多年后，我在寻找相同的解决方案时偶然发现了这个问题。我以上述答案的组合结束，尤其是以下答案：agent-j 骑自行车和Thomas Levesque 使用队列

public class SlidingBuffer<T> : IEnumerable<T>
{
    protected T[] items;
    protected int index = -1;
    protected bool hasCycled = false;

    public SlidingBuffer(int windowSize) 
    {
        items = new T[windowSize];
    }

    public void Add(T item)
    {
        index++;
        if (index >= items.Length) {
            hasCycled = true;
            index %= items.Length;
        }

        items[index] = item;
    }

    public IEnumerator<T> GetEnumerator()
    {
        if (index == -1)
            yield break;

        for (int i = index; i > -1; i--)
        {
            yield return items[i];
        }

        if (hasCycled) 
        {
            for (int i = items.Length-1; i > index; i--)
            {
                yield return items[i];
            }
        }
    }

    IEnumerator IEnumerable.GetEnumerator() => GetEnumerator();
}

我不得不放弃j-agent非常优雅的单行代码：ct = (ct + 1) % times.Length; 因为我需要检测我们何时回旋（通过hasCycled）才能拥有一个表现良好的枚举器。请注意，枚举器返回的值从最近到最旧的值。

于 2021-02-21T16:19:33.997 回答