Structuring and Cleanup

Author: Shad0wlife

DSP_AssemblerUI/AssemblerSpeedUI/AssemblerSpeedUIMod.cs

@@ -3,6 +3,7 @@
 using HarmonyLib;
 using System;
 using System.Collections.Generic;
+using System.Diagnostics;
 using System.Linq;
 using System.Reflection;
 using System.Reflection.Emit;
@@ -19,6 +20,7 @@ namespace DSP_AssemblerUI.AssemblerSpeedUI
     [BepInPlugin("dsp.assemblerUI.speedMod", "Assembler UI Speed Info Mod", "1.0.0.0")]
     public class AssemblerSpeedUIMod : BaseUnityPlugin
     {
+        #region Main Plugin
         public new static ManualLogSource Logger;
 
         internal void Awake()
@@ -31,8 +33,14 @@ internal void Awake()
             Harmony.CreateAndPatchAll(typeof(AssemblerSpeedUIMod));
         }
 
+        [Conditional("DEBUG")]
+        static void DebugLog(string logMessage)
+        {
+            Logger.LogDebug(logMessage);
+        }
+        #endregion
 
-
+        #region Patcher
         internal class ItemSpeedInfoLabel
         {
             public GameObject gameObject;
@@ -134,9 +142,9 @@ public static void AddSpeedLabel(string id, int num, int ofNum)
         public static void PositionSpeedLabel(GameObject gameObject, int num, int ofNum)
         {
 
-            Logger.LogDebug($"OgPosition:{gameObject.transform.localPosition}");
+            DebugLog($"OgPosition:{gameObject.transform.localPosition}");
             Vector3 shiftVector = getPosShift(num, ofNum);
-            Logger.LogDebug($"ShiftedBy:{shiftVector}");
+            DebugLog($"ShiftedBy:{shiftVector}");
 
             gameObject.transform.localPosition = ogPos.Value + shiftVector;
         }
@@ -164,15 +172,15 @@ public static void UpdateSpeedLabels(float baseSpeed, int[] productCounts)
         [HarmonyTranspiler, HarmonyPatch(typeof(UIAssemblerWindow), "_OnUpdate")]
         public static IEnumerable<CodeInstruction> Transpiler(IEnumerable<CodeInstruction> instructions)
         {
-            Logger.LogDebug("UiTextTranspiler started!");
+            DebugLog("UiTextTranspiler started!");
             CodeMatcher matcher = new CodeMatcher(instructions);
             //find -->
             //ldloc.s 18
             //ldloc.s 19
             //mul
             //stloc.s 18
             //<-- endFind
-            Logger.LogDebug($"UiTextTranspiler Matcher Codes Count: {matcher.Instructions().Count}, Matcher Pos: {matcher.Pos}!");
+            DebugLog($"UiTextTranspiler Matcher Codes Count: {matcher.Instructions().Count}, Matcher Pos: {matcher.Pos}!");
             matcher.MatchForward(
                 true,
                 new CodeMatch(i => i.opcode == OpCodes.Ldloc_S && i.operand is LocalBuilder lb && lb.LocalIndex == 18),
@@ -181,15 +189,15 @@ public static IEnumerable<CodeInstruction> Transpiler(IEnumerable<CodeInstructio
                 new CodeMatch(i => i.opcode == OpCodes.Stloc_S && i.operand is LocalBuilder lb && lb.LocalIndex == 18)
             );
 
-            Logger.LogDebug($"UiTextTranspiler Matcher Codes Count: {matcher.Instructions().Count}, Matcher Pos: {matcher.Pos}!");
+            DebugLog($"UiTextTranspiler Matcher Codes Count: {matcher.Instructions().Count}, Matcher Pos: {matcher.Pos}!");
             matcher.Advance(1); //move from last match to next element
             //insert-->
             //ldloc.s 18
             //ldloca.s 0 //AssemblerComponent local var
             //ldfld int32[] AssemblerComponent::productCounts
             //call update
             //<-- endInsert
-            Logger.LogDebug($"UiTextTranspiler Matcher Codes Count: {matcher.Instructions().Count}, Matcher Pos: {matcher.Pos}!");
+            DebugLog($"UiTextTranspiler Matcher Codes Count: {matcher.Instructions().Count}, Matcher Pos: {matcher.Pos}!");
             matcher.InsertAndAdvance(
                 new CodeInstruction(OpCodes.Ldloc_S, (byte)18),
                 new CodeInstruction(OpCodes.Ldloca_S, (byte)0),
@@ -199,5 +207,6 @@ public static IEnumerable<CodeInstruction> Transpiler(IEnumerable<CodeInstructio
 
             return matcher.InstructionEnumeration();
         }
+        #endregion
     }
 }