[Solved] [Lua] Array of Units issue

[iown_azz]

I am trying to have an array of units that I made manually but it doesn't work. You guys are my last hope!

I tried nextCirc = {"gg_unit_gcop_0039", "gg_unit_gcop_0042", "gg_unit_gcop_0044", "gg_unit_gcop_0049"}
and that without ""

function HeroTele()
        playerTele = {}
        nextCirc  = {"gg_unit_gcop_0039", "gg_unit_gcop_0042", "gg_unit_gcop_0044", "gg_unit_gcop_0049"}
        for i=1,15 do
            playerTele[i] = 0
        end
        local coP = GetUnitsOfPlayerAndTypeId(Player(PLAYER_NEUTRAL_PASSIVE), FourCC("bcop"))
        ForGroup(coP, CircleofPower)
        DestroyGroup(coP)
 end

function CircleofPower()
        local EnumcoP = GetEnumUnit()
        local coPCollision = CreateTrigger() 
        TriggerRegisterUnitInRangeSimple(coPCollision, 86, EnumcoP)
        TriggerAddAction(coPCollision, function() coPTele(EnumcoP) end)
end

function coPTele(a)
        local trigUnit =GetOwningPlayer(GetTriggerUnit()) 
        local playerId = GetPlayerId(trigUnit) + 1
        if trigUnit ~= Player(15) then
            SetUnitOwner(a, trigUnit, TRUE)
            local checkCircs = GetUnitsOfPlayerAndTypeId(trigUnit, FourCC("bcop"))
            if BlzGroupGetSize(checkCircs) >= 2 then
                ForGroup(checkCircs, copOwnerChange)
                DestroyGroup(checkCircs)
                if playerTele[playerId] == 0 then
                    SetUnitPositionLocFacingBJ(GetTriggerUnit(), GetRectCenter(GetUnitLoc(_G[string.format(nextCirc[playerId])]), 75))
                end
            end
        end
end

function copOwnerChange()
    SetUnitOwner(GetEnumUnit(), Player(PLAYER_NEUTRAL_PASSIVE), TRUE)
end

