ボタンを押している間オブジェクトがY軸に動くスクリプトがうまく動きません。

Question

UnityでC#で記述していますズブの初心者です。
ボタンを押している間オブジェクトがY軸に動くスクリプトを作成中どうしてもboolで指定した「PushUpKey」が毎フレーム「false」にされてしまいます。
ボタンを押した瞬間「true」にはなるのですが次のフレームになった瞬間「false」にされてしまいボタンを押しても動いてくれません。
初期値を「true」にした場合は今度は動きっぱなしになってしまい困っています。
void Updateには「PushUpKey」を変化させるものを入れていないのになんでこんなことになるのかわかりません。
もしわかる方いましたらご教授いただければと思います。よろしくお願いいたします。

using System.Collections;
using System.Collections.Generic;
using UnityEngine;
using UnityEngine.UI;

public class Player : MonoBehaviour {
    float ValueY;
    bool PushUpKey;
    Spaceship Spaceship;
    IEnumerator Start(){

        Spaceship = GetComponent<Spaceship>();
        while (true){
            Spaceship.Shot(transform);
            GetComponent<AudioSource>().Play();
            yield return new WaitForSeconds(Spaceship.ShotDelay);
        }
    }
    void Move(Vector2 direction) {
        Vector2 min = Camera.main.ViewportToWorldPoint(new Vector2(0, 0));
        Vector2 max = Camera.main.ViewportToWorldPoint(new Vector2(1, 1));
        Vector2 pos = transform.position;
        pos += direction * Spaceship.Speed * Time.deltaTime;
        pos.x = Mathf.Clamp(pos.x, min.x, max.x);
        pos.y = Mathf.Clamp(pos.y, min.y, max.y);
        transform.position = pos;
    }
    void Update() {   
        if (PushUpKey == true){
            ValueY = 1;
            Debug.Log("1111");//原因特定用
        }
        else if (PushUpKey == false) {
            ValueY = 0;
            Debug.Log("0000");//原因特定用
        }
        Vector2 direction = new Vector2(0, ValueY).normalized;
        Move(direction);
    }
    public void UpKeyDown()//ボタンを押している間
    {
        PushUpKey = true;
 　　　　　Debug.Log(PushUpKey);//原因特定用
    }
    public void UpKeyUp()//ボタンを離したとき
    {
        PushUpKey = false; 
　　　　　　 Debug.Log(PushUpKey);//原因特定用
    }
}

UpKeyDownメソッドとUpKeyUpメソッドはUNITY上でボタン押下時に実行するよう指定しています。

Comments

[退会済みユーザー]

UpKeyDownメソッド、UpKeyDownメソッドを呼び出しているコードも記載したほうがよいかと。

[退会済みユーザー]

> UpKeyDownメソッドとUpKeyUpメソッドはUNITY上でボタン押下時に実行するよう指定しています。
ここがうまくいっていない可能性もありそうですが。

[YANAGI584]

上記二つのメソッドにDebug.Log(PushUpKey)を加えてコンソールで参照しましたが値は変わっているようでした。ただ瞬間的に「false」にされてしまうようで…。

[退会済みユーザー]

値を変えているのは、上記二つのメソッドを呼び出している部分ですよね？なので、そこがおかしいので変わってしまうのでは？ということです。

[YANAGI584]

UNITY側のイベントトリガーというコンポーネントでUpKeyDownメソッドはpointerdown(ボタン押下時に呼び出す)、UpKeyUpメソッドはpointerup(ボタンを離す時に呼び出す)の登録しておりチェックしてみましたがおかしいような感じはしませんでした。「true」変化後、瞬間的に変わってるようなのでvoid Update内が怪しいと踏んでるのですが原因わからずなんです…。

[退会済みユーザー]

なるほど。原因特定用に用意されているDebug.Log("")を、2つのメソッドのほうに用意すると動きがもうすこしわかりやすいと思いますよ。

[YANAGI584]

アドバイスありがとうございます。２つのメソッドにDebug.Log("")を追記したところコンソールに表示されましたので、やはり一度は値が変わっているようです。

[sakura_hana]

１．PushUpKeyを他のクラスから操作している可能性はありませんか？　２．このスクリプトが複数のオブジェクトに付いていたりしませんか？

[退会済みユーザー]

私のコードではうまくいきましたので、こちらのコードでも試してみますね。
あと、private変数でも、その前の行に[SerializeField]と書いておくと、インスペクターで確認できるので楽しいですよ。

[退会済みユーザー]

貼っていただいたコードってビルドエラーしますよね。もしかしたらですが、ビルドできていないこときづかず、前に成功したビルドで走っており、なにかしら行った修正が効いていないとか？コードを見る限り、Debug.Log("1111");(もしくはDebug.Log("0000");)と、 Debug.Log(PushUpKey);が同数連続で出力されるところ、されていないので、そんな気がします。

[退会済みユーザー]

