Проблема с получением игрового объекта из пула объектов в Unity

У меня возникли проблемы с преобразованием сценария пула объектов из UnityScript в C#, и здесь мне очень помогли. Теперь у меня проблема с попыткой получить игровой объект из пула. У меня есть три сценария, взаимодействующих друг с другом, поэтому я не совсем уверен, где что-то не так. Вот два скрипта для пула объектов, которые, как мне кажется, полностью исправлены и не выдают никаких ошибок:

public class EasyObjectPool : MonoBehaviour {
    [System.Serializable]
    public class PoolInfo{
        [SerializeField]
        public string poolName;
        public GameObject prefab;
        public int poolSize;
        public bool canGrowPoolSize = true;

}

[System.Serializable]
public class Pool{

    public List<PoolObject> list = new List<PoolObject>();
    public bool  canGrowPoolSize;

    public void  Add (PoolObject poolObject){
        list.Add(poolObject);
    }

    public int Count (){
        return list.Count;
    }


    public PoolObject ObjectAt ( int index  ){

        PoolObject result = null;
        if(index < list.Count) {
            result = list[index];
        }

        return result;

    }
}
static public EasyObjectPool instance ;

[SerializeField]
PoolInfo[] poolInfo = null;

private Dictionary<string, Pool> poolDictionary  = new Dictionary<string, Pool>();


void Start () {

    instance = this;

    CheckForDuplicatePoolNames();

    CreatePools();

}

private void CheckForDuplicatePoolNames() {

    for (int index = 0; index < poolInfo.Length; index++) {
        string poolName= poolInfo[index].poolName;
        if(poolName.Length == 0) {
            Debug.LogError(string.Format("Pool {0} does not have a name!",index));
        }
        for (int internalIndex = index + 1; internalIndex < poolInfo.Length; internalIndex++) {
            if(poolName == poolInfo[internalIndex].poolName) {
                Debug.LogError(string.Format("Pool {0} & {1} have the same name. Assign different names.", index, internalIndex));
            }
        }
    }
}

private void CreatePools() {

    foreach(PoolInfo currentPoolInfo in poolInfo){

        Pool pool = new Pool();
        pool.canGrowPoolSize = currentPoolInfo.canGrowPoolSize;

        for(int index = 0; index < currentPoolInfo.poolSize; index++) {
            //instantiate
            GameObject go = Instantiate(currentPoolInfo.prefab) as GameObject;
            PoolObject poolObject = go.GetComponent<PoolObject>();
            if(poolObject == null) {
                Debug.LogError("Prefab must have PoolObject script attached!: " + currentPoolInfo.poolName);
            } else {
                //set state
                poolObject.ReturnToPool();
                //add to pool
                pool.Add(poolObject);
            }
        }

        Debug.Log("Adding pool for: " + currentPoolInfo.poolName);
        poolDictionary[currentPoolInfo.poolName] = pool;

    }
}

public PoolObject GetObjectFromPool ( string poolName  ){
    PoolObject poolObject = null;

    if(poolDictionary.ContainsKey(poolName)) {
        Pool pool = poolDictionary[poolName];

        //get the available object
        for (int index = 0; index < pool.Count(); index++) {
            PoolObject currentObject = pool.ObjectAt(index);

            if(currentObject.AvailableForReuse()) {
                //found an available object in pool
                poolObject = currentObject;
                break;
            }
        }


        if(poolObject == null) {
            if(pool.canGrowPoolSize) {
                Debug.Log("Increasing pool size by 1: " + poolName);

                foreach (PoolInfo currentPoolInfo in poolInfo) {    

                    if(poolName == currentPoolInfo.poolName) {

                        GameObject go = Instantiate(currentPoolInfo.prefab) as GameObject;
                        poolObject = go.GetComponent<PoolObject>();
                        //set state
                        poolObject.ReturnToPool();
                        //add to pool
                        pool.Add(poolObject);

                        break;

                    }
                }
            } else {
                Debug.LogWarning("No object available in pool. Consider setting canGrowPoolSize to true.: " + poolName);
            }
        }

    } else {
        Debug.LogError("Invalid pool name specified: " + poolName);
    }

    return poolObject;
}

}

А также:

public class PoolObject : MonoBehaviour {

[HideInInspector]
public bool availableForReuse = true;


void Activate () {

    availableForReuse = false;
    gameObject.SetActive(true);

}


public void ReturnToPool () {

    gameObject.SetActive(false);
    availableForReuse = true;


}

public bool AvailableForReuse () {
    //true when isAvailableForReuse & inactive in hierarchy

    return availableForReuse && (gameObject.activeInHierarchy == false);



}
}

В оригинальном UnityScript сказано, что объект из пула извлекается с помощью этого оператора:

var poolObject : PoolObject = EasyObjectPool.instance.GetObjectFromPool(poolName);

Вот как я пытался сделать это в своем сценарии стрельбы, пытаясь выстрелить сборной пулей из пула:

public class ShootScript : MonoBehaviour {

public PoolObject poolObject;

private Transform myTransform;

private Transform cameraTransform;

private Vector3 launchPosition = new Vector3();

public float fireRate = 0.2f;

public float nextFire = 0;

// Use this for initialization
void Start () {

    myTransform = transform;

    cameraTransform = myTransform.FindChild("BulletSpawn");

}


void Update () {

    poolObject = EasyObjectPool.instance.GetObjectFromPool<poolName>();

    if(Input.GetButton("Fire1") && Time.time > nextFire){

        nextFire = Time.time + fireRate;

        launchPosition = cameraTransform.TransformPoint(0, 0, 0.2f);

        poolObject.Activate();

        poolObject.transform.position = launchPosition;
        poolObject.transform.rotation = Quaternion.Euler(cameraTransform.eulerAngles.x + 90, myTransform.eulerAngles.y, 0);

    }

}
}

