Minor tweaks to simplify game logic

NetAF.Tests/TestGameConfiguration.cs

@@ -20,7 +20,6 @@ internal class TestGameConfiguration(Size displaySize, ExitMode exitMode, IIOAda
                                                             new CustomCommandInterpreter());
 
         public FrameBuilderCollection FrameBuilders { get; set; } = FrameBuilderCollections.Default;
-        public string ErrorPrefix { get; set; } = "Oops";
         public bool DisplayCommandListInSceneFrames { get; set; } = true;
         public KeyType SceneMapKeyType { get; set; } = KeyType.Dynamic;
         public IIOAdapter Adapter { get; private set; } = adapter;

NetAF/Logic/Configuration/ConsoleGameConfiguration.cs

@@ -45,11 +45,6 @@ public sealed class ConsoleGameConfiguration(Size displaySize, ExitMode exitMode
         /// </summary>
         public FrameBuilderCollection FrameBuilders { get; set; } = FrameBuilderCollections.Default;
 
-        /// <summary>
-        /// Get or set the prefix to use when displaying errors.
-        /// </summary>
-        public string ErrorPrefix { get; set; } = "Oops";
-
         /// <summary>
         /// Get or set if the command list is displayed in scene frames.
         /// </summary>

NetAF/Logic/Configuration/IGameConfiguration.cs

@@ -33,11 +33,6 @@ public interface IGameConfiguration
         /// </summary>
         public FrameBuilderCollection FrameBuilders { get; set; }
 
-        /// <summary>
-        /// Get or set the prefix to use when displaying errors.
-        /// </summary>
-        public string ErrorPrefix { get; set; }
-
         /// <summary>
         /// Get or set if the command list is displayed in scene frames.
         /// </summary>

NetAF/Logic/Game.cs

@@ -153,17 +153,8 @@ internal void Execute()
                 // process the input
                 var reaction = ProcessInput(input);
 
-                // if the reaction should be displayed
-                if (reaction.Result != ReactionResult.Silent)
-                {
-                    // display the reaction now
-                    DisplayReaction(reaction);
-                }
-                else if (reaction.Result != ReactionResult.GameModeChanged && Mode.Type == GameModeType.Information)
-                {
-                    // revert back to scene mode as the command didn't change the mode and the current mode is information, essentially the mode has expired
-                    ChangeMode(new SceneMode());
-                }
+                // handle the reaction
+                HandleReaction(reaction);
 
                 // check if the game has ended
                 if (CheckForGameEnd(EndConditions, out endMode))
@@ -192,6 +183,25 @@ internal void Execute()
             state = GameState.Finished;
         }
 
+        /// <summary>
+        /// Handle a reaction.
+        /// </summary>
+        /// <param name="reaction">The reaction to handle.</param>
+        private void HandleReaction(Reaction reaction)
+        {
+            // check if needed to 
+            if (reaction.Result == ReactionResult.Error || reaction.Result == ReactionResult.Inform)
+            {
+                // display the reaction
+                ChangeMode(new ReactionMode(Overworld.CurrentRegion.CurrentRoom.Identifier.Name, reaction.Description));
+            }
+            else if (reaction.Result != ReactionResult.GameModeChanged && Mode.Type == GameModeType.Information)
+            {
+                // revert back to scene mode as the command didn't change the mode and the current mode information, essentially the mode has expired
+                ChangeMode(new SceneMode());
+            }
+        }
+
         /// <summary>
         /// Check to see if the game has ended.
         /// </summary>
@@ -320,29 +330,6 @@ private Reaction ProcessInput(string input)
             return new(ReactionResult.Silent, string.Empty);
         }
 
-        /// <summary>
-        /// Display a reaction.
-        /// </summary>
-        /// <param name="reaction">The reaction to handle.</param>
-        private void DisplayReaction(Reaction reaction)
-        {
-            switch (reaction.Result)
-            {
-                case ReactionResult.Error:
-                    var message = Configuration.ErrorPrefix + ": " + reaction.Description;
-                    ChangeMode(new ReactionMode(Overworld.CurrentRegion.CurrentRoom.Identifier.Name, message));
-                    break;
-                case ReactionResult.Silent:
-                case ReactionResult.GameModeChanged:
-                    break;
-                case ReactionResult.Inform:
-                    ChangeMode(new ReactionMode(Overworld.CurrentRegion.CurrentRoom.Identifier.Name, reaction.Description));
-                    break;
-                default:
-                    throw new NotImplementedException();
-            }
-        }
-
         /// <summary>
         /// End the game.
         /// </summary>

NetAF/Logic/Modes/ReactionMode.cs

@@ -7,7 +7,8 @@ namespace NetAF.Logic.Modes
     /// </summary>
     /// <param name="title">The title.</param>
     /// <param name="message">The message.</param>
