Unity for文の実行結果が目的と異なることについて

Question

前提・実現したいこと

簡潔に言うと、ランダムに選んだ数字によって格納するenemyを決定し、指定の座標でインスタンス化した後、Translateで移動させたい。

①test1flagがtrueのときにEnemyselectnum()で0から1の乱数を一度だけ入手(今後数字の範囲は増やす予定)→数字をselect_enemynumに代入し、void FixedUpdate()内で使えるようにする。falseのときは実行する処理は無しなので、使わない数字の-1を代入した。

②select_enemynumが0のとき→ 配列型のgetenemy[0]の敵をセット。1のとき→getenemy[0]の敵をセットする。格納するenemy1とenemy2はImageのプレハブで、Inspectorに格納した。

③getenemy[i]をenemyobjのGameObjectとして指定の座標でインスタンス化し、x軸負の方向へ移動させ続けたい。

④_②と③をfor文で処理。

発生している問題・エラーメッセージ

エラーなし。①select_enemynumの結果によらず、enemy1しか表示されない。② rect_setenemy.Translate(-0.1f, 0, 0);の処理が実行されない。

該当のソースコード

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

public class Enemy : MonoBehaviour
{
    public GameObject enemy1;
    public GameObject enemy2;
    public GameObject canvas;

    GameObject[] getenemy = new GameObject[2];

    bool test1flag;

    RectTransform enemyrect;

    float enemyposx;
    float enemyposy;
    float enemyposz;

    int select_enemynum;

    // Start is called before the first frame update
    void Start()
    {
        test1flag = true;

        getenemy[0] = enemy1;
        getenemy[1] = enemy2; 
    }

    // Update is called once per frame
    void FixedUpdate()
    {
        Enemyselectnum();

            for (int i = 0; i <= 1; i++)
            {
                if (select_enemynum == i)
                {
                    enemyrect = getenemy[i].GetComponent<RectTransform>();
                    Debug.Log(getenemy[i]);//enemy1しか表示されない

                    enemyposx = enemyrect.localPosition.x;
                    enemyposy = enemyrect.localPosition.y;
                    enemyposz = enemyrect.localPosition.z;

                    enemyposx = 78;
                    enemyposy = 108;
                    enemyposx = 0;

                    Vector3 enemypos = new Vector3(enemyposx, enemyposy, enemyposz);

                    GameObject enemyobj = Instantiate(getenemy[i], enemypos, Quaternion.identity);
                    enemyobj.transform.SetParent(canvas.transform, false);

                    //生成したPrefabを左へ移動させる
                    RectTransform rect_setenemy;
                    rect_setenemy = enemyobj.GetComponent<RectTransform>();
                    rect_setenemy.Translate(-0.1f, 0, 0);
                }
            }
    }

    private int Enemyselectnum()
    {
        if (test1flag == true)
        {
            int select_enemynum = Random.Range(0, 2);
            test1flag = false;
            Debug.Log(select_enemynum);
            return select_enemynum;
        }
        else
        {
            return select_enemynum = -1;
        }      
    }
}

試したこと

デバッグの結果から、select_enemynum == 0のときにenemy1が格納され、select_enemynum == 1のときもenemy1が格納されてしまうことは確認済み。発生している問題①はfor文の認識を何か誤っているのかもしれませんが、エラーが出ていない以上、それ以上の原因追及ができませんでした。

補足情報（FW/ツールのバージョンなど）

Unityバージョン、2020.3.18f1

Comments

[退会済みユーザー]

変数 select_enemynum のスコープの問題では？

[退会済みユーザー]

質問者さん、回答したのでフィードバックを返してください。役に立った/立たなかったぐらいはすぐに返せるのでは？ 役に立たなかったならどこがダメかを書くとより期待に近い回答が出てくるかも。

[退会済みユーザー]

頂いたアドバイスのようにスコープを意識し、int select_enemynum;の記述を削除して問題①と②が解決するか試しましたが、結果は両方ダメでした。select_enemynumはEnemyselectnum()だけに記述している状況です。また、for文の中のif分中のselect_enemynumをEnemyselectnum()に変えました。

また、②について、Translateの-0.1fを例えば-1.0に変えてみると、選ばれたenemyのローカル座標が変化している(-1.0のほうが負の位置にいる)ため、処理は実行されるが、「継続的に移動させる」処理ができていないようです。なので、やはりfor文の使い方に誤りがあるのでは?と推測した次第です

