[JASS] Simple question

[CrueIIntentions]

Im experimenting with natives; Can anyone explain what went wrong with this one please?

function Trig_Map_Initialization_Resort takes nothing returns nothing
    local location t
    local integer i = 0
    loop
      exitwhen i>11
      set t = GetPlayerStartLocation(Player(i))
      call CreateNUnitsAtLoc( 3, 'h002', Player(i), t, 0.00 )
      call RemoveLocation(t)
    endloop
endfunction

Im new at this. Please bear with me.

 

[Axarion]

You need to increase your loop variable.

function Trig_Map_Initialization_Resort takes nothing returns nothing
    local location t
    local integer i = 0
    loop
      set t = GetPlayerStartLocation(Player(i))
      call CreateNUnitsAtLoc( 3, 'h002', Player(i), t, 0.00 )
      call RemoveLocation(t)
      exitwhen i >= 11
      set i = i + 1
    endloop
endfunction

also you should learn to use coordinates instead of locations, because they are faster

 

[CrueIIntentions]

I would love to learn how to use coordinates more appropriately. However, when I use the jass script you just gave me it syntaxed on:

function Trig_Map_Initialization_Resort takes nothing returns nothing
    local location t
    local integer i = 0
    loop
      set t = GetPlayerStartLocation(Player(i)) //This part
      call CreateNUnitsAtLoc( 3, 'h002', Player(i), t, 0.00 )
      call RemoveLocation(t)
      exitwhen i >= 11
      set i = i + 1
    endloop
endfunction

 

[Axarion]

function Trig_Map_Initialization_Resort takes nothing returns nothing
    local location t
    local integer i = 0
    local real x
    local real y
    loop
        set x = GetStartLocationX(GetPlayerStartLocation(Player(i)))
        set y = GetStartLocationY(GetPlayerStartLocation(Player(i)))
        
        call CreateUnit(Player(i), 'h002', x, y, 0)
        call CreateUnit(Player(i), 'h002', x, y, 0)
        call CreateUnit(Player(i), 'h002', x, y, 0)
        
        call RemoveLocation(t)
        exitwhen i >= 11
        set i = i + 1
    endloop
endfunction

Thats it with coordinates. It gave you a syntax because of your own fail.

GetPlayerStartLocation(Player(i)) returns an integer and not a location

 

[jk2pach]

function Trig_Map_Initialization_Resort takes nothing returns nothing
    local integer i = 0
    local integer j = 0
    local real x = 0.
    local real y = 0.
    loop
      exitwhen i>11
      set x = GetStartLocationX(GetPlayerStartLocation(Player(i)))
      set y = GetStartLocationY(GetPlayerStartLocation(Player(i)))
      set j = 0
      loop
          exitwhen j > 2
          call CreateUnit(Player(i), 'h002', x, y, 0)
          set j = j + 1
      endloop
      set i = i + 1
    endloop
endfunction

 

[CrueIIntentions]

Thanks alot. Coordinates seem the way to go. +rep