ボタンを押したタイミングを記録した配列が、Jsonファイルへ出力されない

Question

Jsonに配列を出力する方法について質問です。
以下のコードではOnClick()でtapのタグが付いたボタンを押すたびに配列内へそのタイミングを記録し、配列の大きさを1つ増やすリサイズの処理をしています。そしてFinishのタグが付いたボタンを押すとタイミングが"soundscore"に配列形式で記録されたJsonファイルが出力される仕組みです。これを譜面データとして利用したいのですが、試しに31回tapボタンを押してから出力したところ以下の結果になりました。

{"soundscore":[0.0,0.0]}

何度か修正を繰り返したところ、Makerクラス内で宣言した配列soundscore[2]が値が入っていない状態そのままで書き出しに使用され、mk.soundscore[i]そのものを参照できていないのではないかという考えにたどり着きました。そこで、i回分クリックしたタイミングのデータを格納したmk.soundscore[i]のデータをどのような処理を使えばJsonファイルへ書き出せるか知りたいです。

目指していること
Jsonファイルを出力した時、ボタンを押したタイミングが回数分きちんとファイル内に反映される(以下の状態が理想の出力で、mk.soundscore[0]などにそれぞれ格納されているfloat型の数値が入ります)

{"soundscore":[mk.soundscore[0],mk.soundscore[1]・・・mk.soundscore[i]]}

using UnityEngine;
using System.IO;
using System;
using UnityEngine.UI;

[Serializable]
public class Maker
{
    public float[] soundscore = new float[2];
    //public int[] Notekind;
}

public class timing : MonoBehaviour
{

    float TimeCount = 0;
    //インスタンス作成
    Maker mk = new Maker();
    public GameObject ko;
    public GameObject lp;
    int i=1;
    
    // Start is called before the first frame update
    void Start()
    {
　　　　// 配列を初期化
        mk.soundscore[0] = 0;
        ko = GameObject.FindWithTag("tap");
        lp = GameObject.FindWithTag("Finish");
    }

    // Update is called once per frame
    void Update()
    {
        //開始からの時間を計測
        TimeCount += Time.deltaTime;
    }

    public void OnClick()
    {
        if (tag == "tap")
        {
            //ボタンを押したタイミングを記録し、配列の要素数を増やしつつ
            //配列に代入
            mk.soundscore[i] = TimeCount;
            Array.Resize(ref mk.soundscore, i + 1);
            Debug.Log(mk.soundscore[i]);
        }

        if (tag == "Finish")
        {
            //Jsonファイルで出力
            string json = JsonUtility.ToJson(mk);
            File.WriteAllText("Assets\testj", json);
            //読み込みの確認用 start()内で指定した数値しか返らない
            int u;
            for (u = 0; u < i; u++)
            {
                Debug.Log(mk.soundscore[u]);
            }
        }
    }
}

参考サイト様
https://sawalemounity.hatenablog.com/entry/2017/09/23/223000
ー＞こちらのサイト様にあるソースコードの一部分たちを試行錯誤して組み込んだものが上のコードになります。

使用環境
Unity2019,1,5f1 C#のみ使用しています

Answer 1

こんにちは。
OnClick()にてtagが"tap"だった場合の処理にて
配列の要素数は増えていますが、iの変数の値は増えていないようです。
iをインクリメントすることで解決できると思います。

public void OnClick()
{
    if (tag == "tap")
    {
        //ボタンを押したタイミングを記録し、配列の要素数を増やしつつ
        //配列に代入
        mk.soundscore[i] = TimeCount;

        // iの値を更新する
        i++;

        // 現在の配列数に +1 する
        Array.Resize(ref mk.soundscore, mk.soundscore.Length + 1);
        Debug.Log(TimeCount);
    }
}

余談ですが、このような場合は配列を使うよりListを使ったほうがいいかもしれません。
Listでしたら、'Add()'関数を使うことで最後尾に要素を追加できるので、
インデックス用の変数を所持しなくてすみます。
ご参考までに。

using UnityEngine;
using System;
using System.Collections.Generic; // Listを使う際の名前空間

[Serializable]
public class Maker
{
    public List<float> soundscore = new List<float>();
}

public class timing : MonoBehaviour
{
    Maker mk = new Maker();

    // Start is called before the first frame update
    void Start()
    {
        // 配列を初期化
        mk.soundscore.Add(0);
    }
    public void OnClick()
    {
        if (tag == "tap")
        {
            mk.soundscore.Add(TimeCount)
            Debug.Log(TimeCount);
        }

        if (tag == "Finish")
        {
            string json = JsonUtility.ToJson(mk);
            File.WriteAllText("Assets\testj", json);
            for (int u = 0; u < mk.soundscore.Count; u++)
            {
                Debug.Log(mk.soundscore[u]);
            }
        }
    }
}

Comments

[tuna-uniko]

YASU_jrt様
ありがとうございます。Listを利用するとよいことは初めて知りました。
実際にあれこれ試してみましたが、前者では「範囲外やぞお前」とエラーが出る他、エラーが消えても出力は変わりませんでした。Listを使った書き方では、エラーこそ一度も出なかったですが出力は初期化で加えた0のみでした(コンソールには要素数1、値0が出力)。
このことから、初期化された状態のmkがなぜかOnClick()内 tag=="tap"の処理で上書きされずそのままになっていると考えているのですが、なぜなのでしょうか。

[YASU_jrt]

前者の「範囲外やぞお前」エラーの件は修正しました。

OnClick()処理内の tag で判定している件について質問があるのですが、
tag が "tap" か "Finish" かで判定していますが、
もしかして tap用のGameObjectと、finish用のGameObject それぞれにこの timing コンポーネントはアタッチされていますか？
Hierarchy上に timing コンポーネントは複数存在していますか？

[tuna-uniko]

tapとFinishのタグが付いたGameObjectそれぞれにtiming.csがアタッチされています。
実は先ほど思い切ってOnClick()のtagがFinishの時の処理でiのみ表示するよう書き換えるとiが0のままでした。またUpdate()内から新しく作ったTes()とEnd()をキー入力で呼び出す仕組みに変えたところ、iが増加していることも確認できました。
こうなったのはtimingが処理を担当する複数のGameObjectにアタッチされているために、tapとFinishそれぞれのGameObjectが参照するiの値が増加後のiの値と初期化処理をされただけのi=0とで異なっていたからですか？

[YASU_jrt]

はい、その通りです。

tapのGameObjectにアタッチされている timing コンポーネントと
finishのGameObjectにアタッチされている timing コンポーネントは
別々のもの扱いになっています。

そのため、tap の timing コンポーネントは OnClick()が呼ばれるたびに soundscore は増えていきますが、
finish の timing コンポーネント の soundscore は増えていません。
そのため finish の timing コンポーネント の OnClick() が呼ばれても soundscore は初期化されたままのため、
json に書き込んでも数値が増えていないということになります。

[tuna-uniko]

回答ありがとうございます。ずっと上手くいかず頭を悩ませていたのでスッキリしました。Listについても教えていただき感謝しております。これからはコンポーネントが別々になることにも注意してプログラムを組んでいこうと思います。

YASU_jrt様、本当にありがとうございました！