Storing all world tiles

[Dcrew]

I have this in my loop:

local tiles_refreshed
local function refresh_tiles()
	if tiles_refreshed then
		return
	end
	tiles = {}
	for x = 0, world_w do
		for y = 0, world_h do
			local tile = TheWorld.Map:GetTile(x, y)
			if tile ~= GROUND.IMPASSABLE and tile < GROUND.UNDERGROUND then
				if not tiles[tile] then
					tiles[tile] = {}
				end
				table.insert(tiles[tile], { tile_x = x, tile_y = y })
			end
		end
	end
	tiles_refreshed = true
end

I only call it when I need the tiles to save as much performance as I can, I also need to do this to get the number of each tile type in the world, and a random tile in the world of that type, so, for example, I can use this like so:

if tiles[GROUND.FOREST] > 10 then
  local idx = math.random(#tiles[GROUND.FOREST])
  local x, y = tiles[GROUND.FOREST][idx].tile_x, tiles[GROUND.FOREST][idx].tile_y
end

I need the number of tiles and a random tile for various reasons in my script.

So what I'm wondering is, if there's a way to optimize this, I was thinking to only call that at the start and then listen for a tile changed event (if there is one), is this possible?

Note: I am not fluent with DST API, Lua, DST Events (not creating them nor listening for them)

[Ultroman]

The map has a SetTile(x, y, tile) function, but it's not part of the code we have access to. You can try extending it to push an event on all the players or whatever prefab you have that needs information about it, but I don't think it'll work.

I'm not sure how to solve this one.

[zarklord_klei]

local _SetTile = Map.SetTile
function Map:SetTile(x, y, ground, ...)
	--do your proccessing to update your tile map here
	return _SetTile(self, x, y, ground, ...)
end

 

Edited July 30, 2019 by Zarklord

[Ultroman]

18 minutes ago, Zarklord said:

local _SetTile = Map.SetTile
function Map:SetTile(x, y, ground, ...)
	--do your proccessing to update your tile map here
	return _SetTile(self, x, y, ground, ...)
end

 

Yeah, that's what I suggested. Just wasn't sure we had access to extend it, since the original function isn't in the game files we have access to. Are you sure this works?

[zarklord_klei]

4 hours ago, Ultroman said:

Yeah, that's what I suggested. Just wasn't sure we had access to extend it, since the original function isn't in the game files we have access to. Are you sure this works?

Positive I do this all the time.

[Ultroman]

44 minutes ago, Zarklord said:

Positive I do this all the time.

That's awesome! Thanks for confirming! Ooooh, the power grows