[Solved] [LUA] Need a little help

[iown_azz]

So i am trying to remove every unit of Demon Hunter(Edem) of player slot that is unused. I know this can easily be done in the gui trigger but I am trying to learn LUA.

function HelloWorld()
    CheckPlayers3()
    CheckPlayers()  
    print("Hello World")
end
function CheckPlayers3()
    print("checking3")
end
function CheckPlayers()
    print("checking...")
    local playerUnit = GetUnitsOfTypeIdAll("Edem")
    local playerSlot = GetPlayerSlotState(GetOwningPlayer(GetEnumUnit()))
    print(playerUnit)
    ForGroupBJ(playerUnit, HeroKill(playerUnit))
  
end
function HeroKill(x)
if playerSlot == FASLE then
    print("var var 2")
        print(x)
        RemoveUnit(x)
  
end
end

I get all the print and group number/id, whatever it is. However, non of the units gets removed.

 

[Drake53]

Use [code=lua] so your code is easier to read.

 

[Uncle]

In Lua you need to use the prefix: FourCC()
Before typing your code

CreateUnit(Player(0), FourCC("H000"), 0, 0, 0) --This creates a Paladin for Player 1

 

[iown_azz]

In Lua you need to use the prefix: FourCC()
Before typing your code

CreateUnit(Player(0), FourCC("H000"), 0, 0, 0) --This creates a Paladin for Player 1

I see but it didn't help... are there any good tutorials or documentation on LUA for warcraft 3? I couldn't find any...

 

[TriggerHappy]

I see but it didn't help... are there any good tutorials or documentation on LUA for warcraft 3? I couldn't find any...

Any standard Lua tutorial should apply to Warcraft III as well. You can look at the common.j if you don't know how various natives work.

Your code doesn't make sense in many places. I figure you are basically trying to do this:

function CheckPlayers()
    ForGroup(GetUnitsOfTypeIdAll(FourCC("Edem")), HeroKill)
end

function HeroKill()
    local x = GetEnumUnit()
    local playerSlot = GetPlayerSlotState(GetOwningPlayer(x))
    if playerSlot == PLAYER_SLOT_STATE_EMPTY then
        RemoveUnit(x)
    end
end

 

[iown_azz]

Any standard Lua tutorial should apply to Warcraft III as well. You can look at the common.j if you don't know how various natives work.

Your code doesn't make sense in many places. I figure you are basically trying to do this:

function CheckPlayers()
    ForGroup(GetUnitsOfTypeIdAll(FourCC("Edem")), HeroKill)
end

function HeroKill()
    local x = GetEnumUnit()
    local playerSlot = GetPlayerSlotState(GetOwningPlayer(x))
    if playerSlot == PLAYER_SLOT_STATE_EMPTY then
        RemoveUnit(x)
    end
end

I was just plugging in different things to see if it worked....

But thanks, it worked! Ill do some more studying