お返事遅くなり申し訳ありません。本業ではなく、現在早急な返信が難しい状況になってしまっており、ご迷惑をお掛けします。

[退会済みユーザー]

回答に対するコメントは回答欄下のコメント欄にお願いします。

>int select_enemynum;の記述を削除して問題①と②が解決するか試しましたが・・・

public class Enemy : MonoBehaviour 下の int select_enemynum; を削除したのですか？　それではコンパイルも通らないはずですけど。

int select_enemynum = Random.Range(0, 2); の int を削除したらどうなりますか？

Answer 1

変数 select_enemynum のスコープの問題では？

public class Enemy : MonoBehaviour 下の int select_enemynum; はクラス Enemy のフィールド変数です。

一方、Enemyselectnum メソッド内の int select_enemynum = Random.Range(0, 2); で定義されている変数 select_enemynum はメソッド内のローカル変数です。

それらは別物です。だからでは？

Comments

[退会済みユーザー]

>int select_enemynum = Random.Range(0, 2); の int を削除したらどうなりますか？

デバッグより、enemy1とenemy2はgetenemy[i]に格納されるので、GameObject enemyobj = Instantiate(getenemy[i], enemypos, Quaternion.identity);
enemyobj.transform.SetParent(canvas.transform, false);
の処理までは意図通り実行できます。ただし、
//生成したPrefabを左へ移動させる
RectTransform rect_setenemy;
rect_setenemy = enemyobj.GetComponent<RectTransform>();
rect_setenemy.Translate(-0.1f, 0, 0);
のところで、UIのImageが表示されるだけで移動し続けません。試しに-0.1fを-1に変更すると、生成される座標が変化するため、処理自体は行われるが、Translate(-0.1f, 0, 0)で連続的に移動させる、という意図を実現できずにいるのが現状です。

[退会済みユーザー]

C# のコードとして明らかに間違っていると思われるところだけ指摘しましたが、それを直してもダメということですと、実際に動かしてデバッグできない自分にはお手上げです。お役に立てずすみませんが、他の方の回答をお待ちください。

Answer 2

こちらのスクリプトで解決できたので記します。格納までの処理をfor文中で行い、Translateの処理はfor文の外で実行。やはりfor文の使い方を一部誤っていたようです。

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

public class Enemy_ : MonoBehaviour
{
    public GameObject enemy1;
    public GameObject enemy2;
    public GameObject canvas;

    GameObject[] getenemy = new GameObject[2];
    //test
    GameObject enemyobj;

    bool test1flag;

    RectTransform enemyrect;

    float enemyposx;
    float enemyposy;
    float enemyposz;

    int select_enemynum;

    // Start is called before the first frame update
    void Start()
    {
        test1flag = true;

        getenemy[0] = enemy1;
        getenemy[1] = enemy2;
    }

    // Update is called once per frame
    void FixedUpdate()
    {
        Enemyselectnum();

        for (int i = 0; i <= 1; i++)
        {
            if (select_enemynum == i)
            {
                enemyrect = getenemy[i].GetComponent<RectTransform>();
                Debug.Log(getenemy[i]);//enemy1しか表示されない

                enemyposx = enemyrect.localPosition.x;
                enemyposy = enemyrect.localPosition.y;
                enemyposz = enemyrect.localPosition.z;

                enemyposx = 78;
                enemyposy = 108;
                enemyposx = 0;

                Vector3 enemypos = new Vector3(enemyposx, enemyposy, enemyposz);


                enemyobj = Instantiate(getenemy[i], enemypos, Quaternion.identity);
                //GameObject enemyobj = Instantiate(getenemy[i], enemypos, Quaternion.identity);
                enemyobj.transform.SetParent(canvas.transform, false);               
            }
        }

        //生成したPrefabを左へ移動させる
        RectTransform rect_setenemy;
        rect_setenemy = enemyobj.GetComponent<RectTransform>();
        rect_setenemy.Translate(-0.01f, 0, 0);
    }

    private int Enemyselectnum()
    {
        if (test1flag == true)
        {
            select_enemynum = Random.Range(0, 2);
            test1flag = false;
            Debug.Log(select_enemynum);
            return select_enemynum;
        }
        else
        {
            return select_enemynum = -1;
        }
    }
}