Unity(C#)：ある回数分スイッチが押されたときに反応するようにしたい

Question

前提・実現したいこと

Unity(C#)を使用しています。
実現したい内容 …
JキーとFキーが押された場合で異なる処理をしたいと考えています。

• 　Jキーの場合：csvファイルで読み込んだ値とJキーが押された回数が一致した場合、"Jキー"というログを表示したいです。csvファイルは[3, 3, 5, 5, 10, 10, 15]のようになっています。具体的には、Jキーを3回押したらログが表示、Jキーを3回押したらログが表示、Jキーを5回押したらログが表示...のようにしたいです。

Jキーが押されると1ずつ増えるJCounterを用意して、csvファイルで読み込んだ値の回数分Jキーが押されるとJCounterが0になるようにして、csvファイルで読み込んだ値をfor文で繰り返すことで、Jキーを3回押したらログが表示、Jキーを3回押したらログが表示、Jキーを5回押したらログが表示...となるようにしました。」

• 　Fキーが押された場合：範囲を設定したランダムな値とFキーが押された回数が一致した場合、"Fキー"というログを表示したいです。

　Fキーが押されると1ずつ増えるFCounterを用意して、範囲を設定したランダムな値の回数分Fキーが押されるとFCounterが0になるようにして、Fキーをランダム回数押したらログが表示するようにしました。」

使用しているcsvファイルの詳細は下図のとおりです。

実行したコードとその結果

実行したコード

このようなコードを実行しました。

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using System.IO;

public class Botton_Script : MonoBehaviour
{
    // Jキーが押された場合
    private List<string> CsvRandom = new List<string>();
    int i;
    int JCounter = 0;

    // Fキーが押された場合
    int min = 2;
    int max = 5;
    int Ramdom;
    int FCounter = 0;


    void Start()
    {
        TextAsset csv = Resources.Load("RandomTimes") as TextAsset;
        StringReader reader = new StringReader(csv.text);
        while (reader.Peek() > -1)
        {
            string line = reader.ReadLine();
            string[] values = line.Split(',');

            for (i = 0; i < values.Length; i++)
            {
                CsvRandom.Add(values[i]);
            }
        }
    }


    private void Update()
    {
        // Jキーが押された場合
        if (Input.GetKeyDown(KeyCode.J))
        {
            JCounter += 1;
            Debug.Log(JCounter);

            for (i = 0; i < CsvRandom.Count; i++)
            {
                if (Counter == int.Parse(CsvRandom[i]))
                {
                    Debug.Log("Jキー");
                    JCounter = 0;
                }

            }
        }

        // Fキーが押された場合
        Random = Random.Range(min, max);
        if (Input.GetKeyDown(KeyCode.F))
        {
            FCounter += 1;
            Debug.Log(FCounter);

            if (Counter == Random)
            {
                Debug.Log("Fキー");
                FCounter = 0;
            }

        }
    }
}

実行した結果

上記のコードを実行した結果、
以下のようになりました。
「Jキーの場合：for文を使って[3, 3, 5, 5, 10, 10, 15]が順番に来るようにしたのですが、最初の[3]だけしか抽出できていません。」
「Fキーの場合：何回かはFキーをランダム回数押したらログが表示され、FCounterが0になるのですが、途中からFCounterが0になりません。」

Answer 1

Jキーの判断について
JCounterがJキーが押された回数なのは分かると思います。
何回押されたらログを出すか判断する為の数字のCsvRandomの添え字はJキーが押された回数と同じになったらログを出力し、添え字を進めるべきです。
ループで判断するものではありません。

Fキーの判断について
Updateのたびに押さなくてはならない回数がランダムで選択されてしまっています。
なので以下のようなことになっているのではないでしょうか。

１回目、Random = 2、Fキー=1
２回目、Random = 5、Fキー=2
３回目、Random = 1、Fキー=3

という感じです。押すべき回数をいつ求めるかについては、ご自身で仕様を考えてください。

また、Randomという名前の変数はやめたほうがいいかと思います。
乱数を求めるRandomクラスのことなのか、乱数を入れるためのintの変数なのかがぱっと見区別がつきませんのでバグを作りこみやすくなります。

Comments

[TeRa_130]

ご回答ありがとうございます。
・Jキー
Jキーが押された場合のif分岐を下記のようにしました。それにより、csvファイルで読み込んだ値が順番に呼び出されるようになりました。
```C#
if (JCounter == int.Parse(CsvRandom[CsvCounter]))
{
Debug.Log("Jキー");
JCounter = 0;
CsvCounter += 1;
}
```

・Fキー
「RandomTimes = Random.Range(min, max);」をStart()で確定させてから、Fキーが押された場合のif分岐を作成しました。それにより、途中からFCounterが0にならないという不具合が解消しました。