Merge branch 'SpartaqS-AddingSoldiersToSquad' into 'Development'

made usage of adding/removing soldiers to squad easier

See merge request gskowron/eopsy-lab-567-project!13

theProject/Assets/Scripts/Managers/SoldierSpawning.cs

@@ -0,0 +1,36 @@
+using System;
+using System.Collections;
+using System.Collections.Generic;
+using UnityEngine;
+
+//[RequireComponent]
+public class SoldierSpawning : MonoBehaviour
+{
+	[SerializeField] int spawnInterval = 1000; // ticks between spawning soldiers
+	[SerializeField] int lastSpawnTick = 0;
+	[SerializeField] Squad squad;
+	[SerializeField] TilemapManager tilemapManager;
+	[SerializeField] Vector2Int spawnCoords = Vector2Int.one * -1;
+	public Vector2Int GetSpawnCoords() { return spawnCoords; }
+	public void SetSpawnCoords(Vector2Int newSpawnCoords) { spawnCoords = newSpawnCoords; }
+
+	private void Awake()
+	{
+		squad = GetComponent<Squad>();
+		TickSystem.OnTick += HandleTick;
+		tilemapManager = FindObjectOfType<TilemapManager>();//DEPENDENCY_INJECTION
+	}
+
+	private void HandleTick(TickSystem.OnTickEventArgs tickEventArgs)
+	{
+		if (lastSpawnTick + spawnInterval > tickEventArgs.tickNumber)
+			return;
+
+		Soldier spawnedSoldier = tilemapManager.SpawnSoldier(spawnCoords.x, spawnCoords.y, squad.GetOwnTeam() == Entity.Team.Ally) as Soldier;
+		if (spawnedSoldier == null)
+			return;
+		// managed to spawn the soldier
+		lastSpawnTick = tickEventArgs.tickNumber;
+		squad.AddSoldierToSquad(spawnedSoldier);
+	}
+}

theProject/Assets/Scripts/Managers/SoldierSpawning.cs.meta

@@ -0,0 +1,11 @@
+fileFormatVersion: 2
+guid: 46491169607191d40bf2df92f03ef895
+MonoImporter:
+  externalObjects: {}
+  serializedVersion: 2
+  defaultReferences: []
+  executionOrder: 0
+  icon: {instanceID: 0}
+  userData: 
+  assetBundleName: 
+  assetBundleVariant: 

theProject/Assets/Scripts/Managers/SquadManager.cs

@@ -8,15 +8,21 @@ public class SquadManager : MonoBehaviour
 	[SerializeField] GameObject squadPrefab;
 	Squad playerSquad;
 	Squad enemySquad;
+
+	Vector2Int playerSpawnCoords = Vector2Int.up; //TEMP SPAWN BY BASE
 	private void Awake()
 	{
 		playerSquad = Instantiate(squadPrefab).GetComponent<Squad>();
 		playerSquad.gameObject.name = "Player Squad";
 		playerSquad.transform.SetParent(transform);
+		playerSquad.SetOwnTeam(Entity.Team.Ally);
+		playerSquad.gameObject.AddComponent<SoldierSpawning>();
+		playerSquad.GetComponent<SoldierSpawning>().SetSpawnCoords(playerSpawnCoords);
 
 		enemySquad = Instantiate(squadPrefab).GetComponent<Squad>();
 		enemySquad.gameObject.name = "Enemy Squad";
 		enemySquad.transform.SetParent(transform);
+		enemySquad.SetOwnTeam(Entity.Team.Enemy);
 	}
 	// Update is called once per frame
 	void Update()
@@ -24,14 +30,20 @@ public class SquadManager : MonoBehaviour
 		Debug.Log("Added initial soldiers to squad");
 		// add all ally soldiers to squad
 		var soldiers = FindObjectsOfType<Soldier>();
-		foreach(var soldier in soldiers)
-		{
-			if(soldier.GetOwnTeam() == Soldier.Team.Ally)
-			{
-				playerSquad.TempAddSoldierToSquad(soldier);
-			}
-		}
+		var squads = new List<Squad>();
+		squads.Add(playerSquad);
+		squads.Add(enemySquad);
 
