From 8b33f3ff5a3bf7e2524818b6644f609448166e57 Mon Sep 17 00:00:00 2001
From: henriquenakashima <henrique.nakashima@gmail.com>
Date: Sat, 19 Sep 2020 21:18:27 -0400
Subject: [PATCH] Show cards in PlayArea

---
 Assets/Battle/BattleController.cs          |  8 +--
 Assets/Battle/BattlePlayerControls.cs      | 74 ++++++++++++++++++++--
 Assets/Battle/Cards/CardCover.cs           |  1 -
 Assets/Battle/Cards/CardCrashingPunch.cs   |  1 -
 Assets/Battle/Cards/CardLightningStrike.cs |  1 -
 Assets/Battle/Cards/CardThunderstorm.cs    |  1 -
 6 files changed, 70 insertions(+), 16 deletions(-)

diff --git a/Assets/Battle/BattleController.cs b/Assets/Battle/BattleController.cs
index 405ce0e..4ba2e7d 100644
--- a/Assets/Battle/BattleController.cs
+++ b/Assets/Battle/BattleController.cs
@@ -74,12 +74,12 @@ public class BattleController : MonoBehaviour, IBattleEffectApplier, IBattleEffe
         }
         this.heroDeck = new HeroDeck(this.cardIds, cardOwners);
 
-        this.battlePlayerControls.Init(this.heroDeck);
+        this.playArea = new PlayArea();
+
+        this.battlePlayerControls.Init(this.heroDeck, this.playArea);
         this.battlePlayerControls.OnCardUsed += this.NotifyCardUsed;
         this.battlePlayerControls.OnTurnPassed += this.NotifyTurnPassed;
         this.heroDeck.OnHandPileChanged += this.battlePlayerControls.NotifyHandPileChanged;
-
-        this.playArea = new PlayArea();
     }
 
     private BattleParticipant FindHeroByClass(HeroId heroId) {
@@ -139,7 +139,6 @@ public class BattleController : MonoBehaviour, IBattleEffectApplier, IBattleEffe
     }
 
     public void NotifyCardUsed(BaseCard card) {
-        Debug.Log("NotifyCardUsed " + card);
         if (!CanUseCard(card)) {
             throw new System.InvalidOperationException("Cannot use card " + card.GetDisplayName());
         }
@@ -159,7 +158,6 @@ public class BattleController : MonoBehaviour, IBattleEffectApplier, IBattleEffe
 
     public void NotifyTurnPassed() {
         if (this.heroTurn) {
-            Debug.Log("NotifyTurnPassed");
             StartEnemyTurn();
         }
     }
diff --git a/Assets/Battle/BattlePlayerControls.cs b/Assets/Battle/BattlePlayerControls.cs
index d83ec80..386b1e2 100644
--- a/Assets/Battle/BattlePlayerControls.cs
+++ b/Assets/Battle/BattlePlayerControls.cs
@@ -16,15 +16,21 @@ public class BattlePlayerControls : MonoBehaviour
 
     private HeroDeck heroDeck;
     private List<GameObject> cardsInHandUIs;
+    private PlayArea playArea;
+    private Dictionary<BaseCard, GameObject> cardsInPlayUIs;
 
-    public void Init(HeroDeck heroDeck) {
+    public void Init(HeroDeck heroDeck, PlayArea playArea) {
         this.heroDeck = heroDeck;
+        this.playArea = playArea;
+
+        this.cardsInPlayUIs = new Dictionary<BaseCard, GameObject>();
     }
 
     // Start is called before the first frame update
     void Start()
     {
-        
+        NotifyPlayAreaChanged();
+        this.playArea.OnPlayAreaChanged += NotifyPlayAreaChanged;
     }
 
     // Update is called once per frame
@@ -45,19 +51,23 @@ public class BattlePlayerControls : MonoBehaviour
             BaseCard card = hand[i];
             GameObject cardUI = Instantiate(this.cardPrefab, this.uiCanvas.transform);
             this.cardsInHandUIs.Add(cardUI);
-            cardUI.transform.position = CalculateCardPosition(i, hand.Count);
+            cardUI.transform.position = CalculateHandCardPosition(i, hand.Count);
             cardUI.GetComponent<CardUI>().Init(card);
             int cardIndex = i;
             cardUI.GetComponent<Button>().onClick.AddListener(delegate {OnCardClicked(card);});
         }
     }
 
