unityでGameobject型のリストのcountが0になる。

Question

unityでアイテム(下の画像で選択されているオブジェクト)がバー（アイテムの真下にある棒状のオブジェクト）に当たったら現在のボールの数を倍に増やすようなコードを書いています。

BallManager.csというファイルで、まずヒエラルキーに最初からあるFirstBallというGamaObjectを取得してGameObject型のList(ballと定義)に追加しています。
この時点では当たり前ですがball.Count=1です。しかしこれをItemManager.csという別のファイルからball.Countを含む関数(DoubleBall)を呼び出した時なぜかball.Count=0とコンソールに表示されてしまいます。
どのようにしたらballの要素数をItemManager.csから参照できるのでしょうか。
以下BallManagerとItemManagerのファイルで、BallManagerはBallのPrefabとFirstBallに、ItemManagerはヒエラルキーのItemにそれぞれアタッチしています。

作業環境はUnity 2019.4.10f
VS 2019
Win 10
です。

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

public class BallManager : MonoBehaviour
{  

    // Start is called before the first frame update
    //ボールを格納するリスト

   public List<GameObject> ball = new List<GameObject>();
    public GameObject instanceBall;
   

    List<Rigidbody> rb = new List<Rigidbody>();

    Vector3 ballInitSpeed;

    void Start()
    {
        //最初のボール
        GameObject firstBall = GameObject.Find("FirstBall");
       ball.Add(firstBall);
        Debug.Log("最初の"+ball.Count);
       
   //最初のボールの初速
        ballInitSpeed = new Vector3(100, 300, 0);

        rb.Insert(0, firstBall.GetComponent<Rigidbody>());
        rb[0].AddForce(ballInitSpeed);

       instanceBall=  Resources.Load("Ball") as GameObject;

      //  ballCount = 1;


    }

    // Update is called once per frame
    void Update()
    {


        // Debug.Log("ballCount" + ball.Count);


    }

    
    public void DoubleBall()
    {
       Debug.Log("呼び出された後のカウント"+ball.Count);
        
        //増える分のボールをballに格納
       foreach (var item in ball)
        {
          //  Debug.Log("呼び出された");

            //ボールを現在存在するボールのそれぞれから分裂させる
            Instantiate(instanceBall,item.transform.position,Quaternion.identity);

            //ballリストのインデックスを倍にする
            ball.Add(instanceBall);
      

        }

        //分裂したボールに初速度を与える
        for (int i = ball.Count/2; i < ball.Count-1; i++)
        {
            rb.Insert(i, instanceBall.GetComponent<Rigidbody>());

            ballInitSpeed = new Vector3(Random.Range(-1000, 1000), Random.Range(-1000, 1000), 0);
            rb[i].AddForce(ballInitSpeed);
        }
            

    }    

    
}

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

public class ItemManager : MonoBehaviour
{
    public GameObject item;
  //  public GameObject[] balls;   

    BallManager ballManager;
    Vector3 itemPos;

    public GameObject ball;
   
    
    // Start is called before the first frame update
    void Start()
    {
        itemPos = new Vector3(-0.15f,2.2f,0);

       // Instantiate(item,itemPos,Quaternion.identity);     
       
    }

    // Update is called once per frame
    void Update()
    {

        
}

    //バーにアイテムがぶつかったら
   public void OnCollisionEnter(Collision other)
    {
        Debug.Log("aaaa");
        if (other.gameObject.tag=="Bar")
        {
            ballManager = ball.GetComponent<BallManager>();
            //ボールを増やす
            ballManager.DoubleBall();


           Destroy(gameObject);

        

        }
    }
    
}

また当方unityもc#も初心者であるため、もっといい,ボールを2倍にする実装方法などあればご教授いただきたいです。
よろしくお願いします。

Comments

[退会済みユーザー]

BallManagerを複製されるballにつけるのですか？
Countが0になる原因はこれっぽい。
ItemMamagerでCollision判定がするんですか？ItemManagerがどういう意味で作ったスクリプトなのかは分かりませんが…
いろいろおかしい気がします

[lain62]

ItemManagerはアイテムの生成（まだ実装していない）とアイテムの破棄を担当しています。衝突判定のできるスクリプトがアタッチされていないオブジェクトの衝突検知の仕方が分からなかったのでアイテムに直接スクリプトをアタッチしています。
BallManagerはprefabにつけるとよくないのですか。

[退会済みユーザー]

シーンに元から存在しているFirstBallそれにアタッチされているBallManagerスクリプトと
prefabでInstantiateされたballにアタッチされるBallManagerスクリプトは別のインスタンスっていう部分は大丈夫ですか？

