Changeset 455 for binary-improvements/AllocsCommands/Commands/RemoveLandProtection.cs

Timestamp:

Jul 31, 2023, 4:06:13 PM (16 months ago)

Author:

alloc

Message:

25_30_44

• Got rid (mostly) of custom JSON serialization
• Some code cleanup

File:

• binary-improvements/AllocsCommands/Commands/RemoveLandProtection.cs (modified) (4 diffs)

binary-improvements/AllocsCommands/Commands/RemoveLandProtection.cs

@@ -2 +2 @@
 using System.Collections.Generic;
 using AllocsFixes.PersistentData;
+using JetBrains.Annotations;
 
 namespace AllocsFixes.CustomCommands {
+        [UsedImplicitly]
         public class RemoveLandProtection : ConsoleCmdAbstract {
                 protected override string getDescription () {
@@ -47 +49 @@
                                 GameManager.Instance.SetBlocksRPC (changes);
 
-                                SdtdConsole.Instance.Output ("Tried to remove #" + changes.Count +
-                                                             " land protection blocks for player \"" + _id + "\". Note " +
-                                                             "that only blocks in chunks that are currently loaded (close to any player) could be removed. " +
-                                                             "Please check for remaining blocks by running:");
-                                SdtdConsole.Instance.Output ("  listlandprotection " + _id);
+                                SdtdConsole.Instance.Output (
+                                        $"Tried to remove #{changes.Count} land protection blocks for player \"{_id}\". Note that only blocks in chunks that are currently loaded (close to any player) could be removed. Please check for remaining blocks by running:");
+                                SdtdConsole.Instance.Output ($"  listlandprotection {_id}");
                         } catch (Exception e) {
-                                Log.Out ("Error in RemoveLandProtection.removeById: " + e);
+                                Log.Out ($"Error in RemoveLandProtection.removeById: {e}");
                         }
                 }
 
-                private void removeByPosition (List<string> _coords) {
+                private void removeByPosition (IReadOnlyList<string> _coords) {
                         int.TryParse (_coords [0], out int x);
                         int.TryParse (_coords [1], out int y);
@@ -84 +84 @@
                         GameManager.Instance.SetBlocksRPC (changes);
 
-                        SdtdConsole.Instance.Output ("Land protection block at (" + v + ") removed");
+                        SdtdConsole.Instance.Output ($"Land protection block at ({v}) removed");
                 }
 
@@ -125 +125 @@
                                         } catch (Exception e) {
                                                 SdtdConsole.Instance.Output ("Error removing claims");
-                                                Log.Out ("Error in RemoveLandProtection.Run: " + e);
+                                                Log.Out ($"Error in RemoveLandProtection.Run: {e}");
                                         }
                                 } catch (Exception e) {
                                         SdtdConsole.Instance.Output ("Error getting current player's position");
-                                        Log.Out ("Error in RemoveLandProtection.Run: " + e);
+                                        Log.Out ($"Error in RemoveLandProtection.Run: {e}");
                                 }
                         } else if (_params.Count == 1) {