-    private Vector3 CalculateCardPosition(int index, int count) {
-        const int CARD_SIZE = 120;
+    private const int CARD_WIDTH = 122;
+    private const int CARD_HEIGHT = 155;
+
+    private Vector3 CalculateHandCardPosition(int index, int count) {
         const int SPACING = 8;
-        const int DELTA = CARD_SIZE + SPACING;
+        const int DELTA = CARD_WIDTH + SPACING;
         
-        return new Vector3(Screen.width * 0.5f - DELTA / 2 * (count - 1) + DELTA * index, 160, 0);
+        return new Vector3(Screen.width * 0.5f - DELTA / 2 * (count - 1) + DELTA * index,
+                160,
+                0);
     }
 
     private void OnCardClicked(BaseCard card) {
@@ -95,4 +105,54 @@ public class BattlePlayerControls : MonoBehaviour
     public void NotifyHandPileChanged() {
         SetHand(this.heroDeck.GetHandPile());
     }
+
+    private void NotifyPlayAreaChanged() {
+        DestroyCardsThatLeftPlayArea();
+        CreateCardsThatEnteredPlayArea();
+        RepositionCardsInPlayArea();
+    }
+
+    private void DestroyCardsThatLeftPlayArea() {
+        List<BaseCard> cardsToRemove = new List<BaseCard>();
+        foreach (KeyValuePair<BaseCard, GameObject> pair in this.cardsInPlayUIs) {
+            if (!this.playArea.GetHeroPermanents().Contains(pair.Key)) {
+                Destroy(pair.Value);
+                cardsToRemove.Add(pair.Key);
+            }
+        }
+
+        foreach (BaseCard cardToRemove in cardsToRemove) {
+            this.cardsInPlayUIs.Remove(cardToRemove);
+        }
+    }
+
+    private void CreateCardsThatEnteredPlayArea() {
+        foreach (BaseCard card in this.playArea.GetHeroPermanents()) {
+            if (!this.cardsInPlayUIs.ContainsKey(card)) {
+                GameObject cardUI = Instantiate(this.cardPrefab, this.uiCanvas.transform);
+                // Leave position to be set by RepositionCardsInPlayArea();
+                cardUI.GetComponent<CardUI>().Init(card);
+                this.cardsInPlayUIs.Add(card, cardUI);
+            }
+        }
+    }
+
+    private void RepositionCardsInPlayArea() {
+        List<BaseCard> permanentList = this.playArea.GetHeroPermanents();
+        for (int i = 0; i < permanentList.Count; i++) {
+            BaseCard card = permanentList[i];
+            GameObject cardUI = this.cardsInPlayUIs[card];
+            cardUI.transform.position = CalculatePlayAreaCardPosition(i, permanentList.Count);
+        }
+    }
+    
+    private Vector3 CalculatePlayAreaCardPosition(int index, int count) {
+        const int SPACING = 8;
+        const int DELTA = CARD_HEIGHT + SPACING;
+        const int PADDING = 8;
+        
+        return new Vector3(PADDING + CARD_WIDTH / 2,
+                Screen.height - PADDING - DELTA * index - CARD_HEIGHT / 2,
+                0);
+    }
 }
diff --git a/Assets/Battle/Cards/CardCover.cs b/Assets/Battle/Cards/CardCover.cs
index 1579cab..bbee993 100644
--- a/Assets/Battle/Cards/CardCover.cs
+++ b/Assets/Battle/Cards/CardCover.cs
@@ -14,7 +14,6 @@ public class CardCover : BaseCard
         SingleTargeting singleTargeting = new SingleTargeting(new OtherHeroTargetPolicy(this.owner));
         yield return playerControls.StartCoroutine(playerControls.ChooseTarget(singleTargeting));
         BattleParticipant target = singleTargeting.GetTarget();
-        Debug.Log("Covering " + target);
         target.AddBlock(3);
     }
 }
diff --git a/Assets/Battle/Cards/CardCrashingPunch.cs b/Assets/Battle/Cards/CardCrashingPunch.cs
index 1e694fa..43a0e47 100644
--- a/Assets/Battle/Cards/CardCrashingPunch.cs
+++ b/Assets/Battle/Cards/CardCrashingPunch.cs
@@ -13,7 +13,6 @@ public class CardCrashingPunch : BaseCard
         SingleTargeting singleTargeting = new SingleTargeting(new AnyTargetPolicy());
         yield return playerControls.StartCoroutine(playerControls.ChooseTarget(singleTargeting));
         BattleParticipant target = singleTargeting.GetTarget();
-        Debug.Log("Punching " + target);
         target.DealDamage(3);
     }
 }
diff --git a/Assets/Battle/Cards/CardLightningStrike.cs b/Assets/Battle/Cards/CardLightningStrike.cs
index 31f05b6..3f3914c 100644
--- a/Assets/Battle/Cards/CardLightningStrike.cs
+++ b/Assets/Battle/Cards/CardLightningStrike.cs
@@ -13,7 +13,6 @@ public class CardLightningStrike : BaseCard
         SingleTargeting singleTargeting = new SingleTargeting(new AnyTargetPolicy());
         yield return playerControls.StartCoroutine(playerControls.ChooseTarget(singleTargeting));
         BattleParticipant target = singleTargeting.GetTarget();
-        Debug.Log("Striking " + target + " with lightning.");
         target.DealDamage(3);
     }
 }
diff --git a/Assets/Battle/Cards/CardThunderstorm.cs b/Assets/Battle/Cards/CardThunderstorm.cs
index dcb339d..da7a5cd 100644
--- a/Assets/Battle/Cards/CardThunderstorm.cs
+++ b/Assets/Battle/Cards/CardThunderstorm.cs
@@ -10,7 +10,6 @@ public class CardThunderstorm : BaseCard
     }
 
     public override IEnumerator Use(IBattleEffectApplier effectApplier, BattlePlayerControls playerControls) {
-        Debug.Log("A storm is brewing.");
         effectApplier.Apply(new IncreaseMaxEnergyEffect(1));
         effectApplier.Apply(new AcquireCardsEffect(new CardId[] {CardId.LightningStrike, CardId.LightningStrike}, BattleZone.DrawPile, this.owner));
         yield break;