【UnityとC#でゲーム制作】deltatimeを使ったスクリプトがうまく機能しません。

Question

ご覧いただきありがとうございます。

Unityで3Dゲームを作っています。

今、キャラクターが光る剣のようなもので
攻撃するモーションを作成しています。

剣を振るときだけ光る刃（ブレード）が表示されるようにしたいのですが、
うまくいきません。

スクリプトで制御して
「BladeOnのあと1秒経ったらBladeOff」
という動作をさせたいです。

以下のスクリプトだと
Onは上手く行っても
「1秒待ってOff」がうまく機能しません。

OnのままでOffになりません。
光りっぱなしです。

どこがおかしいのでしょうか。。。

ちなみにですが、一部スクリプトを変えて
「ボタンを押している間はブレードオン」
「ボタンを離したらブレードオフ」
には成功しています。

    private void Update()
    {



        //　Vが押されたら
        if (Input.GetKeyDown(KeyCode.V))
        {
            //ブレードをオンにする
            bbm.GetComponent<BladeOnOff>().BladeOn();

            //時間カウント開始
            seconds += Time.deltaTime;


        }

        //1秒経ったら
        if (seconds >= 1.0f)
        {
            //時間計測を0に戻す
            seconds = 0f;

            //ブレードをオフにする
            bbm.GetComponent<BladeOnOff>().BladeOff();
        }
    }
}

Comments

[ku__ra__ge]

コードは```とだけ書いた行で囲むように質問文を修正してください。
普通の文章として提示されるとインデントが消えて読みづらいです。

[senkiru]

アドバイスありがとうございます。

修正いたしました。

Answer 1

(ku__ra__geさんの回答への 2022/01/09 21:26 コメントを受けて回答します。)
vを押して時間計測を始めること自体は間違いではなく、コードが間違っています。
vを押したときの処理としては
1.secondsを0にする
2.時間計測を始める
で実現できると思います。ストップウォッチのイメージです。

時間計測は
seconds += Time.deltaTime; //時間を加算
if(seconds >= 1.0f) //その結果が規定値以上なら
までが1セットなので　seconds += Time.deltaTime;　がvを押したならのif中にあると時間計測が成立しません。

Comments

[senkiru]

ありがとうございます。
検討させていただきます。

Answer 2

まずはじめに、Update()が呼ばれた時には２種類の状態があることを認識しないといけません。

１つ目は、ブレードが表示されていない場合で、
２つ目は、ブレードが表示されている場合です。

当たり前のことを言っていますが、このことをきちんと理解しておかなければいけません。

Update()では、２つの状態をわけてコードを書かなければいけません。
となると、Update()は以下のような形になるはずですよね。

void Update(){
    if(ブレードが表示されている）{
        // ブレードが表示されている時の処理
    }else{
        // ブレードが表示されていない時の処理
    }
}

次に、質問者さんが書いたコードを見て、上記２つのどちらの状態の場合の処理か考えていきます。

    //　Vが押されたら
    if (Input.GetKeyDown(KeyCode.V))
    {
        //ブレードをオンにする
        bbm.GetComponent<BladeOnOff>().BladeOn();

        //時間カウント開始
        seconds += Time.deltaTime;
    }

「Vキーが押されたらブレードをオンにする」という処理ですが、これは当然、「ブレードが表示されていない場合」の処理になります。
ところが、「時間カウント開始」のところの seconds += Time.deltaTime; はカウント開始を表していません。経過時間を計算しています。
経過時間の計算とカウント開始をごっちゃに考えてしまっているのが、混乱の元になっていると思われます。

ここでは一旦、カウント開始に焦点を絞って（vann_2921さんの考えと同様に）、secondsを０にするのが良いでしょう。また、それとは別に「ブレードが表示されている」ことを表すフラグも導入したほうが良いです。
以上をふまえて、「ブレードが表示されていない」ときの処理を書くと、

    //　Vが押されたら
    if (Input.GetKeyDown(KeyCode.V))
    {
        //ブレードをオンにする
        bbm.GetComponent<BladeOnOff>().BladeOn();

        //時間カウント開始
        isBladeOn = true;
        seconds = 0;
    }

のような感じになるでしょう。

続いて、

    //1秒経ったら
    if (seconds >= 1.0f)
    {
        //時間計測を0に戻す
        seconds = 0f;

        //ブレードをオフにする
        bbm.GetComponent<BladeOnOff>().BladeOff();
    }

のコードを考えていきます。

こちらは「1秒経ったら、ブレードをオフにする」処理ですので、「ブレードが表示されている」時の処理になります。

「 時間計測を0に戻す」処理は、ブレードが表示されていない時の処理に移動したので、ここでは記述する必要がなくなっていますので、それを省くと、「ブレードが表示されている」時の処理は、

    if (seconds >= 1.0f)  //1秒経ったら
    {
        //ブレードをオフにする
        bbm.GetComponent<BladeOnOff>().BladeOff();
        isBladeOn = false;
    }else{  // １秒経っていない時
        // 経過時間の計算
        seconds += Time.deltaTime;
    }

となります。

以上をまとめると、Update()のコードは次のようになるでしょう。

bool isBladeOn = false;

void Update(){
    if(isBladeOn）{ // ブレードが表示されている時の処理
        if (seconds >= 1.0f) //1秒経ったら
        {
            //ブレードをオフにする
            bbm.GetComponent<BladeOnOff>().BladeOff();
            isBladeOn = false;
        }else{  // １秒経っていない時は
            // 経過時間を計算する。
            seconds += Time.deltaTime;
        }
    }else{ // ブレードが表示されていない時の処理
        //　Vキーが押されたら
        if (Input.GetKeyDown(KeyCode.V))
        {
            //ブレードをオンにする
            bbm.GetComponent<BladeOnOff>().BladeOn();

            //時間カウント開始
            isBladeOn = true;
            seconds = 0;
        }
    }
}

このように、Unityでは、Update()関数内の処理は状態ごとに場合分けして考える必要があります。

Comments

[senkiru]

私は社会人なのですが、趣味で（小学生の頃からの夢で）たまにunityやC#、blenderを構って3Dゲームを作っています。教えていただいて本当にありがとうございました。

Answer 3

どこがおかしいのでしょうか。。。

「Vが押されたら」のif文の中で「1秒経ったら」の判定をしていることです。
これではVが押されてなおかつ1秒経っていないとブレードをオフにする処理が実行されません。

Comments

[senkiru]

アドバイスありがとうございます・
質問文のスクリプトを訂正いたしました。
「1秒経ったら」のif分を外に出しました。

残念ながら、
やはり私の望むようには挙動してくれません。。。

[YAmaGNZ]

時間足すのもVが押されたらですけど・・・

[senkiru]

YAmaGNZさん
コメントありがとうございます。

不慣れではございますが解説させていただきます。

Vを押す
↓
キャラクターが剣を振るモーション開始
＆ブレード光らせたい
＆ここで時間計測開始
↓
1秒で剣を降る動作を終える
＆ここで1秒経ってるので剣が光るのを終える

みたいなイメージなのですが、
Vを押してから時間計測を始めるというのは
不適切でしょうか。。。

何分にも初心者でしてスミマセン。。。

[YAmaGNZ]

ログを出力するとかで実際にどのような動作をしているのか確認することをお勧めします。
Vが押された瞬間に加算されますが、それ以外は加算されません。

[senkiru]

ありがとうございます。
検討させていただきます。