[Trigger] How to destroy leak if i have UnitGroup[Array]

[IgneousRed]

Since i have UnitGroup(array[4])
This does not work:

• Custom script: call DeleteGroup(udg_UnitGroup)

also this:

• For each (Integer A) from 0 to 3, do (Actions)
  • Loop - Actions
  • Custom script: call DeleteGroup(udg_UnitGroup[GetForLoopIndexA])

than i tried:

• Custom script: call DeleteGroup(udg_UnitGroup[0])
• Custom script: call DeleteGroup(udg_UnitGroup[1])
• Custom script: call DeleteGroup(udg_UnitGroup[2])
• Custom script: call DeleteGroup(udg_UnitGroup[3])

and none of these work.
What is the correct way?

 

[Chaosy]

It's DestroyGroup not DeleteGroup

 

[IgneousRed]

Whoops

• For each (Integer A) from 0 to 3, do (Actions)
  • Loop - Actions
    • Custom script: call DestroyGroup(udg_T_US_TR_Selected([GetForLoopIndexA])

Error "Expected array index"

 

[Ardenian]

It has to be:

• Custom script: call DestroyGroup(udg_T_US_TR_Selected[GetForLoopIndexA])

This is very simple, maybe try to find it yourself next time.

In this case, you get the error message 'expected array index'.
So what does this mean ?
This means the array is not found although there has to be one, as the variable udg_T_US_TR_Selected uses an array.
If you take a close look yo see you added a not required '(' before '['.

 

[Wietlol]

isnt "GetForLoopIndexA" a function?
Thus: udg_T_US_TR_Selected[GetForLoopIndexA()]

 

[IgneousRed]

I see, you ware right Wietlol.
It looks like it is a function.
and ty Ardenian, both rep+

Just one last thing:
if in gui i have group 1,2,3,4
i have to write Clear(0,1,2,3)
everything starts from 0 right?

 

[Meatmuffin]

No, only player indexes start from 0 (Player 0 being Player (Red)).

If in GUI you have set up 1/2/3/4 unit group array indexes, then in JASS script you
also refer to them with the same indexes.

 

[Almia]

No, only player indexes start from 0 (Player 0 being Player (Red)).

Nope

basically, you just iterate the values of the arrays where you put the values at.

Arrays start at 0(indeed true) but still it depends on what index you started

 

[Meatmuffin]

It was literally the only thing I encountered in JASS, but yeah, your explanation is way more correct

 

[IgneousRed]

So in my case since gui declared UnitGroup as 1-4 i don't need to start with 0,
but since game players are always declared i need to start with 0.
Or what?

 

[Meatmuffin]

If you want to refer to Player 1 (Red) you refer to it as Player(0);
If you want to refer to Player 9 (Gray) you refer to it as Player(8).

Simple as that.

 

[Almia]

^ you don't do that in GUI lol

Player Red is still Player 1 in GUI. it is adjusted in GUI

//===========================================================================

function GetConvertedPlayerId takes player whichPlayer returns integer

    return GetPlayerId(whichPlayer) + 1

endfunction



//===========================================================================

function ConvertedPlayer takes integer convertedPlayerId returns player

    return Player(convertedPlayerId - 1)

endfunction

 

[Meatmuffin]

Oh, I thought he asked for it in JASS. My bad.

 

[IgneousRed]

I did
Ok ty both rep+
if you would like to know what i need it for check this https://www.youtube.com/watch?v=JyJfwCaB-V4

 

[Almia]

Nice

Reminds me of Mineralz

 

[IgneousRed]

"Mineralz"
Can you link that?
and ty
I plan to animate them further.
i will post in 1h i guess

 

[Almia]

http://www.hiveworkshop.com/forums/maps-564/mineralz-1-3-a-218690/?prev=search=mineralz&d=list&r=20

it also shows a range indicator and a custom fog (i think)

 

[IgneousRed]

Ups maybe another hour stay tuned

Added rainbow color
https://youtu.be/AYp5U1ZBYoE
How is it?
now effects this will take a long time

YT makes the color look bad :/

 

[Almia]

Please don't double-post, use the edit button next time

OT :
looks good, tho I can't see that "rainbow" effect that well, maybe you should slowdown the rainbow effect.

 

[IgneousRed]

YT is making it look bad
the rainbow resets after 3 sec
I'l make it 6