[Trigger] Point Array Destroing Help?

[stonneash]

i need you help in a Spell:

• Events
  • Unit - A unit starts the effect of an ability
• Conditions
  • (Ability being cast) Equal to Arcane Circle
• Actions
  • ....
  • ....
  • Set PS_Caster_Position[PS_Intger]
  • ....
  • ....
  • ....
  • Custom Script: call RemoveLocation(udg_PS_Caster_Postion[bj_PS_IntgerIndex])

The " Custom Script: call RemoveLocation(udg_PS_Caster_Postion[bj_PS_IntgerIndex])"Doesn't Work What to do??

 

[Arhowk]

not nearly enough information.
also, PS_intger isnt a BJ..
it should be udg_PS_Intger

 

[Chaosy]

As I wrote in your pm it seems that BJ_INTEGER_VARIABLE should be udg_INTEGER_VARIABLE.

also when you post triggers use the copy as text when right clicking on the trigger you want to show

 

[defskull]

I think he should be preached about using "Temp" variables.

 

[Almia]

bj are variables made by Blizzard.

So examples of these are bj_lastCreatedUnit w/c is a unit variable or bj_wantDestroyGroup is boolean variable.Because you are using your own variable,it should be named udg_YOUR_VARIABLE.

I think you are confused w/ bj_forLoopAIndex and bj_forLoopBIndex

 

[ap0calypse]

I think he should be preached about using "Temp" variables.

No! They should be "temp" variables.
Would be cool if there were some kind of naming convention for GUI, because right now it's pretty horrible. So many variables that could all be merged.

 

[defskull]

I hate camel-casing, no !

 

[ap0calypse]

I hate camel-casing, no !

To be completely fair, you're the one who's doing 100% camel-casing (since the beginning of each word has to be a capital letter, but my "temp" is lower-case).
So for a regular variable like "tempInt", it's only 50% camel-cased. "TempInt" is 100% Camel-cased.

(And you know... standard naming conventions for variables).

 

[Dr Super Good]

All caps is a constant
THIS_IS_A_CONSTANT

Normal global variables take naming similar to functions.
ThisIsAGlobal

Local variables are all lower case.
thisisalocal

 

[defskull]

You heard what the Dr. said apo ?
You can't fight with the Doctor.

 

[Almia]

^ right

 

[ap0calypse]

I personally don't like all-lower for local variables.
I'll give in to upper-camel case for globals (I'll go change my systems if anyone needs 'em), but locals have to be lower-camel case for me.

All-caps with underscore for constants is a given.
Upper-camel case for functions as well.

This is all off-topic though (maybe create another topic for this?)