[Trigger] Maze Trigger not working

[brad.dude03]

This is my trigger for my Maze that makes it so that if you step into certain area off the path, then you die. It's not working though and I don't see what's wrong. Can anyone help me out? Thanks.

• [LEFT]Kill
  • Events
    • Time - Every 0.05 seconds of game time
  • Conditions
  • Actions
    • Player Group - Pick every player in Noobs and do (Actions)
      • Loop - Actions
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • (Terrain type at (Position of (Picked unit))) Equal to Village - Thick Grass
          • Then - Actions
            • Unit - Kill (Picked unit)
          • Else - Actions
            • Do nothing[/LEFT]

I also tried this one. It didn't work either.

• Kill2
  • Events
    • Time - Every 0.05 seconds of game time
  • Conditions
  • Actions
    • Unit Group - Pick every unit in (Units in (Playable map area)) and do (Actions)
      • Loop - Actions
        • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
          • If - Conditions
            • And - All (Conditions) are true
              • Conditions
                • (Terrain type at (Position of (Triggering unit))) Equal to Village - Thick Grass
                • (Owner of (Triggering unit)) Not equal to Player 12 (Brown)
          • Then - Actions
            • Unit - Kill (Triggering unit)
          • Else - Actions
            • Custom script: call DestroyGroup(GetLastCreatedGroup())

 

[VampireHeart]

uh? y dont u make it simple and do this:

 

[brad.dude03]

Uh because it's a MAZE and I happen to have 392 regions. Do you want to do that trigger 392 times over?! That's what I thought.

 

[ragingspeedhorn]

• Events
  • Time - Every 0.12 seconds of game time
• Conditions
• Actions
  • Set tempGroup = (Units of type |cFF469191Random Person)
  • Unit Group - Pick every unit in tempGroup and do (Actions)
    • Loop - Actions
      • Set tempPoint = (Position of (Picked unit))
      • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
        • If - Conditions
          • ((Picked unit) is alive) Equal to True
          • Or - Any (Conditions) are true
            • Conditions
              • (Terrain type at tempPoint) Equal to Dalaran Ruins - Square Tiles
        • Then - Actions
          • Unit - Kill (Picked unit)
        • Else - Actions
      • Custom script: call RemoveLocation(udg_tempPoint)
  • Custom script: call DestroyGroup(udg_tempGroup)

 

[Tonks]

Or you could use that program that auto-enters the data. You enter a starting and ending number (Ex, region 1 - 999) and it creates a trigger that kills any unit that steps in them.

I forgot what it's called. Terrain type kill triggers work, too.

 

[ragingspeedhorn]

Regions take up more space, but besides from that so-so same result.

 

[HappyTauren]

picking unit group every 0.12 secounds is bad thing to do, indeed

 

[ragingspeedhorn]

No it isen't?

 

[brad.dude03]

No it isn't at all.... you want it to constantly be running through so that way the second someone steps off they die. This allows for more accuracy as well because it doesn't rely on regions that try to match the terrain. Thanks Rage!

 

[Tonks]

Picking unit groups generally causes a memory leak. If the trigger with the leak is running periodically, then yes, that's a very bad idea.
However, as far as I can tell, it doesn't leak, since he created and destroyed the variable.

 

[HappyTauren]

the fact is... each time you pick unit group it is cleared from memory if you use bj_wantDestroyGroup = true just before it. thing is... if it is runing every 0.12 and if that is battle net map, it is gonna lag, it even lags in single player if there are more then 50 units.

 

[ragingspeedhorn]

I have used this (and others aswell) in, as far as I know, 20-25 maps and it has never caused any lag what so ever.

 

[brad.dude03]

Yeah, using it in my map singleplayer and there's no lag at all.... you die instantly, no lag before you die, it's great.