[JASS] Remove Location error

[CodeBlack]

...Hey guys...i just made a triggered spell, but my editor doesnt like this line:

• Custom script: call RemoveLocation (udg_LocC)

When i try to save the map it always says "expected name".....

does someone know how to fix this?

 

[OmagnaT]

Post your map

 

[Vinz]

Remove the space between RemoveLocation and the parentheses.

 

[OmagnaT]

I thought the same thing, but tested it with the space and ran no error

 

[PurgeandFire]

Show the full trigger, or post a screenshot of the error. I'm guessing your variable is not named LocC. Either that, or it is a different line that is the problem.

 

[CodeBlack]

Show the full trigger, or post a screenshot of the error. I'm guessing your variable is not named LocC. Either that, or it is a different line that is the problem.

It is definetly this line, as the error says line XYZ and marks the line in another colour. Anyways...here are the triggers:

• Staff of Lightning
  • Events
    • Unit - A unit Is attacked
  • Conditions
    • ((Attacking unit) has an item of type Staff of Lightning) Equal to True
  • Actions
    • If (All Conditions are True) then do (Then Actions) else do (Else Actions)
      • If - Conditions
        • (Random integer number between 1 and 100) Less than or equal to 25
      • Then - Actions
        • Set locC = (Position of (Attacking unit))
        • Unit - Create 1 Dummy for (Owner of (Attacking unit)) at locC facing (Position of (Attacking unit))
        • Unit - Add a 2.00 second Generic expiration timer to (Last created unit)
        • Unit - Order (Last created unit) to Orc Far Seer - Chain Lightning (Attacked unit)
        • Custom script: call RemoveLocation(udg_LocC)
      • Else - Actions
        • Do nothing

and here is the Error:

 

[Vinz]

You are capitalizing "LocC" when your global variable name is "locC".

Also, on a side note, do not use the "Do Nothing" action. Just leave it empty.

 

[deathismyfriend]

change this.

• Custom script: call RemoveLocation(udg_LocC)

to this

• Custom script: call RemoveLocation(udg_locC)

Also you have to change this (Position of (Attacking unit)) to the variable.

Also remove the do nothing action.

 

[Vinz]

Wait, creating a unit at loc facing the same loc would result in weird behavior. I suggest (Unit - Facing angle of Attacking Unit). Also cache attacking unit into a variable since you use it more than once.

 

[OmagnaT]

Wait, creating a unit at loc facing the same loc would result in weird behavior. I suggest (Unit - Facing angle of Attacking Unit)..

it's a dummy unit, i dont think that matters, i always use bj_UNIT_FACING

 

[Vinz]

IIRC 0. is better than bj_UNIT_FACING since it is faster. But they lead to the same angle. It's weird that he doesn't. o.o

 

[CodeBlack]

Damn...didnt notice that XD

L or l

THX

EDIT: dont get everything of what you are talking about...to what should i change (Attacking Unit) ??

 

[Vinz]

Cache it into a unit varìable since you use it morè than once. Vars are faster than functions, like GetAttacker.

 

[CodeBlack]

Cache it into a unit varìable since you use it morè than once. Vars are faster than functions, like GetAttacker.

ok thx =D