更新される値の移動平均と、Unityのプログラムが動作しないことを解決したい

Question

前提・実現したいこと

更新し続ける「rssi」を古い値から最新の値を含めて5つの値を配列に格納して移動平均を行い、最新の値が移動平均より10大きいと平均にをとる値に含めないようにしたい

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

現在移動平均を行うプログラムの制作したが、Unityでは、エラーは出てはいないが、UnityのRegionからrssiの値を取得し、テキストに表示させるプログラムなのだが、それが実行されないためか、表示されず「New Text」のままになっている。

該当のソースコード

省略
public int rssi;
int cnt = 5;
    int[] R_rssi = new int[5];
　　double total=0;
    public enum BroadcastMode1
    {
        send = 0,
        receive = 1,
        unknown = 2
    }
    public enum BroadcastState1
    {
        inactive = 0,
        active = 1
    }

    private BroadcastMode1 bm_Mode;
    private BroadcastState1 bs_State;

    [SerializeField]
    private GameObject go_FoundBeacon;
    List<GameObject> go_FoundBeaconCloneList = new List<GameObject>();

    private List<Beacon> mybeacons = new List<Beacon>();
    // Start is called before the first frame update
    void Start()
    {
        BluetoothState.EnableBluetooth();
        BluetoothState.Init();
        iBeaconReceiver.Scan();
        iBeaconReceiver.BeaconRangeChangedEvent += OnBeaconRangeChanged;
        iBeaconReceiver.BluetoothStateChangedEvent += OnBluetoothStateChanged;
       
    }

    void onDestroy()
    {
        iBeaconReceiver.Stop();
        iBeaconReceiver.BeaconRangeChangedEvent -= OnBeaconRangeChanged;
        iBeaconReceiver.BluetoothStateChangedEvent -= OnBluetoothStateChanged;
    }


   
    void Onbi()
    {
           //iBeaconReceiver.Scan();
        Region = iBeaconReceiver.regions[0].regionName;
        UUID = iBeaconReceiver.regions[0].beacon.UUID;
        Major = iBeaconReceiver.regions[0].beacon.major;
        Minor = iBeaconReceiver.regions[0].beacon.minor;
        Range = mybeacons[0].range;
        Strength = mybeacons[0].strength;
        Accuracy = mybeacons[0].accuracy;
        rssi = mybeacons[0].rssi;

        for(int i=0; i<cnt-1;i++)
        {
            total += R_rssi[i];
        }
        //Debug.Log("ok0");
        
        total /= cnt;

        for(int i=cnt-1; i>0; i++)
        {
            R_rssi[i] = R_rssi[i - 1];
        }
         R_rssi[0] = rssi;
        
        

        textUI.text = "RegionName:" + Region + "\nUUID:" + UUID.ToString() + "\nMajor:" + Major.ToString() + "\nMinor:" + Minor.ToString() + "\nRange:" + Range + "\nStrength:" + Strength.ToString() + "db" + "\nAccuracy:" + Accuracy.ToString()
                           + "m" + "\nRssi:" + rssi.ToString() + "db";
｝

 private void OnBeaconRangeChanged(Beacon[] beacons)
    { // 
        foreach (Beacon b in beacons)
        {
            //mybeacons = iBeaconReceiver.regions[0].beacon;
            var index = mybeacons.IndexOf(b);
            
            if (index == -1)
            {

                mybeacons.Add(b);
            }
            else {
                mybeacons[index] = b;
            }
        }
        for (int i = mybeacons.Count - 1; i >= 0; --i)
        {
            if (mybeacons[i].lastSeen.AddSeconds(10) < DateTime.Now)
            {
                mybeacons.RemoveAt(i);
            }
        }
        Onbi();
    }

private void OnBluetoothStateChanged(BluetoothLowEnergyState state) { 
			switch (state) {
			case BluetoothLowEnergyState.TURNING_OFF:
			case BluetoothLowEnergyState.TURNING_ON:
				break;
			case BluetoothLowEnergyState.UNKNOWN:
			case BluetoothLowEnergyState.RESETTING:
                break;

			case BluetoothLowEnergyState.UNAUTHORIZED:
				break;
            case BluetoothLowEnergyState.UNSUPPORTED:
				break;
			case BluetoothLowEnergyState.POWERED_OFF:
				break;
			case BluetoothLowEnergyState.POWERED_ON:
                //iBeaconReceiver.Scan();
                break;
			case BluetoothLowEnergyState.IBEACON_ONLY:
				break;
			default:
				break;
			}
		}
}

試したこと

似たソースを探したが、見つけることができなかった

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

Unity2019.4.15f

Comments

[YAmaGNZ]

「Unityのプログラムが実行されない」とはどういうことですか？

[hoshi-takanori]

Onbi の最初の　for ループで R_rssi[i] = rssi; するのはおかしいのでは。

[wing]

YAmaGNZさん
このプログラムは、Regionからrssiの値をテキストに表示させるプログラムなのですが、その値が表示されないということです。

[YAmaGNZ]

実行されてるけど目的の部分の動作が確認できない。
（コメントアウトされているDebug.Logの出力がされない）ということですか？
Onbiメソッドはどこから呼ばれているのですか？

[wing]

YAmaGNZさん
その通りです。
指摘された部分のソースを追加しました。

[YAmaGNZ]

そもそもOnBeaconRangeChanged自体が呼ばれているのかなど確認されたのでしょうか？
ソースを見る限りOnBeaconRangeChangedが呼ばれていてエラーが発生していないのであればtextUI.textは更新されるかと思います。
ステップ実行などでどこまで実行されているのかなど確認されるべきかと思います。

Answer 1

for(int i=0; i<cnt-1;i++)

    {
        R_rssi[i] = rssi;
        total += R_rssi[i];
    }
    total += rssi;

これを、

    for(int i=0; i<cnt;i++)
    {
        total += R_rssi[i];
    }

に。

Comments

[wing]

R_rssiにrssiの数字を格納するにはどうしたらいいのでしょうか

[y_waiwai]

それは提示のコードのままでいいです
で、どういう不具合があるんでしょうか。それがわからないために、答えようがないです

[wing]

ｙ＿waiwaiさん
不具合の文章を書き直しました。

[y_waiwai]

元のコードのままなら私の修正で移動平均値がとれてたのに、修正されてシッチャカメッチャカになってますな

VisualStudioを使ってるなら、好きなところで止めたり１行づつ実行できたりします
それで変数のナカミも見れます。
そうやって動作を見ていけばどうでしょう

[wing]

無事
解決することができました。
Unityが動作しなかったのは、私が演算子を間違えていたからでした。