harvest detection

[Chaosy]

In short, I want to give a player +1 lumber each time a worker attacks/harvests a tree.

I thought of a few ideas but none of them are good.

1. every X seconds check health of every single tree in the game and compare it to the hp when last checked. However I think a destructable group got a limit of 64 and I got a lot of trees in my map. This would also be very inefficient.

2. check when unit is ordered to harvest a tree, store the targeted tree into a variable and then keep track of the hp of that specific tree in a loop. This is what I am rolling with atm but it seems kinda.. bad.. somehow.

3. check when unit attacks and then give +1 lumber. I am not even sure if the event works on destructables and even so it's buggy since you could get lumber by canceling the animation.

4. checking for the current order id in a loop trigger and give player lumber every -animation length- seconds. This would be buggy since the issued order can be given from far away and the player would therefore be getting lumber by running to the tree.

 

[edo494]

only the GUI destructable enumeration has 64 trees limit(can be seen in the BJ function that is called), but yes, 1. is very very bad, cause checking every tree is so heavy your map would drop to literally 0 fps.

Second one sounds a lot better.

Im pretty sure third one wont work, cause unit has order Harvest, not Attack when harvesting from trees.

You named the problem in the fourth yourself

 

[Chaosy]

• get target
  • Events
    • Unit - A unit Is issued an order targeting an object
  • Conditions
  • Actions
    • Set order = (Current order of (Triggering unit))
    • Set target_tree = (Target destructible of issued order)
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • Or - Any (Conditions) are true
          • Conditions
            • order Equal to (Order(harvest))
            • order Equal to (Order(attack))
            • order Equal to (Order(smart))
      • Then - Actions
        • Game - Display to (All players) the text: ((Name of (Triggering unit)) + (will go and harvest + (Name of (Target destructible of issued order))))
        • Set tree_last_hp = (Current life of target_tree)
      • Else - Actions

• Untitled Trigger 001
  • Events
    • Time - Every 0.05 seconds of game time
  • Conditions
  • Actions
    • Set tree_new_hp = (Current life of target_tree)
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • tree_new_hp Less than tree_last_hp
      • Then - Actions
        • Game - Display to (All players) the text: +1
      • Else - Actions
    • Set tree_last_hp = tree_new_hp

Maybe it's just me who thinks this feels odd. Why couldn't Blizzard just add a Destructable takes damage event -.-

 

[IcemanBo]

Just as info: The GUI destructable enumeration also is not limited by 64. It's only for the registration function for event.

 

[Wietlol]

Attack indexing?

 

[edo494]

Thanks to Iceman for correcting me on that.

You cant index attack for harvest order, because no attack happens to begin with.