移動平均の誤差を失くし、直前の値がなぜか表示されてしまう動作を改善したい

Question

前提・実現したいこと

移動平均を行うプログラムの制作して、実行させると誤差が出てしまうため、誤差をなくせるようにしたいこととそのプログラムの中で変数を宣言するとおかしな動作をしてしまうため解決したいです。
なお、移動平均の1~4回はdistanceの値をそのまま表示させたいと考えています。
5回目以降は移動平均を行いたいと考えており、0.12589...、0.14125…、0.11220…、0.1、0.112589と5つのdistanceの値をとり平均をと表示させ、データをずらして再計算したいと考えております。

発生している問題・エラーメッセージ

「前提・実現したいこと」で書いたような数値の移動平均をとっています。5回目以降の最新のデータが0，125895の時は、0，1573656…となってしまいます。正しくは、0.125895…に近い値が表示されるようにしたいです。決して前後に飛びぬけている値があるとかではないです。
下記のソースのB2の距離の移動平均のソースの部分に「sum1=0」と値の合計を0にしている部分があるのですが、これを記載してしまうと、移動平均が実行されず直前の「distance2」の値がそのまま表示されてしまいます。

デバッグしてみた結果、3つある配列ma、ma1,ma2の中身、5つ全部がmaはdistance、ma1はdistance１、ma2はdistance２の値になっていました。同じ値を5回足して5で割っている状態でした。この配列が全部同じdistanceのように更新されていく値を保持するためにはどうすれば良いのでしょうか。自分でも調べたり、ほかの変数に格納して、実行してみたのですがうまくは行きませんでした。
何度も質問し、それに付き合っていただき誠にありがとうございます。

該当のソースコード

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using System;
using UnityEngine.UI;
using Mathd;
using System.Linq;

public class Pointer : MonoBehaviour
{
    
    //入る変数
    GameObject iBeacon;
    //取得したscriptが入る変数
    Test001 script;

    double speed = 2.0f;
    /*
    //研究室は縦8.5ｍ、横10.5m
    //Unityは10倍
    //iBeacon[B1]の座標
    Vector3 B1 = new Vector3(-52.5,0,42.5);
    //iBeacon[B2]の座標
    Vector3 B2 = new Vector3(52.5, 0, 42.5);
    //iBeacon[B3]の座標
    Vector3 B3 = new Vector3(0, 0, -42.5);
    */


    //自宅は縦3m、横3.5m
    //Unityは10倍
    //iBeacon[B1]の座標
      Vector3 B1 = new Vector3d(-15, 0, 17.5);
    //iBeacon[B2]の座標
      Vector3 B2 = new Vector3d(15, 0, 17.5);
    //iBeacon[B3]の座標
      Vector3 B3 = new Vector3d(0, 0, -17.5);
    

    //RSSI
    private int RSSI;
    private int RSSI1;
    private int RSSI2;

    //TxPower
    private int TxPower;
    private int TxPower1;
    private int TxPower2;

    //distance
    public double distance;
    public double distance1;
    public double distance2;

    public static double Pow;
    double Va, Vb, x, z, y;
    //int cnt = 5;

    double[] ma = new double[5];
    //double mat = 0;
    double[] ma1 = new double[5];
    double sum1 = 0;
    //double t1 = 0;

    double[] ma2 = new double[5];
    double sum2 = 0;
    //double t2 = 0;

    public Text textUI;
    public Text textUUI;
    public Text textUUUI;

    Vector3d Basepoint;
    public static int count;
    int a;
    int b = 2;
    // Start is called before the first frame update
    void Start()
    {
        iBeacon = GameObject.Find("IBeacon");
        script = iBeacon.GetComponent<Test001>();
        count = 0;

        //RSSI持ってくるかも
        RSSI = script.total;
        RSSI1 = script.total1;
        RSSI2 = script.total2;

        TxPower = script.Strength;
        TxPower1 = script.Strength1;
        TxPower2 = script.Strength2;
        a = 1;
        //iBeaconMath();
        b = 1;
       
        //現在地取得かも～～
        //transform.position = new Vector3d(x, 0.5, z);

    }

