Unity C# kod hatası

Hazeyn_44 dedi:

Unity ile flappy bird oyunu yapıyordum, arka arkaya gelen borular için bir script yazıyordum, scripti yazdım, gerekli ayarları yaptım ama oyunu çalıştırınca sadece 1 boru geliyor. Arka arkaya borular spawn olmuyor.

C#:Panoya kopyala

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

public class spawner : MonoBehaviour
{
    public birdcontroller Birdscript;

    public GameObject Borular;

    public float height;

    private void start()
    {
        StartCoroutine(SpawnObject());
    }

    public IEnumerator SpawnObject()
    {
        Instantiate(Borular, new Vector3(3, Random.Range(-height, height), 0), Quaternion.identity);
      
        yield return new WaitForSeconds(2f);

    }

}

Genişletmek için tıkla...

private void update()
    {
        StartCoroutine(SpawnObject());
    }

AhmetKnKc dedi:

start() metodunun içine yazdığınızdan dolayı böyle. Start() başta bir kere çalışır sonra durur.
update veya fixedUpdate metodlarından birisine yazmanız gerekiyor, aşağıdaki gibi;

C#:Panoya kopyala

private void update()
    {
        StartCoroutine(SpawnObject());
    }

Genişletmek için tıkla...

    public IEnumerator SpawnObject()
    {
        while (true)
        {
            Instantiate(Borular, new Vector3(3, Random.Range(-height, height), 0), Quaternion.identity);
      
            yield return new WaitForSeconds(2f);
        }
    }

Foxie dedi:

Hocam bu ciddi anlamda belleği şişirebilir, böyle daha mantıklı

C#:Panoya kopyala

 public IEnumerator SpawnObject()
 {
 while (true)
 {
 Instantiate(Borular, new Vector3(3, Random.Range(-height, height), 0), Quaternion.identity);

 yield return new WaitForSeconds(2f);
 }
 }

Genişletmek için tıkla...

Hazeyn_44 dedi:

Son komut bu ama yine çalışmıyor.

C#:Panoya kopyala

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

public class spawner : MonoBehaviour
{
    public birdcontroller Birdscript;

    public GameObject Borular;

    public float height;

    private void update()
    {
        StartCoroutine(SpawnObject());
    }

    public IEnumerator SpawnObject()
    {
        while (true)
        {
            Instantiate(Borular, new Vector3(3, Random.Range(-height, height), 0), Quaternion.identity);

            yield return new WaitForSeconds(2f);
        }
    }

}

Genişletmek için tıkla...

using System.Collections;

using System.Collections.Generic;

using UnityEngine;



public class spawner : MonoBehaviour

{

    public birdcontroller Birdscript;



    public GameObject Borular;



    public float height;



    private void Start()

    {

        StartCoroutine(SpawnObject());

    }



    public IEnumerator SpawnObject()

    {

            Instantiate(Borular, new Vector3(3, Random.Range(-height, height), 0), Quaternion.identity);



            yield return new WaitForSeconds(2f);

           StartCoroutine(SpawnObject());

    }



}

Abdullah_AKD dedi:

Kod:Panoya kopyala

using System.Collections;

using System.Collections.Generic;

using UnityEngine;

public class spawner : MonoBehaviour.

{

 public birdcontroller Birdscript;

 public GameObject Borular;

 public float height;

 private void Start()

 {

 StartCoroutine(SpawnObject());

 }

 public IEnumerator SpawnObject()

 {

 Instantiate(Borular, new Vector3(3, Random.Range(-height, height), 0), Quaternion.identity);

 yield return new WaitForSeconds(2f);

 StartCoroutine(SpawnObject());

 }

}

Bu şekilde yapın.

Genişletmek için tıkla...

Hazeyn_44 dedi:

Kod çalışıyor ama geçen borular da silinirse iyi olur. Onu nasıl yapabilirim?

Genişletmek için tıkla...

private List<GameObject> createdPipes = new();

private int maximumPipeCount = 7; //Ekranda max 6 tane pipe gözükecek varsayalım. 1 tane de garanti olsun fazladan tutalım



private IEnumerator CreatePipes()
{
    var pipe = Instantiate(pipePrefab,pipePosition,pipeRotation);
    createdPipes.Add(pipe);
    
    if(createdPipes.Count >= maximumPipeCount)
    {
        Destroy(createdPipes[0]);
        createdPipes.RemoveAt(0);
    }
    
    yield return new WaitForSeconds(2);
    StartCoroutine(CreatePipe());
}

Ömerrrrrr dedi:

Bunu yapmanız çok işinize yarayacaktır. Oluşturduğunuz boruları bir listede tutun. Ekranınızda maksimum kaç tane boru gözüktüğüne bakın ve bundan 1 fazlasını garanti olsun diye ekranda bırakalım. Şu şekilde yapacağız.

C#:Panoya kopyala

private List<GameObject> createdPipes = new();

private int maximumPipeCount = 7; //Ekranda max 6 tane pipe gözükecek varsayalım. 1 tane de garanti olsun fazladan tutalım.

private IEnumerator CreatePipes()
{
 var pipe = Instantiate(pipePrefab,pipePosition,pipeRotation);
 createdPipes.Add(pipe);

 if(createdPipes.Count >= maximumPipeCount)
 {
 Destroy(createdPipes[0]);
 createdPipes.RemoveAt(0);
 }

 yield return new WaitForSeconds(2);
 StartCoroutine(CreatePipe());
}

Bunun yapılabilecek daha iyi bir versiyonu da var. O da şu. En başta toplam 7 tane boru oluşturup en soldaki boru ekrandan silindikten sonra onu en sağa tekrar çizerseniz toplamda sadece 7 tane boru kullanıp bunları döndürerek bellekten çok büyük bir tasarruf etmiş olursunuz. Buna object pooling denir. Araştırırsanız çok iyi şeyler öğrenebilirsiniz. Bu problem için biraz araştırın yapamazsanız onu da beraber yaparız.

Genişletmek için tıkla...