-        enabled = false;
+		foreach (var soldier in soldiers)
+		{
+			foreach (var squad in squads)
+			{
+				if (soldier.GetOwnTeam() == squad.GetOwnTeam())
+				{
+					squad.AddSoldierToSquad(soldier);
+				}
+			}
+			enabled = false;
+		}
     }
 }

theProject/Assets/Scripts/Managers/TilemapManager.cs

@@ -104,10 +104,10 @@ public class TilemapManager : MonoBehaviour
 
     // ---------- public functions
 
-    public bool SpawnSoldier(int x, int y, bool isAlly, bool isBase=false)
+    public Entity SpawnSoldier(int x, int y, bool isAlly, bool isBase=false)
     {
         if (GetTileState(x, y) != TileState.free)
-            return false;
+            return null;
 
         if (isBase)
             tiles[x, y].standingEntity = Instantiate(basePrefab, tilemap.CellToWorld(new Vector3Int(x, y, 0)) + WORLD_SPACE_OFFSET, basePrefab.transform.rotation).GetComponent<Base>();
@@ -115,7 +115,7 @@ public class TilemapManager : MonoBehaviour
             tiles[x, y].standingEntity = Instantiate(soldierPrefab, tilemap.CellToWorld(new Vector3Int(x, y, 0)) + WORLD_SPACE_OFFSET, soldierPrefab.transform.rotation).GetComponent<Soldier>();
 
         if (tiles[x, y].standingEntity == null)
-            return false;
+            return null;
 
         if (isAlly)
             tiles[x, y].standingEntity.SetOwnTeam(Base.Team.Ally);
@@ -124,7 +124,7 @@ public class TilemapManager : MonoBehaviour
 
         tiles[x, y].standingEntity.SetTileCoords(new Vector2Int(x, y));
 
-        return true;
+        return tiles[x, y].standingEntity;
     }
 
     public bool DespawnSoldier(int x, int y)

theProject/Assets/Scripts/Squad.cs

@@ -5,6 +5,9 @@ using UnityEngine;
 
 public class Squad : MonoBehaviour
 {
+	private Entity.Team ownTeam;
+	public Entity.Team GetOwnTeam() { return ownTeam; }
+	public void SetOwnTeam(Entity.Team newTeam) {ownTeam = newTeam; }
 	#region Orders
 	public abstract class Order // generic order (to keep in queue)
 	{
@@ -31,12 +34,20 @@ public class Squad : MonoBehaviour
 	}
 	#endregion
 
-	[SerializeField] private List<Soldier> soldiers = new List<Soldier>(); // soldiers belonging to the squad
+	[SerializeField] private List<Entity> soldiers = new List<Entity>(); // soldiers belonging to the squad
+	public List<Entity> GetSoldiers() { return soldiers; }
     private Queue<Order> orders = new Queue<Order>(); // orders given to the squad
 
-	public void TempAddSoldierToSquad(Soldier soldier)
+	public void AddSoldierToSquad(Entity soldier)
 	{
 		soldiers.Add(soldier);
+		soldier.OnDeath.AddListener(RemoveSoldierFromSquad);
+	}
+
+	public void RemoveSoldierFromSquad(Entity soldier)
+	{
+		soldiers.Remove(soldier);
+		soldier.OnDeath.RemoveListener(RemoveSoldierFromSquad);
 	}
 
 	private void Awake()
@@ -44,12 +55,12 @@ public class Squad : MonoBehaviour
 		TickSystem.OnTick += HandleTick;
 	}
 
-    private void Start()
-    {
-		
-    }
+	private void OnDestroy()
+	{
+		TickSystem.OnTick -= HandleTick;
+	}
 
-    private void HandleTick(TickSystem.OnTickEventArgs eventArgs)
+	private void HandleTick(TickSystem.OnTickEventArgs eventArgs)
 	{// pass a single order to all soldiers
 		if (orders.Count < 1)
 			return; // for now nothing to do here