    // Update is called once per frame
    void Update()
    {
        
        double step = speed * Time.deltaTime;

        //RSSI持ってくるかも
        RSSI = script.total;
        RSSI1 = script.total1;
        RSSI2 = script.total2;

        TxPower = script.Strength;
        TxPower1 = script.Strength1;
        TxPower2 = script.Strength2;

        IBeaconMath();
        //int c = 1;

        /*
        Vector3 devicepoint = new Vector3d(x, 0.5, z);
        if(count == 0)
        {
            Basepoint = devicepoint;
            count = 1;
        }

        transform.position = Vector3d.MoveTowards(Basepoint, devicepoint, step);
        double tmpX = x;
        double tmpZ = z;
        Basepoint = new Vector3d(tmpX, 0.5, tmpZ);
        */
       
    }

    private void IBeaconMath()
    {
       

       int d = 0;
        //B1の距離の移動平均
        distance = Math.Pow(10.0, (TxPower - RSSI) / 20.0);
        int h = 1;
        
        int l = 1;
        double mat = 0;
        for(int j=0; j<5; j++)
        {
            
            mat += ma[j];
        }
        int e = 1;
        mat /= 5;

        int f = 1;
        for (int j=1; j<5; j++)
        {
            //mat = 0;
            ma[j] = ma[j - 1];

        }
        ma[0] = distance;
        int g = 1;
        
        //double result;
        //result = t - distance;


        //B2の距離の移動平均
        distance1 = Math.Pow(10.0, (TxPower1 - RSSI1) / 20.0);
        //sum1=0;
        for (int i = 0; i < 5; i++)
        {
            
            sum1 += ma1[i];
        }

            sum1 /= 5;
        
        for (int i=4; i>0; i--)
        {
            ma1[i] = ma1[i - 1];
        }

        ma1[0] = distance1;
        

        //B3の距離の移動平均
        distance2 = Math.Pow(10.0, (TxPower2 - RSSI2) / 20.0);
        //sum2 = 0;
        for (int k = 0; k < 5; k++)
        {
            sum2 += ma2[k];
        }

            sum2 /= 5;
        
        //sum2 /= 5;
        for (int k = 4; k > 0; k--)
        {
            ma2[k] = ma2[k - 1];
        }
        ma2[0] = distance2;

        textUI.text = "B1との距離" + distance.ToString() + "m" + "\nB2との距離" + distance1.ToString() + "m" + "\nB3との距離" + distance2.ToString() + "m" + "\nB1のRSSI" + RSSI.ToString()+"db"+"\nB2のRSSI"+RSSI1.ToString()+"db"+ "\nB3のRSSI" + RSSI2.ToString() + "db";
        textUUI.text = "B1とのma距離" + mat.ToString() + "m" + "\nB2とのma距離" + sum1.ToString() + "m" + "\nB3とのma距離" + sum2.ToString() + "m";
        textUUUI.text = d.ToString() + h.ToString() + l.ToString() + e.ToString() + f.ToString() + g.ToString();

        /*
        Va = ((Math.Pow(t1, 2) - Math.Pow(t2, 2)) - (Math.Pow(B2.x, 2) - Math.Pow(B3.x, 2)) - (Math.Pow(B2.z, 2) - Math.Pow(B3.z, 2))) / 2;
        Vb = ((Math.Pow(t1, 2) - Math.Pow(t, 2)) - (Math.Pow(B2.x, 2) - Math.Pow(B1.x, 2)) - (Math.Pow(B2.z, 2) - Math.Pow(B1.z, 2))) / 2;

        z = (Vb * (B3.x - B2.x) - Va * (B1.x - B2.x)) / ((B1.z - B2.z) * (B3.x - B2.x) - (B3.z - B2.z) * (B1.x - B2.x));
        x = (Va - z * (B3.z - B2.z)) / (B3.x - B2.x);
        */



    

    
    
}

試したこと

自分が分かる範囲でソースを見直した
移動平均のソースを調べた。

補足情報（FW/ツールのバージョンなど）

C#
Unity2019.4.15f
visual stdio 2015

Comments

[YAmaGNZ]

何故デバッグしないのですか？

[YAmaGNZ]

＞3つある配列ma、ma1,ma2の中身、5つ全部がmaはdistance、ma1はdistance１、ma2はdistance２の値になっていました

とのことですが、現在提示しているソースコードでの話ですか？
もし異なるのであればそれを提示してください。

複数回呼ぶような処理ですから、
1回目のTxPower、RSSI、結果（各変数の値）
2回目のTxPower、RSSI、結果（各変数の値）
といった感じで具体的な値を提示できませんか？

[wing]

現在提示しているソースです。

[YAmaGNZ]

では入力の値は？

[wing]

B1,B2,のTxPowerは常時-75です
B1は
1回目　RSSI：-85　distance　3.16227766016838
2回目　RSSI：-81　distance　1.99526231496888
3回目 RSSI: -86 distance 3.54813389233576
4回目　RSSI:-82 　 distance 2.23872113856834
5回目　RSSI:-82 　distance　2.23872113856834
B2は
1回目　RSSI：-74　distance１：0.891250938133746
2回目　RSSI:-72　 distance1：0.707945784384138
3回目　RSSI:-79　distance１ : 1.58489319246111
4回目　RSSI:-81　distance１：1.99526231496888
5回目　RSSI:-84 　distance１：2.81838293126445
B3
1回目　RSSI：-8２　distance２　2.23872113856834
2回目　RSSI:-84 　distance2　2.81838293126445
3回目　RSSI：-82　distance2　2.23872113856834
４回目 RSSI: -8３ distance２ 3.54813389233576
５回目　RSSI:-82 　 distance２ 2.23872113856834
5回目　RSSI:-8１ 　distance２　1.99526231496888

Answer 1