ちなみに、エラーとなりえる箇所はここ → IEnumerator Start(){ // voidでは？

[YANAGI584]

→sakura_hanaさま　PushUpKeyに関しては他のスクリプトやクラスに記述いていないのでアクセスはしていないかと思います。またチェックしましたがこのスクリプトが他のオブジェクトにはアタッチされてはいませんでした。→ ppnさま　アドバイスありがとうございます。便利ですねこれ。インスペクタ上で確認しましたがPushUpKeyのチェックボックスはボタンの押下と正常に連動していましたので動いているようです。

[退会済みユーザー]

ごめんなさい、上のやつ嘘書きました。ビルドできますね。ただ、Debug.Logの出力が変なのは、解決の糸口になるのじゃないかと気になります。

[退会済みユーザー]

ということは、PushUpKeyの値は特に問題なかったということかな…？

[YANAGI584]

PushUpKeyが正常に動いてるとなるとvoidUpdate内のIF文がオカシイのかもしれません。文法的に間違っているのでしょうか？コンソールで帰ってくるのは”0000”ばかりなのです。

[退会済みユーザー]

gameobjectの名前の一緒に出力することで、本当にplayerから出力されたものなのか確認してみてもいいかもしれません（全然見当違いの調べ方だったらごめんなさい

[YANAGI584]

今のところDebug.Logを打ち込んでいるのはこのスクリプトのみでして、他のスクリプトやオブジェクトがログを出力しているということはなさそうです。

[sakura_hana]

Unity2018ですが、このコードで正常に動作しました。if文も間違いではないです。よりスマートに書くなら→　if (PushUpKey) {〜} else {〜}　　　既にPushUpKeyが正常に動作しているということは本格的に原因不明ですが、Debug.Log("0000");をコメントアウトしてみる・Unity再起動・VisualStudio再起動・スクリプト作り直し等するといいかもしれないです。

[YANAGI584]

検証ありがとうございました。なるほど動きましたか。UnityやVisualStudioを再起動しましたがオブジェクトが動いてくれませんでしたので、また作り直してみるようにします。ありがとうございました。

[Bongo]

もうプロジェクトを作り直してしまったのでしたらかまいませんが、もしまだ前のプロジェクトを削除していなければご確認いただきたいのですが、「Debug.Log("1111");」を「Debug.Log("1111 " + this.GetInstanceID());」に、「Debug.Log("0000");」を「Debug.Log("0000 " + this.GetInstanceID());」に、UpKeyDownとUpKeyUpの「Debug.Log(PushUpKey);」を「Debug.Log(PushUpKey + " " + this.GetInstanceID());」に変えてみると、コンソールの出力はどうなるでしょうか?出力結果の末尾の数字はすべて一致するでしょうか。

[YANAGI584]

→Bongoさま　返信遅れて申し訳ありません。スクリプトは作り直してしまいましたが前の状況を復旧させてご指摘いただいたスクリプトに変更したところ”0000”と”PushUpKey”のIDは異なっていました。これは違うスクリプトの影響を受けているということなのでしょうか？

Answer 1

インスタンスIDが違うということは、どうやらBUTTON_Uで操作しているPlayerと、シーン上でUpdateを実行しているPlayerが異なっている様子ですね。

「BUTTON_Uで操作しているPlayer」が何者なのかということですが、疑わしいのは「プロジェクトビューのPrefabフォルダ内のPlayer」ですかね?
誤ってシーン上のプレイヤーオブジェクトではなく、プレハブをボタンに参照させてしまった...といった心当たりはないでしょうか。

制作時点でシーン上にプレイヤープレハブのインスタンスを配置している場合は、そのシーン上のプレイヤーをボタンにセットすればいいでしょう。
もし実行時にInstantiateでプレイヤーを生成している場合はインスペクタを使ってセットするわけにいきませんので、ちょっと面倒になってしまいますが、下記のような手順でスクリプト上からイベントリスナーを追加することになると思います。

using UnityEngine;
using UnityEngine.EventSystems;
using UnityEngine.UI;

public class PlayerSpawner : MonoBehaviour
{
    public Button ButtonU; // 上ボタン
    public GameObject PlayerPrefab; // プレイヤーのプレハブ

    private void Start()
    {
        var playerPosition = Vector3.zero;
        var playerRotation = Quaternion.identity;

        // もし実行時にInstantiateでプレイヤーオブジェクトを生成しているなら...
        var playerInstance = Instantiate(this.PlayerPrefab, playerPosition, playerRotation);

        // シーン上に生成されたプレイヤーからPlayerスクリプトコンポーネントを取得し...
        var playerScript = playerInstance.GetComponent<Player>();

        // PointerDown用のエントリーを作り...
        var upButtonDownEntry = new EventTrigger.Entry();
        upButtonDownEntry.eventID = EventTriggerType.PointerDown;
        upButtonDownEntry.callback.AddListener(_ => { playerScript.UpKeyDown(); });

        // PointerUp用のエントリーを作り...
        var upButtonUpEntry = new EventTrigger.Entry();
        upButtonUpEntry.eventID = EventTriggerType.PointerUp;
        upButtonUpEntry.callback.AddListener(_ => { playerScript.UpKeyUp(); });

        // ボタンのEventTriggerを取得し...
        var buttonUEventTrigger = this.ButtonU.GetComponent<EventTrigger>();

        // triggersにエントリーを追加する
        buttonUEventTrigger.triggers.Add(upButtonDownEntry);
        buttonUEventTrigger.triggers.Add(upButtonUpEntry);
    }
}

Comments

[YANAGI584]

Bongoさま
ありがとうございました。想定した動きができました！ご指摘の通りInstantiateでプレイヤーを作成していました。これで生成されるとprefabのプレイヤーとは別オブジェクト扱いになるのですね。
勉強になりました。

他の皆様もご協力いただきましてありがとうございました。