[lain62]

その点を忘れていました...。参照型の話ですね。
ということは空のゲームオブジェクトを作ってそこでボールの管理をした方がいいのでしょか

[lain62]

ItemManagerのBallをFirstBallに差し替えたら無事分裂してカウントが増えました。
ただボールが倍々ではなく一つずつしか増えないですね。

[退会済みユーザー]

どの様に管理するか？は自由ですので手法論についての回答は難しいです。
実現したい事(どういった動作をさせたいか？)を書いていただければ私なりのコードは提示できるかもしれません。

[lain62]

ボールが倍々に増えるブロック崩しを作っています。
ブロック（まだ一つしか配置していませんが）を崩すと1/2の確率でアイテムが崩れたブロックから生成し、重力に沿って落ちていきます。アイテムが左右キーで移動できる(実装済み)バーに触れるとアイテムは破棄され現在sceneに存在しているボールがそれぞれ二つずつに分裂し、ドラえもんのバイバインのようにボールが二倍に増えます。下に落ちすぎたボールやバーに触れずに落ちたアイテムは破棄されます。
このボールが増える動作をGameObject型のListで管理しようとしていました。最初からボールは1つ存在していないとゲームは成り立たないためFirstBallを用意してBallManagerを割り当てていました。しかしヒエラルキーのItemにアタッチしていたItemManagerのインスペクタのBallをFirstBallに差し替えたところカウントが正常に働きましたがボールが一つずつしか増えない（倍々にならない）所で詰まっています。動作している様子は以下から確認できます(https://drive.google.com/file/d/1D2DSoGLehpb5XeFh22ppEz75pc-BGNHw/view?usp=sharing)

Answer 1

方法の一つとして回答します。方法なのでこのやり方が正しい、間違っているという話ではないです。

まず、BallManagerを空のGameObject(とりあえずBallManagerという名前にしましょう)にアタッチします。ボールを複製する部分は同じような感じにしたつもりです。
ただ、追加したいballのListを別で作成しています。Listを回している最中にその同一のListに追加するっていう行為は問題があったはず、、、

public class BallManager : MonoBehaviour
{
    public List<GameObject> balls = new List<GameObject>();
    public GameObject ballPrefab;

    public void DoubleBalls()
    {
        var newBalls = new List<GameObject>();
        foreach(var ball in balls) 
        {
            var ballObj = Instantinate(ballPrefab, ball.transform.position, Quaternion.identity);
            newBalls.Add(ballObj);
        }

        foreach(var ball in newBalls) balls.Add(ball);
    }
}

次に生成用のBall(instanceBallのprefab)にBallスクリプトを作ってそれをアタッチします。
ちょっと強引な気もしますが、ヒエラルキーからBallManagerというオブジェクトを見つけて、その中のBallManagerスクリプトのballsという管理用のListに自身から登録させます。
削除された時にBallManagerの中の自身も削除しなければいけません。
存在しない参照がリストに残ってしまいます。

public class Ball : MonoBehaviour
{
    void Start()
    {
        GameObject.Find("BallManager").GetComponent<BallManager>().balls.Add(this.gameObject);
    }

    void OnDestroy() 
    {
        GameObject.Find("BallManager").GetComponent<BallManager>().balls.Remove(this.gameObject);
    }
}

FirstBallが必要であればこのBallスクリプトをアタッチしてください。ball自体を別で生成する(例えばキーが押されたら１つ目のボールを生成する、など)ならFirstBall自体も必要ないです。

発動の条件はitemからDoubleBallsメソッドを呼ぶとして、分裂する(倍々に増える)という部分だけですがこれで出来ると思います。

Comments

[lain62]

ありがとうございます！できました！
ただBallManagerとprefabにくっついてるBall両方でAddしちゃってるらしくボールが3倍ずつ増えてしまったので,Ballスクリプトのstart内をコメントアウトしたら倍々に増えるようになりました。
実は以前のスクリプトでは実行するとunityが停止していたのですが、これはforeach内でListの要素を増やしているために起こっていたんですね。

Answer 2

Instantiate(instanceBall,item.transform.position,Quaternion.identity);
//ballリストのインデックスを倍にする
ball.Add(instanceBall);

この部分を

var createBall = Instantiate();
ball.Add(createBall);

にしてみてください．これでListの数が増えると思います．

上のコードではリストに追加していたものは生成したオブジェクトではなくPrefabのためうまく追加できないのではないかと考えています(Objectの内部IDが同じだから．．？,詳しいことは分かりませんが

Comments

[lain62]

ありがとうございます！
色んな所がおかしかったみたいです。
解決できました。