Instantiate usando foreach con múltiples GameObject no funciona

[IronArthur]

Hombre, tanto como un bug no es. Es una implementación y ya está. Seguramente para otras cosas que no sea juegos no pasará nada ni la gente se habrá percatado de que cree basura a tutiplen.

Un saludo.

Hombre si en el otro hilo Eskema se quejaba que una corutina gastaba mucha memoria, lo del foreach es un apaga y vamonos. pq depende de como lo estes haciendo ahoga el juego que no veas. En mi caso en un proyecto fue quitar los foreach y pasar de tener outofmemorys de vez en cuando a ir pasable.

Salu2

[Eskema]

Yo no he dicho nada porque me trae sin cuidado y paso 3 pueblos de "ayudar" a la gente, mis consejos estan en mi canal de youtube donde tengo un video de optimizacion y buenas practicas. El que lo quiera ver y aplicarlas pues estupendo para él, todo eso que se ganará, él que no pues que siga como va xDD
Total hoy dia nadie se preocupa de nada, siempre hay recursos de sobra o siempre esta la misma excusa, "es que es un proyecto pequeño y asi va bien" xDDD

[Anarchy]

Mira, no sabía lo del foreach. Haré limpieza y cambiaré las funciones para evitarlo.

[Anarchy]

Las corutinas estoy tratando de limitarlas después de lo que comentó eskema, pero hice bastante uso de ellas al comienzo del proyecto y no se si me vale la pena andar cambiando el código. Este juego lo estoy usando para aprender Unity y hacerme al c#, y creo que me está sirviendo de sobra.

[notbad]

Las corutinas estoy tratando de limitarlas después de lo que comentó eskema, pero hice bastante uso de ellas al comienzo del proyecto y no se si me vale la pena andar cambiando el código. Este juego lo estoy usando para aprender Unity y hacerme al c#, y creo que me está sirviendo de sobra.

las coroutinas te saldrán a cuenta quitarlas si las estás usando (disparandolas) en metodos que se ejecutan muy frecuentemente como update, late update, fixedupdate, etc... Si no, pues no vas a notar casi nada. De todas formas si está en medio del desarrollo yo no me pondría a optimizar nada, como mucho a iterar sobre el código para hacerlo más mantenible y entendible.

La gente que sabe de esto suele comentar que ... early optimizations == bullet hell , por algo será.

Un saludo.

[Anarchy]

las coroutinas te saldrán a cuenta quitarlas si las estás usando (disparandolas) en metodos que se ejecutan muy frecuentemente como update, late update, fixedupdate, etc... Si no, pues no vas a notar casi nada.

No se me ocurriría usarlas en el update ni aunque no me hubieran dicho que consumen memoria. Las uso en momentos muy puntuales. Desde luego no he tenido problemas de rendimiento ni de memoria probando el juego en móviles súper cutres de gama bajísima, así que entiendo que tampoco estoy haciendo un uso excesivo de las mismas.

[notbad]

No se me ocurriría usarlas en el update ni aunque no me hubieran dicho que consumen memoria. Las uso en momentos muy puntuales. Desde luego no he tenido problemas de rendimiento ni de memoria probando el juego en móviles súper cutres de gama bajísima, así que entiendo que tampoco estoy haciendo un uso excesivo de las mismas.

No le des más vueltas. Dejalas donde están e invierte el tema en cosas más importantes como terminar el juego. Al final cuando todo esté funcionando ya coges el profiler y miras que movidas están relamente haciendo sufrir el frame rate del juego.

Un saludo.

[chipan]

No le des más vueltas. Dejalas donde están e invierte el tema en cosas más importantes como terminar el juego. Al final cuando todo esté funcionando ya coges el profiler y miras que movidas están relamente haciendo sufrir el frame rate del juego.

Un saludo.

Exactamente, es un juego con pajaros que explotan asi que tiene que ser bueno XD.

[Anarchy]

Y gusanos gigantes que te persiguen mientras escalas plataformas! xDDD

[chipan]

Y gusanos gigantes que te persiguen mientras escalas plataformas! xDDD

¡Mejor me lo pones!

[Drumpi]

Así sin probarlo, yo como lo programaría sería así:

public GameObject Explosion; //(El prefrab con la explosión)
private GameObject[] pajaros;
private GameObject objeto; //(Este lo estoy usando como último recurso para hacer pruebas, originalmente no lo utilizaba).

    void Start () {
        pajaros = GameObject.FindGameObjectsWithTag("Pajaro");
    }

    void OnTriggerEnter2D(Collider2D other)
    {
        foreach (GameObject pajaro in pajaros)
        {
            objeto= (GameObject) Instantiate(Explosion, new Vector3(pajaro.transform.position.x, pajaro.transform.position.y, -1), pajaro.transform.rotation); //-1 simplemente para asegurarme de que la explosión aparezca por delante de los pájaros
            objeto.transform.localScale = new Vector3(2, 2, 0); //No lo necesito, pero estoy haciendo pruebas al no funcionarme. Originalmente lanzaba el Instantitate sin usar el GameObject "objeto"
            StartCoroutine(destruyePajaro(pajaro)); //La rutina que se carga los pájaros tras la explosión, esta se lanza sin problemas
        }
    }

Ya no sé lo que hace la corrutina, pero no recomiendo usarlas SALVO para tareas que ocurren en diferentes frames (generalmente para esperar tiempos, hacer una animación "a mano", y siempre desactivando el propio script en la corrutina para que no ejecute código, la activas al terminar la corrutina o llamando a un método del propio script; sí, puedes ejecutar métodos del script aunque este esté deshabilitado, muy útil para usar con OnTriggerEnter).

Según he entendido, la filosofía de Unity te pide que sea el propio pájaro el que genere la explosión al morir, pero en mi experiencia, lo mejor que puedes hacer es que este script de control llame a un método del pájaro que se llame "morir" o algo así, y que haga dos cosas: instanciar la explosión, y autodestruirse (no uses el método OnDestroy, generaría la explosión al salir de la escena).
Usar grandes códigos que controlan varios elementos parece que va en contra de la filosofía de Unity, que es lo opuesto a lo que nos enseñan en programación, pero te tienes que hacer a la idea que lo que controla el juego es el editor gráfico, y los scripts son pequeñas instrucciones para que los objetos actúen por su cuenta. Otra cosa es que quieras mantener tu estilo, cada uno lo hace como le resulta más cómodo.