Mega Bug In Turret - Help

http://technology.blurst.com/unity-physics-trigger-collider-examples/

An object destroyed inside of a trigger will not send OnTriggerExit events. This link will explain a fix if you think this is your problem.

Err never mind you seem to understand this. Well it is a good link anyway.

Do the dead turrets ever get removed from the list?

I'm not an expert on static but it could mean that there is one DamageReceiver.Dead for all DamageReceiver components. So you should perhaps make Dead public and not static. Also, the unity convention would be dead (lowercase) since it is a variable.

It does not need to be static. Here's an idea:

function OnTriggerEnter (enemy : Collider) { closeObjects.Add(enemy.gameObject); }

Keep a list of gameObjects instead of transforms. Then you can call:

transform.LookAt(target.GetComponent(Transform));

or

if(target.GetComponent(DamageReceiver).dead) { //blah }

Oops again. I guess I assumed DamageReceiver was a component of all target gameobjects.

In any case, there is certainly a solution without that static and an array of gameObjects could help with that.

I dunno I gotta go to sleep.

Looking at your code comments it appears the only reason 'Dead' is static is so that it is visible inside SmoothLookAt. Using statics and BroadcastMessage for communication between classes is not needed when you have an instance of the object. I agree that you should use GetComponent as Kliffy suggested.

Get an instance of your script component from the Transform and call functions on it directly.

I noticed you are using quite a few static variables. Static variables are easy to use because they're accessible everywhere, but they have major drawbacks and should be used sparingly. Having global scope means whenever you have a bug, that variable is in visible so it can't be ruled out, implicitly increasing the complexity of all your code. Another issue is everything can modify it, so if you've discovered that a variable has the wrong value you can't make any assumptions about what changed it, again because everything has access.

I'm not sure what causes that error based on what I have in front of me. But, I would use your ontriggerenter/exit to make a list of GameObjects of targets. I would make dead a public var. You could just make a function

function IsDead() { return dead; }

and you can call that from the turret script. In that case dead could even be private, and you could make a function for the turret to call to change dead from true to false. And then each target will have its own dead variable.

What is dead receiver attached to currently? When you tell it to Destroy(gameObject) I'm not sure how the subsequent stuff gets called although I guess the destruction happens at the end of the frame?

If you are worried with my implementation that the gameobject will be destroyed before the turret can see that they are dead, you can send a time variable to Destroy(gameobject, time) to have it be destroyed after a certain length of time which would allow a dying animation to be played and while the turret realizes the target is no longer worth shooting / looking at because it is dead.

My guess is that you have multiple turrets changing that static dead back and forth, as well as different targets changing it back and forth and at some point it gets changed in a way / order that you did not intend and causes the bug.

Also if you try to do my suggestion, dead would be a var called in the constructor (aka at the top and outside of functions) of DamageReceiver and it would originally be set to false. And then I presume that the detonation script would just send the message and set dead to true.

I've been working on a similar thing for my tower defense, I got this code up and working last night with the use of GameObject.FindGameObjectsWithTag to just search for anything tagged as Enemy (just tag your enemy prefab and it should work out). Oh and you'll probably have to swap out the GetComponent for the decrement of enemy health with whatever your enemy script name and health value in the firing function

 var closest : GameObject;
 var rotSpeed : float = 3;
 var enemyFound : boolean = false;
 var shootRange : float = 35.0;
 
 
 function Update(){
     FindClosestEnemy();
 
     if(enemyFound){
         var targetRot = Quaternion.LookRotation(closest.transform.position-transform.position);
         transform.rotation = Quaternion.Slerp(transform.rotation, targetRot, Time.deltaTime * rotSpeed);
 
         if(Vector3.Distance(transform.position, closest.transform.position) < shootRange){
             FireAtEnemy();
         }
     }
 }
 
 function FindClosestEnemy () : GameObject {
     var distance = Mathf.Infinity; 
     var position = transform.position; 
 
     var gos : GameObject[];
     gos = GameObject.FindGameObjectsWithTag("Enemy");
 
     for (var go : GameObject in gos)  { 
         var diff = (go.transform.position - position);
         var curDistance = diff.sqrMagnitude; 
         if (curDistance < distance) { 
             closest = go; 
             distance = curDistance;
             enemyFound = true;
         } 
     } 
     return closest;
 }
 
 function FireAtEnemy(){
     var enemyHP = closest.GetComponent("enemyScript");
     enemyHP.health = enemyHP.health -1;
 }