I tried to create the the array "nextCirc" without the quotes around the units and ran it without the _g[string.format and it didn't work.


Edit: I added GetRectCenter() infront of GetUnitLoc() but it sends me way offset

 

[pr114]

You need to remove the parantheses from gg_unit as these are unit variables.

_G[string.format(nextCirc[playerId])]

This part is not correct. What you are trying to do is find an object called 'nextCirc[1]' etc, which doesn't exist.
The statement nextCirc = {...} creates one list object called nextCirc.

You can find element in the nextCirc list like this:

nextCirc[playerId]

_G["object name"..Id] will only be useful when iterating over several named variables.


Also, you cannot pass a location to GetRectCenter.

 

[iown_azz]

You need to remove the parantheses from gg_unit as these are unit variables.

_G[string.format(nextCirc[playerId])]

This part is not correct. What you are trying to do is find an object called 'nextCirc[1]' etc, which doesn't exist.
The statement nextCirc = {...} creates one list object called nextCirc.

You can find element in the nextCirc list like this:

nextCirc[playerId]

_G["object name"..Id] will only be useful when iterating over several named variables.


Also, you cannot pass a location to GetRectCenter.

I made the changes but for nextCirc[playerId] I keep getting nil if I print it out or print out
GetUnitLoc(nextCirc[playerId])

and what do you mean by nextCirc[1] doesn't exist?

 

[Uncle]

playerTele = {} can be changed to playerTele = __jarray(0)

This gives it the default value of 0.

And you probably want to create the Array like this:
nextCirc = {}
nextCirc[1] = gg_unit_gcop_0039
nextCirc[2] = gg_unit_gcop_0042
nextCirc[3] = gg_unit_gcop_0044
nextCirc[4] = gg_unit_gcop_0049

What's the goal of this function anyway? I don't really get what you're trying to do.

And I THINK you need to replace TRUE with true:

SetUnitOwner(a, trigUnit, TRUE)
--That should be:
SetUnitOwner(a, trigUnit, true)

 

[pr114]

Are you sure the unit variables gg_unit_gcop_0039 etc are valid?

 

[iown_azz]

playerTele = {} can be changed to playerTele = __jarray(0)

This gives it the default value of 0.

And you probably want to create the Array like this:
nextCirc = {}
nextCirc[1] = gg_unit_gcop_0039
nextCirc[2] = gg_unit_gcop_0042
nextCirc[3] = gg_unit_gcop_0044
nextCirc[4] = gg_unit_gcop_0049

What's the goal of this function anyway? I don't really get what you're trying to do.

And I THINK you need to replace TRUE with true:

SetUnitOwner(a, trigUnit, TRUE)
--That should be:
SetUnitOwner(a, trigUnit, true)

Are you sure the unit variables gg_unit_gcop_0039 etc are valid?

I think Pr114 is right uncle, it is still nil.

But I am trying to get it so a unit has to enter range with 2 circle of powers and once both are touched, the unit teleport to the next respected one. But if you want to try my map be my guest. You basically helped me with a lot of triggers so you are going to get credit on my map. Don't hate tho because it is a maze... This is my first map since 2010-2011 and I'm trying to learn LUA as I go.

 

[IcemanBo]

I think Pr114 is right uncle, it is still nil.

Is the problem the ggs don't exist?
The gg_s are only created when they were used in GUI at least once. When does Editor create or remove gg (generated global variables)
If you want access the preplaced unit vars without GUI, you could generate the globals yourself using W3X Tools.

 

[iown_azz]

Is the problem the ggs don't exist?
The gg_s are only created when they were used in GUI at least once. When does Editor create or remove gg (generated global variables)
If you want access the preplaced unit vars without GUI, you could generate the globals yourself using W3X Tools.

that most likely be that... If i make a trigger with no event, would it still work just calling those units for anything?

or is there another way?

 

[IcemanBo]

You ask if gg's get created if one trigger without an event is created? I'm not sure I understand, can you elaborate?
But the answer is no, I guess, as long as there's no GUI reference, see the link above.

or is there another way?

I linked to w3x tools, which you could use to generate those globals.

 

[iown_azz]

You ask if gg's get created if one trigger without an event is created? I'm not sure I understand, can you elaborate?
But the answer is no, I guess, as long as there's no GUI reference, see the link above.


I linked to w3x tools, which you could use to generate those globals.

So I did some testing, GUI trigger needs to be triggered and if you delete the GUI trigger, it no longer exist.

Also, W3X doesn't have a Lua option so ;(

 

[Uncle]

Can't you just store the units in an Array (using GUI variables) and reference them in your script.

 

[iown_azz]

Can't you just store the units in an Array (using GUI variables) and reference them in your script.

Does it need anything special or can it be just units[1]

in Lua that is if I make the variable in GUI

Edit: the answer to my question is udg_

 

[IcemanBo]

Also, W3X doesn't have a Lua option so ;(

Plain text is created, it's not JASS specific. It works depending on input. If you tell me what output code you need exactly, I could help.

So I did some testing, GUI trigger needs to be triggered and if you delete the GUI trigger, it no longer exist.

So what I tested in the linked thread is wrong, that reference is kept if it's being used in code at same time?

 

[iown_azz]

So what I tested in the linked thread is wrong, that reference is kept if it's being used in code at same time?

I don't know if it is a JASS thing or what but I tried however I did not save the map upon testing so it might be invalid.

I put it in a gui variable and called it with udg_

 

[pr114]

A preplaced unit is only bound to a 'gg_unit' variable' if it is referenced in one of your world editor Unit variables.

Oops, saw this mentioned earlier in this topic.

 

[IcemanBo]

not entirely

When does Editor create or remove gg (generated global variables)

 

[iown_azz]

What's the goal of this function anyway? I don't really get what you're trying to do.

this is what I am trying to accomplish...

well what I accomplished.... but unit doesn't turn instantly if you know any way around that issue?

My Code

function HeroTele()
        playerTele = {}
        for i=1,15 do
            playerTele[i] = 1
        end
        local coP = GetUnitsOfPlayerAndTypeId(Player(PLAYER_NEUTRAL_PASSIVE), FourCC("bcop"))
        ForGroup(coP, CircleofPower)
        DestroyGroup(coP)
end

function CircleofPower()
        local EnumcoP = GetEnumUnit()
        local coPCollision = CreateTrigger()
        TriggerRegisterUnitInRangeSimple(coPCollision, 86, EnumcoP)
        TriggerAddAction(coPCollision, function() coPTele(EnumcoP) end)
end

function coPTele(a)
        local trigUnit =GetOwningPlayer(GetTriggerUnit())
        local playerId = GetPlayerId(trigUnit) + 1
        local trigUnitTele = playerTele[playerId]
        if trigUnit ~= Player(15) then
            print(GetUnitLoc(udg_TeleCOP[playerId]))
            SetUnitOwner(a, trigUnit, TRUE)
            local checkCircs = GetUnitsOfPlayerAndTypeId(trigUnit, FourCC("bcop"))
            if BlzGroupGetSize(checkCircs) >= 2 then
                ForGroup(checkCircs, copOwnerChange)
                DestroyGroup(checkCircs)
             
                if playerTele[playerId] <=5 then
                    SetUnitFacingTimed(GetTriggerUnit(), 270, 0.00)
                    SetCameraPositionLocForPlayer(trigUnit, GetUnitLoc(udg_TeleCOP[trigUnitTele]))
                    SetUnitPositionLoc(GetTriggerUnit(), GetUnitLoc(udg_TeleCOP[trigUnitTele]))
                    ReviveHeroLoc(GetTriggerUnit(), GetUnitLoc(udg_TeleCOP[trigUnitTele]), false)
                    playerTele[playerId] = playerTele[playerId] + 1
                end
            end
        end
end

function copOwnerChange()
    SetUnitOwner(GetEnumUnit(), Player(PLAYER_NEUTRAL_PASSIVE), TRUE)
end

and TRUE works just so you know and also false and FALSE. I don't think Lua cares

playerTele = __jarray(0)

I didn't try this as well because I would have to change the whole +1 and array[1] to array[0]

 

[Uncle]

I believe they added a make unit face angle instantly function.

And i'm not sure what you mean about changing array[1] to array[0]. All __jarray(0) does is create a table (just like using Var = {}) with the added benefit of setting the default value of all Elements in that table to your listed value. You can set it to any value you want meaning it doesn't have to be an Integer either.

Example:
SomeBoolean = __jarray(false)

Will result in:
SomeBoolean[1] = false
SomeBoolean[2] = false
SomeBoolean[3] = false
etc...

Otherwise, their default values would be Nil.

 

[iown_azz]

I believe they added a make unit face angle instantly function.

And i'm not sure what you mean about changing array[1] to array[0]. All __jarray(0) does is create a table (just like using Var = {}) with the added benefit of setting the default value of all Elements in that table to your listed value. You can set it to any value you want meaning it doesn't have to be an Integer either.

Example:
SomeBoolean = __jarray(false)

Will result in:
SomeBoolean[1] = false
SomeBoolean[2] = false
SomeBoolean[3] = false
etc...

Otherwise, their default values would be Nil.

okay that good to know I can put any value in it.... I just thought it was 0 but that makes it much easier and faster game wise

ps: im talking out my ass

BlzSetUnitFacingEx(GetTriggerUnit(), 270)
is the trigger