Question regarding delegates and events

Question

I have been recently learning about delegates & events. I would like to know performance wise if there is another way to do this. Or should I just write events for every single command like so:

 public delegate void IndividualUnitAttackingDelegate(GlobalAISensor aiSensor);
 public static event IndividualUnitAttackingDelegate MoveEnemyUnitToAttack1;
 public static event IndividualUnitAttackingDelegate MoveEnemyUnitToAttack2;
 public static event IndividualUnitAttackingDelegate MoveEnemyUnitToAttack3;

 public delegate void IndividualDefendingUnitDelegate(Vector3 destination);
 public static event IndividualDefendingUnitDelegate MoveEnemyUnitToDefend1;
 public static event IndividualDefendingUnitDelegate MoveEnemyUnitToDefend2;
 public static event IndividualDefendingUnitDelegate MoveEnemyUnitToDefend3;

here is how it's being used in this main event script if for whatever reason you need that information:

 void DetermineTargetDestination()
     {
         if (Enemy1DefendPlanet)
             MoveEnemyUnitToDefend1(defendPosition1);
         if (Enemy2DefendPlanet)
             MoveEnemyUnitToDefend2(defendPosition2);
         if (Enemy3DefendPlanet)
             MoveEnemyUnitToDefend3(defendPosition3);
         if (!Enemy1DefendPlanet && aiSensor1 != null && MoveEnemyUnitToAttack1 != null)
             MoveEnemyUnitToAttack1(aiSensor1);
         if (!Enemy2DefendPlanet && aiSensor2 != null && MoveEnemyUnitToAttack2 != null)
             MoveEnemyUnitToAttack2(aiSensor2);
         if (!Enemy3DefendPlanet && aiSensor3 != null && MoveEnemyUnitToAttack3 != null)
             MoveEnemyUnitToAttack3(aiSensor3);
         Enemy1DefendPlanet = false;
         Enemy2DefendPlanet = false;
         Enemy3DefendPlanet = false;
     }

Answer 1

Answer by YoungDeveloper · Sep 15, 2015 at 09:02 PM

Why not pass the command as secondary param?

0 Show 3 · Share

Ziron999 · Sep 15, 2015 at 09:05 PM 0

Share

do you mean create something like IndividualUnitDelegate and then add both of these delegates to that delegate? I'm not adding a 2nd param and just using 1 delegate because then it would be processing that 2nd param in code where it's not needed.

YoungDeveloper · Sep 15, 2015 at 09:11 PM 0

Share

I mean passing in function everything you need MyDelegate(Sensor sensor, AttackType AttackType);

And or receive

 switch(attackType){
  case AttackType.Melee:
     //work
 break;
 }

Ziron999 YoungDeveloper · Sep 15, 2015 at 09:30 PM 0

Share

that's where my issue is sensor would only be used in attack and the vector3 would only be used in defend. maybe I should just do MyDelegate(AttackType AttackType)? Would this actually process information faster this way? performance matters a lot here it's 1000 units of data passing through scripts.

Question regarding delegates and events

1 Reply

Your answer

Welcome to Unity Answers

Follow this Question

Related Questions