Reward menu approach

New approach that allows for player to choose from a selection of relics after the battle is won

Author: Rmojarro1

Reward.cs

@@ -2,6 +2,8 @@
 using System.Linq;
 using Godot;
 
+//it's very messy, feel free to clean up as much as necessary
+
 public static class Reward
 {
     private static readonly Random _rng = new Random();
@@ -46,4 +48,19 @@ public static void AddRelic(PlayerStats player, RelicTemplate relic)
         player.CurRelics = player.CurRelics.Append(relic).ToArray();
         GD.Print("Adding relic: " + relic.Name);
     }
+
+    public static RelicTemplate[] GetMultipleRelics(RelicTemplate[] ownedRelics, int count)
+    {
+        var availableRelics = Scribe
+            .RelicDictionary.Where(r => !ownedRelics.Any(o => o.Name == r.Name))
+            .ToArray();
+        if (availableRelics.Length == 0)
+            return new RelicTemplate[0];
+
+        return availableRelics
+            .OrderBy(_ => _rng.Next())
+            .Take(count)
+            .Select(r => r.Clone())
+            .ToArray();
+    }
 }

RewardSelect.cs

@@ -0,0 +1,41 @@
+using System;
+using System.Linq;
+using Godot;
+
+public partial class RewardSelect : CanvasLayer
+{
+    [Export]
+    public VBoxContainer ButtonContainer;
+
+    private PlayerStats _player;
+    private RelicTemplate[] _choices;
+
+    public void Initialize(PlayerStats player)
+    {
+        _player = player;
+        GenerateRelicChoices();
+    }
+
+    private void GenerateRelicChoices()
+    {
+        //should probably change this so that the amount of relics offered can be changed when BD calls it
+        //i.e less options when killing trash mobs/basic/weak enemies
+        _choices = Reward.GetMultipleRelics(_player.CurRelics, 3);
+
+        foreach (var relic in _choices)
+        {
+            var button = new Button();
+            button.Text = relic.Name;
+            button.Pressed += () => OnRelicSelected(relic);
+            ButtonContainer.AddChild(button);
+        }
+    }
+
+    private void OnRelicSelected(RelicTemplate choiceRelic)
+    {
+        Reward.AddRelic(_player, choiceRelic);
+        GD.Print("Relic selected: " + choiceRelic.Name);
+
+        QueueFree();
+    }
+}

RewardSelectionUI.tscn

@@ -0,0 +1,18 @@
+[gd_scene load_steps=2 format=3 uid="uid://c6icx2yriud6y"]
+
+[ext_resource type="Script" path="res://RewardSelect.cs" id="1_ts2x3"]
+
+[node name="CanvasLayer" type="CanvasLayer" node_paths=PackedStringArray("ButtonContainer")]
+script = ExtResource("1_ts2x3")
+ButtonContainer = NodePath("PanelContainer/VBoxContainer")
+
+[node name="PanelContainer" type="PanelContainer" parent="."]
+offset_left = 34.0
+offset_top = 67.0
+offset_right = 600.0
+offset_bottom = 264.0
+
+[node name="VBoxContainer" type="VBoxContainer" parent="PanelContainer"]
+layout_mode = 2
+
+[node name="RewardSelect" type="Node2D" parent="."]

scenes/BattleDirector/scripts/BattleDirector.cs

@@ -119,23 +119,25 @@ private void Begin()
 
     public override void _Process(double delta)
     {
+        //check if either one is dead until one is true, feel free to remove if we have a more efficent way of checking
+        //alternatively, stop the other process since the battle is over
         if (!battleLost || !battleWon)
         {
             CheckBattleStatus();
         }
         TimeKeeper.CurrentTime = Audio.GetPlaybackPosition();
         CD.CheckMiss();
-        //CheckBattleStatus();
     }
     #endregion
 
     #region Input&Timing
 
     public override void _UnhandledInput(InputEvent @event)
     {
+        //this one is for calling a debug key to insta-kill the enemy
         if (@event is InputEventKey eventKey && eventKey.Pressed && !eventKey.Echo)
         {
-            if (eventKey.Keycode == Key.Key0) // Adjust if you prefer a different key code.
+            if (eventKey.Keycode == Key.Key0)
             {
                 DebugKillEnemy();
             }
@@ -239,13 +241,20 @@ private void CheckBattleStatus()
             return;
         }
 
+        //will have to adjust this to account for when we have multiple enemies at once
         if (Enemy.GetCurrentHealth() <= 0)
         {
             GD.Print("Enemy is dead");
             battleWon = true;
 
-            Reward.GiveRandomRelic(Player.Stats);
-            EventizeRelics(); //literally just here for debugging, ignore later
+            //below, old method that just adds a random relic to the inventory
+            //Reward.GiveRandomRelic(Player.Stats);
+            //EventizeRelics(); //literally just here to force the ui to update and see if it was added, remove with the proper ui update
+            //probably won't even need it since we'll be loading to seperate scene anyways
+
+            //new method that allows player to choose a relic
+            ShowRewardSelection();
+
             return;
         }
     }
@@ -254,4 +263,12 @@ private void DebugKillEnemy()
     {
         Enemy.TakeDamage(1000);
     }
+
+    private void ShowRewardSelection()
+    {
+        var rewardUI = GD.Load<PackedScene>("res://RewardSelectionUI.tscn")
+            .Instantiate<RewardSelect>();
+        AddChild(rewardUI);
+        rewardUI.Initialize(Player.Stats);
+    }
 }