        //B1の距離の移動平均
        distance = Math.Pow(10.0, (TxPower - RSSI) / 20.0);

        for(int j=0; j<5; j++)
        {
            mat += ma[j];
        }

        mat /= 5;


        for (int j=4; j>0; j--)
        {
            ma[j] = ma[j - 1];
        }
        ma[0] = distance;

今回求めたdistanceは平均に含まれませんがいいのですか？
この関数が呼ばれた回数が1～4回目は値が1～4個しかないのに5で割って明らかに平均じゃありませんがいいのですか？

こういったことが関係なく値がおかしいといわれるのであれば、入力値と出力値とご自身で正しいと思われる値を記載してください。

また、固定長の配列を使うよりList<T>を使用したほうがコードがすっきりするかと思います。

追記

    //sum1=0;
    for (int i = 0; i < 5; i++)
    {
        sum1 += ma1[i];
    }
    sum1 /= 5;

    for (int i=4; i>0; i--)
    {
        ma1[i] = ma1[i - 1];
    }
    ma1[0] = distance1;

この部分がどう動いているか
distance1は仮定の値、ma1の値はsum1を計算するときの値とします。

回数	distance1	ma1[0]	ma1[1]	ma1[2]	ma1[3]	ma1[4]	sum1
1	1	0	0	0	0	0	0
2	2	1	0	0	0	0	0.2
3	3	2	1	0	0	0	0.64

sum1は(sum1+ma1[0]+ma1[1]+ma1[2]+ma1[3]+ma1[4])/5となっています。
なので3回目は2回目で求めた平均の0.2にma1の合計の3が加算され3.2/5という計算になっています。
また、求めたい平均は1回目から1、1.5、2となるのではないですか？

---

さらに追記

2021/07/30 02:49編集時点でのソースでの検証です。

B1計算部分の動作は以下のようになります。

1回目
入力値 RSSI = -85 : TxPower = -75
distance = 3.16227766016838
配列の和を求める部分
ma[0]=0 : ma[1]=0 : ma[2]=0 : ma[3]=0 : ma[4]=0
mat = 0  :  （参考）配列の合計=0
平均を計算 (mat / 5)
mat = 0
配列をシフト後
ma[0]=3.16227766016838 : ma[1]=0 : ma[2]=0 : ma[3]=0 : ma[4]=0

2回目
入力値 RSSI = -81 : TxPower = -75
distance = 1.99526231496888
配列の和を求める部分
ma[0]=3.16227766016838 : ma[1]=0 : ma[2]=0 : ma[3]=0 : ma[4]=0
mat = 3.16227766016838  :  （参考）配列の合計=3.16227766016838
平均を計算 (mat / 5)
mat = 0.632455532033676
配列をシフト後
ma[0]=1.99526231496888 : ma[1]=3.16227766016838 : ma[2]=3.16227766016838 : ma[3]=3.16227766016838 : ma[4]=3.16227766016838

3回目
入力値 RSSI = -86 : TxPower = -75
distance = 3.54813389233576
配列の和を求める部分
ma[0]=1.99526231496888 : ma[1]=3.16227766016838 : ma[2]=3.16227766016838 : ma[3]=3.16227766016838 : ma[4]=3.16227766016838
mat = 14.6443729556424  :  （参考）配列の合計=14.6443729556424
平均を計算 (mat / 5)
mat = 2.92887459112848
配列をシフト後
ma[0]=3.54813389233576 : ma[1]=1.99526231496888 : ma[2]=1.99526231496888 : ma[3]=1.99526231496888 : ma[4]=1.99526231496888

4回目
入力値 RSSI = -82 : TxPower = -75
distance = 2.23872113856834
配列の和を求める部分
ma[0]=3.54813389233576 : ma[1]=1.99526231496888 : ma[2]=1.99526231496888 : ma[3]=1.99526231496888 : ma[4]=1.99526231496888
mat = 11.5291831522113  :  （参考）配列の合計=11.5291831522113
平均を計算 (mat / 5)
mat = 2.30583663044225
配列をシフト後
ma[0]=2.23872113856834 : ma[1]=3.54813389233576 : ma[2]=3.54813389233576 : ma[3]=3.54813389233576 : ma[4]=3.54813389233576

5回目
入力値 RSSI = -82 : TxPower = -75
distance = 2.23872113856834
配列の和を求める部分
ma[0]=2.23872113856834 : ma[1]=3.54813389233576 : ma[2]=3.54813389233576 : ma[3]=3.54813389233576 : ma[4]=3.54813389233576
mat = 16.4312567079114  :  （参考）配列の合計=16.4312567079114
平均を計算 (mat / 5)
mat = 3.28625134158227
配列をシフト後
ma[0]=2.23872113856834 : ma[1]=2.23872113856834 : ma[2]=2.23872113856834 : ma[3]=2.23872113856834 : ma[4]=2.23872113856834

配列のシフト処理がおかしいので配列の値が正しくありません。
2021/07/21 20:31のコメントにて
B1のバッファ処理がおかしいので2回目以降の値がおかしくなります。（前回のソースだとここは正しかったはず）
と指摘している内容です。

平均を求める部分に関してはmatはローカル変数として定義され毎回0からの加算となるので
平均の計算自体は正常に計算されます。
（今回求めたdistanceが平均に含まれないのは前から疑問として投げかけた部分なので無視）

B2計算部分の動作は以下のようになります。（B3も同じなので省略します）

1回目
入力値 RSSI = -74 : TxPower = -75
distance1 = 0.891250938133746
配列の和を求める部分
ma1[0]=0 : ma1[1]=0 : ma1[2]=0 : ma1[3]=0 : ma1[4]=0
sum1 = 0  :  （参考）配列の合計=0
平均を計算 (sum1 / 5)
sum1 = 0
配列をシフト後
ma1[0]=0.891250938133746 : ma1[1]=0 : ma1[2]=0 : ma1[3]=0 : ma1[4]=0

2回目
入力値 RSSI = -72 : TxPower = -75
distance1 = 0.707945784384138
配列の和を求める部分
ma1[0]=0.891250938133746 : ma1[1]=0 : ma1[2]=0 : ma1[3]=0 : ma1[4]=0
sum1 = 0.891250938133746  :  （参考）配列の合計=0.891250938133746
平均を計算 (sum1 / 5)
sum1 = 0.178250187626749
配列をシフト後
ma1[0]=0.707945784384138 : ma1[1]=0.891250938133746 : ma1[2]=0 : ma1[3]=0 : ma1[4]=0

3回目
入力値 RSSI = -79 : TxPower = -75
distance1 = 1.58489319246111
配列の和を求める部分
ma1[0]=0.707945784384138 : ma1[1]=0.891250938133746 : ma1[2]=0 : ma1[3]=0 : ma1[4]=0
sum1 = 1.77744691014463  :  （参考）配列の合計=1.59919672251788
平均を計算 (sum1 / 5)
sum1 = 0.355489382028926
配列をシフト後
ma1[0]=1.58489319246111 : ma1[1]=0.707945784384138 : ma1[2]=0.891250938133746 : ma1[3]=0 : ma1[4]=0

4回目
入力値 RSSI = -81 : TxPower = -75
distance1 = 1.99526231496888
配列の和を求める部分
ma1[0]=1.58489319246111 : ma1[1]=0.707945784384138 : ma1[2]=0.891250938133746 : ma1[3]=0 : ma1[4]=0
sum1 = 3.53957929700792  :  （参考）配列の合計=3.184089914979
平均を計算 (sum1 / 5)
sum1 = 0.707915859401585
配列をシフト後
ma1[0]=1.99526231496888 : ma1[1]=1.58489319246111 : ma1[2]=0.707945784384138 : ma1[3]=0.891250938133746 : ma1[4]=0

5回目
入力値 RSSI = -84 : TxPower = -75
distance1 = 2.81838293126445
配列の和を求める部分
ma1[0]=1.99526231496888 : ma1[1]=1.58489319246111 : ma1[2]=0.707945784384138 : ma1[3]=0.891250938133746 : ma1[4]=0
sum1 = 5.88726808934946  :  （参考）配列の合計=5.17935222994788
平均を計算 (sum1 / 5)
sum1 = 1.17745361786989
配列をシフト後
ma1[0]=2.81838293126445 : ma1[1]=1.99526231496888 : ma1[2]=1.58489319246111 : ma1[3]=0.707945784384138 : ma1[4]=0.891250938133746

B1の動作と違い配列のシフト処理は正常に動作します。
しかし、sum1のクリアが無いため、平均を正常に求めることが出来ません。
これは2021/07/25 10:16に回答に追記した部分で指摘しています。

例）
3回目の計算の場合、配列の合計値は1.59919672251788となります。
しかし、平均を求めるためのsum1は前回計算で求めたsum1の0.178250187626749+配列合計の1.59919672251788の1.77744691014463となってしまっています。

また、質問を編集されて追記された

デバッグしてみた結果、3つある配列ma、ma1,ma2の中身、5つ全部がmaはdistance、ma1はdistance１、ma2はdistance２の値になっていました。同じ値を5回足して5で割っている状態でした。

という現象は見受けられません。

Comments

[wing]

distanceは、Txpowerが-75、RSSIが-54なら距離が0.089・・・となり、この値に近いものが連続して入力され、移動平均で0.08•••に近い値が出力さてほしいです。しかし現在は、0.1114と出力されています。
1~4回目の処理は移動平均がうまく出力されてから追加したいと考えています。

[YAmaGNZ]

その説明では不十分です。
私の指摘した問題点に関してはどうなのでしょうか？
1つ目の「今回求めたdistanceは平均に含まれませんがいいのですか？」に関しては
1回目に呼ばれた場合、distanceを求めているが平均計算には利用していない為、平均は0となりますがこれでいいのですか？
2つ目の「この関数が呼ばれた回数が1～4回目は値が1～4個しかないのに5で割って」に関しては
１回目のdistanceが１，2回目のdistanceが２だった場合、maの合計値は３でそれを５で割っていることになります。
平均を求めるのであれば２つの値なので3/2となるはずです。
これらは私の指摘は的外れでプログラムは仕様通りに作成されているということなのですか？

私の指摘が違うのであれば具体的に
TxPower、RSSI、distance、ma[0]、ma[1]、ma[2]、ma[3]、ma[4]、mat
の値を1回目に呼んだとき、2回目に呼んだときといった感じで実際にプログラムを動作させたときの値を確認して提示してください。
そしてその値から例えば「X回目のmatの値が0.～となっているが、正しくは0.～となって欲しい」とか具体的に問題点を提示してください。

[YAmaGNZ]

他にもB3の計算部分でif文ではtを使っているのにインクリメントしているのはt2であったりとおかしいのでは？といった部分がありますし、変数定義部分も書いていないので変数スコープが正しいのかも分かりません。

こういったことはデバッガでステップ実行したり、どのように動いているのかDebug.Logを入れたりすれば分かることです。
なので私は厳しく「デバッグしないのですか？」とお聞きしました。

[wing]

非常に分かりずらいソースコードで、すいませんでした。
Android端末で行っているため、どこまでプログラムが動いているか確認するために、自分で決めたわかりやすい変数を置いてそれをテキスト表示されるようにしています。
疑問にお答えできるように説明を追加しました。

[YAmaGNZ]

正確な値は実機ではないと得られないでしょうが、ロジックテストなのですから実機でなくても適当な値でPC上で動作させてテストできるはずです。

で、現状で提示されているソースを動かした結果
B1のバッファ処理がおかしいので2回目以降の値がおかしくなります。（前回のソースだとここは正しかったはず）
B2、B3に関してはsum1とsum2はクリアされずに合計され続けるので計算結果が正しくありません。

私は実機を持っていないので当然実際のTxPowerやRSSIの値は得られませんが、TxPower=-75、RSSI=-54と一定の値を入力して複数回実行するだけでこれだけ分かります。

Answer 2

