Only require settings where necessary

Brain/Brain.cs

@@ -145,27 +145,27 @@ private string ProcessGlobal(string command, Bomb bomb)
 
         private string ProcessModule(string command, Bomb bomb)
         {
-            if (!bomb.Ready)
-            {
-                return "Bomb not set up.";
-            }
-
             string module = command[(command.IndexOf(' ') + 1) ..].ToUpperInvariant();
 
             if (bomb.ActiveModule != null)
             {
                 this.ResetProcessors();
             }
 
-            SpeechProcessor speechModule = this.processors[module];
+            BombModule bombModule = this.processors[module] as BombModule;
 
-            if (!speechModule.IsGlobal)
+            if (!bombModule.IsGlobal)
             {
-                speechModule = Activator.CreateInstance(this.processors[module].GetType()) as SpeechProcessor;
-                this.processors[module] = speechModule;
+                bombModule = Activator.CreateInstance(this.processors[module].GetType()) as BombModule;
+                this.processors[module] = bombModule;
+
+                if (!bombModule.TryDefuse(bomb, out string response))
+                {
+                    return response;
+                }
             }
 
-            bomb.ActiveModule = speechModule as BombModule;
+            bomb.ActiveModule = bombModule;
             this.Recognition.Grammars.Where(g => g.Name == module)
                 .First()
                 .Enabled = true;

Game/Bomb.cs

@@ -2,6 +2,7 @@ namespace KTANE.Game
 {
     using System;
     using System.Collections.Generic;
+    using System.ComponentModel;
     using System.Linq;
     using System.Speech.Recognition;
     using KTANE.Brain;
@@ -25,16 +26,22 @@ internal class Bomb : SpeechProcessor
 
         public bool AutoReset { get; set; }
 
+        [DisplayName("Batteries")]
         public int? Batteries { get; set; }
 
+        [DisplayName("Parallel Port")]
         public bool? HasParallelPort { get; set; }
 
+        [DisplayName("FRK")]
         public bool? HasLitFRK { get; set; }
 
+        [DisplayName("CAR")]
         public bool? HasLitCAR { get; set; }
 
+        [DisplayName("Vowel")]
         public bool? HasVowel { get; set; }
 
+        [DisplayName("Last Digit")]
         public bool? LastDigitEven { get; set; }
 
         public int Strikes { get; set; }
@@ -83,10 +90,6 @@ public override GrammarBuilder Grammar
             }
         }
 
-        public bool Ready => typeof(Bomb).GetProperties()
-            .Where(p => p.PropertyType.IsGenericType && p.PropertyType.GetGenericTypeDefinition() == typeof(Nullable<>))
-            .All(p => p.GetValue(this) != null);
-
         public static Bomb CreateRandom()
         {
             Random rng = new ();

Game/BombModule.cs

@@ -1,9 +1,36 @@
 namespace KTANE.Game
 {
+    using System.Collections.Generic;
+    using System.ComponentModel;
+    using System.Linq;
+    using System.Reflection;
     using KTANE.Brain;
 
     internal abstract class BombModule : SpeechProcessor
     {
         public abstract string PreInfo { get; }
+
+        public bool TryDefuse(Bomb bomb, out string response)
+        {
+            response = null;
+
+            if (this.GetType()
+                    .GetMethod(nameof(this.Process))
+                    .GetCustomAttribute(typeof(RequiredBombSettingAttribute), true) is RequiredBombSettingAttribute attr)
+            {
+                List<string> missing = typeof(Bomb).GetProperties()
+                    .Where(p => attr.Settings.Contains(p.Name)
+                        && p.GetValue(bomb) == null)
+                    .Select(p => (p.GetCustomAttribute(typeof(DisplayNameAttribute), false) as DisplayNameAttribute).DisplayName)
+                    .ToList();
+
+                if (missing.Count > 0)
+                {
+                    response = $"Set {string.Join(", ", missing)}!";
+                }
+            }
+
+            return response == null;
+        }
     }
 }

Game/Modules/Button.cs

@@ -36,6 +36,7 @@ public override GrammarBuilder Grammar
 
         public override string PreInfo => string.Empty;
 
+        [RequiredBombSetting(nameof(Bomb.Batteries), nameof(Bomb.HasLitCAR), nameof(Bomb.HasLitFRK))]
         public override string Process(string command, Bomb bomb)
         {
             string[] parts = command.Split(' ');

Game/Modules/ComplexWires.cs

@@ -46,6 +46,7 @@ public override GrammarBuilder Grammar
             }
         }
 
+        [RequiredBombSetting(nameof(Bomb.Batteries), nameof(Bomb.HasParallelPort), nameof(Bomb.LastDigitEven))]
         public override string Process(string command, Bomb bomb)
         {
             bool red = command.Contains("red");

Game/Modules/Wires.cs

@@ -23,6 +23,7 @@ public override GrammarBuilder Grammar
             }
         }
 
+        [RequiredBombSetting(nameof(Bomb.LastDigitEven))]
         public override string Process(string command, Bomb bomb)
         {
             string[] wires = command.Split(' ');

Game/RequiredBombSettingAttribute.cs

@@ -0,0 +1,17 @@
+namespace KTANE.Game
+{
+    using System;
+    using System.Collections.Generic;
+    using System.Linq;
+
+    [AttributeUsage(AttributeTargets.Method)]
+    internal class RequiredBombSettingAttribute : Attribute
+    {
+        public RequiredBombSettingAttribute(params string[] settings)
+        {
+            this.Settings = settings.ToList();
+        }
+
+        public List<string> Settings { get; private set; }
+    }
+}