[Trigger] Basic Income Trigger

[Silverfish423]

Hello everyone, I need help creating an income trigger than will give 15gold per capital. A capital is any building that has the ability "Income - Capital" which gives off a buff of the same type. For each of these buildings that the player would own, 15 gold would be received.
I attempted creating this trigger on my own, however, it did not work. This is what I made:

• Capitals
  • Events
    • Time - Every 60.00 seconds of game time
  • Conditions
    • ((Triggering unit) has buff Income - Capital ) Equal to True
  • Actions
    • Player - Add 15 to (Owner of (Triggering unit)) Current gold

Would anyone know how i can fix my trigger so that it does what I'd like it to do? If you have any questions, let me know, thanks!

 

[HesitationSnow]

Make actions like, pick all units with buff, then do actions. Inside the actions, make an if then else statement (multiple), move the condition into this one, and add gold action into it. Remove unit group leak. Change triggering unit to picked unit. I would show you but im typing this on my cellphone.

 

[deathismyfriend]

in ur case there is no triggering units since the trigger is every 60 seconds so if u add the structure to a group and do what hesitation suggested tht would be ur best bet

 

[Wrda]

I guess you want only user-players receive gold right? So here it is the solution:

• Income
  • Events
    • Time - Every 60.00 seconds of game time
  • Conditions
  • Actions
    • Unit Group - Pick every unit in (Units in (Playable map area) matching (((Owner of (Matching unit)) controller) Equal to User)) and do (Actions)
      • Loop - Actions
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • (Level of Income - Capital for (Picked unit)) Greater than 0
          • Then - Actions
            • Player - Add 15 to (Owner of (Picked unit)) Current gold
          • Else - Actions
    • Custom script: call DestroyGroup(GetLastCreatedGroup())

 

[Chaosy]

use a unit group

 

[-Kobas-]

Check this http://www.hiveworkshop.com/forums/...-2-gui-218904/?prev=search=income&d=list&r=20

 

[Assassins-Creed]

I made one here but it adds a timer to each farm right when its built so instead of just getting *all farms x 60* gold when the timer ends, each farm has it's own timer of 8 seconds. http://www.hiveworkshop.com/forums/triggers-scripts-269/income-system-help-226072/#post2248971

 

[nedio95]

• ((Triggering unit) has buff Income - Capital ) Equal to True

You have no triggering unit.
If the unit is a building and is preset in yourmap just do

• Set Capital = Capital 001gen

And replace triggering unit in your trigger with the variabe.


Edit//
If its not preset or unique:

use a unit group

3triggers:
1st-When an income building is created add it to unit group.
2nd-Evry xx seconds add yy gold depending on the number of units in yhe unit group.
3rd-When an income building dies remove it from the unit group

 

[Wrda]

• ((Triggering unit) has buff Income - Capital ) Equal to True

You have no triggering unit.
If the unit is a building and is preset in yourmap just do

• Set Capital = Capital 001gen

And replace triggering unit in your trigger with the variabe.


Edit//
If its not preset or unique:
3triggers:
1st-When an income building is created add it to unit group.
2nd-Evry xx seconds add yy gold depending on the number of units in yhe unit group.
3rd-When an income building dies remove it from the unit group

LOL.
That would be done slower than setting a unit group with 3 conditions, unit is a building, it is alive and has capital ability. Your coding knowledge should be improved nedio...

 

[Cokemonkey11]

LOL.
That would be done slower than setting a unit group with 3 conditions, unit is a building, it is alive and has capital ability. Your coding knowledge should be improved nedio...

Actually, it shouldn't be slower. Enumerating all the units in the whole map and then checking if each one matches 3 conditions is just as slow.

A better method would be to store player incomes as an integer array and then increment/decrement values when income buildings are created/destroyed.

 

[Assassins-Creed]

LOL.
That would be done slower than setting a unit group with 3 conditions, unit is a building, it is alive and has capital ability. Your coding knowledge should be improved nedio...

I thought the Hive was friendly? Why are you here? Be gone demon spawn!

 

[nedio95]

A better method would be to store player incomes as an integer array and then increment/decrement values when income buildings are created/destroyed.

Agreed.

 

[Wrda]

I thought the Hive was friendly? Why are you here? Be gone demon spawn!

This

Your coding knowledge should be improved nedio...

is supposed to be an advice, not an inflamatory post.

Actually, it shouldn't be slower. Enumerating all the units in the whole map and then checking if each one matches 3 conditions is just as slow.

A better method would be to store player incomes as an integer array and then increment/decrement values when income buildings are created/destroyed.

It doesn't enumerate all the units in the map, it enumerates the units in map who match the conditions. Anyway it doesn't slow.

 

[Cokemonkey11]

It doesn't enumerate all the units in the map, it enumerates the units in map who match the conditions. Anyway it doesn't slow.

While that would be nice, it does actually have to check all units in the whole map (enumerating them) and checking each one individually if it matches the condition before storing it in the group.

Sorry!

 

[Wrda]

True ^^ (makes sense)

 

[Silverfish423]

Thanks for the help everyone! I am definitely not a coder, so all the help is much appreciated. I am going to first try the system that -Kobas- suggested and then the others if that one doesn't work the way I wish it to.
Thanks again!