[Solved] C# Tile array trouble

Natey214

I am creating an RTS and am trying to write a method to check the position of each tile within an array, and whether it is occupied. hasBuilding is a public boolean declared in the tile's script. I get errors at the array value in the if and return statements that Tile cannot be converted to an int. I feel like I need a new approach but I'm stumped.

	using Godot;
	using System;

	public class Map : Node
	{
		Tile[] allTiles;
		Tile[] tilesWithBuildings;
		float tileSize = 32.0f;
	

		public override void _Ready()
		{

		}

		Tile get_tile_at_position (int position)
		{

			foreach(Tile x in allTiles)
			{
				if (allTiles[x].Position == position && allTiles[x].hasBuilding == false)
				return allTiles[x];
			}
			return null;
			}

xyz

Inside foreach loop x is a reference to an actual Tile object, it's not an index.

Natey214

@xyz said: Inside foreach loop x is a reference to an actual Tile object, it's not an index.

Correct, for each tile object I am checking and returning the Tile.Position and whether the hasBuilding boolean is false.

xyz

Yes, but in the loop body you're treating it as it was an index, which is expected to be of integer type not of Tile type, hence the error. x should be directly used as a Tile object. When you iterate through a collection in this manner your iterator variable represents the object itself, not its index. Note that you even declared it as such in the foreach statement (Tile x)

foreach(Tile x in allTiles){
	if (x.Position == position && x.hasBuilding == false){
		 return x;
	}
}

Of course, some name other than x would be more suitable here.

Natey214

Ah, thank you! This also revealed that I declared the wrong parameter in the method as well, should have been a Vecor2.