不能让数据定义耦合具体实现逻辑

Author: lindexi

src/MediaConverters/MediaConverters.Tool/Contexts/IImageConvertTask.cs

@@ -1,12 +1,8 @@
-using SixLabors.ImageSharp;
-using SixLabors.ImageSharp.PixelFormats;
-
-using System.Text.Json.Serialization;
+using System.Text.Json.Serialization;
 
 namespace DotNetCampus.MediaConverters.Contexts;
 
 [JsonDerivedType(typeof(ReplaceColorTask))]
 public interface IImageConvertTask
 {
-    void Run(Image<Rgba32> image);
 }

src/MediaConverters/MediaConverters.Tool/Contexts/ReplaceColorTask.cs

@@ -1,32 +1,6 @@
-using DotNetCampus.MediaConverters.Imaging.Effects;
-using DotNetCampus.MediaConverters.Imaging.Effects.Colors;
-using DotNetCampus.MediaConverters.Utils;
-
-using SixLabors.ImageSharp;
-using SixLabors.ImageSharp.PixelFormats;
-
-namespace DotNetCampus.MediaConverters.Contexts;
+namespace DotNetCampus.MediaConverters.Contexts;
 
 public class ReplaceColorTask : IImageConvertTask
 {
     public List<ReplaceColorInfo>? ReplaceColorInfoList { get; init; }
-
-    public void Run(Image<Rgba32> image)
-    {
-        if (ReplaceColorInfoList is null || ReplaceColorInfoList.Count == 0)
-        {
-            return;
-        }
-
-        var dictionary = new List<(ColorMetadata SourceColor, ColorMetadata TargetColor)>(ReplaceColorInfoList.Count);
-        foreach (var replaceColorInfo in ReplaceColorInfoList)
-        {
-            var (oldColor, newColor) = replaceColorInfo.ToRgba32Pair();
-            var oldMetadata = new ColorMetadata(oldColor);
-            var newMetadata = new ColorMetadata(newColor);
-            dictionary.Add((oldMetadata, newMetadata));
-        }
-
-        image.ReplaceColor(dictionary);
-    }
 }

src/MediaConverters/MediaConverters.Tool/Program.cs

@@ -1,13 +1,15 @@
 // See https://aka.ms/new-console-template for more information
 
-using System.Diagnostics;
-using System.Text.Json;
-
 using DotNetCampus.MediaConverters.Contexts;
 using DotNetCampus.MediaConverters.Imaging.Optimizations;
+using DotNetCampus.MediaConverters.Workers;
 
 using SixLabors.ImageSharp;
 using SixLabors.ImageSharp.PixelFormats;
+
+using System.Diagnostics;
+using System.Text.Json;
+
 using SourceGenerationContext = DotNetCampus.MediaConverters.Contexts.SourceGenerationContext;
 
 namespace DotNetCampus.MediaConverters;
@@ -44,16 +46,12 @@ public static async Task<int> Main(string[] args)
                     break;
                 case ImageFileOptimizationFailureReason.UnknownImageFormat:
                     return ErrorCode.UnknownImageFormat;
-                    break;
                 case ImageFileOptimizationFailureReason.InvalidImageContent:
                     return ErrorCode.InvalidImageContent;
-                    break;
                 case ImageFileOptimizationFailureReason.FileNotFound:
                     return ErrorCode.ImageFileNotFound;
-                    break;
                 case ImageFileOptimizationFailureReason.NotSupported:
                     return ErrorCode.NotSupported;
-                    break;
                 default:
                     throw new ArgumentOutOfRangeException();
             }
@@ -66,10 +64,11 @@ public static async Task<int> Main(string[] args)
         if (imageConvertContext.ImageConvertTaskList is { } list)
         {
             using var image = await Image.LoadAsync<Rgba32>(optimizedImageFile.FullName);
+            var workerProvider = new WorkerProvider();
 
-            foreach (var imageConvertTask in list)
+            foreach (IImageConvertTask imageConvertTask in list)
             {
-                imageConvertTask.Run(image);
+                workerProvider.Run(image, imageConvertTask);
             }
         }
 

src/MediaConverters/MediaConverters.Tool/Workers/IWorker.cs

@@ -0,0 +1,11 @@
+using DotNetCampus.MediaConverters.Contexts;
+using SixLabors.ImageSharp;
+using SixLabors.ImageSharp.PixelFormats;
+
+namespace DotNetCampus.MediaConverters.Workers;
+
+interface IWorker
+{
+    Type TaskType { get; }
+    void Run(Image<Rgba32> image, IImageConvertTask task);
+}

src/MediaConverters/MediaConverters.Tool/Workers/ReplaceColorWorker.cs

@@ -0,0 +1,31 @@
+using DotNetCampus.MediaConverters.Contexts;
+using DotNetCampus.MediaConverters.Imaging.Effects;
+using DotNetCampus.MediaConverters.Imaging.Effects.Colors;
+using DotNetCampus.MediaConverters.Utils;
+
+using SixLabors.ImageSharp;
+using SixLabors.ImageSharp.PixelFormats;
+
+namespace DotNetCampus.MediaConverters.Workers;
+
+internal class ReplaceColorWorker : WorkerBase<ReplaceColorTask>
+{
+    protected override void RunCore(Image<Rgba32> image, ReplaceColorTask task)
+    {
+        if (task.ReplaceColorInfoList is null || task.ReplaceColorInfoList.Count == 0)
+        {
+            return;
+        }
+
+        var dictionary = new List<(ColorMetadata SourceColor, ColorMetadata TargetColor)>(task.ReplaceColorInfoList.Count);
+        foreach (var replaceColorInfo in task.ReplaceColorInfoList)
+        {
+            var (oldColor, newColor) = replaceColorInfo.ToRgba32Pair();
+            var oldMetadata = new ColorMetadata(oldColor);
+            var newMetadata = new ColorMetadata(newColor);
+            dictionary.Add((oldMetadata, newMetadata));
+        }
+
+        image.ReplaceColor(dictionary);
+    }
+}

src/MediaConverters/MediaConverters.Tool/Workers/WorkerBase.cs

@@ -0,0 +1,17 @@
+using DotNetCampus.MediaConverters.Contexts;
+using SixLabors.ImageSharp;
+using SixLabors.ImageSharp.PixelFormats;
+
+namespace DotNetCampus.MediaConverters.Workers;
+
+abstract class WorkerBase<T> : IWorker
+    where T : IImageConvertTask
+{
+    public void Run(Image<Rgba32> image, IImageConvertTask task)
+    {
+        RunCore(image, (T) task);
+    }
+
+    protected abstract void RunCore(Image<Rgba32> image, T task);
+    public Type TaskType => typeof(T);
+}

src/MediaConverters/MediaConverters.Tool/Workers/WorkerProvider.cs

@@ -0,0 +1,25 @@
+using DotNetCampus.MediaConverters.Contexts;
+using SixLabors.ImageSharp;
+using SixLabors.ImageSharp.PixelFormats;
+
+namespace DotNetCampus.MediaConverters.Workers;
+
+class WorkerProvider
+{
+    private readonly IReadOnlyList<IWorker> _workerList =
+    [
+        new ReplaceColorWorker(),
+    ];
+
+    public void Run(Image<Rgba32> image, IImageConvertTask task)
+    {
+        var taskType = task.GetType();
+        foreach (var worker in _workerList)
+        {
+            if (worker.TaskType == taskType)
+            {
+                worker.Run(image, task);
+            }
+        }
+    }
+}