Is it possible to get the TileSet Source Id by tile's name?

Help Programming
, , ,
1

Godot Version

4.4

Question

All my tiles have a name specified. Right now I’m getting the source id with a for loop, but in the back of my mind I’m thinking that this could be a lot faster and more “correct” than this.

The relevant code:

var source_id := get_or_create_tile_source(tile_data["id"])
set_cell(cell_pos, source_id, Vector2i.ZERO, transforms)

func get_or_create_tile_source(tile_id: String) -> int:
	for source_id in tile_set.get_source_count():
		if tile_set.get_source(source_id).resource_name == tile_id:
			return source_id
			
	return 0
2

If you need a quick map of names to ids, my suggestion would be a dictionary:

var TileNameToID = {}

func init_tile_mapping():
    for sid in tile_set.get_source_count():
        TileNameToID[tile_set.get_source(sid).resource_name] = sid

Once you’ve done that, TileNameToID["foo"] will give you the id for "foo", and you can do things like existance checks with TileNameToID.has("thing").

That said, strings tend to be slow, so if you can use an enum instead to name your tiles, you’d probably find that faster and cleaner.

1 Like
3

Hmm I thought about this approach before aswell, it must be the best I can do right now. Are there any actual good reasons for why godot doesn’t have a “get_source_id(“name”)”?

4

I can’t speak for the devs, but my suspicion is that there isn’t really a common use case for that. Given the source IDs are integer, unless you’re loading these names at runtime it probably makes more sense to just name the integers, either via const or enum. Most people are probably expected to either use baked tilemaps, or generate them procedurally; the former case doesn’t need names, and in the later case it’s more efficient to work with integers.

1 Like