fix json Serializer #60

profcomff · May 17, 2023 · 108f79a · 108f79a
1 parent 7f4b87c
commit 108f79a
Show file tree

Hide file tree

Showing 2 changed files with 44 additions and 14 deletions.
diff --git a/PrinterApp/Marketing.cs b/PrinterApp/Marketing.cs
@@ -28,26 +28,26 @@ private static void Post(string action, string status, string pathFrom,
         string pathTo)
     {
         var body = new MarketingBody(action: action,
-            additional_data:
-            $"{{\"status\": \"{status}\",\"app_version\": \"{AssemblyVersion}\",\"terminal_user_id\": \"{TerminalUserId}\"}}",
+            additional_data: new MarketingBody.AdditionalData(status, AssemblyVersion,
+                TerminalUserId),
             path_from: pathFrom, path_to: pathTo);
         SharedClient.PostAsJsonAsync("action", body);
     }
 
     private static void Post(string action, string status)
     {
         var body = new MarketingBody(action: action,
-            additional_data:
-            $"{{\"status\": \"{status}\",\"app_version\": \"{AssemblyVersion}\",\"terminal_user_id\": \"{TerminalUserId}\"}}");
+            additional_data: new MarketingBody.AdditionalData(status, AssemblyVersion,
+                TerminalUserId));
         SharedClient.PostAsJsonAsync("action", body);
     }
 
     private static void Post(string action, string status, float availableMem,
         float currentMem)
     {
         var body = new MarketingBody(action: action,
-            additional_data:
-            $"{{\"status\": \"{status}\",\"available_mem\": \"{availableMem}\",\"current_mem\": \"{currentMem}\",\"app_version\": \"{AssemblyVersion}\",\"terminal_user_id\": \"{TerminalUserId}\"}}");
+            additional_data: new MarketingBody.AdditionalData(status, availableMem, currentMem,
+                AssemblyVersion, TerminalUserId));
         SharedClient.PostAsJsonAsync("action", body);
     }
 

diff --git a/PrinterApp/MarketingBody.cs b/PrinterApp/MarketingBody.cs
@@ -1,20 +1,50 @@
+using Newtonsoft.Json;
+
 namespace PrinterApp;
 
 public class MarketingBody
 {
-    public MarketingBody(string action, string additional_data, string path_from,
-        string path_to)
+    public class AdditionalData
     {
-        this.action = action;
-        this.additional_data = additional_data;
-        this.path_from = path_from;
-        this.path_to = path_to;
+        public AdditionalData(string status, string app_version, string terminal_user_id)
+        {
+            this.status = status;
+            this.app_version = app_version;
+            this.terminal_user_id = terminal_user_id;
+        }
+
+        public AdditionalData(string status, float available_mem, float current_mem,
+            string app_version, string terminal_user_id) : this(status, app_version,
+            terminal_user_id)
+        {
+            this.available_mem = available_mem;
+            this.current_mem = current_mem;
+        }
+
+        public string? status { get; }
+        public string? app_version { get; }
+        public string? terminal_user_id { get; }
+        public float? available_mem { get; }
+        public float? current_mem { get; }
     }
 
-    public MarketingBody(string action, string additional_data)
+    public MarketingBody(string action, AdditionalData additional_data)
     {
         this.action = action;
-        this.additional_data = additional_data;
+        JsonSerializerSettings jsonSerializerSettings = new()
+        {
+            NullValueHandling = NullValueHandling.Ignore,
+            StringEscapeHandling = StringEscapeHandling.Default
+        };
+        this.additional_data =
+            JsonConvert.SerializeObject(additional_data, jsonSerializerSettings);
+    }
+
+    public MarketingBody(string action, AdditionalData additional_data, string path_from,
+        string path_to) : this(action, additional_data)
+    {
+        this.path_from = path_from;
+        this.path_to = path_to;
     }
 
     public int user_id { get; set; } = -1;