Hey im looking for some help on triggers

[ZKAmedic]

i need a trigger that will make it, so anytime that i build a building (say a baracks) it will spawn a unit every 10 seconds?

 

[GhostThruster]

• Spawn
  • Events
    • Time - Every 10.00 seconds of game time
  • Conditions
  • Actions
    • Custom script: set bj_wantDestroyGroup = true
    • Unit Group - Pick every unit in (Units of type Barracks) and do (Actions)
      • Loop - Actions
        • Set temp_point = (Position of (Picked unit))
        • Unit - Create 1 Footman for (Owner of (Picked unit)) at temp_point facing Default building facing degrees
        • Custom script: call RemoveLocation(udg_temp_point)

 

[ZKAmedic]

thanks mate

 

[ZKAmedic]

• Spawn
  • Events
    • Time - Every 10.00 seconds of game time
  • Conditions
  • Actions
    • Custom script: set bj_wantDestroyGroup = true
    • Unit Group - Pick every unit in (Units of type Barracks) and do (Actions)
      • Loop - Actions
        • Set temp_point = (Position of (Picked unit))
        • Unit - Create 1 Footman for (Owner of (Picked unit)) at temp_point facing Default building facing degrees
        • Custom script: call RemoveLocation(udg_temp_point)

hey mate
how do i create this? Set temp_point = (Position of (Picked unit))
im currently testing this out for lag reasons, an yeah i cant find this command?

 

[GhostThruster]

Create an action, then find 'Set Variable'. It is near the top of the list.

 

[ZKAmedic]

i found it it looks like this.
Set Variable = Value
??

 

[AL0NE]

go to trigger editor, click the yellow X icon. create a new variable called temp_point. then set temp_point = (position of (picked unit)).

 

[ZKAmedic]

Custom script: set bj_wantDestroyGroup = true
This is picked up as a unknown name??
bj_wantDestroyGroup = true

 

[GhostThruster]

Use Custom Script. It is also an option, nearby 'Set Variable'.

 

[ZKAmedic]

function InitGlobals takes nothing returns nothing
endfunction

function Trig_Melee_1_spawn_Func002A takes nothing returns nothing
set udg_temp_point = GetUnitLoc(GetEnumUnit())
call CreateNUnitsAtLoc( 1, 'h002', Player(0), GetRectCenter(GetPlayableMapRect()), bj_UNIT_FACING )
call RemoveLocation(udg_temp_point)
endfunction

function Trig_Melee_1_spawn_Actions takes nothing returns nothing
set bj_wantDestroyGroup = True
call ForGroupBJ( GetUnitsOfTypeIdAll('h003'), function Trig_Melee_1_spawn_Func002A )
endfunction

//===========================================================================
function InitTrig_Melee_1_spawn takes nothing returns nothing
set gg_trg_Melee_1_spawn = CreateTrigger( )
call TriggerRegisterTimerEventPeriodic( gg_trg_Melee_1_spawn, 10.00 )
call TriggerAddAction( gg_trg_Melee_1_spawn, function Trig_Melee_1_spawn_Actions )
endfunction

This is the error i am getting

 

[GhostThruster]

What version is your Wc3?

 

[TheWarchief]

Why not just like this:
[trigger=Barracks Spawn]Barracks Spawn
Events
Time - Every 10.00 seconds of game time
Conditions
Actions
Unit Group - Pick every unit in (Units in (Playable map area) matching ((Unit-type of (Matching unit)) Equal to Barracks)) and do (Actions)
Loop - Actions
Unit - Create 1 Footman for (Owner of (Picked unit)) at (Position of (Picked unit)) facing Default building facing degrees[/trigger]

That creates a footman every 10 second at the feet of any barracks in the map for the player who owns the barracks.

 

[TKF]

Why not just like this:
[trigger=Barracks Spawn]Barracks Spawn
Events
Time - Every 10.00 seconds of game time
Conditions
Actions
Unit Group - Pick every unit in (Units in (Playable map area) matching ((Unit-type of (Matching unit)) Equal to Barracks)) and do (Actions)
Loop - Actions
Unit - Create 1 Footman for (Owner of (Picked unit)) at (Position of (Picked unit)) facing Default building facing degrees[/trigger]

That creates a footman every 10 second at the feet of any barracks in the map for the player who owns the barracks.

That is fail, you create 2 leaks every time the trigger runs. Creating a unit at position creates a leak, so you need to appoint a temppoint variable and destroy it afterwards to null the leak. Also there is a leak with unit group.

 

[GhostThruster]

If you want to know what leaks, are, look at the tutorials.
Otherwise all I can tell you is that leaks are bad.

 

[ZKAmedic]

yeah i understood what leaks are
unfortunatly my version of wc3 is like the most basic i think, i havnt updated it ever.
i think im just going to make the map and let someone put the triggers in,

 

[GhostThruster]

Why don't you just update? This is probably why the WE can't recognise "set bj_wantDestroyGroup = true".

 

[ZKAmedic]

yeah proberly good idea