Question:How to level up all heros

[brodygs9630]

I've been having trouble with this for a bit...
I've tried making a trigger that at 23 seconds into the game,all heros gain 200 exp(to level up),but it doesnt work.Please Help

PS:Help = +Rep

 

[branno]

show your trigger u made

 

[Emu.Man00]

pick every unit in playable map area matching (matching unit is a hero == true)
Hero- add 200 experience to (picked unit)

 

[brodygs9630]

• Level Up
  • Events
    • Time - Elapsed game time is 23.00 seconds
  • Conditions
  • Actions
    • Hero - Add 200 experience to (Picked unit), Show level-up graphics

That didn't work,so i tried this:

• Level Up
  • Events
    • Time - Elapsed game time is 23.00 seconds
  • Conditions
  • Actions
    • Player Group - Pick every player in (All players) and do (Hero - Add 200 experience to (Picked unit), Show level-up graphics)

But that didn't work also

EDIT:

pick every unit in playable map area matching (matching unit is a hero == true)
Hero- add 200 experience to (picked unit)

Didn't work

 

[WolfieeifloW]

• Level Up
  • Events
    • Time - Elapsed game time is 23.00 seconds
  • Conditions
  • Actions
    • Unit Group - Pick every unit in (Units in (Entire map) matching (((Matching unit) is A Hero) Equal to (==) True)) and do (Actions)
      • Loop - Actions
        • Hero - Add 200 experience to (Picked unit), Hide level-up graphics

Try that .

 

[Defi4nc3]

• Event
  • Time Elasped - 23.00 Seconds

• Action
  • Set Unit Group - Units in Playable Map Area matching Matching Unit is a Hero equal to True
  • Pick Every Unit in Unit Group
    • Set Hero Level to (Level of Picked Unit + 1)

something like that~

 

[Emu.Man00]

>(Triggering unit) is A Hero

Matching unit, not triggering

edit:

If you're going to use a variable, like in Defi4nc3's code, you might as well destroy the group to clean leaks. At the very end put in this line:
custom script: call DestroyGroup(udg_variable)

 

[WolfieeifloW]

I don't think you have to set Entire Map or Playable Map Area into a variable?

 

[Emu.Man00]

Everything i know about leaks i get from here, and he thinks you do :3
For playable map atleast
http://www.thehelper.net/forums/showthread.php?t=27219&highlight=leak

 

[PrisonLove]

Everything i know about leaks i get from here, and he thinks you do :3
For playable map atleast
http://www.thehelper.net/forums/showthread.php?t=27219&highlight=leak

From what I understand, if you set either all players or entire map into a variable and then destroy it as a leak it messes everything up, and you can no longer reference the map or all players again. I've had this happen to me with all players, not entirely sure about entire map though. Besides, if he only uses this trigger once (which is all he can do, since it's time elapsed), then the leak is negligible.

However, I believe this is what you're looking for:

• add experience
  • Events
    • Time - Elapsed game time is 23.00 seconds
  • Conditions
  • Actions
    • Set TempGroup = (Units in (Playable map area) matching (((Matching unit) is A Hero) Equal to True))
    • Unit Group - Pick every unit in TempGroup and do (Actions)
      • Loop - Actions
        • Hero - Add 200 experience to (Picked unit), Show level-up graphics
    • Custom script: call DestroyGroup(udg_TempGroup)

Bear in mind that TempGroup is a Unit Group Variable, and you can change show level-up graphics to hide, I didn't know what you wanted.

Edit: By the way Brody, I like your avatar lol.

 

[brodygs9630]

Thanks so much Wolfie(as usual):thup:

Finally working,finally finished Beta of my map i've been working on!

 

[Defi4nc3]

Sheesh I didn't have my editor open, of course you wouldn't have to.

• Events
  • Time - Elapsed game time is 23.00 seconds
  • Conditions
  • Actions
    • Set TUGS[1] = (Units in (Playable map area) matching (((Matching unit) is A Hero) Equal to True))
    • Unit Group - Pick every unit in TUGS[1] and do (Actions)
      • Loop - Actions
        • Hero - Set (Picked unit) Hero-level to ((Hero level of (Picked unit)) + 1), Hide level-up graphics
    • Custom script: call DestroyGroup(udg_TUGS[1])
    • Custom script: call DestroyTrigger(GetTriggeringTrigger())

Happy now?

 

[WolfieeifloW]

From what I understand, if you set either all players or entire map into a variable and then destroy it as a leak it messes everything up, and you can no longer reference the map or all players again. I've had this happen to me with all players, not entirely sure about entire map though.

Yeah;
I'm pretty sure that happens for Entire Map too.
It's just Playable Map Area plus the outer bounds.

Thanks so much Wolfie(as usual):thup:

Finally working,finally finished Beta of my map i've been working on!

No problem;
Good luck with the map!


EDIT: Why an array ?
And you should use [0], the first spot in the array .

 

[Defi4nc3]

Looks ugly going through so many different temp unit groups variables...

If your clean you would keep a reference sheet of where you use each different array that way you can just use one variable for every unit group function you need...

Unless of course if you wanna keep making variables... It would look hectic by the time you finish any big project...

 

[WolfieeifloW]

This is true.
You could also use JASS FTW .