My village idiot =/

[WastedSavior]

Okay, I have a prob, I have a trigger, that spawns villagers this -


Villager 2
Events
Time - Every 50.00 seconds of game time
Conditions
Actions
Unit - Create 5 Villager (Male) for Player 10 (Light Blue) at (Random point in Rect 001 <gen>) facing (Random angle) degrees


It works like a CHARM! but they started building up.. so i added this trigger

Limit Copy
Events
Conditions
(Number of units in (Units in Rect 001 <gen>)) Equal to 10
Actions
Unit - Remove (Last created unit) from the game
Unit - Remove (Last created unit) from the game
Unit - Remove (Last created unit) from the game
Unit - Remove (Last created unit) from the game
Unit - Remove (Last created unit) from the game

Is there something wrong with my second trigger? cuz it doesn't seem to be doing its job...

 

[ThySpektre]

Yes, your trigger has no event and therefore it never runs.

Make it a Periodic Event (under time) that also runs every 50 seconds...

...or better yet take your condition from the 2nd trigger and put it in your first trigger and do away with the 2nd trigger all together.

Also I would change the test to greater than or equal to 10 instead of just equal to.

ThySpektre

 

[WastedSavior]

if i change it to greater than or equal to, do i only have to do one "remove last placed unit" ? or still all 5 of them on same trigger?

 

[AgentPaper]

(Last created unit) is one unit, removing it 5 times does nothing. ike he said, add a condition to the first trigger
Number of units in (spawn region) less than 10

 

[WastedSavior]

His thing is all and well, but they still are building up.. it goes from 5.. to 10.. to 14..

 

[WastedSavior]

Anyways, if i combine remove, and spawn, Then it'll need 10 units before it'll start spawning villagers.. kinda a contridiction no?

Limit
Events
Time - Every 50.00 seconds of game time
Conditions
(Number of units in (Units in Rect 001 <gen>)) Greater than or equal to 10
Actions
Unit - Remove (Last created unit) from the game


Villager
Events
Time - Every 50.00 seconds of game time
Conditions
Actions
Unit - Create 1 Villager (Male) for Player 10 (Light Blue) at (Random point in Rect 001 <gen>) facing (Random angle) degrees

is what i have right now.. i have 5 diff villager spawns, so there more random in the rect, then i have the Limit..

 

[AgentPaper]

What triggers do you have now? and don't double post, edit them.

 

[UnknowVector]

You need to use If/Then/Else to detect if you want villegers to spawn, "Last Created Unit" isn't particarly reliable anyway becuase it would destroy any unit last made, it would need to be in the trigger with the made unit and no wait in between for it to be usefull.

EDIT: Il'd do it like this in bold means its a vaule you change, i knocked out the leak otherwise you have 5 point leaks everytime it spawns units.

You need

TempPoint - Point Array Size <however many villgers you want to spawn every 5 seconds
TempPointIndex - Integer

but "Integer A 1 - 5" things need to have the same end vaule

Untitled Trigger 001
    Events
        Time - Every [B]5.00[/B] seconds of game time
    Conditions
    Actions
        If (All Conditions are True) then do (Then Actions) else do (Else Actions)
            If - Conditions
                (Number of units in (Units in ([B]Playable map area[/B]))) Less than or equal to [B]10[/B]
            Then - Actions
                For each (Integer A) from 1 to [B]5[/B], do (Actions)
                    Loop - Actions
                        Set TempPoint[TempPointIndex] = (Random point in ([B]Playable map area[/B]))
                        Unit - Create 1 [B]Villager[/B] (Male 2) for [B]Player 1 (Red)[/B] at TempPoint[TempPointIndex] facing ([B]Random angle[/B]) degrees
                        Set TempPointIndex = (TempPointIndex + 1)
                Set TempPointIndex = 0
                For each (Integer A) from 1 to [B]5[/B], do (Actions)
                    Loop - Actions
                        Custom script:   call RemoveLocation( udg_TempPoint[udg_TempPointIndex] )
                        Set TempPointIndex = (TempPointIndex + 1)
            Else - Actions
                Do nothing

 

[AgentPaper]

Why are you using arrays and custom script? just have 1 variable, TempPoint, and in one loop set it, then spawn the unit at the point. No leaks.

 

[WastedSavior]

Seriously.. someones an over achiever Oo