 if (Math.Abs(distance1 - tmp[1]) > 0)
        {
            sum1 = 0;

            total[1] = 0;
            //3つの配列足す
            for (int j = 0; j < 3; j++)
            {
                sum1 += ma1[j];
            }

            //配列をずらす
            for (int j = 2; j > 0; j--)
            {
                ma1[j] = ma1[j - 1];

            }
            //最新の値を入れる
            tmp[1] = distance1;
            ma1[0] = distance1;
            count[1]++;

            if (count[1] == 1)
            {
                total[1] = distance1;
            }

            else if (count[1] < 3)
            {
                total[1] = sum1 / (count[1] - 1);
            }

            else
            {
                total[1] = sum1 / 3;
            }
        }

Comments

[YAmaGNZ]

平均を取るってことは1回の測定だと誤差が大きいから何回かの平均をとってなるべく正確な距離としましょうってことだと思うのですが、同一値が続いた時無視していいのですか？
測定距離が１，１，２となった場合、１と測定した回数が多いのだから実際の距離は１に近いと考えるべきで実際平均値は1.33となる。貴方の考えだと1.5となりより誤差が大きくなると思います。
さらに１，１，２って計測と１，２，１って計測で結果が変わるって話にもなります。

まぁその仕様でいいって話なら忘れてください。

Answer 3

移動平均を求める箇所が3つもあるのですから共通化しましょう。
同じところに3つも書いてあるのでミスが仮にあっても分かりづらいですし、
好きな値を入れて動作を検証するということもできなくなっています。

こんな感じ:

public class MovingAverage{
    private uint length;
    private uint validLength = 0;
    private uint writePointer = 0;
    private double[] arr;
    public MovingAverage(uint length){
        this.length = length;
        this.arr = new double[length];
    }
    