Мой сценарий съемки дает мне две ошибки:

<сильный>1. Не удалось найти имя типа или пространства имен «poolName». Вам не хватает директивы using или ссылки на сборку?

Для линии:

poolObject = EasyObjectPool.instance.GetObjectFromPool<poolName>();

<сильный>2. 'PoolObject.Activate()' недоступен из-за его уровня защиты

Для линии:

poolObject.Activate();

Я неправильно перевел UnityScript или упустил что-то еще? Любой вклад приветствуется


person user3776884    schedule 27.06.2014    source источник


Ответы (2)


То, что вы пишете внутри ‹>, должно быть именем класса, например PoolObject, если функция является универсальной, а это не так. Итак, чтобы решить эту проблему, вам просто нужно изменить

poolObject = EasyObjectPool.instance.GetObjectFromPool<poolName>();

to

string poolName = "thePoolNameYouWant";
poolObject = EasyObjectPool.instance.GetObjectFromPool(poolName);

Функция является закрытой по умолчанию, поэтому для устранения ошибки «недоступна из-за ее уровня защиты» вам необходимо сделать функцию общедоступной, изменив это

void Activate () {
    availableForReuse = false;
    gameObject.SetActive(true);
}

к этому

public void Activate () {
    availableForReuse = false;
    gameObject.SetActive(true);
}
person Imapler    schedule 27.06.2014
comment
Спасибо за вашу помощь. Изменение строки poolObject дало мне еще несколько ошибок: - person user3776884; 27.06.2014
comment
Имя `poolName' не существует в текущем контексте - person user3776884; 27.06.2014
comment
Лучшее совпадение перегруженного метода для `EasyObjectPool.GetObjectFromPool(string)' имеет несколько недопустимых аргументов. - person user3776884; 27.06.2014
comment
Аргумент выражения #1' cannot convert object для ввода строки - person user3776884; 27.06.2014
comment
poolName должно быть строковой переменной с нужным вам именем пула. я обновил ответ, я считаю, что все три ошибки вызваны этим. Все эти ошибки очень распространены среди новичков, и вы должны найти много информации, просто погуглив сообщение об ошибке. - person Imapler; 29.06.2014

Я думаю, что ваш пул объектов неправильно запрограммирован. Когда у вас есть такой метод, как CheckForDuplicatePoolNames(), спросите себя, нужна ли вам другая коллекция, например словарь.

Будь проще. Недавно я реализовал один и протестировал. Это работает довольно хорошо.

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

[System.Serializable]
public class PoolObject{
    public GameObject prefab;
    public int startNum;
    public int allocateNum;
    public Transform anchor;
}

public class ObjectPool<T> : MonoBehaviour where T:Component{
    public PoolObject[] poolObjects;
    public bool ______________;
    public Dictionary<string,Queue<T>> pool = new Dictionary<string, Queue<T>> ();
    public Dictionary<string, List<T>> spawned = new Dictionary<string, List<T>> ();
    public Transform anchor;


    public void Init(){
        InitPool ();
        StartAllocate ();
    }


    public T Spawn(string type){
        if (pool [type].Count == 0) {
            int i = FindPrefabPosition(type);
            if(poolObjects[i].allocateNum == 0)
                return null;
            Allocate(poolObjects[i], poolObjects[i].allocateNum);
        }
        T t = pool [type].Dequeue ();
        spawned[type].Add (t);
        t.gameObject.SetActive (true);
        return t;
    }


    public void Recycle(T t){
        if (spawned [t.name].Remove (t)) {
            pool[t.name].Enqueue(t);
            t.gameObject.SetActive(false);
        }
    }


    private void Allocate(PoolObject po, int number){
        for (int i=0; i<number; i++) {
            GameObject go = Instantiate (po.prefab) as GameObject;
            go.name = po.prefab.name;
            go.transform.parent = po.anchor != null? po.anchor : anchor;
            T t = go.GetComponent<T>();
            pool[t.name].Enqueue(t);
            t.gameObject.SetActive(false);  
        }
    }

    private int FindPrefabPosition(string type){
        int position = 0;
        while (poolObjects[position].prefab.name != type) {
            position++;     
        }
        return position;
    }

    void InitPool(){
        if(anchor == null)
            anchor = new GameObject("AnchorPool").transform;
        for (int i =0; i<poolObjects.Length; i++) {
            T t = poolObjects[i].prefab.GetComponent<T>();  
            t.name = poolObjects[i].prefab.name;
            pool[t.name] = new Queue<T>();
            spawned[t.name] = new List<T>();
        }
    }

    void StartAllocate(){
        for (int i=0; i<poolObjects.Length; i++) {
                Allocate(poolObjects[i], poolObjects[i].startNum );
        }
    }
}

Пример:

public class CoinsPool : ObjectPool<CoinScript>{}

В инспекторе единства настройте префаб монеты, startNum и allocateNum.

Где-то:

void Awake(){
  coinsPool = GetComponent<CoinsPool>();
  coinsPool.Init();
}

CoinScript cs = coinsPool.Spawn("Coin"); //Coin is the name of the coin prefab.

Позже

coinsPool.Recycle(cs);
person Roger Garzon Nieto    schedule 05.03.2015