Interest Ability

[CBBPhoenix]

Hey guys

I am making a hero which should be based on economy-skills and one of its abilities should give interest of current gold every five minutes. I have tried using this action for the periodic event:
Player -Add property - add(Owner of(random unit from(units in playable map area matching(level of Interest for Matching Unit equal to 1) current gold X (1/7) to (owner of (Matching Unit's)) current gold. I would perfer using reals and not integers for the interest as it would give at least 100% in that case.
The problem is that no gold is added using this trigger; not even if I use a set value instead of a function

Anyone has any ideas? Thanks in advance

 

[kirinelf]

Hmm... I THINK I know a way to solve your problem, but I'm not sure. Gimme a while to get things in order and I'll give it a shot. In the meantime, let's hope someone else pops in who can help.

Edit: I haven't tested this yet, but try this:

• Interest
  • Events
    • Time - Every 300.00 seconds of game time
  • Conditions
    • (Level of [Interest] for [Hero]) Greater than 0
  • Actions
    • Set [BonusIncome] = (((Owner of [Hero]) Current gold) x (7 / 10))
    • Player - Add BonusIncome to (Owner of [Hero]) Current gold

[Interest] is the ability that gives you the Interest. It should be a passive skill with everything (damage, AoE, everything) set to 0.
[Hero] is a unit type variable that is assigned to the hero with the skill.
[BonusIncome] is an integer variable. All you have to do is make it.

As far as I can see, this isn't MUI (Multi Unit uh... something that starts with I, which means that more than one unit can use it at the same time). So if my method works (Not sure if it actually does), it only does for one unit. Try it and see if it works.

 

[Joccaren]

MUI: Multiple Unit Instance.

To make MUI, just use arrays or If, then, elses

 

[DK's Pride]

Add variables for each player to make it MPI - - - - Multi Player Instance

(I dont think its important that u have it MUI as u only have 1 of each Hero)

 

[Komaqtion]

@ kirinelf: Well, the way you put it up, makes it non-MUI, but by just changing it a little, it'll become MUI

• Interest
  • Events
    • Time - Every 300.00 seconds of game time
  • Conditions
  • Actions
    • Set TempGroup = (Units in (Playable map area) matching ((((Matching unit) is A Hero) Equal to True) and ((Level of Acid Bomb for (Matching unit)) Greater than 0)))
    • Unit Group - Pick every unit in TempGroup and do (Actions)
      • Loop - Actions
        • Player - Add (((Owner of (Picked unit)) Current gold) x (1 / 7)) to (Owner of (Picked unit)) Current gold
    • Custom script: call DestroyGroup(udg_TempGroup)

@ DK's Pride: Please don't just assume stuff... Do some research before saying stuff like that.

 

[CBBPhoenix]

Mui

Does it matter at all whether it's MUI or not? I only allow one of each hero type in the map. These triggers look a lot like my own but I'll try them the problem is that when the hero level-ups the skill, it should give 1/6 X Currrent gold, 1/5 X Current gold.. etc income.. and level higher than 0 only works for level 1... But I assume I can simply use an "equal to" integer comparrision?

 

[Komaqtion]

No, I'll change it to suit your needs... Just give me a sec

EDIT: Here

• Interest
  • Events
    • Time - Every 300.00 seconds of game time
  • Conditions
  • Actions
    • Set TempGroup = (Units in (Playable map area) matching ((((Matching unit) is A Hero) Equal to True) and ((Level of "Interest Ability" for (Matching unit)) Greater than 0)))
    • Unit Group - Pick every unit in TempGroup and do (Actions)
      • Loop - Actions
        • Player - Add (((Owner of (Picked unit)) Current gold) x (1 / (8 - (Level of "Interest Ability" for (Picked unit))))) to (Owner of (Picked unit)) Current gold
    • Custom script: call DestroyGroup(udg_TempGroup)

 

[CBBPhoenix]

okay

okay sounds good I am going to test it now. I just hope it wont return 0 gold each interval as the old one :-/

I've tried it but it does not add anything to the gold amount of the player. I am posting a screenshot. Might have missed something.

 

[Joccaren]

The only thing I can see you've missed is the 'Matching unit is a Hero' part which I don't think is required. BTW, it may be easier to use WC3 tags rather than post screenshots (I used to do that until I found out about WC3 tags). To use the tags, right click the page Icon at the top of your trigger, select copy as text. Then just click the WC3 tags icon (The yellow a) in your post then paste your trigger as text between the two tags.

 

[CBBPhoenix]

So you can copy directly from the editor? Nice. still my ability problem is not solved yet, I can't believe that it should have anything to do with the Hero comparison...

 

[Joccaren]

Two things
1. does WE recognise that 1/8 still counts as a number? I can think of multiple reasons why it wouldn't
2. 1/(8-level of interest for Player X))???
Thats making you get less interest each level of the ability. Try + level of interest

 

[CBBPhoenix]

hmm

Well I might check up on the number thing, could be a solution. The reason why 1/(8 - level of interest) gets better at higher levels is because it is a reciprocal number. It will (or should very likely) be lvl1 = 1/7 = 0.14. lvl2 = 1/6 = 0.17 etc

 

[Joccaren]

Got it

 

[Joccaren]

• Interest
  • Events
    • Time - Every 300.00 seconds of game time
  • Conditions
  • Actions
    • Set Interest = ((Player X (Colour) Current gold) / (8 - (Level of Interest for (Triggering unit))))
    • Player - Add Interest to Player X (Colour) Current gold

Try that. Not MPI but that can be changed by using arrays for the variables
Oops, forgot to add 'Level of Interest for Unit X is greater than 0'

Pretty much what has already been posted by Kirinelf, only solution to how to make it MPI and multi leveled.