我试着用C++编写我自己的简单移动平均值

I tried coding my own simple moving average in C++

本文关键字:自己的 我自己 简单 移动 平均值 C++      更新时间:2023-10-16
  1. 我想要一个有效的函数
  2. 我相信我的逻辑是正确的,因此我的(向量超出范围的错误(一定是由于不熟悉和正确使用代码造成的
  3. 我知道这个相当简单的算法有很长的代码。如果可以的话,请帮忙

基本上,我把长度作为"移动"窗口,因为它通过j循环到向量大小的末尾。这个矢量充满了股票价格。

如果数字1、2、3、4的2天移动平均线的长度等于2。我应该能够输出1.5、2.5和3.5。然而,我得到了一个超出范围的错误。

代码中显示了逻辑。如果一位专家能帮助我创建这个简单的移动平均函数,那就太好了!谢谢

void Analysis::SMA()
{
    double length;
    cout << "Enter number days for your Simple Moving Average:" << endl;
    cin >> length;
    double sum = 0;
    double a;
    while (length >= 2){
        vector<double>::iterator it;
        for (int j = 0; j < close.size(); j++){
                sum = vector1[length + j - 1] + vector1[length + j - 2];
                a = sum / length;
                vector2.push_back(a);
                vector<double>::iterator g;
                for (g = vector2.begin(); g != vector2.end(); ++g){
                    cout << "Your SMA: " << *g;
            }
        }
    }
}

计算一个数据数组的移动平均值不需要3个循环,只需要1个。您对数组进行迭代,并跟踪最后n个项的总和,然后针对每个新值对其进行调整,每次添加一个值并移除一个值。

例如,假设您有一个数据集:

4 8 1 6 9

如果你想计算一个窗口大小为3的移动平均线,那么你就可以保持这样的运行总数:

iteration add subtract running-total output average
0         4   -        4             - (not enough values yet)
1         8   -        12            -
2         1   -        13            13 / 3
3         6   4        15            15 / 3
4         9   8        16            16 / 3

注意,我们每次相加,在迭代3时开始相减(对于窗口大小为3(,在迭代2时开始输出平均值(窗口大小减1(。

所以代码将是这样的:

double runningTotal = 0.0;
int windowSize = 3;
for(int i = 0; i < length; i++)
{
     runningTotal += array[i];   // add
     if(i >= windowSize)
         runningTotal -= array[i - windowSize];   // subtract
     if(i >= (windowSize - 1))  // output moving average
         cout << "Your SMA: " << runningTotal / (double)windowSize;
}

您可以对此进行调整以使用矢量数据结构。

在最外层的while循环中,您永远不会更改length,因此您的函数将永远运行。

然后,请注意,如果length是2,closes.size()是4,那么length + j - 1将是5,所以我的心理调试技能告诉我,你的vector1太短了,你把结尾索引掉了。

这个问题已经得到了回答,但我想我会为将来寻求信息的人发布完整的代码。

#include <iostream>
#include <vector>
using namespace std;
int main() {
vector<double> vector1 { 2, 4, 6, 8, 10, 12, 14, 16, 18, 20 };
double length;
cout << "Enter number days for your Simple Moving Average:" << endl;
cin >> length;
double sum = 0;
int cnt = 0;
for (int i = 0; i < vector1.size(); i++) {
    sum += vector1[i];
    cnt++;
    if (cnt >= length) {
        cout << "Your SMA: " << (sum / (double) length) << endl;
        sum -= vector1[cnt - length];           
        }
    }
return 0;
}

这与答案略有不同。中引入了一个"cnt"变量,以避免附加if语句。