    public void Push(double x){
        this.arr[writePointer++] = x;
        writePointer %= length;
        validLength = validLength < length ? validLength+1 : length;
    }
    
    public double Value {
        get{
            return arr.Sum()/validLength;
        }
    }
    
}

Comments

[fana]

原因究明の前にこのように変更してしまうと

> B2の距離の移動平均のソースの部分に「sum1=0」と値の合計を0にしている部分があるのですが、これを記載してしまうと、移動平均が実行されず直前の「distance2」の値がそのまま表示されてしまいます。

という不思議な（sum1とdistance2とは別の移動平均に関わるものなのに互いに謎の作用を生じるという）話自体が消え失せてしまいそうなので，
現状の謎動作の原因を把握してから→このようにする　と良いのかな，と．

[wing]

ありがとうございます！

[wing]

もし、これを導入する場合、複数あるdistanceをどのように組み込めばよいのでしょうか。

[ozwk]

複数インスタンスを作ってください
何言ってるかわからないならC#の入門書か何か読んでください

[wing]

プログラムの解説を頂いてもよろしいでしょうか。
lengthとvaildlengthとwritepointerがなにをさしているのか教えていただきたいです。

[ozwk]

lengthはarrの長さ
validLengthは有効なデータ数
writePointerは次に配列のどこに書くか

[wing]

ありがとうございます

Answer 4

こちらのfor文は 0,1,2,3,4 計５回繰り返されますが

//B3の距離の移動平均
distance2 = Math.Pow(10.0, (TxPower2 - RSSI2) / 20.0);
for (int k = 0; k < 5; k++)
{
    t2 += 1;
    sum2 += ma2[k];
}

その後のfor文　4,3,2,1 計４回しか繰り返されません

//sum2 /= 5;
for (int k = 4; k > 0; k--)
{
    ma2[k] = ma2[k - 1];
}
ma2[0] = distance2;

平均値がずれているのはそのためではないでしょうか？

Comments

[wing]

最初に0~4を足して、それを5で割り、1〜4をずらして、0に新しい値を入れるようにしています

[nonoshu]

なるほど。そういうことでしたか。
ちなみに開示頂いているコードは変数宣言がなかったりしてますので
一部抜粋されているのでしょうか？
色々ご都合もあるでしょうが、現状開示頂いたコードですと私の環境ではビルドも通らないため
検証が出来ません。

ブレークポイントを使って１つずつ手動計算していくのが良いのではないでしょうか？

[wing]

一部を抜粋しております。

Answer 5

    for(int j=0; j<5; j++)
    {
        mat += ma[j];
    }

まず、matが初期化されてません

    for (int k = 0; k < 5; k++)
    {
        t2 += 1;
        sum2 += ma2[k];
    }

これも同様

Comments

[wing]

初期化すると直前のdistanceの値にになってしまいます

[y_waiwai]

ちと意味不明ですが、
初期化しとかないと積算値がむちゃぐちゃになりますよ

[wing]

分かりにくくてすいません。
説明しなおすと、直前のソースコードのdistanceの計算結果と同じものが移動平均の計算結果として表示されてしまいます。

[y_waiwai]

Update関数中では読み出し行ってないじゃないですか。
なにやってんだか。
常に同じ値でループ回してるだけだから同じ値が出てくるのは当たり前ですがな

[wing]

失礼を承知でお伺いします。自分の知識不足ですいません。関数中で読み出しを行なっていないとはどういうことなのでしょうか？