CodeQA (again)

Halp, please make it the last

Author: neon-nyan

CollapseLauncher/Classes/CachesManagement/Honkai/Fetch.cs

@@ -339,14 +339,10 @@ private byte[] GetAssetIndexSalt(string data)
         private static bool IsValidRegionFile(string input, string lang)
         {
             // If the path contains regional string, then move to the next check
-            if (input!.Contains(CacheRegionalCheckName!))
-            {
-                // Check if the regional string has specified language string
-                return input.Contains($"{CacheRegionalCheckName}_{lang}");
-            }
-
+            return !input.Contains(CacheRegionalCheckName!) ||
+                   // Check if the regional string has specified language string
+                   input.Contains($"{CacheRegionalCheckName}_{lang}");
             // If none, then pass it as true (non-regional string)
-            return true;
         }
 
         public KianaDispatch GetCurrentGateway() => GameGateway;

CollapseLauncher/Classes/EventsManagement/BackgroundActivityManager.cs

@@ -41,14 +41,15 @@ public static void Detach(int hashID)
             if (BackgroundActivities.Remove(hashID))
             {
                 DetachEventFromNotification(hashID);
-#if DEBUG
+            #if DEBUG
                 Logger.LogWriteLine($"Background activity with ID: {hashID} has been detached", LogType.Debug, true);
-#endif
+                return;
+            #endif
             }
 
-#if DEBUG
+        #if DEBUG
             Logger.LogWriteLine($"Cannot detach background activity with ID: {hashID} because it doesn't attached", LogType.Debug, true);
-#endif
+        #endif
         }
 
         private static void AttachEventToNotification(int hashID, IBackgroundActivity activity, string activityTitle, string activitySubtitle)