How do I prevent a infinite loop in my procedural generation

PaintedBirds

I am trying to take pockets of Perlin noise, and generate a dungeon inside of it, but to keep it to one pocket, I tried to make a self-duplicaating object that would spread thoughout and die out, without going back on itself. It would generate a sphere where it was that would delete any new objects. exept it doesn't work. I get an infinite loop that freezes up Unity and breaks.

Here is my code:

1. using System.Collections;
2. using System.Collections.Generic;
3. using UnityEngine
4. 
   
5. public class RoomGen : MonoBehaviour
6. {
7. public float RoomValue;
8. public float seed = 1.457582f;
9. public GameObject Marker;
10. public GameObject Self;
11. // Start is called before the first frame update
12. void Start()
13. {
14. }
15. void LateUpdate()
16. {
17. RoomValue = Mathf.PerlinNoise(transform.position.x * seed, transform.position.z * seed);
18. if (RoomValue >= .4f)
19. {
20. Instantiate(Marker, transform.position, transform.rotation);
21. Instantiate(Self, transform.position + new Vector3(-1, 0 ,0), transform.rotation);
22. Instantiate(Self, transform.position + new Vector3(1, 0, 0), transform.rotation);
23. Instantiate(Self, transform.position + new Vector3(0, 0, -1), transform.rotation);
24. Instantiate(Self, transform.position + new Vector3(0, 0, 1), transform.rotation);
25. Destroy(this.gameObject);
26. }
27. else
28. {
29. Destroy(this.gameObject);
30. }
31. }
32. 
    
33. private void OnCollisionStay(Collision collision)
34. {
35. Destroy(this.gameObject);
36. }
38. }

In theory the collison should trigger before the late update, then destroying the object before it can spread. In practice it all breaks down into an infinite loop. Any advice would be appreaciated

Sander

It might be an idea to run it in a coroutine with a delay on it. That way, you can more easily see what is causing the break.

PaintedBirds

@Sander I know where the problem is, but not how to fix it. The OnCollisionStay() is not triggering before the LateUpdate(), causing an inifnite loop