UintyVR オブジェクト（銃）を傾けると音が多重になる

銃を傾けると弾を補充するコードを記述したのですが、
角度に入ると音が多重になり続けてしまいます。
角度に入ったら一回だけ鳴らしたいのですが、
対処法が分かる方がいらっしゃいましたら教えて頂きたいですm(_ _)m

C#
1    void Update()
2    {
3        if(ovrGrabbable.isGrabbed && OVRInput.GetDown(shootingButton, ovrGrabbable.grabbedBy.GetController()) && maxNumberOfBullet > 0)
4        {
5            //Shoot !!!
6            simpleShoot.TriggerShoot();
7            maxNumberOfBullet--;
8            bulletText.text = maxNumberOfBullet.ToString();
9        }
10        if(Vector3.Angle(transform.up,Vector3.up) > 100)　
11        {
12            maxNumberOfBullet = 100;
13            bulletText.text = maxNumberOfBullet.ToString();
14            audioSource.PlayOneShot(GunSE);
15        }
16    }
17}

行動規範の内容に同意します

回答1件

ベストアンサー

Update内でこれを回しているので角度が指定された以上になれば常に実行されます。
なので一定角度以上にすると常時リロードされて音が再生されています。

cs
1        if(Vector3.Angle(transform.up,Vector3.up) > 100)　
2        {
3            maxNumberOfBullet = 100;
4            bulletText.text = maxNumberOfBullet.ToString();
5            audioSource.PlayOneShot(GunSE);
6        }

解決方法としては、「if文の中身を見直して、常にここが走らないようにする」ことで改善できます。

どんな条件にしたいかによるので、そこを考えて詰めてみるのはどうでしょうか。
例えば、1回角度をかえてリロードしたらbool型などでフラグを立てて、銃をもとの角度に戻すまではリロードできないようにするとかですね。

cs
1        bool isReloaded;
2        if(Vector3.Angle(transform.up,Vector3.up) > 100 && !isReloaded)　
3        {
4            isReloaded = true;
5            maxNumberOfBullet = 100;
6            bulletText.text = maxNumberOfBullet.ToString();
7            audioSource.PlayOneShot(GunSE);
8        }else if(Vector3.Angle(transform.up,Vector3.up) < 100 && isReloaded){
9            isReloaded = false;
10        }