[Solved] Loop Location Custom Script Error

[CroMoX]

Hey guys I'm trying to make a trigger where I loop some point variables so I don't have to make 4 triggers, simplifying it down to one trigger. However I get an error every time saying "Expected Name". I have tried to fix the text in the custom script but I still get the error. I even used my own integer variable for it.

Trigger

• Spawn 1
  • Events
    • Time - Every 1.00 seconds of game time
  • Conditions
  • Actions
    • For each (Integer A) from 1 to 4, do (Actions)
      • Loop - Actions
        • Set Arena1Loc[(Integer A)] = (Center of Arena1[(Integer A)])
        • Unit - Create 1 Swordsman for Neutral Hostile at Arena1Loc[(Integer A)] facing Arena1Loc[0]
        • Unit - Order (Last created unit) to Attack-Move To Arena1Loc[0]
        • Custom script: call RemoveLocation(udg_Arena1Loc[Integer A])

I don't see the problem.

Also I tried adding the brackets on the inside of the [] but it still gives me an error.

 

[Doomlord]

It should be

• Custom script: call RemoveLocation(udg_Arena1Loc[bj_forLoopAIndex])

 

[CroMoX]

Oh ok I will try in a sec. Thanks for showing me that I have never had to use loc or groups within a loop before

 

[Doomlord]

Oh ok I will try in a sec. Thanks for showing me that I have never had to use loc or groups within a loop before

No problem. Still, it is advised that you use your own unique integer variable for each looping because looping with integer A/B can cause issues sometimes due to them being global variables.

 

[CroMoX]

No problem. Still, it is advised that you use your own unique integer variable for each looping because looping with integer A/B can cause issues sometimes due to them being global variables.

Yeah I was aware about that issue, and I was planning on it, cause I will be using this for several things (dungeons), and don't want units of a stronger type spawning in the wrong dungeon

Anyway thank you for expanding and clarifying.

 

[jonhysone]

If Integer A ===> [bj_forLoopAIndex]
If Integer B ===> [bj_forLoopBIndex]
If A Variable Integer ===> [udg_'Name_Of_Variable']

Also BJs are slow and inefficient such as Integer A/B. Don't use them . Try to use A variable integer as possible .

 

[CroMoX]

If Integer A ===> [bj_forLoopAIndex]
If Integer B ===> [bj_forLoopBIndex]
If A Variable Integer ===> [udg_'Name_Of_Variable']

Also BJs are slow and inefficient such as Integer A/B. Don't use them . Try to use A variable integer as possible .

Yeah I was aware about that issue, and I was planning on it

____