From e6a9b410c9765b0dd7c4256a3ba3abed179f65ca Mon Sep 17 00:00:00 2001
From: mike <compmichaelb90@gmail.com>
Date: Mon, 6 Mar 2023 20:48:04 -0500
Subject: [PATCH] Instant upgrade working

---
 .../src/ca/cosc3p91/a2/game/GameEngine.java   | 29 +++++++++++--------
 Assignment 2/src/ca/cosc3p91/a2/game/Map.java | 16 ++++++++--
 2 files changed, 30 insertions(+), 15 deletions(-)

diff --git a/Assignment 2/src/ca/cosc3p91/a2/game/GameEngine.java b/Assignment 2/src/ca/cosc3p91/a2/game/GameEngine.java
index 2f27a25..3cf0f46 100644
--- a/Assignment 2/src/ca/cosc3p91/a2/game/GameEngine.java	
+++ b/Assignment 2/src/ca/cosc3p91/a2/game/GameEngine.java	
@@ -81,7 +81,7 @@ public class GameEngine<T> implements Runnable {
     }
 
     private void printMenuOptions() {
-        System.out.println("~ Player Options:\n" +
+        System.out.println("\n~ Player Options:\n" +
                 "1. Build {command: '1 <building name>'}\n" +
                 "2. Train inhabitants {command: '2 <unit name>'}\n"+
                 "3. Upgrade Building {command: '3 i<index>'} / {command: '3 b<index>'}\n"+
@@ -219,7 +219,7 @@ public class GameEngine<T> implements Runnable {
                     String in = sc.nextLine();
                     String[] args = in.split(" ");
                     System.out.println("Your Input: ");
-                    System.out.println("\t->" + in);
+                    System.out.println("\t->" + in + '\n');
                     // reset the map if they aren't exploring
                     if (in.charAt(0) != '4')
                         deleteMyHeart = true;
@@ -253,16 +253,21 @@ public class GameEngine<T> implements Runnable {
                             if (args.length < 2) {
                                 System.err.println("Args must include type!");
                             } else {
-                                if (args[1].contains("i")) {
-                                    Inhabitant unit = map.inhabitants.get(Integer.parseInt(args[1].substring(1)));
-                                    if (map.upgradeInhabitant(unit) ){
-                                        System.out.println("successfully trained a(n)"+unit.getClass().getSimpleName());
-                                    } else System.out.println("Missing Resources to train "+unit.getClass().getSimpleName());
-                                } else if (args[1].contains("b")) {
-                                    Building bUnit = map.contains.get(Integer.parseInt(args[1].substring(1)));
-                                    if (map.upgradeBuilding(bUnit) ){
-                                        System.out.println("successfully trained a(n)"+bUnit.getClass().getSimpleName());
-                                    } else System.out.println("Missing Resources to train "+bUnit.getClass().getSimpleName());
+                                int unitIndex = Integer.parseInt(args[1].substring(1));
+
+                                if (unitIndex < 0) {
+                                    System.err.println("Invalid Index");
+                                    break;
+                                }
+
+                                if (args[1].contains("i") && (unitIndex < map.inhabitants.size()) ) {
+                                    if ( map.upgradeInhabitant(unitIndex) ) {
+                                        System.out.println("successfully upgraded a(n) "+map.inhabitants.get(unitIndex).getClass().getSimpleName());
+                                    } else System.out.println("Missing Resources to upgrade "+map.inhabitants.get(unitIndex).getClass().getSimpleName());
+                                } else if (args[1].contains("b") && (unitIndex < map.contains.size()) ) {
+                                    if ( map.upgradeBuilding(unitIndex) ) {
+                                        System.out.println("successfully upgraded a(n) "+map.contains.get(unitIndex).getClass().getSimpleName());
+                                    } else System.out.println("Missing Resources to upgrade "+map.contains.get(unitIndex).getClass().getSimpleName());
                                 } else {
                                     System.err.println("Args are not a valid unit!");
                                 }
diff --git a/Assignment 2/src/ca/cosc3p91/a2/game/Map.java b/Assignment 2/src/ca/cosc3p91/a2/game/Map.java
index 8b5dcd4..2a7b664 100644
--- a/Assignment 2/src/ca/cosc3p91/a2/game/Map.java	
+++ b/Assignment 2/src/ca/cosc3p91/a2/game/Map.java	
@@ -53,12 +53,17 @@ public class Map {
             return false;
     }
 
-    public boolean upgradeBuilding(Building b) {
+    public boolean upgradeBuilding(int buildingIndex) {
+
+        if (buildingIndex >= contains.size()) return false;
+
+        Building b = contains.get(buildingIndex);
+
         int currentLevel = b.getLevel();
         CasaDeNarino hall = getTownHall();
 
         if (currentLevel >= 2) return false;
-        if (b instanceof Farm) return true;
+        else if (b instanceof Farm) return true;
 
         int goldCost = b.getUpgradeStage().getCost(SaulGoodMine.resource);
         int ironCost = b.getUpgradeStage().getCost(IronMine.resource);
@@ -85,7 +90,12 @@ public class Map {
         return true;
     }
 
-    public boolean upgradeInhabitant(Inhabitant i) {
+    public boolean upgradeInhabitant(int inhabitantIndex) {
+
+        if (inhabitantIndex >= inhabitants.size()) return false;
+
+        Inhabitant i = inhabitants.get(inhabitantIndex);
+
         int currentLevel = i.getLevel();
         CasaDeNarino hall = getTownHall();