-    public sealed class ReactionMode(string title, string message) : IGameMode
+    /// <param name="isError">If the message is an error..</param> 
+    public sealed class ReactionMode(string title, string message, bool isError) : IGameMode
     {
         #region Implementation of IGameMode
 
@@ -27,7 +28,7 @@ public sealed class ReactionMode(string title, string message) : IGameMode
         /// <param name="game">The game.</param>
         public void Render(Game game)
         {
-            var frame = game.Configuration.FrameBuilders.ReactionModeFrameBuilder.Build(title, message, game.Configuration.DisplaySize);
+            var frame = game.Configuration.FrameBuilders.ReactionModeFrameBuilder.Build(title, message, isError, game.Configuration.DisplaySize);
             game.Configuration.Adapter.RenderFrame(frame);
         }
 

NetAF/Rendering/FrameBuilders/Console/ConsoleConversationFrameBuilder.cs

@@ -58,6 +58,11 @@ public sealed class ConsoleConversationFrameBuilder(GridStringBuilder gridString
         /// </summary>
         public AnsiColor InputColor { get; set; } = AnsiColor.White;
 
+        /// <summary>
+        /// Get or set the command title.
+        /// </summary>
+        public string CommandTitle { get; set; } = "You can:";
+
         #endregion
 
         #region StaticMethods
@@ -149,7 +154,7 @@ public IFrame Build(string title, IConverser converser, CommandHelp[] contextual
             if (contextualCommands?.Any() ?? false)
             {
                 gridStringBuilder.DrawHorizontalDivider(lastY + linePadding, BorderColor);
-                gridStringBuilder.DrawWrapped("You can:", leftMargin, lastY + 4, availableWidth, ResponseColor, out _, out lastY);
+                gridStringBuilder.DrawWrapped(CommandTitle, leftMargin, lastY + 4, availableWidth, ResponseColor, out _, out lastY);
 
                 var maxCommandLength = contextualCommands.Max(x => x.Command.Length);
                 const int padding = 4;

NetAF/Rendering/FrameBuilders/Console/ConsoleReactionFrameBuilder.cs

@@ -38,6 +38,11 @@ public sealed class ConsoleReactionFrameBuilder(GridStringBuilder gridStringBuil
         /// </summary>
         public AnsiColor MessageColor { get; set; } = AnsiColor.White;
 
+        /// <summary>
+        /// Get or set the error message color.
+        /// </summary>
+        public AnsiColor ErrorMessageColor { get; set; } = AnsiColor.White;
+
         #endregion
 
         #region Implementation of IReactionFrameBuilder
@@ -47,8 +52,9 @@ public sealed class ConsoleReactionFrameBuilder(GridStringBuilder gridStringBuil
         /// </summary>
         /// <param name="title">The title to display to the user.</param>
         /// <param name="message">The message to display to the user.</param>
+        /// <param name="isError">If the message is an error.</param>
         /// <param name="size">The size of the frame.</param>
-        public IFrame Build(string title, string message, Size size)
+        public IFrame Build(string title, string message, bool isError, Size size)
         {
             gridStringBuilder.Resize(size);
 
@@ -67,8 +73,8 @@ public IFrame Build(string title, string message, Size size)
                 lastY += 3;
             }
 
-            gridStringBuilder.DrawWrapped(message.EnsureFinishedSentence(), leftMargin, lastY, availableWidth, MessageColor, out _, out _);
-
+            gridStringBuilder.DrawWrapped(message.EnsureFinishedSentence(), leftMargin, lastY, availableWidth, isError ? ErrorMessageColor : MessageColor, out _, out _);
+            
             return new GridTextFrame(gridStringBuilder, 0, 0, BackgroundColor) { ShowCursor = false };
         }
 

NetAF/Rendering/FrameBuilders/Console/ConsoleRegionMapFrameBuilder.cs

@@ -52,6 +52,11 @@ public sealed class ConsoleRegionMapFrameBuilder(GridStringBuilder gridStringBui
         /// </summary>
         public AnsiColor InputColor { get; set; } = AnsiColor.White;
 
+        /// <summary>
+        /// Get or set the command title.
+        /// </summary>
+        public string CommandTitle { get; set; } = "You can:";
+
         #endregion
 
         #region Implementation of IRegionMapFrameBuilder
@@ -90,7 +95,7 @@ public IFrame Build(Region region, Point3D focusPosition, CommandHelp[] contextu
                 var requiredYToFitAllCommands = size.Height - commandSpace;
 
                 gridStringBuilder.DrawHorizontalDivider(requiredYToFitAllCommands, BorderColor);
-                gridStringBuilder.DrawWrapped("You can:", leftMargin, requiredYToFitAllCommands + 2, availableWidth, CommandsColor, out _, out lastY);
+                gridStringBuilder.DrawWrapped(CommandTitle, leftMargin, requiredYToFitAllCommands + 2, availableWidth, CommandsColor, out _, out lastY);
 
                 var maxCommandLength = contextualCommands.Max(x => x.Command.Length);
                 const int padding = 4;

NetAF/Rendering/FrameBuilders/Console/ConsoleSceneFrameBuilder.cs

@@ -56,6 +56,11 @@ public sealed class ConsoleSceneFrameBuilder(GridStringBuilder gridStringBuilder
         /// </summary>
         public bool DisplayMessagesInIsolation { get; set; } = true;
 
+        /// <summary>
+        /// Get or set the command title.
+        /// </summary>
+        public string CommandTitle { get; set; } = "You can:";
+
         #endregion
 
         #region Implementation of ISceneFrameBuilder
@@ -119,7 +124,7 @@ public IFrame Build(Room room, ViewPoint viewPoint, PlayableCharacter player, Co
                 lastY = yStart;
 
                 gridStringBuilder.DrawHorizontalDivider(lastY + linePadding, BorderColor);
-                gridStringBuilder.DrawWrapped("You can:", leftMargin, lastY + 4, availableWidth, CommandsColor, out _, out lastY);
+                gridStringBuilder.DrawWrapped(CommandTitle, leftMargin, lastY + 4, availableWidth, CommandsColor, out _, out lastY);
 
                 var maxCommandLength = contextualCommands.Max(x => x.Command.Length);
                 const int padding = 4;

NetAF/Rendering/FrameBuilders/IReactionFrameBuilder.cs

@@ -13,7 +13,8 @@ public interface IReactionFrameBuilder
         /// </summary>
         /// <param name="title">The title to display to the user.</param>
         /// <param name="message">The message to display to the user.</param>
+        /// <param name="isError">If the message is an error.</param>
         /// <param name="size">The size of the frame.</param>
-        IFrame Build(string title, string message, Size size);
+        IFrame Build(string title, string message, bool isError, Size size);
     }
 }