[C#] Jak zamienić to na while?

Question

public TextMesh TextOfLoad;
private GameObject Blod;

void DAB(){
	DestroyAllBlocks();
}

public void DestroyAllBlocks(){
		Blod = GameObject.FindWithTag("block");
		if(Blod!=null){
			TextOfLoad.text = Blod.ToString();
			Destroy(Blod);
			DAB();
		}
		else
		{
			Blod=null;
			TextOfLoad.text = "Null";
		}
}

Taki kod w unity wywala błąd. Myślałem nad zastąpieniem tego pętlą While, ale coś nie działa. Pomógłby ktoś?

Comments

Hej, dużo łatwiej będzie jak podasz jaki błąd wywala ci w konsoli :d

---

Hej, tak na logikę co może wywalać błąd, to że pętle się zapętlają. Unity zatrzymuje program i wywala błąd.

 

StackOverflowException
UnityEngine.GameObject.FindWithTag (System.String tag) (at C:/buildslave/unity/build/artifacts/generated/bindings_old/common/Core/GameObjectBindings.gen.cs:299)
LevelGenerator.DestroyAllBlocks () (at Assets/Scripts/LevelGenerator.cs:62)
LevelGenerator.DAB () (at Assets/Scripts/LevelGenerator.cs:58)
LevelGenerator.DestroyAllBlocks () (at Assets/Scripts/LevelGenerator.cs:66)
LevelGenerator.DAB () (at Assets/Scripts/LevelGenerator.cs:58)
LevelGenerator.DestroyAllBlocks () (at Assets/Scripts/LevelGenerator.cs:66)
LevelGenerator.DAB () (at Assets/Scripts/LevelGenerator.cs:58)
LevelGenerator.DestroyAllBlocks () (at Assets/Scripts/LevelGenerator.cs:66)
... ( za dlugi tekst ale tu jest to samo co wyzej)
LevelGenerator.DAB () (at Assets/Scripts/LevelGenerator.cs:58)
LevelGenerator.DestroyAllBlocks () (at Assets/Scripts/LevelGenerator.cs:66)
LevelGenerator.DAB () (at Assets/Scripts/LevelGenerator<message truncated>

 

Answer 1

Jeśli dobrze rozumiem to może łatwiej będzie ci w ten sposób:

1) Znajdź wszystkie obiekty z danym tagiem i zapisz do listy/tablicy

https://docs.unity3d.com/ScriptReference/GameObject.FindGameObjectsWithTag.html <= tutaj masz funkcję, która znajdzie Ci wszystkie obiekty z danym tagiem

2) Przejdź po liście/tablicy pętlą `for` i w każdej iteracji rób to co tam musisz z tym obiektem

Comments

Unikasz rozwiązania problemu. Ja chcę skorzystać z while.

---

Jak nie patrzeć próbuję podsunąć Ci lepsze rozwiązanie niż while. Jak użyjesz while to będziesz w każdym obrocie pętli szukał obiektu na scenie -> duże obciążenie maszyny. Zrobisz jak uważasz :)

---

-_- Szkoda tylko że każdy obiekt ma taką samą nazwe i taki sam tag. Więc dodanie 10 takich samych obiektów do tablicy nic nie da bo i tak się usunie jeden.

---

Wiesz, z chęcią bym ci pomógł ale jesteś strasznym ignorantem i nie chce mi się tu tracić czasu :)

Answer 2

Nie wiem czy dobrze zrozumiałem, ale może użyj pętli foreach, coś w tym stylu:

public void DestroyAllBlocks(){
	    Blods = GameObject.FindWithTag("block");
		if(Blods) {
			foreach(var blod in Blods) {			
				TextOfLoad.text = blod.ToString();
                                Destroy(blod);			 
			}
 		} else {
				Blods = null;
				TextOfLoad.text = "Null";
			}		
}