OnDisable with Instantiating Prefabs

This doesn't seem to fire for me:

Script "A" gets a prefab passed to it and create 20 instances of it through a loop:

Code (csharp):

1. var o = GameObject.Instantiate(_antibody, new Vector3(Random.Range(-10, 10), 0, Random.Range(-10,10)), Quaternion.identity);

Script "A" has OnDisable() specified:

Code (csharp):

1.     void OnDisable()
2.     {
3.         Debug.Log ("Destroyed");
4.     }

Script "B" destroys the item:

Code (csharp):

1.     void OnControllerColliderHit(ControllerColliderHit hit){
2.         Destroy(hit.gameObject);
3.     }

I appreciate any help! Thanks!

 

Does script B destroy the newly instantiated prefabs? Or does it destroy the object creating the prefabs?

 

It destroys the prefabs that are instantiated. I am unsure how to tie the event to the instantiated prefabs vs the script...(new to Unity)

 

Looks like your OnDisable() is attached to the A script that instantiates from prefabs. You are not destroying script A (based on above) - you are destroying the object instantiated from prefab. So attach it to that prefab. Also it should not be OnDisable() - it should be OnDestroy().

 

OnDisable() is valid, and I'm pretty sure it gets called upon destroying an object since it implicitly gets disabled along the way.

 

Well, it is purely a question of intent. Does the functionality between destroy and disable need to be shared. If yes then you are right. Otherwise, it is just an unclean way of doing things.

I agree though, I should have state that that part was optional.

 

I do get the difference between OnDisable and OnDestroy. So outside of creating a separate script and attaching it to the prefab, is there any way for me to tie an event to the objects called from Instantiate? Since "o" is the output of the object being created, can I do something like nDestroy += (I know this doesn't work, but I am looking for a way to bind an event like I would in normal C#).

Thanks.

 

As far as I know, creating a separate script and attaching it to prefab is the only way to go. Off course, you can attach it to prefab from your code dynamically by using AddComponent as well.