[LUA] getFreeHouses

[Crypton3]

Simple function for getting all free houses. Works well with TFS 0.4. Can be useful in script that teleports player to random / first free house.

function getFreeHouses()
   local towns = getTownList()
   local houses = {}

   for i = 1, #towns do
     local list = getTownHouses(towns[i].id)
     for j = 1, #list do
       if list[j] then
         if getHouseOwner(list[j]) == 0 then
           table.insert(houses, list[j])
         end
       end
     end
   end

   if #houses == 0 then
     return nil
   end

   return houses
end

 

[Printer]

Nicely done

 

[WibbenZ]

Please learn how to tab..

function getFreeHouses()
    local towns = getTownList()
    local houses = {}
   
    for i = 1, #towns do
        local list = getTownHouses(towns[i].id)
        for j = 1, #list do
            if list[j] then
                if getHouseOwner(list[j]) == 0 then
                    table.insert(houses, list[j])
                end
            end
        end
    end
    if(#houses == 0) then
        return nil
    end
return houses
end

But nicely done, even tho Cykotitan made this a while ago, for donation castles.

 

[Ninja]

@WibbenZ

Is there any reason why you added redundant parentheses around #houses == 0?

 

[WibbenZ]

@WibbenZ

Is there any reason why you added redundant parentheses around #houses == 0?

Haha its just a thing I feel makes it easier to read, and according to Elf(somewhere on the forum - was some troll thread with cyko about making smaller scripts I think) he wrote that you should always use if() and not without brakets.

 

[Crypton3]

Please learn how to tab..

Fixed, in forgottenmapeditor we are using two spaces instead of tab and I forgot to change this option in notepad++