Creep respawn trigger

[TfauxT]

I can't get monsters to respawn correctly, heres the trigger I'm using.

function Trig_Unit_respawnsNORM_Copy_Conditions takes nothing returns boolean
    if ( not ( IsUnitType(GetDyingUnit(), UNIT_TYPE_HERO) == false ) ) then
        return false
    endif
    return true
endfunction

function Trig_Unit_respawnsNORM_Copy_Actions takes nothing returns nothing
    set udg_MobSpawn = GetUnitTypeId(GetDyingUnit())
    call TriggerSleepAction( 230.00 )
    call CreateNUnitsAtLoc( 1, udg_MobSpawn, Player(11), GetUnitLoc(GetDyingUnit()), bj_UNIT_FACING )
endfunction

//===========================================================================
function InitTrig_Unit_respawnsNORM_Copy takes nothing returns nothing
    set gg_trg_Unit_respawnsNORM_Copy = CreateTrigger(  )
    call TriggerRegisterPlayerUnitEventSimple( gg_trg_Unit_respawnsNORM_Copy, Player(11), EVENT_PLAYER_UNIT_DEATH )
    call TriggerAddCondition( gg_trg_Unit_respawnsNORM_Copy, Condition( function Trig_Unit_respawnsNORM_Copy_Conditions ) )
    call TriggerAddAction( gg_trg_Unit_respawnsNORM_Copy, function Trig_Unit_respawnsNORM_Copy_Actions )
endfunction

For GUI users, this trigger would like this:
Trigger - Unit owned by player 12 brown dies
Condition - dying unit is a hero equal to false
Actions- set MobSpawn= unit type of dying unit.
wait 150 seconds
Unit- create 1 Mobspawn at position of dying unit.

Any assistance is appreciated

 

[Ghan]

You probably want a local variable.
Example:

function Trig_Unit_respawnsNORM_Copy_Actions takes nothing returns nothing
    local unittype MobSpawn = GetUnitTypeId(GetDyingUnit())
    local location point = GetUnitLoc(GetTriggerUnit())
    call TriggerSleepAction( 230.00 )
    call CreateNUnitsAtLoc( 1, MobSpawn, Player(11), point, bj_UNIT_FACING )
    call RemoveLocation(point)
    set point = null
endfunction

 

[TfauxT]

Thank you Ghan, again you are so helpful
I will try that out and see if it functions the way I want it to. Thank you.

Is there a command to reset/delete a variable after it has been used?

 

[Ghan]

> Is there a command to reset/delete a variable after it has been used?

Depends on the type of variable.
What are you trying to do?

 

[TfauxT]

To make things quick, this is what I'm trying to do with my creeps:

Mob Respawn
    Events
        Unit - A unit owned by Player 12 (Brown) Dies
    Conditions
        ((Dying unit) is A Hero) Equal to False
    Actions
        Set MobSpawn = (Unit-type of (Dying unit))
        Set DeathPoint = (Position of (Dying unit))
        Wait 150.00 seconds
        Unit - Create 1 MobSpawn for Player 12 (Brown) at DeathPoint facing Default building facing degrees

The problem is I do not know how to reset or remove the variables after the trigger is done, so it ends up repsawning the same mobs over and over at each point you kill a new mob. I'm new to jass and the like, so sorry if this is such a simple fix

 

[Ghan]

That's why you use a local variable.
Local variables are only used in the trigger in which they are created.
So, there will be no confusion.
Of course, the easiest thing to do would be to put the

Set MobSpawn = (Unit-type of (Dying unit))

after the wait like this:

Wait 150.00 seconds
Set DeathPoint = (Position of (Dying unit))
Set MobSpawn = (Unit-type of (Dying unit))
Unit - Create 1 MobSpawn for Player 12 (Brown) at DeathPoint facing Default building facing degrees

 

[TfauxT]

Oh! so the local variables must be after the wait for it to work correct? :banghead: I did not think the position would cause such a fuss.

 

[Ghan]

> so the local variables must be after the wait for it to work correct?

No.
But that variable in the GUI version was not local.
If it were, it wouldn't matter where you put it.
The reason you have a problem is because that was a global variable.

 

[TfauxT]

Oh, then how would I create a local variable?

 

[Ghan]

> how would I create a local variable?

In JASS.
Or, if you insist, with a Custom Script:

Custom Script: local unit TempUnit

Note, however, that a local variable will be unusable in GUI events, conditions and actions.

 

[TfauxT]

thank you for all the help today Ghan