From e01eab28a2316a57366c2df794d9dc573f216cdb Mon Sep 17 00:00:00 2001
From: BruceChen <MrChen131217@gmail.com>
Date: Thu, 29 Sep 2022 22:49:07 +0800
Subject: [PATCH] Add Location.Parse

---
 MinecraftClient/Commands/Move.cs    |  9 +--
 MinecraftClient/Mapping/Location.cs | 89 +++++++++++++++++++++++++++++
 2 files changed, 92 insertions(+), 6 deletions(-)

diff --git a/MinecraftClient/Commands/Move.cs b/MinecraftClient/Commands/Move.cs
index da58e76b..62f4a983 100644
--- a/MinecraftClient/Commands/Move.cs
+++ b/MinecraftClient/Commands/Move.cs
@@ -91,14 +91,11 @@ namespace MinecraftClient.Commands
                     try
                     {
                         Location current = handler.GetCurrentLocation(), currentCenter = current.ToCenter();
-                        
-                        double x = args[0].StartsWith('~') ? current.X + (args[0].Length > 1 ? double.Parse(args[0][1..]) : 0) : double.Parse(args[0]);
-                        double y = args[1].StartsWith('~') ? current.Y + (args[1].Length > 1 ? double.Parse(args[1][1..]) : 0) : double.Parse(args[1]);
-                        double z = args[2].StartsWith('~') ? current.Z + (args[2].Length > 1 ? double.Parse(args[2][1..]) : 0) : double.Parse(args[2]);
-                        Location goal = new(x, y, z);
+
+                        Location goal = Location.Parse(current, args[0], args[1], args[2]);
 
                         if (!Movement.CheckChunkLoading(handler.GetWorld(), current, goal))
-                            return Translations.Get("cmd.move.chunk_not_loaded", x, y, z);
+                            return Translations.Get("cmd.move.chunk_not_loaded", goal.X, goal.Y, goal.Z);
 
                         if (takeRisk || Movement.PlayerFitsHere(handler.GetWorld(), goal))
                         {
diff --git a/MinecraftClient/Mapping/Location.cs b/MinecraftClient/Mapping/Location.cs
index edc8404f..339ec562 100644
--- a/MinecraftClient/Mapping/Location.cs
+++ b/MinecraftClient/Mapping/Location.cs
@@ -73,6 +73,95 @@ namespace MinecraftClient.Mapping
             Z = chunkZ * Chunk.SizeZ + blockZ;
         }
 
+        /// <summary>
+        /// Parse location from the string.
+        /// return NULL if the parsing fails.
+        /// </summary>
+        /// <param name="x">The string representation of the X-axis coordinate.</param>
+        /// <param name="y">The string representation of the Y-axis coordinate.</param>
+        /// <param name="z">The string representation of the Z-axis coordinate.</param>
+        /// <returns>The location represented by the string.</returns>
+        public static Location Parse(string x, string y, string z)
+        {
+            Location.TryParse(x, y, z, out Location? res);
+            if (res == null)
+                throw new FormatException();
+            else
+                return (Location)res;
+        }
+
+        public static bool TryParse(string x, string y, string z, out Location? location)
+        {
+            string[] coord_str = new string[] { x.Trim(), y.Trim(), z.Trim() };
+            double[] coord_res = new double[3];
+
+            for (int i = 0; i < 3; ++i)
+            {
+                if (!double.TryParse(coord_str[i], out coord_res[i]))
+                {
+                    location = null;
+                    return false;
+                }
+            }
+
+            location = new Location(coord_res[0], coord_res[1], coord_res[2]);
+            return true;
+        }
+
+        /// <summary>
+        /// Parse location from the string (relative coordinate representation is supported).
+        /// return NULL if the parsing fails.
+        /// </summary>
+        /// <param name="current">Relative position base point.</param>
+        /// <param name="x">The string representation of the X-axis coordinate.</param>
+        /// <param name="y">The string representation of the Y-axis coordinate.</param>
+        /// <param name="z">The string representation of the Z-axis coordinate.</param>
+        /// <returns>The location represented by the string.</returns>
+        public static Location Parse(Location current, string x, string y, string z)
+        {
+            Location.TryParse(current, x, y, z, out Location? res);
+            if (res == null)
+                throw new FormatException();
+            else
+                return (Location)res;
+        }
+
+        public static bool TryParse(Location current, string x, string y, string z, out Location? location)
+        {
+            string[] coord_str = new string[] { x.Trim(), y.Trim(), z.Trim() };
+            double[] coord_res = new double[3];
+            double[] coord_cur = new double[] { current.X, current.Y, current.Z };
+
+            for (int i = 0; i < 3; ++i)
+            {
+                if (coord_str[i].StartsWith('~'))
+                {
+                    if (coord_str[i].Length > 1)
+                    {
+                        if (!double.TryParse(coord_str[i][1..], out coord_res[i]))
+                        {
+                            location = null;
+                            return false;
+                        }
+                        coord_res[i] += coord_cur[i];
+                    }
+                    else
+                        coord_res[i] = coord_cur[i];
+                }
+                else
+                {
+                    if (!double.TryParse(coord_str[i], out coord_res[i]))
+                    {
+                        location = null;
+                        return false;
+                    }
+                }
+            }
+
+            location = new Location(coord_res[0], coord_res[1], coord_res[2]);
+            return true;
+        }
+
         /// <summary>
         /// Round coordinates
         /// </summary>