Improve code quality

Author: Jack251970

Flow.Launcher.Infrastructure/Storage/BinaryStorage.cs

@@ -81,27 +81,27 @@ private static async ValueTask<T> DeserializeAsync(Stream stream, T defaultData)
             }
         }
 
-        public async ValueTask SaveAsync()
-        {
-            await using var stream = new FileStream(FilePath, FileMode.Create);
-            await MemoryPackSerializer.SerializeAsync(stream, Data);
-        }
-
-        // For SavePluginSettings function
         public void Save()
         {
             var serialized = MemoryPackSerializer.Serialize(Data);
 
             File.WriteAllBytes(FilePath, serialized);
         }
 
-        // ImageCache need to be converted into concurrent dictionary, so it does not need to cache loading results into Data
+        public async ValueTask SaveAsync()
+        {
+            await SaveAsync(Data.NonNull());
+        }
+
+        // ImageCache need to convert data into concurrent dictionary for usage,
+        // so we would better to clear the data
         public void ClearData()
         {
             Data = default;
         }
 
-        // ImageCache storages data in its class, so it needs to pass it to SaveAsync
+        // ImageCache storages data in its class,
+        // so we need to pass it to SaveAsync
         public async ValueTask SaveAsync(T data)
         {
             await using var stream = new FileStream(FilePath, FileMode.Create);